[Erp5-report] r26180 - /erp5/trunk/products/ERP5/Document/OrderBuilder.py

[nobody at svn.erp5.org]

Author: luke
Date: Fri Mar 27 12:00:32 2009
New Revision: 26180

URL: http://svn.erp5.org?rev=26180&view=rev
Log:
 - do not allow to use more than one key in property dict in set of movement groups

Modified:
    erp5/trunk/products/ERP5/Document/OrderBuilder.py

Modified: erp5/trunk/products/ERP5/Document/OrderBuilder.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/OrderBuilder.py?rev=26180&r1=26179&r2=26180&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/OrderBuilder.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/OrderBuilder.py [utf8] Fri Mar 27 12:00:32 2009
@@ -40,6 +40,7 @@
 
 class CollectError(Exception): pass
 class MatrixError(Exception): pass
+class DuplicatedPropertyDictKeysError(Exception): pass
 
 class OrderBuilder(XMLObject, Amount, Predicate):
   """
@@ -259,7 +260,11 @@
     property_dict = {}
     if not len(instance_list):
       for movement_group_node in movement_group_node_list:
-        property_dict.update(movement_group_node.getGroupEditDict())
+        for k,v in movement_group_node.getGroupEditDict().iteritems():
+          if k in property_dict:
+            raise DuplicatedPropertyDictKeysError(k)
+          else:
+            property_dict[k] = v
     else:
       # we want to check the original delivery first.
       # so sort instance_list by that current is exists or not.