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

targetDocument is not always available. #175

Merged
merged 1 commit into from Sep 12, 2012

Conversation

Projects
None yet
2 participants
Contributor

simensen commented Sep 12, 2012

The documentation implied that targetDocument was optional but this was causing an undefined index notice when targetDocument is not specified.

dbu added a commit that referenced this pull request Sep 12, 2012

Merge pull request #175 from simensen/undefined-index
targetDocument is not always available.

@dbu dbu merged commit fa22ffd into doctrine:master Sep 12, 2012

Member

dbu commented Sep 12, 2012

thanks. surprised we did not see that before. there is a test having no targetDocument (most tests do have one, but not: https://github.com/doctrine/phpcr-odm/blob/master/tests/Doctrine/Tests/ODM/PHPCR/Functional/ProtectedPropertyTest.php)

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