DDC-1505: EntityManager->find returning null for entity on the inverse side of a OneToOne association #1884

Closed
opened 2026-01-22 13:29:43 +01:00 by admin · 5 comments
Owner

Originally created by @doctrinebot on GitHub (Nov 23, 2011).

Originally assigned to: @asm89 on GitHub.

Jira issue originally created by user chrisrichard:

BasicEntityPersister.getJoinSQLForJoinColumns treats joinColumn nullable setting as defaulting to false:

         if(isset($joinColumn['nullable']) && $joinColumn['nullable']){
             return 'LEFT JOIN';
         }

I think it should be:

         if(!isset($joinColumn['nullable']) ](| $joinColumn['nullable')){
             return 'LEFT JOIN';
         }
Originally created by @doctrinebot on GitHub (Nov 23, 2011). Originally assigned to: @asm89 on GitHub. Jira issue originally created by user chrisrichard: BasicEntityPersister.getJoinSQLForJoinColumns treats joinColumn nullable setting as defaulting to false: ``` if(isset($joinColumn['nullable']) && $joinColumn['nullable']){ return 'LEFT JOIN'; } ``` I think it should be: ``` if(!isset($joinColumn['nullable']) ](| $joinColumn['nullable')){ return 'LEFT JOIN'; } ```
admin added the Bug label 2026-01-22 13:29:43 +01:00
admin closed this issue 2026-01-22 13:29:44 +01:00
Author
Owner

@doctrinebot commented on GitHub (Dec 15, 2011):

Comment created by @beberlei:

Assigned to asm

@doctrinebot commented on GitHub (Dec 15, 2011): Comment created by @beberlei: Assigned to asm
Author
Owner

@doctrinebot commented on GitHub (Dec 18, 2011):

Comment created by @beberlei:

This issue is referenced in Github Pull-Request GH-226
https://github.com/doctrine/doctrine2/pull/226

@doctrinebot commented on GitHub (Dec 18, 2011): Comment created by @beberlei: This issue is referenced in Github Pull-Request GH-226 https://github.com/doctrine/doctrine2/pull/226
Author
Owner

@doctrinebot commented on GitHub (Dec 18, 2011):

Comment created by @asm89:

Should be fixed as soon as this is pulled:
https://github.com/doctrine/doctrine2/pull/226

@doctrinebot commented on GitHub (Dec 18, 2011): Comment created by @asm89: Should be fixed as soon as this is pulled: https://github.com/doctrine/doctrine2/pull/226
Author
Owner

@doctrinebot commented on GitHub (Dec 18, 2011):

Comment created by @beberlei:

Related Pull Request was closed: https://github.com/doctrine/doctrine2/pull/226

@doctrinebot commented on GitHub (Dec 18, 2011): Comment created by @beberlei: Related Pull Request was closed: https://github.com/doctrine/doctrine2/pull/226
Author
Owner

@doctrinebot commented on GitHub (Dec 18, 2011):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Dec 18, 2011): Issue was closed with resolution "Fixed"
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#1884