Merge pull request #5681 from martenson/set-from-list

library manager - use a set instead of list
galaxyproject · Mar 13, 2018 · 7ad5dba · 7ad5dba
2 parents ccc5cfc + f808879
commit 7ad5dba
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/galaxy/managers/libraries.py b/lib/galaxy/managers/libraries.py
@@ -139,10 +139,10 @@ def list(self, trans, deleted=False):
             library_add_action = trans.app.security_agent.permitted_actions.LIBRARY_ADD.action
             library_modify_action = trans.app.security_agent.permitted_actions.LIBRARY_MODIFY.action
             library_manage_action = trans.app.security_agent.permitted_actions.LIBRARY_MANAGE.action
-            accessible_restricted_library_ids = []
-            allowed_library_add_ids = set([])
-            allowed_library_modify_ids = set([])
-            allowed_library_manage_ids = set([])
+            accessible_restricted_library_ids = set()
+            allowed_library_add_ids = set()
+            allowed_library_modify_ids = set()
+            allowed_library_manage_ids = set()
             for action in all_actions:
                 if action.action == library_access_action:
                     accessible_restricted_library_ids.append(action.library_id)