diff options
| author | Fabien Potencier <fabien.potencier@gmail.com> | 2013-11-22 18:20:31 +0100 |
|---|---|---|
| committer | Fabien Potencier <fabien.potencier@gmail.com> | 2013-11-22 18:20:31 +0100 |
| commit | 63ffd37014000fcd75585b98f8b08a83c05de7b5 (patch) | |
| tree | 3fc9717829f1c9c35dcfec34f15540e331ab4f11 /Tests/Core/Encoder/MessageDigestPasswordEncoderTest.php | |
| parent | 689e28551f445853c2e6f2430b30dcf65de8db30 (diff) | |
| parent | ad60b880267e83089e8e614927c2d3a4cfc1f205 (diff) | |
| download | symfony-security-63ffd37014000fcd75585b98f8b08a83c05de7b5.zip symfony-security-63ffd37014000fcd75585b98f8b08a83c05de7b5.tar.gz symfony-security-63ffd37014000fcd75585b98f8b08a83c05de7b5.tar.bz2 | |
bug #9485 [Acl] Fix for issue #9433 (guilro)
This PR was squashed before being merged into the 2.2 branch (closes #9485).
Discussion
----------
[Acl] Fix for issue #9433
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9433
| License | MIT
| Doc PR |
Two new test for issue #9433 :
`testUpdateAclInsertingMultipleObjectFieldAcesThrowsDBConstraintViolations()`
`testUpdateAclDeletingObjectFieldAcesThrowsDBConstraintViolations()`
The change to `updateAces()` line 857 is enough to make the first test succeed. When changing the `order` field value to a higher value, we must first change the value of the next entry (and all the next entries recursively) to preserve uniqueness of the `order` field in the database.
All the other changes are for the second test. In the former `updateAcl()` method, we commit the changes of the existing ACEs to the database before deleting or adding the new ones. We must delete the old ACEs before changing the existing ACEs in order to preserve uniqueness of the `order` field in the database.
Commits
-------
a38fab9 [Acl] Fix for issue #9433
Diffstat (limited to 'Tests/Core/Encoder/MessageDigestPasswordEncoderTest.php')
0 files changed, 0 insertions, 0 deletions