DDC-725: Consider removing onUpdate attribute from join column mappings #893

Closed
opened 2026-01-22 12:54:19 +01:00 by admin · 4 comments
Owner

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

Jira issue originally created by user romanb:

If onUpdate is only ever used in cases where a primary key changes, which is bad design and not supported by doctrine anyway, there is no reason to support this attribute.

Originally created by @doctrinebot on GitHub (Jul 29, 2010). Jira issue originally created by user romanb: If onUpdate is only ever used in cases where a primary key changes, which is bad design and not supported by doctrine anyway, there is no reason to support this attribute.
admin closed this issue 2026-01-22 12:54:21 +01:00
Author
Owner

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

Comment created by @beberlei:

ah yes, this is probably a good idea.

@doctrinebot commented on GitHub (Jul 30, 2010): Comment created by @beberlei: ah yes, this is probably a good idea.
Author
Owner

@doctrinebot commented on GitHub (Jun 21, 2011):

Comment created by mridgway:

Do you plan on removing the annotation or just ignoring it? Removing it will cause a backwards compatibility issue for people that did use it.

I will work on a patch for this then.

@doctrinebot commented on GitHub (Jun 21, 2011): Comment created by mridgway: Do you plan on removing the annotation or just ignoring it? Removing it will cause a backwards compatibility issue for people that did use it. I will work on a patch for this then.
Author
Owner

@doctrinebot commented on GitHub (Jul 26, 2011):

Comment created by @beberlei:

Merged Pull Request, closing this issue.

@doctrinebot commented on GitHub (Jul 26, 2011): Comment created by @beberlei: Merged Pull Request, closing this issue.
Author
Owner

@doctrinebot commented on GitHub (Jul 26, 2011):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Jul 26, 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#893