Problematic/ProblematicAclManagerBundle — Issue #9: ACEs are not "updated"
Author: khepin
Created: 2012-01-24 08:25:59+00:00
State: open

In the case where a permission should go from a higher one to a lower one say from DELETE to only EDIT, the permissions are not updated.
This is because the ACL component will just add new entries whenever they are different from existing ones. Also, for that matter I think the check in "doApplyPermission" to see if a permission already matches is not useful as this is taken care of by the Acl object already.

I forked it to see how to do the update and the code I got is now at https://github.com/khepin/ProblematicAclManagerBundle/tree/update_permissions if you're interested. I don't think that this should be the way the code actually looks like in the end, but I have too many different opinions on what the api for this should look like to actually modify all the code.

Also your code says "addPermission" but the comments say "updates a permission" so I don't know if the current behavior is the one you were actually aiming for or if the comments were actually what you wanted so I did this in a different setPermission method.