Commit 9a26a9ba authored by Kazuhiko Shiozaki's avatar Kazuhiko Shiozaki

Revert "ERP5Security.ERP5LoginUserManager: enumerateUsers() should not include...

Revert "ERP5Security.ERP5LoginUserManager: enumerateUsers() should not include Persons without any Login."

This reverts commit caefbddd.
parent 06c3f2cd
......@@ -226,7 +226,7 @@ class ERP5LoginUserManager(BasePlugin):
'id': user['reference'],
# Note: PAS forbids us from returning more than one entry per given id,
# so take any available login.
'login': login_dict[user['uid']][0]['reference'],
'login': login_dict.get(user['uid'], [None])[0]['reference'],
'pluginid': plugin_id,
# Extra properties, specific to ERP5
......@@ -237,10 +237,10 @@ class ERP5LoginUserManager(BasePlugin):
'path': login['path'],
'uid': login['uid'],
}
for login in login_dict[user['uid']]
for login in login_dict.get(user['uid'], [])
],
}
for user in user_list if login_dict.get(user['uid'], []) and user['reference']
for user in user_list if user['reference']
])
classImplements(ERP5LoginUserManager, IAuthenticationPlugin, IUserEnumerationPlugin)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment