[Zope-Checkins] SVN: Zope/trunk/ When you add roles in manage_access, roles are now stripped of any leading or trailing

[Lennart Regebro]

Log message for revision 67839:
  When you add roles in manage_access, roles are now stripped of any leading or trailing 
  spaces.
  
  

Changed:
  U   Zope/trunk/doc/CHANGES.txt
  U   Zope/trunk/lib/python/AccessControl/Role.py

-=-
Modified: Zope/trunk/doc/CHANGES.txt
===================================================================
--- Zope/trunk/doc/CHANGES.txt	2006-05-02 14:09:09 UTC (rev 67838)
+++ Zope/trunk/doc/CHANGES.txt	2006-05-02 14:22:06 UTC (rev 67839)
@@ -240,6 +240,8 @@
         from the Zope 3 source tree (to get rid of redundant packages)
 
     Bugs Fixed
+      - When you add roles in manage_access, roles are now stripped of
+        any leading or trailing spaces.
 
       - Collector #2062: Fix manage_historyCopy, which was broken, and write
         tests for it.

Modified: Zope/trunk/lib/python/AccessControl/Role.py
===================================================================
--- Zope/trunk/lib/python/AccessControl/Role.py	2006-05-02 14:09:09 UTC (rev 67838)
+++ Zope/trunk/lib/python/AccessControl/Role.py	2006-05-02 14:22:06 UTC (rev 67839)
@@ -473,7 +473,7 @@
         """
 
         if submit=='Add Role':
-            role=reqattr(REQUEST, 'role')
+            role=reqattr(REQUEST, 'role').strip()
             return self._addRole(role, REQUEST)
 
         if submit=='Delete Role':