Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@alexander-schranz
Copy link
Member

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no yes
Fixed tickets fixes #
Related issues/PRs #
License MIT
Documentation PR sulu/sulu-docs#

What's in this PR?

Add support for Doctrine DBAL 4 and Doctrine Persistence 4.

Why?

ORM 3 supports new DBAL and Persistence component versin.


<one-to-many field="routes" target-entity="Sulu\CustomUrl\Domain\Model\CustomUrlRouteInterface" mapped-by="customUrl" fetch="EXTRA_LAZY" orphan-removal="true">
<cascade>
<cascade-all/>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/cc @wachterjohannes not sure why this was added this is normal a nogo from performance view.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure to be honest - but i think persist should be enough - have you tried if removing custom-url is working and remove the route?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cascade on database level is still set and so removes the route.

@alexander-schranz alexander-schranz added the DX Affecting the end developer label Dec 11, 2025
@alexander-schranz alexander-schranz merged commit f45de8f into sulu:3.0 Dec 12, 2025
9 checks passed
@alexander-schranz alexander-schranz deleted the enhancement/doctrine-dbal-persistence-update branch December 12, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DX Affecting the end developer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants