Ticket #13661 (new Bug)

Opened 3 years ago

Last modified 3 years ago

PlonePas group plugins getGroupsByUserId() requires username, not userid.

Reported by: regebro Owned by:
Priority: minor Milestone: 4.x
Component: Unknown Version: 4.3
Keywords: Cc:

Description

PlonePas group plugins getGroupsByUserId() requires the username, not userid, meaning anyone who actually reads the documentation will see their software stop working if you migrate to using email as login.

Change History

comment:1 Changed 3 years ago by davisagli

That is confusing.

I'm not quite clear from your report on where the problem lies. Is there a particular add-on or code in Plone core that is using getGroupsByUserId incorrectly?

comment:2 Changed 3 years ago by davisagli

Is this a duplicate of #13660?

comment:3 Changed 3 years ago by regebro

It's in Plone core. And they are not duplicates, but they do the same mistake of using the user name, when they should use the User ID.

The problem is hence that if you call getGroupsByUserId() with the user id, it will fail. You have to call it with the user name. This is incorrect.

Note: See TracTickets for help on using tickets.