From 572b003e299428238546813f956ebbb23f800946 Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <alex.aubin@mailoo.org>
Date: Tue, 29 Oct 2019 17:17:43 +0100
Subject: [PATCH] We gotta return a permission structure here in all case,
 otherwise stuff like app_addaccess will miserably fail in these case

---
 src/yunohost/permission.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/yunohost/permission.py b/src/yunohost/permission.py
index 226cc9050..c53804d49 100644
--- a/src/yunohost/permission.py
+++ b/src/yunohost/permission.py
@@ -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