Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[Docs] Remove reference to deprecated ORM hydrator #150

Merged
merged 1 commit into from Jan 17, 2013

Conversation

Projects
None yet
3 participants
Contributor

davidwindell commented Jan 17, 2013

Remove reference to deprecated ORM hydrator.

Question @bakura10 @Ocramius - do we now always do find() calls? Can this/should this be optimised back to using getReference() for ORM users?

Member

bakura10 commented Jan 17, 2013

Ask @Ocramius about the find calls... he removed it :D. I tried so hard, fight against him... but I lose on this battle :(. getReference is definitely gone.

Contributor

davidwindell commented Jan 17, 2013

@Ocramius I'd be interested to hear the reasons why

Owner

Ocramius commented Jan 17, 2013

@davidwindell problems are:

  1. data types discovery (the long story of '' !== null)
  2. inheritances (getReference does not work on inheritances)

If the problem is performance, any input filter can convert '' to null and there you go: no call to find

Contributor

davidwindell commented Jan 17, 2013

Thanks for the summary @Ocramius

Ocramius added a commit that referenced this pull request Jan 17, 2013

Merge pull request #150 from davidwindell/patch-5
[Docs] Remove reference to deprecated ORM hydrator

@Ocramius Ocramius merged commit 31d75b3 into doctrine:master Jan 17, 2013

1 check passed

default The Travis build passed
Details

gencer pushed a commit to gencer/DoctrineModule that referenced this pull request Apr 11, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment