FriendsOfSymfony/FOSUserBundle — comment on commit e4ef0db6bf72b860717e5346bbb189bc34057f66
File:  line 0
By stof:

well, there is 2 things to consider here:

- currently, the interface does not enforce the getId() method so it would be a BC break. And it could be an issue with other bundles using an interface with a getId method as it would make them incompatible (isn't OrnicarMessageBundle doing this ?)
- adding the method could improve performances a bit here as retrieving the object by primary key is optimized in doctrine ORM (and probably in the ODM too)

And this will not solve the ACL issue: the security component always uses the username for the ACL. There is no way to change this if the username is not immutable. @schmittjoh wouldn't it make sense to add an interface to get the acl identifier ? If the user class does not implement it, the acl system would keep using the username (so fully BC) but the dev would be able to use another field as immutable identifier (typically the primary key)