Merge lp:~openerp-dev/openerp-web/trunk-account_cash_flow-jap into lp:openerp-web

Proposed by Sanjay Gohel (Open ERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/trunk-account_cash_flow-jap
Merge into: lp:openerp-web
Diff against target: 11 lines (+1/-1)
1 file modified
addons/web/controllers/main.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/trunk-account_cash_flow-jap
Reviewer Review Type Date Requested Status
Thibault Delavallée (OpenERP) (community) Needs Information
OpenERP R&D Web Team Pending
Review via email: mp+183142@code.launchpad.net

Description of the change

Hello,

  financial report can not be open due to some changes fixes that.

Thanks
Sanjay

To post a comment you must log in.
Revision history for this message
Thibault Delavallée (OpenERP) (tde-openerp) wrote :

Could you please give a way to reproduce the bug (or, better, when you're at it, fill a bug report) ?

Thibault.

review: Needs Information
Revision history for this message
Sanjay Gohel (Open ERP) (sgo-openerp) wrote :

Hello,

     Try in latest trunk runbot go into Accounting > Reporting > Legal Reporting open any of report wizard and click on print it gives traceback. This patch solves that issue and print all reports.

Sanjay.

3820. By Sanjay Gohel (Open ERP)

[MERGE]with trunk

3821. By Sanjay Gohel (Open ERP)

[MERGE]with trunk

3822. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

3823. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

3824. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

Unmerged revisions

3824. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

3823. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

3822. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

3821. By Sanjay Gohel (Open ERP)

[MERGE]with trunk

3820. By Sanjay Gohel (Open ERP)

[MERGE]with trunk

3819. By Sanjay Gohel (Open ERP)

[MERGE]sync with trunk

3818. By Sanjay Gohel (Open ERP)

[IMP]when click on all generic report print button it gives traceback this solves that issues for which we can not test and see all financial reports

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/controllers/main.py'
2--- addons/web/controllers/main.py 2013-10-30 18:33:14 +0000
3+++ addons/web/controllers/main.py 2013-11-01 06:33:12 +0000
4@@ -1684,7 +1684,7 @@
5 if 'report_type' in action:
6 report_data['report_type'] = action['report_type']
7 if 'datas' in action:
8- if 'ids' in action['datas']:
9+ if action['datas'].get('ids'):
10 report_ids = action['datas'].pop('ids')
11 report_data.update(action['datas'])
12