[PR #458] [MERGED] Use cascade=all if all cascade options set #8228

New Issue

admin · 2026-01-22T15:58:58+01:00

admin commented

2026-01-22 15:58:58 +01:00

📋 Pull Request Information

Original PR: https://github.com/doctrine/orm/pull/458
Author: @carlos-granados
Created: 9/29/2012
Status: ✅ Merged
Merged: 10/5/2012
Merged by: @beberlei

Base: master ← Head: master

📝 Commits (4)

c97eff9 Output cascade={"all"} if all cascade options set
cd37ec4 Set 'cascade' => 'all' if all cascade options set
3b27216 add empty lines around if block
919cf85 Remove trailing whitespace

📊 Changes

2 files changed (+9 additions, -0 deletions)

View changed files

📝 lib/Doctrine/ORM/Tools/EntityGenerator.php (+4 -0)
📝 lib/Doctrine/ORM/Tools/Export/Driver/PhpExporter.php (+5 -0)

📄 Description

The YAML and XML metadata exporters output cascade=all if all five cascade options are set. The php and annotation exporter do not, they output cascade=list of all 5 cascades instead.

In order that all 4 exporters offer the same behavior, I updated the php and annotations exporters so that they export cascade=all

_{🔄 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/458 **Author:** [@carlos-granados](https://github.com/carlos-granados) **Created:** 9/29/2012 **Status:** ✅ Merged **Merged:** 10/5/2012 **Merged by:** [@beberlei](https://github.com/beberlei) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (4) - [`c97eff9`](https://github.com/doctrine/orm/commit/c97eff94f56929238ace052af4b5e77df93c9317) Output cascade={"all"} if all cascade options set - [`cd37ec4`](https://github.com/doctrine/orm/commit/cd37ec47d55dd5d8300e8cb8a2c8fb1ea120bc7f) Set 'cascade' => 'all' if all cascade options set - [`3b27216`](https://github.com/doctrine/orm/commit/3b27216c5153b962a4436b0549c8c59986e0483e) add empty lines around if block - [`919cf85`](https://github.com/doctrine/orm/commit/919cf8558ba0170025132108c2f9da36612f5860) Remove trailing whitespace ### 📊 Changes **2 files changed** (+9 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `lib/Doctrine/ORM/Tools/EntityGenerator.php` (+4 -0) 📝 `lib/Doctrine/ORM/Tools/Export/Driver/PhpExporter.php` (+5 -0) </details> ### 📄 Description The YAML and XML metadata exporters output cascade=all if all five cascade options are set. The php and annotation exporter do not, they output cascade=list of all 5 cascades instead. In order that all 4 exporters offer the same behavior, I updated the php and annotations exporters so that they export cascade=all --- <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:58:58 +01:00

admin referenced this issue

2026-01-22 16:08:57 +01:00

[PR #8228] Update SchemaTool.php #10867

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8228