ERP5Type: set Getter guarantees the order with just removing the duplicates.

product/ERP5Type/Accessor/Acquired.py

@@ -27,6 +27,7 @@ from __future__ import absolute_import
 #
 ##############################################################################
 
+from collections import OrderedDict
 from .Base import func_code, type_definition, list_types, ATTRIBUTE_PREFIX, Getter as BaseGetter
 from Products.ERP5Type.PsycoWrapper import psyco
 
@@ -208,4 +209,4 @@ class SetGetter(ListGetter):
     Gets a category value set
     """
     def __call__(self, instance, *args, **kw):
-      return list(set(ListGetter.__call__(self, instance, *args, **kw)))
+      return list(OrderedDict.fromkeys(ListGetter.__call__(self, instance, *args, **kw)))

product/ERP5Type/Accessor/List.py

@@ -273,6 +273,6 @@ class SetGetter(ListGetter):
     def __call__(self, instance, *args, **kw):
       result_list = ListGetter.__call__(self, instance, *args, **kw)
       if result_list is not None:
-        return list(set(result_list))
+        return list(OrderedDict.fromkeys(result_list))
 
 Tester = Base.Tester

product/ERP5Type/Accessor/Related.py

@@ -27,6 +27,7 @@ from __future__ import absolute_import
 #
 ##############################################################################
 
+from collections import OrderedDict
 from .Base import func_code, type_definition, list_types, ATTRIBUTE_PREFIX, Getter as BaseGetter, Setter as BaseSetter
 from Products.ERP5Type.PsycoWrapper import psyco
 from zLOG import LOG
@@ -106,4 +107,4 @@ class SetGetter(ListGetter):
   Gets a category value set
   """
   def __call__(self, instance, *args, **kw):
-    return list(set(ListGetter.__call__(self, instance, *args, **kw)))
+    return list(OrderedDict.fromkeys(ListGetter.__call__(self, instance, *args, **kw)))

product/ERP5Type/Accessor/RelatedValue.py

@@ -27,6 +27,7 @@ from __future__ import absolute_import
 #
 ##############################################################################
 
+from collections import OrderedDict
 from .Base import func_code, type_definition, list_types, \
                  ATTRIBUTE_PREFIX, Getter as BaseGetter, Setter as BaseSetter
 from Products.ERP5Type.PsycoWrapper import psyco
@@ -112,7 +113,7 @@ class SetGetter(ListGetter):
   Gets a category value set
   """
   def __call__(self, instance, *args, **kw):
-    return list(set(ListGetter.__call__(self, instance, *args, **kw)))
+    return list(OrderedDict.fromkeys(ListGetter.__call__(self, instance, *args, **kw)))
 
 
 class DefaultPropertyGetter(BaseGetter):
@@ -192,7 +193,7 @@ class PropertySetGetter(PropertyListGetter):
   Gets a category value set
   """
   def __call__(self, instance, *args, **kw):
-    return list(set(PropertyListGetter.__call__(self, instance, *args, **kw)))
+    return list(OrderedDict.fromkeys(PropertyListGetter.__call__(self, instance, *args, **kw)))
 
 
 class DefaultIdGetter(PropertyGetter):

product/ERP5Type/Accessor/Value.py

@@ -232,7 +232,7 @@ class UidSetSetter(BaseSetter):
     def __call__(self, instance, *args, **kw):
       if self._warning:
         LOG("ERP5Type Deprecated Getter Id:",0, self._id)
-      instance._setValueUidList(self._key, set(args[0]),
+      instance._setValueUidList(self._key, list(OrderedDict.fromkeys(args[0])),
                                                  spec=kw.get('spec',()),
                                                  filter=kw.get('filter', None),
                                                  portal_type=kw.get('portal_type',()),
@@ -298,4 +298,4 @@ class PropertySetGetter(PropertyListGetter):
     """
     def __call__(self, instance, *args, **kw):
       r = PropertyListGetter.__call__(self, instance, **kw)
-      return list(set(r)) if r or not args else args[0]
+      return list(OrderedDict.fromkeys(r)) if r or not args else args[0]