Merge pull request #384 from keitaroinc/fix/notifications-to-active-s…

…ysadmins-only Retrieves only active sysadmin users.
keitaroinc · May 20, 2020 · 4335bc4 · 4335bc4
2 parents 92a8e24 + 66a4a5b
commit 4335bc4
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/ckanext/knowledgehub/logic/action/create.py b/ckanext/knowledgehub/logic/action/create.py
@@ -1953,6 +1953,7 @@ def _find_sysadmins():
     '''
     q = model.Session.query(model.User)
     q = q.filter(model.user_table.c.sysadmin == True)
+    q = q.filter(model.user_table.c.state == 'active')
     sysadmins = []
     for user in q.all():
         sysadmins.append(user)