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 ) ) _______________________________________________ Zope-Checkins maillist - Zope-Checkins@zope.org http://mail.zope.org/mailman/listinfo/zope-checkins