commit c30833957368b7cde6b12a795ffcccade8b992f5
author Bogdan Kyryliuk <bogdan.kyryliuk> Fri Nov 18 15:56:22 2016 -0800
committer Bogdan Kyryliuk <bogdan.kyryliuk> Mon Nov 21 13:31:43 2016 -0800
tree 3b767991907b96fdcd515ecbd12f152c1d24cba0
parent bd6a439e0b2a3a76f8aece91f11a7eee2ebf6d29

Admin / Alpha permission cleanup and fixes.

superset/security.py

diff --git a/superset/security.py b/superset/security.py
index 10bdd9a..8e97a0c 100644
--- a/superset/security.py
+++ b/superset/security.py
@@ -54,6 +54,7 @@
     'muldelete',
     'all_datasource_access',
 ])
+
 READ_ONLY_PRODUCT = set(
     product(READ_ONLY_PERMISSION, READ_ONLY_MODELVIEWS))
 
@@ -126,8 +127,7 @@
                     p.view_menu.name not in ADMIN_ONLY_VIEW_MENUES and
                     p.permission.name not in ADMIN_ONLY_PERMISSIONS
                 ) or
-                (p.permission.name, p.view_menu.name) in READ_ONLY_PRODUCT
-        ):
+                (p.permission.name, p.view_menu.name) in READ_ONLY_PRODUCT):
             sm.add_permission_role(alpha, p)
         else:
             sm.del_permission_role(alpha, p)