We gotta return a permission structure here in all case, otherwise stuff like app_addaccess will miserably fail in these case

This commit is contained in:
Alexandre Aubin 2019-10-29 17:17:43 +01:00
parent 71bc6a0faf
commit 572b003e29

View file

@ -155,7 +155,7 @@ def user_permission_update(operation_logger, permission, add=None, remove=None,
# Don't update LDAP if we update exactly the same values
if set(new_allowed_groups) == set(current_allowed_groups):
logger.warning(m18n.n("permission_already_up_to_date"))
return
return existing_permission
# Commit the new allowed group list