Replace `__cmp__` with Python 3 compatible ordering methods.

Replace `cmp` with Python 3 compatible ordering methods.
3c669886 · Hanno Schlichting · ecc18276 · 3c669886 · 3c669886 · 3c669886
Commit 3c669886 authored Sep 12, 2016 by Hanno Schlichting
7 changed files
--- a/src/App/Extensions.py
+++ b/src/App/Extensions.py
@@ -14,6 +14,7 @@
 Extensions currently include external methods.
 """
+from functools import total_ordering
 import imp
 import os
@@ -23,20 +24,24 @@ import Products
 from zExceptions import NotFound
-class FuncCode:
+@total_ordering
+class FuncCode(object):
    def __init__(self, f, im=0):
        self.co_varnames = f.func_code.co_varnames[im:]
        self.co_argcount = f.func_code.co_argcount - im
-    def __cmp__(self, other):
+    def __eq__(self, other):
-        if other is None:
+        if not isinstance(other, FuncCode):
-            return 1
+            return False
-        try:
+        return ((self.co_argcount, self.co_varnames) ==
-            return cmp((self.co_argcount, self.co_varnames),
+                (other.co_argcount, other.co_varnames))
-                       (other.co_argcount, other.co_varnames))
-        except Exception:
+    def __lt__(self, other):
-            return 1
+        if not isinstance(other, FuncCode):
+            return False
+        return ((self.co_argcount, self.co_varnames) <
+                (other.co_argcount, other.co_varnames))
 def _getPath(home, prefix, name, suffixes):

--- a/src/Products/Five/browser/tests/test_menu.py
+++ b/src/Products/Five/browser/tests/test_menu.py
@@ -47,7 +47,8 @@ def test_menu():
    Sort menu items by title so we get a stable testable result:
-      >>> menu.sort(lambda x, y: cmp(x['title'], y['title']))
+      >>> from operator import itemgetter
+      >>> menu.sort(key=itemgetter('title'))
      >>> from pprint import pprint
      >>> pprint(menu[0])
      {'action': '@@cockatiel_menu_public.html',
@@ -99,7 +100,7 @@ def test_menu():
      >>> len(menu)
      7
-      >>> menu.sort(lambda x, y: cmp(x['title'], y['title']))
+      >>> menu.sort(key=itemgetter('title'))
      >>> pprint(menu[0])
      {'action': '@@cockatiel_menu_protected.html',
       'description': u'',

--- a/src/Products/Five/viewlet/README.txt
+++ b/src/Products/Five/viewlet/README.txt
@@ -754,6 +754,8 @@ two features are not part of the view logic, they should be treated with
 independent components. In this example, we are going to only implement
 sorting using a simple utility:
+  >>> from operator import attrgetter
  >>> class ISorter(zope.interface.Interface):
  ...
  ...     def sort(values):
@@ -763,7 +765,7 @@ sorting using a simple utility:
  ...     zope.interface.implements(ISorter)
  ...
  ...     def sort(self, values):
-  ...         return sorted(values, lambda x, y: cmp(x.__name__, y.__name__))
+  ...         return sorted(values, key=attrgetter('__name__'))
  >>> zope.component.provideUtility(SortByName(), name='name')
@@ -771,10 +773,9 @@ sorting using a simple utility:
  ...     zope.interface.implements(ISorter)
  ...
  ...     def sort(self, values):
-  ...         return sorted(
+  ...         def _key(value):
-  ...             values,
+  ...             return size.interfaces.ISized(value).sizeForSorting()
-  ...             lambda x, y: cmp(size.interfaces.ISized(x).sizeForSorting(),
+  ...         return sorted(values, key=_key)
-  ...                              size.interfaces.ISized(y).sizeForSorting()))
  >>> zope.component.provideUtility(SortBySize(), name='size')

--- a/src/Products/PageTemplates/tests/rr.pt
+++ b/src/Products/PageTemplates/tests/rr.pt
 <html>
  <body>
    <ul tal:define="refs options/refs;
-                    refs python:sorted(refs, lambda x,y: cmp(x.order, y.order))">
+                    refs python:sorted(refs, key=lambda x: x.order)">
      <li tal:repeat="ref refs" tal:content="ref" />
    </ul>
  </body>

--- a/src/Shared/DC/Scripts/Signature.py
+++ b/src/Shared/DC/Scripts/Signature.py
@@ -15,26 +15,31 @@
 This provides support for simulating function signatures
 """
+from functools import total_ordering
-class FuncCode:
+@total_ordering
+class FuncCode(object):
    def __init__(self, varnames, argcount):
        self.co_varnames = varnames
        self.co_argcount = argcount
-    def __cmp__(self, other):
+    def __eq__(self, other):
-        if other is None:
+        if not isinstance(other, FuncCode):
-            return 1
+            return False
-        try:
+        return ((self.co_argcount, self.co_varnames) ==
-            return cmp((self.co_argcount, self.co_varnames),
+                (other.co_argcount, other.co_varnames))
-                       (other.co_argcount, other.co_varnames))
-        except Exception:
-            return 1
-# This is meant to be imported directly into a class.
+    def __lt__(self, other):
+        if not isinstance(other, FuncCode):
+            return False
+        return ((self.co_argcount, self.co_varnames) <
+                (other.co_argcount, other.co_varnames))
 def _setFuncSignature(self, defaults=None, varnames=(), argcount=-1):
+    # This is meant to be imported directly into a class.
    # Simplify calls.
    if argcount < 0 and varnames:
        argcount = len(varnames)

--- a/src/ZPublisher/HTTPRequest.py
+++ b/src/ZPublisher/HTTPRequest.py
@@ -1743,7 +1743,7 @@ def parse_cookie(text,
    return parse_cookie(text[l:], result)
-class record:
+class record(object):
    # Allow access to record methods and values from DTML
    __allow_access_to_unprotected_subobjects__ = 1
@@ -1766,21 +1766,19 @@ class record:
        return self.__dict__[key]
    def __str__(self):
-        L1 = self.__dict__.items()
+        return ", ".join("%s: %s" % item for item in
-        L1.sort()
+                         sorted(self.__dict__.items()))
-        return ", ".join("%s: %s" % item for item in L1)
    def __repr__(self):
        # return repr( self.__dict__ )
-        L1 = self.__dict__.items()
-        L1.sort()
        return '{%s}' % ', '.join(
-            "'%s': %s" % (item[0], repr(item[1])) for item in L1)
+            "'%s': %s" % (item[0], repr(item[1])) for item in
+            sorted(self.__dict__.items()))
-    def __cmp__(self, other):
+    def __eq__(self, other):
-        return (cmp(type(self), type(other)) or
+        if not isinstance(other, record):
-                cmp(self.__class__, other.__class__) or
+            return False
-                cmp(self.__dict__.items(), other.__dict__.items()))
+        return self.__dict__.items() == other.__dict__.items()
 def _filterPasswordFields(items):

--- a/src/ZTUtils/Tree.py
+++ b/src/ZTUtils/Tree.py
@@ -300,6 +300,7 @@ def decodeExpansion(s, nth=None, maxsize=8192):
    nth_pair = None
    if nth is not None:
        nth_pair = (None, None)
+    obid = None
    for step in s.split(':'):
        if step.startswith('_'):
            pop = len(step) - 1