Improve check before permission update

This commit is contained in:
Josué Tille 2019-12-30 13:28:56 +01:00
parent dc39b9a5af
commit e2def74997
No known key found for this signature in database
GPG key ID: 716A6C99B04194EF

View file

@ -166,7 +166,8 @@ def user_permission_update(operation_logger, permission, add=None, remove=None,
raise YunohostError('permission_cannot_remove_all_users_while_visitors_allowed') raise YunohostError('permission_cannot_remove_all_users_while_visitors_allowed')
# Don't update LDAP if we update exactly the same values # Don't update LDAP if we update exactly the same values
if set(new_allowed_groups) == set(current_allowed_groups) and protected is None: if set(new_allowed_groups) == set(current_allowed_groups) and \
(protected is None or protected == existing_permission["protected"]):
logger.warning(m18n.n("permission_already_up_to_date")) logger.warning(m18n.n("permission_already_up_to_date"))
return existing_permission return existing_permission