Merge lp:~ovnicraft/web-addons/6.1-web_report_name into lp:~webaddons-core-editors/web-addons/6.1
Proposed by
Cristian Salamea
Status: | Work in progress |
---|---|
Proposed branch: | lp:~ovnicraft/web-addons/6.1-web_report_name |
Merge into: | lp:~webaddons-core-editors/web-addons/6.1 |
Diff against target: |
173 lines (+150/-0) 4 files modified
web_report_name/__init__.py (+20/-0) web_report_name/__openerp__.py (+41/-0) web_report_name/controllers.py (+61/-0) web_report_name/static/src/js/views.js (+28/-0) |
To merge this branch: | bzr merge lp:~ovnicraft/web-addons/6.1-web_report_name |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Holger Brunn (Therp) | Needs Information | ||
Web-Addons Core Editors | Pending | ||
Review via email: mp+169662@code.launchpad.net |
Description of the change
Add module to change report file name, now system gives report.service.name i consider not user friendly and redefine index method to return Action Human-readable Name.pdf defined in report.
So i want help to test.
Regards,
To post a comment you must log in.
Unmerged revisions
- 15. By Cristian Salamea
-
[FIX] new cp_path report and filename function
- 14. By Cristian Salamea
-
[IMP] function to define filename
- 13. By Cristian Salamea
-
[IMP] module to add action name to file report instead report.service.name
index is mainly a copy of Reports' index. Is there a reason why you don't call super(...).index here and apply your changes in its return value?
Further: Why don't you hijack the original's _cp_path, as this would make the js-part unnecessary?