[6.1][7.0][trunk] Default report filename is useless in daily use
Bug #1261322 reported by
Yann Papouin
on 2013-12-16
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | ||
|---|---|---|---|---|---|---|
| | Odoo Web (MOVED TO GITHUB) |
New
|
Undecided
|
Unassigned | ||
| OpenERP Community Backports (Web) | Status tracked in 7.0 | |||||
| | 6.1 |
Medium
|
Yann Papouin | |||
| | 7.0 |
Medium
|
Ronald Portier (Therp) | |||
Bug Description
This bug report is related to a question asked [1] on stackoverflow.
The main idea is to allow the user to download its report with a reliable filename:
-"PO00006.pdf" instead of "Request For Quotation.pdf" for a single object report
-"PO00006-
My proposal is to use the object field name if exists and else to use the current behavior to create the report filename.
(Note that the browser will trim invalid filename data like slash, backslash, etc.)
[1] http://
Related branches
lp:~yann-papouin/ocb-web/6.1-bug-1261322-useful-report-filename
- Pedro Manuel Baeza: Approve (code review) on 2014-05-19
- Ronald Portier (Therp): Approve (code review and partial testing) on 2014-05-16
- Holger Brunn (Therp): Approve (code review) on 2014-01-20
- Yann Papouin: Needs Information on 2014-01-08
-
Diff: 37 lines (+19/-1)1 file modifiedaddons/web/controllers/main.py (+19/-1)
lp:~therp-nl/ocb-web/7.0_lp1261332_better_report_name
- Yannick Vaucher @ Camptocamp: Approve on 2014-05-23
- Lara (Therp) (community): Approve (test) on 2014-05-22
- Yann Papouin: Approve on 2014-05-19
- Pedro Manuel Baeza: Approve (code review and test) on 2014-05-19
-
Diff: 37 lines (+17/-9)1 file modifiedaddons/web/controllers/main.py (+17/-9)
lp:~therp-nl/openerp-web/7.0_lp1261322_better_report_name
- OpenERP Core Team: Pending requested 2014-05-23
-
Diff: 37 lines (+17/-9)1 file modifiedaddons/web/controllers/main.py (+17/-9)
| Jean Weisbuch (m-jea0-p) wrote : | #1 |
Ronald Portier (Therp) (rportier1962)
on 2014-05-16
| tags: | added: 6.1 |
To post a comment you must log in.
I "adapted" (move the code a little bit further and remove 2 lines) the patch for OpenERP7 if it might interest anyone.