[Erp5-report] r22887 - /erp5/trunk/products/ERP5/Document/Order.py

nobody at svn.erp5.org nobody at svn.erp5.org
Wed Aug 6 11:03:29 CEST 2008


Author: jerome
Date: Wed Aug  6 11:03:26 2008
New Revision: 22887

URL: http://svn.erp5.org?rev=22887&view=rev
Log:
When we delete an order, the related applied rules are deleted in activities
using SQLDict, so only the first one is deleted, also it can create problems
if we delete the applied rule before the activity is executed (which usually
happens in tests teardown); Anyway users never delete orders directly, so
there's no need to use activities here. Delete the related applied rules
synchronously instead.

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

Modified: erp5/trunk/products/ERP5/Document/Order.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/Order.py?rev=22887&r1=22886&r2=22887&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/Order.py (original)
+++ erp5/trunk/products/ERP5/Document/Order.py Wed Aug  6 11:03:26 2008
@@ -131,7 +131,7 @@
           Delete related Applied Rule
       """
       for o in self.getCausalityRelatedValueList(portal_type='Applied Rule'):
-        o.getParentValue().activate().deleteContent(o.getId())
+        o.getParentValue().deleteContent(o.getId())
       Delivery.manage_beforeDelete(self, item, container)
 
     ##########################################################################




More information about the Erp5-report mailing list