From a959a97e8c7ba736bd87cf1c8a8e46ea8181dcee Mon Sep 17 00:00:00 2001 From: Kay0u Date: Wed, 20 Nov 2019 13:28:10 +0900 Subject: [PATCH] [Fix] permission create --- src/yunohost/permission.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/yunohost/permission.py b/src/yunohost/permission.py index 170019ee5..3f187a48c 100644 --- a/src/yunohost/permission.py +++ b/src/yunohost/permission.py @@ -273,9 +273,8 @@ def permission_create(operation_logger, permission, url=None, allowed=None, sync if allowed: if not isinstance(allowed, list): to_add = [allowed] - # For main permission, we add all users by default - elif permission.endswith(".main"): - to_add = "all_users" + else: + to_add = allowed new_permission = _update_ldap_group_permission(permission=permission, allowed=to_add, sync_perm=sync_perm) @@ -295,6 +294,10 @@ def permission_url(operation_logger, permission, url=None, sync_perm=True): from yunohost.utils.ldap import _get_ldap_interface ldap = _get_ldap_interface() + # By default, manipulate main permission + if "." not in permission: + permission = permission + ".main" + # Fetch existing permission existing_permission = user_permission_list(full=True)["permissions"].get(permission, None)