[Erp5-report] r32706 kazuhiko - /erp5/trunk/products/ERP5/Document/DeliveryBuilder.py

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Thu Feb 18 02:39:54 2010
New Revision: 32706

URL: http://svn.erp5.org?rev=32706&view=rev
Log:
use the getter method instead of accessing the attribute directly.

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

Modified: erp5/trunk/products/ERP5/Document/DeliveryBuilder.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Document/DeliveryBuilder.py?rev=32706&r1=32705&r2=32706&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Document/DeliveryBuilder.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Document/DeliveryBuilder.py [utf8] Thu Feb 18 02:39:54 2010
@@ -117,10 +117,10 @@
     # Add resource query
     if self.getResourcePortalType() not in ('', None):
       kw['resourceType'] = self.getResourcePortalType()
-    if self.simulation_select_method_id in ['', None]:
+    if self.getSimulationSelectMethodId() in ['', None]:
       movement_list = [x.getObject() for x in self.portal_catalog(**kw)]
     else:
-      select_method = getattr(self.getPortalObject(), self.simulation_select_method_id)
+      select_method = getattr(self.getPortalObject(), self.getSimulationSelectMethodId())
       movement_list = select_method(**kw)
     # XXX Use buildSQLQuery will be better
     movement_list = [x for x in movement_list if \
@@ -134,7 +134,7 @@
       if mvt_dict.has_key(movement):
         raise SelectMethodError, \
               "%s return %s twice (or more)" % \
-              (str(self.simulation_select_method_id),
+              (str(self.getSimulationSelectMethodId()),
                str(movement.getRelativeUrl()))
       else:
         mvt_dict[movement] = 1