[Erp5-report] r17083 - in /erp5/trunk/products/ERP5: Tool/ bootstrap/erp5_core/SkinTemplate...

[nobody at svn.erp5.org]

Author: jerome
Date: Mon Oct 22 11:42:51 2007
New Revision: 17083

URL: http://svn.erp5.org?rev=17083&view=rev
Log:
Default group_by parameters for getMovementHistoryList should be handled in the tool, not in the ZSQL Method.

Modified:
    erp5/trunk/products/ERP5/Tool/SimulationTool.py
    erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Resource_zGetMovementHistoryList.xml
    erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision

Modified: erp5/trunk/products/ERP5/Tool/SimulationTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Tool/SimulationTool.py?rev=17083&r1=17082&r2=17083&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Tool/SimulationTool.py (original)
+++ erp5/trunk/products/ERP5/Tool/SimulationTool.py Mon Oct 22 11:42:51 2007
@@ -832,6 +832,7 @@
         group_by_variation=0, group_by_sub_variation=0,
         group_by_movement=0, group_by_date=0,
         group_by_resource=None,
+        movement_list_mode=0,
         **ignored):
       """
       Set defaults group_by parameters
@@ -840,7 +841,10 @@
 
       If any group-by is provided, automatically group by resource aswell
       unless group_by_resource is explicitely set to false.
-      If no group by is provided, group by movement, node and resource.
+      If no group by is provided, use the default group by: movement, node and
+      resource, unless movement_list_mode is true, in that case, group by
+      movement, node, resource and date (this is historically the default in
+      getMovementHistoryList).
       """
       new_group_by_dict = {}
       if not ignore_group_by:
@@ -855,6 +859,8 @@
           new_group_by_dict['group_by_movement'] = 1
           new_group_by_dict['group_by_node'] = 1
           new_group_by_dict['group_by_resource'] = 1
+          if movement_list_mode:
+            new_group_by_dict['group_by_date'] = 1
       return new_group_by_dict
 
     security.declareProtected(Permissions.AccessContentsInformation,
@@ -1398,6 +1404,8 @@
       brains. The initial values can be passed, in case you want to have an
       "initial summary line".
       """
+      kw['movement_list_mode'] = 1
+      kw.update(self._getDefaultGroupByParameters(**kw))
       sql_kw = self._generateSQLKeywordDict(**kw)
       return self.Resource_zGetMovementHistoryList(
                          src__=src__, ignore_variation=ignore_variation,

Modified: erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Resource_zGetMovementHistoryList.xml
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Resource_zGetMovementHistoryList.xml?rev=17083&r1=17082&r2=17083&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Resource_zGetMovementHistoryList.xml (original)
+++ erp5/trunk/products/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/Resource_zGetMovementHistoryList.xml Mon Oct 22 11:42:51 2007
@@ -762,12 +762,10 @@
   AND <dtml-var "portal_selections.buildSQLExpressionFromDomainSelection(selection_report, strict_membership=1)">\n
 </dtml-if>\n
 \n
+<dtml-if group_by_expression>\n
 GROUP BY\n
-  <dtml-if group_by_expression>\n
     <dtml-var group_by_expression>\n
-  <dtml-else>\n
-    stock.uid, stock.node_uid, stock.resource_uid, stock.date\n
-  </dtml-if>\n
+</dtml-if>\n
 \n
 <dtml-if order_by_expression>\n
 ORDER BY\n
@@ -920,12 +918,10 @@
   AND <dtml-var "portal_selections.buildSQLExpressionFromDomainSelection(selection_report, strict_membership=1)">\n
 </dtml-if>\n
 \n
+<dtml-if group_by_expression>\n
 GROUP BY\n
-  <dtml-if group_by_expression>\n
     <dtml-var group_by_expression>\n
-  <dtml-else>\n
-    stock.uid, stock.node_uid, stock.resource_uid, stock.date\n
-  </dtml-if>\n
+</dtml-if>\n
 \n
 <dtml-if order_by_expression>\n
 ORDER BY\n

Modified: erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision?rev=17083&r1=17082&r2=17083&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision (original)
+++ erp5/trunk/products/ERP5/bootstrap/erp5_core/bt/revision Mon Oct 22 11:42:51 2007
@@ -1,1 +1,1 @@
-538
+539