reverted r65903:

- I still believe this was the Right Thing to do, but not everybody agrees

doc/CHANGES.txt

@@ -18,9 +18,6 @@ Zope Changes
 
     Bugs fixed
 
-      - OFS ObjectManager: Fixed 'checkValidId'.
-        Names starting with '@' are reserved for views and not allowed in IDs.
-
       - Missing import of NotFound in webdav.Resource.
 
   Zope 2.9.1  (2006/02/25)

lib/python/OFS/ObjectManager.py

@@ -82,9 +82,6 @@ def checkValidId(self, id, allow_dup=0):
         '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:

lib/python/OFS/tests/testObjectManager.py

@@ -378,7 +378,6 @@ class ObjectManagerTests(PlacelessSetup, unittest.TestCase):
         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_list_imports(self):
         om = self._makeOne()
@@ -390,7 +389,6 @@ class ObjectManagerTests(PlacelessSetup, unittest.TestCase):
             self.failUnless(filename.endswith('.zexp') or
                             filename.endswith('.xml'))
 
-
 def test_suite():
     suite = unittest.TestSuite()
     suite.addTest( unittest.makeSuite( ObjectManagerTests ) )