Commit ff0eafe3 authored by Kazuhiko Shiozaki's avatar Kazuhiko Shiozaki

Revert "py2/py3: define OrderableList that uses legacy cmp for ordering."

This reverts commit d4ab7bb5.
parent dbfcb4be
...@@ -144,26 +144,6 @@ else: ...@@ -144,26 +144,6 @@ else:
# Generic sort method # Generic sort method
##################################################### #####################################################
class OrderableList(object):
def __init__(self, *args):
self.__slots__ = args
def __lt__(self, other):
for i, v in enumerate(self.__slots__):
try:
other_v = other.__slots__[i]
except IndexError:
return False
c = cmp(v, other_v)
if c == -1:
return True
elif c == 1:
return False
return True
def __repr__(self):
return repr(self.__slots__)
sort_kw_cache = {} sort_kw_cache = {}
def sortValueList(value_list, sort_on=None, sort_order=None, **kw): def sortValueList(value_list, sort_on=None, sort_order=None, **kw):
...@@ -221,10 +201,7 @@ def sortValueList(value_list, sort_on=None, sort_order=None, **kw): ...@@ -221,10 +201,7 @@ def sortValueList(value_list, sort_on=None, sort_order=None, **kw):
except TypeError: except TypeError:
pass pass
value_list.append(x) value_list.append(x)
if six.PY2:
return value_list return value_list
else:
return OrderableList(*value_list)
sort_kw = {'key':sortValue, 'reverse':reverse} sort_kw = {'key':sortValue, 'reverse':reverse}
sort_kw_cache[(sort_on, sort_order)] = sort_kw sort_kw_cache[(sort_on, sort_order)] = sort_kw
else: else:
......
...@@ -38,7 +38,7 @@ from Acquisition import aq_base ...@@ -38,7 +38,7 @@ from Acquisition import aq_base
from Products.ERP5Type.tests.ERP5TypeTestCase import ERP5TypeTestCase from Products.ERP5Type.tests.ERP5TypeTestCase import ERP5TypeTestCase
from AccessControl.ZopeGuards import guarded_import from AccessControl.ZopeGuards import guarded_import
from Products.ERP5Type.tests.utils import LogInterceptor, createZODBPythonScript from Products.ERP5Type.tests.utils import LogInterceptor, createZODBPythonScript
from Products.ERP5Type.Utils import OrderableList, cmp from Products.ERP5Type.Utils import cmp
class TestERP5Type(ERP5TypeTestCase, LogInterceptor): class TestERP5Type(ERP5TypeTestCase, LogInterceptor):
""" """
...@@ -296,16 +296,6 @@ class TestERP5Type(ERP5TypeTestCase, LogInterceptor): ...@@ -296,16 +296,6 @@ class TestERP5Type(ERP5TypeTestCase, LogInterceptor):
self.assertEqual(cmp(None, ''), -1) self.assertEqual(cmp(None, ''), -1)
self.assertEqual(cmp(0, ''), -1) self.assertEqual(cmp(0, ''), -1)
def test_OrderableList(self):
o1 = OrderableList(None, 1)
o2 = OrderableList(0, 0)
self.assertEqual(sorted([o1, o2]), [o1, o2])
self.assertEqual(sorted([o2, o1]), [o1, o2])
o3 = OrderableList(1,2,3)
o4 = OrderableList(1,2,3,4)
self.assertEqual(sorted([o3, o4]), [o3, o4])
self.assertEqual(sorted([o4, o3]), [o3, o4])
def test_suite(): def test_suite():
suite = unittest.TestSuite() suite = unittest.TestSuite()
suite.addTest(unittest.makeSuite(TestERP5Type)) suite.addTest(unittest.makeSuite(TestERP5Type))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment