Skip to content

Fix a condition that would not allow diffing of foreign keys for Sqlite#4011

Closed
camilleterol wants to merge 35 commits intodoctrine:2.10.xfrom
camilleterol:2.11.x
Closed

Fix a condition that would not allow diffing of foreign keys for Sqlite#4011
camilleterol wants to merge 35 commits intodoctrine:2.10.xfrom
camilleterol:2.11.x

Conversation

@camilleterol
Copy link

@camilleterol camilleterol commented May 17, 2020

Q A
Type bug
BC Break yes
Fixed issues #1204, #4010

Summary

After the changes introduced by #1204, the SchemaDiff would throw an error when trying to diff an Sqlite-backed schema with foreign keys modifications.

BenMorel and others added 30 commits January 27, 2020 20:45
Rename andX() / orX() methods
First parameter of ExpressionBuilder::and/or() mandatory
Prepare CompositeExpression for immutability
Deprecate calling QueryBuilder methods with an array argument
Deprecated the usage of the Version class
…eexpression

CompositeExpression and()/or() factory methods
Deprecate the usage of the legacy platforms and drivers
1. Removed no longer existing entries
2. Added new entries
3. Sorted using `sort` (ignores case and the leading dot, same as `ls`)
Actualize the content of the .gitattributes file
….11.x_5e9de30c262e49.66926046

Merge release 2.10.2 into 2.11.x
@greg0ire
Copy link
Member

greg0ire commented May 17, 2020

This looks a lot like #3994 … maybe there are more occurrences of supportsForeignKeyConstraints that should be re-evaluated?

In any case, please add a test.

Also, since this is a patch, you should target 2.10.x IMO.

@greg0ire greg0ire added the Bug label May 17, 2020
@camilleterol
Copy link
Author

camilleterol commented May 17, 2020

Thanks for the swift reply. I'll look into adding a test for this specific use case.
I'll reopen a PR targeting the correct branch then.

@greg0ire
Copy link
Member

git rebase --onto origin/2.10.x origin/2.11.x 2.11.x would have done the trick ;)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants