diff --git a/.coveralls.yml b/.coveralls.yml index 273f84b694..e916d8ed05 100644 --- a/.coveralls.yml +++ b/.coveralls.yml @@ -1 +1 @@ -repo_token: eESbYiv4An6KEvjpmguDs4L7YkubXbqn1 +repo_token: 4P9MpvLrZfJKzHdGZsdV3MzO43OZJgYFn diff --git a/superset/security.py b/superset/security.py index 5878ea0ba2..9a01d6b652 100644 --- a/superset/security.py +++ b/superset/security.py @@ -164,7 +164,8 @@ def create_missing_perms(): "Fetching a set of all perms to lookup which ones are missing") all_pvs = set() for pv in sm.get_session.query(sm.permissionview_model).all(): - all_pvs.add((pv.permission.name, pv.view_menu.name)) + if pv.permission and pv.view_menu: + all_pvs.add((pv.permission.name, pv.view_menu.name)) def merge_pv(view_menu, perm): """Create permission view menu only if it doesn't exist"""