DDC-685: Bug in BasicEntityPersister #844

New Issue

admin · 2026-01-22T12:52:26+01:00

admin commented

2026-01-22 12:52:26 +01:00

Originally created by @doctrinebot on GitHub (Jul 12, 2010).

Jira issue originally created by user avi:

I have Entity:

class Comments
{
/****
* @Column(type="integer")
* @Id
* @GeneratedValue(strategy="IDENTITY")
*/
private $commentId;

/****
* @Column (type="integer", nullable=true)
*/
private $userId;
...

/****
 * @ManyToOne(targetEntity="Entities\Passport\Clients")
 * @JoinColumn(name="userId", referencedColumnName="clientID")
 */
private $Clients;

...

here i have field userId, and relation with another table on this field.

Where i`m trying to insert (flush) new record in table, i get null value in field userId.

In method _prepareUpdateData (class BasicEntityPersister) field userId overlaps by field of "attachable" table Clents - userId, and sets to null.

Originally created by @doctrinebot on GitHub (Jul 12, 2010). Jira issue originally created by user avi: I have Entity: class Comments { /**** \* @Column(type="integer") \* @Id \* @GeneratedValue(strategy="IDENTITY") */ private $commentId; /**** \* @Column (type="integer", nullable=true) */ private $userId; ... ``` /**** * @ManyToOne(targetEntity="Entities\Passport\Clients") * @JoinColumn(name="userId", referencedColumnName="clientID") */ private $Clients; ... ``` here i have field userId, and relation with another table on this field. Where i`m trying to insert (flush) new record in table, i get null value in field userId. In method _prepareUpdateData (class BasicEntityPersister) field userId overlaps by field of "attachable" table Clents - userId, and sets to null.

admin added the Bug label 2026-01-22 12:52:26 +01:00

admin closed this issue

2026-01-22 12:52:27 +01:00

admin commented

2026-01-22 12:52:28 +01:00

@doctrinebot commented on GitHub (Jul 12, 2010):

Comment created by @beberlei:

First: your mapping is wrong, you are not supposed to map foreign keys. UserId is supposed to be a ManyToOne relation with the User class.

Second: If you want to do this anyways, there is arleady two issues for it: DDC-419 and DDC-522

@doctrinebot commented on GitHub (Jul 12, 2010): Comment created by @beberlei: First: your mapping is wrong, you are not supposed to map foreign keys. UserId is supposed to be a ManyToOne relation with the User class. Second: If you want to do this anyways, there is arleady two issues for it: [DDC-419](http://www.doctrine-project.org/jira/browse/DDC-419) and [DDC-522](http://www.doctrine-project.org/jira/browse/DDC-522)

admin commented

2026-01-22 12:52:29 +01:00

@doctrinebot commented on GitHub (Jul 12, 2010):

Issue was closed with resolution "Invalid"

@doctrinebot commented on GitHub (Jul 12, 2010): Issue was closed with resolution "Invalid"

admin referenced this issue

2026-01-22 16:01:44 +01:00

[PR #844] [MERGED] Teach orm:validate-schema to --skip-mapping and --skip-sync #8791

Sign in to join this conversation.