slapos_erp5: Suggested by vincent (squash)

master/bt5/slapos_erp5/SkinTemplateItem/portal_skins/slapos_core/ERP5Type_asSecurityGroupIdSet.py

@@ -3,14 +3,13 @@ def handleSort(_category_dict):
   # generates the same Security ID. This workarround is required because
   # the value already arrives at this point pre-sorted on various locations
   # and it is not possible to change since it is widely used.
-  if _category_dict.keys() == ['function', 'source_project']:
+  category_list = _category_dict.keys()
+  if category_list == ['function', 'source_project']:
     return ['source_project', 'function']
-
-  if _category_dict.keys() == ['function', 'destination_project']:
+  if category_list == ['function', 'destination_project']:
     return ['destination_project', 'function']
-
   # Enforce return list in case
-  return list(_category_dict)
+  return category_list
 
 return context.portal_skins.erp5_core.ERP5Type_asSecurityGroupIdSet(
   category_dict=category_dict, key_sort=handleSort)