Skip to content

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Aug 18, 2025

Q A
Branch? 6.4
Bug fix? no
New feature? no
Deprecations? no
Issues
License MIT

see doctrine/orm#12126

@carsonbot carsonbot added this to the 6.4 milestone Aug 18, 2025
@carsonbot carsonbot changed the title [DoctrineBridge] remove deprecated nullable option from primary key columns [DoctrineBridge] remove deprecated nullable option from primary key columns Aug 18, 2025
@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit b3b7847 into symfony:6.4 Aug 18, 2025
11 checks passed
@xabbuh xabbuh deleted the doctrine-orm-12126 branch August 18, 2025 09:42
@derrabus
Copy link
Member

Did these attributes trigger the deprecation? It was my understanding that we've deprecated that setting for many-to-many relations only because it does not make sense to have nullable join columns on a join table. /cc @greg0ire

@xabbuh
Copy link
Member Author

xabbuh commented Aug 18, 2025

Did these attributes trigger the deprecation?

yes, see https://github.com/symfony/symfony/actions/runs/17033331265/job/48280266743#step:8:155

@greg0ire
Copy link
Contributor

@derrabus it was not only nullable join columns on a join table, it was also to-one join columns that are part of the identifier (and so end up being part of the primary key).

@derrabus
Copy link
Member

Right, they join column is part of the PK here. I missed hat. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants