Amit Dodiya (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/6.0-opw-383665-ado into 
lp:openobject-addons/6.0.

Requested reviews:
  nel (nel-tinyerp)
  Naresh(OpenERP) (nch-openerp)
Related bugs:
  Bug #932710 in OpenERP Addons: "Account profit loss report periods filter not 
working"
  https://bugs.launchpad.net/openobject-addons/+bug/932710

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-383665-ado/+merge/93361

Hello,

The period filter in the profit and loss report doesn't seem to be working.

If you look in account/report/account_profit_loss.py (around line 101) :
        if data['form']['filter'] == 'filter_period':
            ctx['periods'] = data['form'].get('periods', False)
Periods will always gives you blank list.

ctx should be updated like:
        ctx['period_from']= data['form'].get('period_from', False)
        ctx['period_to']= data['form'].get('period_to', False)


Thanks,
Amit
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-383665-ado/+merge/93361
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-383665-ado.
=== modified file 'account/report/account_balance_sheet.py'
--- account/report/account_balance_sheet.py	2011-06-29 05:29:28 +0000
+++ account/report/account_balance_sheet.py	2012-02-16 09:43:19 +0000
@@ -102,7 +102,8 @@
         ctx['fiscalyear'] = data['form'].get('fiscalyear_id', False)
 
         if data['form']['filter'] == 'filter_period':
-            ctx['periods'] = data['form'].get('periods', False)
+            ctx['period_from']= data['form'].get('period_from', False)
+            ctx['period_to']= data['form'].get('period_to', False)
         elif data['form']['filter'] == 'filter_date':
             ctx['date_from'] = data['form'].get('date_from', False)
             ctx['date_to'] =  data['form'].get('date_to', False)

=== modified file 'account/report/account_profit_loss.py'
--- account/report/account_profit_loss.py	2011-08-04 13:15:39 +0000
+++ account/report/account_profit_loss.py	2012-02-16 09:43:19 +0000
@@ -99,7 +99,8 @@
         ctx['fiscalyear'] = data['form'].get('fiscalyear_id', False)
 
         if data['form']['filter'] == 'filter_period':
-            ctx['periods'] =  data['form'].get('periods', False)
+            ctx['period_from']= data['form'].get('period_from', False)
+            ctx['period_to']= data['form'].get('period_to', False)
         elif data['form']['filter'] == 'filter_date':
             ctx['date_from'] = data['form'].get('date_from', False)
             ctx['date_to'] =  data['form'].get('date_to', False)

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to