[Zope3-checkins] CVS: Zope3/src/zope/products/securitypolicy/browser - principalroleview.py:1.3

Steve Alexander steve at cat-box.net
Thu Jan 15 18:25:23 EST 2004


Update of /cvs-repository/Zope3/src/zope/products/securitypolicy/browser
In directory cvs.zope.org:/tmp/cvs-serv27581/src/zope/products/securitypolicy/browser

Modified Files:
	principalroleview.py 
Log Message:
Fixed Role that should have been a "Role".

The fact that this was not picked up before shows that we need a unit-
test here.


=== Zope3/src/zope/products/securitypolicy/browser/principalroleview.py 1.2 => 1.3 ===
--- Zope3/src/zope/products/securitypolicy/browser/principalroleview.py:1.2	Wed Jan 14 17:55:33 2004
+++ Zope3/src/zope/products/securitypolicy/browser/principalroleview.py	Thu Jan 15 18:25:22 2004
@@ -60,7 +60,8 @@
                 roles = self.getAllRoles()
             else:
                 # Ugh, we have ids, but we want objects
-                role_service = getService(self.context, Roles)
+                # XXX This code path needs a test
+                role_service = getService(self.context, 'Roles')
                 roles = [role_service.getRole(role)
                          for role in roles]
 




More information about the Zope3-Checkins mailing list