[Zope-Checkins] CVS: Zope/lib/python/OFS - PropertyManager.py:1.44 PropertySheets.py:1.83

[Casey Duncan]

Update of /cvs-repository/Zope/lib/python/OFS
In directory cvs.zope.org:/tmp/cvs-serv10643/lib/python/OFS

Modified Files:
	PropertyManager.py PropertySheets.py 
Log Message:
Fixed PropertyManager/PropertySheets so that you can add a property named
'ids' without the sky falling.


=== Zope/lib/python/OFS/PropertyManager.py 1.43 => 1.44 ===
     def manage_delProperties(self, ids=None, REQUEST=None):
         """Delete one or more properties specified by 'ids'."""
+        if REQUEST:
+            # Bugfix for property named "ids" (Casey)
+            if ids == self.getProperty('ids', None): ids = None
+            ids = REQUEST.get('_ids', ids)
         if ids is None:
             return MessageDialog(
                    title='No property specified',


=== Zope/lib/python/OFS/PropertySheets.py 1.82 => 1.83 ===
     def manage_delProperties(self, ids=None, REQUEST=None):
         """Delete one or more properties specified by 'ids'."""
+        if REQUEST:
+            # Bugfix for properties named "ids" (casey)
+            if ids == self.getProperty('ids', None): ids = None
+            ids = REQUEST.get('_ids', ids)
         if ids is None:
             return MessageDialog(
                    title='No property specified',