zerodine / MongoDBAclBundle

This bundle allows the use of MongoDB as the storage layer for the Symfony ACLs component.
21 stars 32 forks source link

Adds the ability to update ACEs when username changes #37

Closed cristiangraz closed 6 years ago

cristiangraz commented 9 years ago

This commit adds an updateUserSecurityIdentity method, which is needed when a user updates their username. In my case, the username is just an alias to an email address, so this was necessary if the user happens to update their email address. See this issue: https://github.com/symfony/symfony/issues/5787 as well as this commit: https://github.com/symfony/symfony/commit/8d39213f4cca19466f84a5656a199eee98602ab1

A deleteAceBySecurityIdentity was also added. This allows you to pass in a UserSecurityIdentity object and delete all ACEs associated with it. This is useful when editing a user's permissions via checkboxes, for example, and being able to simply delete all their ACEs and recreate them on edit.

cristiangraz commented 9 years ago

@iampersistent Could you trigger a refresh on this travis build? PHP 5.3 and 5.5. failed, but 5.4 passed. The build failure looks like a Travis issue vs an issue with the commit: "MongoConnectionException: Failed to connect to: localhost:27017: Connection refused".

Hoping someone with write access to the repo and signed into Travis can trigger a refresh. Let me know if there's someone else who I should direct this to.

http://stackoverflow.com/a/17624403/1527587

iampersistent commented 9 years ago

Sorry, I don't have write access on the repo. @tspycher or @philippspinnler can you handle this?

ghost commented 9 years ago

ping @tspycher @philippspinnler

mmeloni commented 8 years ago

News??Acl stil bugging without this commit