[Erp5-report] r34542 tatuya - /erp5/trunk/products/ERP5/Document/
nobody at svn.erp5.org
nobody at svn.erp5.org
Wed Apr 14 12:59:22 CEST 2010
Author: tatuya
Date: Wed Apr 14 12:59:09 2010
New Revision: 34542
URL: http://svn.erp5.org?rev=34542&view=rev
Log:
- Prevent to update properties because the movement group is not an assignment movement group.
- Prevent an argument error when getting a property of an object.
Modified:
erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py
Modified: erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py?rev=34542&r1=34541&r2=34542&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/ParentDeliveryPropertyMovementGroup.py [utf8] Wed Apr 14 12:59:09 2010
@@ -28,8 +28,6 @@
from Products.ERP5.Document.PropertyMovementGroup import PropertyMovementGroup
from Products.ERP5Type.Utils import UpperCase
-_MARKER = []
-
class ParentDeliveryPropertyMovementGroup(PropertyMovementGroup):
"""
Parent Delivery Property Movement Group is similar to Property
@@ -47,7 +45,7 @@
parent_delivery = self._getParentDelivery(movement)
if parent_delivery is not None:
for prop in self.getTestedPropertyList():
- property_dict['_%s' % prop] = self._getProperty(parent_delivery, prop, None)
+ property_dict['_%s' % prop] = self._getProperty(parent_delivery, prop)
return property_dict
def test(self, document, property_dict, property_list=None, **kw):
@@ -57,9 +55,9 @@
else:
target_property_list = self.getTestedPropertyList()
for prop in target_property_list:
- if property_dict['_%s' % prop] != self._getProperty(document, prop, None):
- return False, property_dict
- return True, property_dict
+ if property_dict['_%s' % prop] != self._getProperty(document, prop):
+ return False, {}
+ return True, {}
def _getParentDelivery(self, movement):
# try to find local payment conditions from the upper level delivery
@@ -72,9 +70,9 @@
delivery = movement.getDeliveryValue()
return delivery
- def _getProperty(self, document, property_id, d=_MARKER):
+ def _getProperty(self, document, property_id):
# XXX here we don't use Base.getProperty() but try to call accessor
# directly to make acquired property
# (eg. payment_condition_efficiency) working.
accessor_name = 'get' + UpperCase(property_id)
- return getattr(document, accessor_name)(d)
+ return getattr(document, accessor_name)()
More information about the Erp5-report
mailing list