[Zope-Checkins] SVN: Zope/branches/Zope-2_8-branch/ reverted r65902:

[Yvo Schubbe]

Log message for revision 66022:
  reverted r65902:
  - I still believe this was the Right Thing to do, but not everybody agrees

Changed:
  U   Zope/branches/Zope-2_8-branch/doc/CHANGES.txt
  U   Zope/branches/Zope-2_8-branch/lib/python/OFS/ObjectManager.py
  U   Zope/branches/Zope-2_8-branch/lib/python/OFS/tests/testObjectManager.py

-=-
Modified: Zope/branches/Zope-2_8-branch/doc/CHANGES.txt
===================================================================
--- Zope/branches/Zope-2_8-branch/doc/CHANGES.txt	2006-03-15 08:12:10 UTC (rev 66021)
+++ Zope/branches/Zope-2_8-branch/doc/CHANGES.txt	2006-03-15 08:16:56 UTC (rev 66022)
@@ -18,9 +18,6 @@
 
     Bugs fixed
 
-      - OFS ObjectManager: Fixed 'checkValidId'.
-        Names starting with '@' are reserved for views and not allowed in IDs.
-
       - Collector #2039: 'ZPublisher.HTTPRequest.HTTPRequest._authUserPW'
         choked on passwords which contained colons.
 

Modified: Zope/branches/Zope-2_8-branch/lib/python/OFS/ObjectManager.py
===================================================================
--- Zope/branches/Zope-2_8-branch/lib/python/OFS/ObjectManager.py	2006-03-15 08:12:10 UTC (rev 66021)
+++ Zope/branches/Zope-2_8-branch/lib/python/OFS/ObjectManager.py	2006-03-15 08:16:56 UTC (rev 66022)
@@ -72,9 +72,6 @@
         'The id "%s" is invalid because it begins with "aq_".' % id)
     if id.endswith('__'): raise BadRequest, (
         'The id "%s" is invalid because it ends with two underscores.' % id)
-    if id[0] == '@':
-        raise BadRequest('The id "%s" is invalid because it begins with '
-                         '"@".' % id)
     if not allow_dup:
         obj = getattr(self, id, None)
         if obj is not None:

Modified: Zope/branches/Zope-2_8-branch/lib/python/OFS/tests/testObjectManager.py
===================================================================
--- Zope/branches/Zope-2_8-branch/lib/python/OFS/tests/testObjectManager.py	2006-03-15 08:12:10 UTC (rev 66021)
+++ Zope/branches/Zope-2_8-branch/lib/python/OFS/tests/testObjectManager.py	2006-03-15 08:16:56 UTC (rev 66022)
@@ -354,9 +354,7 @@
         self.assertRaises(BadRequest, om._setObject, '111', si)
         self.assertRaises(BadRequest, om._setObject, 'REQUEST', si)
         self.assertRaises(BadRequest, om._setObject, '/', si)
-        self.assertRaises(BadRequest, om._setObject, '@@view', si)
 
-
 def test_suite():
     suite = unittest.TestSuite()
     suite.addTest( unittest.makeSuite( ObjectManagerTests ) )