getUserIdentifier() instead of getUsername() in SF5.3 and higher?

[rsvitak]

I am sorry, this is likely a question than an issue report. An answer will actually help me to understand things.

On several places like this,

security-acl/Dbal/MutableAclProvider.php

Line 563 in f24881e

$identifier = $sid->getClass().'-'.$sid->getUsername();

there is the getUsername call, but, if the UserInterface::getUsername is deprecated since SF5.3, is it going to be replaced by getUserIdentifier() in the security-acl bundle too?

[wouterj]

Hi! As far as I'm concerned, the ACL component and bundle are treated as "maintained by the community".

The core team does their best to merge new PRs, keep the build green and release versions, but we don't actively maintain or change anything in these packages. So even more than on the main repository, there is no roadmap and there are no plans for the ACL packages.

That said, if you think it makes sense to rename the method, you're free to start a PR imho :)