Use the interface of AccountProxy

Would there be problems with using AccountInterface instead?

See https://www.drupal.org/project/drupal/issues/3372939#comment-15172031 🐛 Use AccountProxyInterface instead of AccountInterface Needs work

Please use an issue fork and merge request.

Whatever the status of https://www.drupal.org/project/drupal/issues/3372939 🐛 Use AccountProxyInterface instead of AccountInterface Needs work , the only method this module calls on AccountProxy is hasPermission and that's in AccountInterface. So, in the issue fork I changed AccountProxy to AccountInterface. There appears to be crosstalk with the other issue fork I created so I can do this again if needed.

As I understand it, there are two things in play here:

• Using an interface instead of a class.
• Choosing between AccountProxyInterface and AccountInterface

Is that right? I totally agree with using an interface. ::hasPermission() is in both because AccountProxyInterface extends AccountInterface, so it makes sense to use AccountInterface.

I just found your commits in the issue fork. They were not on the feature branch. I have put them on the feature branch and created a merge request. Please review that I did what you intended.

Looking at it again, I am satisfied that this is the correct change. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.