[PR #79] [MERGED] [DDC-725] Removed onUpdate property on join columns #7684

Closed
opened 2026-01-22 15:56:21 +01:00 by admin · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/79
Author: @mridgway
Created: 6/28/2011
Status: Merged
Merged: 7/26/2011
Merged by: @beberlei

Base: masterHead: DDC-725


📝 Commits (2)

  • 9ae3042 Removed onUpdate property on join columns
  • c19d7fe Missed one

📊 Changes

18 files changed (+4 additions, -46 deletions)

View changed files

📝 lib/Doctrine/ORM/Mapping/Driver/AnnotationDriver.php (+0 -4)
📝 lib/Doctrine/ORM/Mapping/Driver/DoctrineAnnotations.php (+0 -1)
📝 lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php (+0 -4)
📝 lib/Doctrine/ORM/Mapping/Driver/YamlDriver.php (+0 -4)
📝 lib/Doctrine/ORM/Tools/ConvertDoctrine1Schema.php (+0 -1)
📝 lib/Doctrine/ORM/Tools/EntityGenerator.php (+0 -4)
📝 lib/Doctrine/ORM/Tools/Export/Driver/XmlExporter.php (+0 -9)
📝 lib/Doctrine/ORM/Tools/Export/Driver/YamlExporter.php (+0 -3)
📝 lib/Doctrine/ORM/Tools/SchemaTool.php (+0 -4)
📝 tests/Doctrine/Tests/ORM/Functional/Ticket/DDC448Test.php (+1 -1)
📝 tests/Doctrine/Tests/ORM/Mapping/AbstractMappingDriverTest.php (+2 -4)
📝 tests/Doctrine/Tests/ORM/Mapping/php/Doctrine.Tests.ORM.Mapping.User.php (+0 -1)
📝 tests/Doctrine/Tests/ORM/Mapping/yaml/Doctrine.Tests.ORM.Mapping.User.dcm.yml (+0 -1)
📝 tests/Doctrine/Tests/ORM/Tools/Export/AbstractClassMetadataExporterTest.php (+0 -1)
📝 tests/Doctrine/Tests/ORM/Tools/Export/annotation/Doctrine.Tests.ORM.Tools.Export.User.php (+1 -1)
📝 tests/Doctrine/Tests/ORM/Tools/Export/php/Doctrine.Tests.ORM.Tools.Export.User.php (+0 -1)
📝 tests/Doctrine/Tests/ORM/Tools/Export/yaml/Doctrine.Tests.ORM.Tools.Export.User.dcm.yml (+0 -1)
📝 tests/Doctrine/Tests/ORM/Tools/doctrine1schema/schema.yml (+0 -1)

📄 Description

This removes the onUpdate property on join columns. This breaks BC for any metadata that has an onUpdate property. The exception being thrown right now is:

BadMethodCallException: Unknown property 'onUpdate' on annotation 'Doctrine\ORM\Mapping\JoinColumn'


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/doctrine/orm/pull/79 **Author:** [@mridgway](https://github.com/mridgway) **Created:** 6/28/2011 **Status:** ✅ Merged **Merged:** 7/26/2011 **Merged by:** [@beberlei](https://github.com/beberlei) **Base:** `master` ← **Head:** `DDC-725` --- ### 📝 Commits (2) - [`9ae3042`](https://github.com/doctrine/orm/commit/9ae30421dd80683e1c26b09de7d86000ddeadd1c) Removed onUpdate property on join columns - [`c19d7fe`](https://github.com/doctrine/orm/commit/c19d7fe2ebe465f5cba4e4a45ba8ea119a18a4d9) Missed one ### 📊 Changes **18 files changed** (+4 additions, -46 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Mapping/Driver/AnnotationDriver.php` (+0 -4) 📝 `lib/Doctrine/ORM/Mapping/Driver/DoctrineAnnotations.php` (+0 -1) 📝 `lib/Doctrine/ORM/Mapping/Driver/XmlDriver.php` (+0 -4) 📝 `lib/Doctrine/ORM/Mapping/Driver/YamlDriver.php` (+0 -4) 📝 `lib/Doctrine/ORM/Tools/ConvertDoctrine1Schema.php` (+0 -1) 📝 `lib/Doctrine/ORM/Tools/EntityGenerator.php` (+0 -4) 📝 `lib/Doctrine/ORM/Tools/Export/Driver/XmlExporter.php` (+0 -9) 📝 `lib/Doctrine/ORM/Tools/Export/Driver/YamlExporter.php` (+0 -3) 📝 `lib/Doctrine/ORM/Tools/SchemaTool.php` (+0 -4) 📝 `tests/Doctrine/Tests/ORM/Functional/Ticket/DDC448Test.php` (+1 -1) 📝 `tests/Doctrine/Tests/ORM/Mapping/AbstractMappingDriverTest.php` (+2 -4) 📝 `tests/Doctrine/Tests/ORM/Mapping/php/Doctrine.Tests.ORM.Mapping.User.php` (+0 -1) 📝 `tests/Doctrine/Tests/ORM/Mapping/yaml/Doctrine.Tests.ORM.Mapping.User.dcm.yml` (+0 -1) 📝 `tests/Doctrine/Tests/ORM/Tools/Export/AbstractClassMetadataExporterTest.php` (+0 -1) 📝 `tests/Doctrine/Tests/ORM/Tools/Export/annotation/Doctrine.Tests.ORM.Tools.Export.User.php` (+1 -1) 📝 `tests/Doctrine/Tests/ORM/Tools/Export/php/Doctrine.Tests.ORM.Tools.Export.User.php` (+0 -1) 📝 `tests/Doctrine/Tests/ORM/Tools/Export/yaml/Doctrine.Tests.ORM.Tools.Export.User.dcm.yml` (+0 -1) 📝 `tests/Doctrine/Tests/ORM/Tools/doctrine1schema/schema.yml` (+0 -1) </details> ### 📄 Description This removes the onUpdate property on join columns. This breaks BC for any metadata that has an onUpdate property. The exception being thrown right now is: BadMethodCallException: Unknown property 'onUpdate' on annotation 'Doctrine\ORM\Mapping\JoinColumn' --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
admin added the pull-request label 2026-01-22 15:56:21 +01:00
admin closed this issue 2026-01-22 15:56:21 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#7684