Account profit loss report periods filter not working

Bug #932710 reported by David Lefever @ Taktik
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Undecided
OpenERP Publisher's Warranty Team

Bug Description

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)

but data['form']['periods'] is an empty list.
Shouldn't the ctx be updated like so :

            ctx['period_from']= data['form'].get('period_from', False)
            ctx['period_to']= data['form'].get('period_to', False)

Thank you very much,
David.

Tags: maintenance

Related branches

Revision history for this message
David Lefever @ Taktik (dl-taktik) wrote :

It seems the bug is also present in the Balance Sheet report.
In the file account_balance_sheet.py, line 101 :
            ctx['periods'] = data['form'].get('periods', False)

Thank you,
David.

Changed in openobject-addons:
assignee: nobody → OpenERP Publisher's Warranty Team (openerp-opw)
tags: added: maintenance
Changed in openobject-addons:
status: New → Confirmed
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Amit Dodiya (OpenERP) (ado-openerp) wrote :

Hello David,

Thanks for the contribution.

This issue is fixed in following branch :
Branch : lp:~openerp-dev/openobject-addons/6.0-opw-383665-ado
Revision-id : <email address hidden>
Revision-no : 5059

Thanks,
Amit Dodiya

Changed in openobject-addons:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.