From 27d57647e96329a326b484249dbe5abbf4b4df2f Mon Sep 17 00:00:00 2001 From: Leonardo Rochael Almeida <leonardo@nexedi.com> Date: Thu, 7 Jul 2011 17:47:17 +0200 Subject: [PATCH] Remove unnecessary comment simulation_movement.setPropertyList( 'variation_category', movement.getProperty('variation_category_list')) Does work after all. Note the absence of the _list suffix on the setting property. --- product/ERP5/Document/MappedProperty.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/product/ERP5/Document/MappedProperty.py b/product/ERP5/Document/MappedProperty.py index f1483e61a1..b9d9716250 100644 --- a/product/ERP5/Document/MappedProperty.py +++ b/product/ERP5/Document/MappedProperty.py @@ -100,10 +100,6 @@ class MappedProperty(XMLObject): security.declarePublic('setMappedProperty') def setMappedProperty(self, document, property, value): if property.endswith('_list'): - # XXX-Leo: This won't work for 'variation_category_list' - # calling document.setPropertyList('variation_category_list', value) - # breaks with: - # TypeError: A mono valued property must be set with a list of len 1 property = property[:-5] setProperty = document.setPropertyList else: -- 2.30.9