Moved subset id calculation in `OFS.OrderSupport.moveObjectsByDelta` to a new...

Moved subset id calculation in `OFS.OrderSupport.moveObjectsByDelta` to a new helper method, patch by Tom Gross. This allows CMFPlone and Archetypes to re-use the OFS implementation instead of using their own copy of the modified Zope 2.7 codebase :)

doc/CHANGES.rst

@@ -19,6 +19,9 @@ Bugs Fixed
 Features Added
 ++++++++++++++
 
+- Moved subset id calculation in `OFS.OrderSupport.moveObjectsByDelta` to a
+  new helper method, patch by Tom Gross.
+
 - Updated to Zope Toolkit 1.0.1.
 
 - Use cProfile where possible for the

src/OFS/OrderSupport.py

@@ -134,12 +134,12 @@ class OrderSupport(object):
                            suppress_events=False):
         """ Move specified sub-objects by delta.
         """
-        if type(ids) is str:
+        if isinstance(ids, basestring):
             ids = (ids,)
         min_position = 0
         objects = list(self._objects)
-        if subset_ids == None:
-            subset_ids = [ obj['id'] for obj in objects ]
+        if subset_ids is None:
+            subset_ids = self.getIdsSubset(objects)
         else:
             subset_ids = list(subset_ids)
         # unify moving direction
@@ -283,4 +283,12 @@ class OrderSupport(object):
                 r.reverse()
         return r
 
+    #
+    #   Helper methods
+    #
+
+    def getIdsSubset(self, objects):
+        return [obj['id'] for obj in objects]
+
+
 InitializeClass(OrderSupport)