[Checkins] SVN: Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/ Don't fail on users defined in multiple user sources on the ZODBGroupManager listing page.
Hanno Schlichting
plone at hannosch.info
Tue Mar 3 10:09:53 EST 2009
Log message for revision 97440:
Don't fail on users defined in multiple user sources on the ZODBGroupManager listing page.
Changed:
U Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/doc/CHANGES.txt
U Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/plugins/ZODBGroupManager.py
-=-
Modified: Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/doc/CHANGES.txt
===================================================================
--- Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/doc/CHANGES.txt 2009-03-03 14:12:41 UTC (rev 97439)
+++ Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/doc/CHANGES.txt 2009-03-03 15:09:53 UTC (rev 97440)
@@ -4,6 +4,8 @@
PluggableAuthService 1.5.5 (unreleased)
---------------------------------------
+- Don't fail on users defined in multiple user sources on the ZODBGroupManager
+ listing page.
PluggableAuthService 1.5.4 (2008-11-20)
---------------------------------------
Modified: Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/plugins/ZODBGroupManager.py
===================================================================
--- Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/plugins/ZODBGroupManager.py 2009-03-03 14:12:41 UTC (rev 97439)
+++ Products.PluggableAuthService/branches/1.5/Products/PluggableAuthService/plugins/ZODBGroupManager.py 2009-03-03 15:09:53 UTC (rev 97440)
@@ -250,14 +250,12 @@
for k, v in self._principal_groups.items():
if group_id in v:
- # should be one and only one mapping to 'k'
-
parent = aq_parent( self )
info = parent.searchPrincipals( id=k, exact_match=True )
- assert( len( info ) in ( 0, 1 ) )
if len( info ) == 0:
title = '<%s: not found>' % k
else:
+ # always use the title of the first principal found
title = info[0].get( 'title', k )
result.append( ( k, title ) )
More information about the Checkins
mailing list