[Collective-checkins] r41370 - in PlonePAS/trunk: browser plugins

[wichert <svn...@pl...>]

Author: wichert
Date: Mon Apr 30 22:43:35 2007
New Revision: 41370

Modified:
   PlonePAS/trunk/browser/search.py
   PlonePAS/trunk/plugins/role.py
Log:
Fix return value

Modified: PlonePAS/trunk/browser/search.py
==============================================================================
--- PlonePAS/trunk/browser/search.py	(original)
+++ PlonePAS/trunk/browser/search.py	Mon Apr 30 22:43:35 2007
@@ -31,10 +31,10 @@
     def sort(self, results, key):
         self.pas=getToolByName(aq_inner(self.context), "acl_users")
         
-        def compare(a,b):
-            return cmp(a.get(key, "").lower(), b.get(key, "").lower())
+        def key(a):
+            return a.get(key, "").lower()
 
-        results.sort(cmp=compare)
+        results.sort(key=key)
 
         return results
 

Modified: PlonePAS/trunk/plugins/role.py
==============================================================================
--- PlonePAS/trunk/plugins/role.py	(original)
+++ PlonePAS/trunk/plugins/role.py	Mon Apr 30 22:43:35 2007
@@ -122,17 +122,17 @@
 
     def listRoleIds(self):
         self.updateRolesList()
-        ZODBRoleManager.listRoleIds(self)
+        return ZODBRoleManager.listRoleIds(self)
 
     def getRoleInfo(self, role_id):
         if role_id not in self._roles:
             self.updateRolesList()
-        ZODBRoleManager.getRoleInfo(self, role_id)
+        return ZODBRoleManager.getRoleInfo(self, role_id)
 
     def getRoleInfo(self, role_id):
         if role_id not in self._roles:
             self.updateRolesList()
-        ZODBRoleManager.getRoleInfo(self, role_id)
+        return ZODBRoleManager.getRoleInfo(self, role_id)
 
 
 classImplements(GroupAwareRoleManager,