Fixed bug 1224: property sheet edit bug. manage_editProperties now

leaves existing properties that do not have explicit values in the
REQUEST alone rather than blindly giving them a null value.

lib/python/OFS/PropertyManager.py

@@ -84,7 +84,7 @@
 ##############################################################################
 
 """Property management"""
-__version__='$Revision: 1.22 $'[11:-2]
+__version__='$Revision: 1.23 $'[11:-2]
 
 import ExtensionClass, Globals
 import ZDOM
@@ -302,8 +302,9 @@ class PropertyManager(ExtensionClass.Base, ZDOM.ElementWithAttributes):
         """Edit object properties via the web."""
         for prop in self._properties:
             name=prop['id']
-            value=REQUEST.get(name, '')
-            self._setPropValue(name, value)
+            if REQUEST.has_key(name):
+                value=REQUEST.get(name)
+                self._setPropValue(name, value)
         return MessageDialog(
                title  ='Success!',
                message='Your changes have been saved',

lib/python/OFS/PropertySheets.py

@@ -84,7 +84,7 @@
 ##############################################################################
 
 """Property sheets"""
-__version__='$Revision: 1.45 $'[11:-2]
+__version__='$Revision: 1.46 $'[11:-2]
 
 import time, string, App.Management, Globals
 from ZPublisher.Converters import type_converters
@@ -428,8 +428,9 @@ class PropertySheet(Persistent, Implicit):
         """Edit object properties via the web."""
         for prop in self.propertyMap():
             name=prop['id']
-            value=REQUEST.get(name, '')
-            self._updateProperty(name, value)
+            if REQUEST.has_key(name):
+                value=REQUEST.get(name)
+                self._updateProperty(name, value)
         return MessageDialog(
                title  ='Success!',
                message='Your changes have been saved',