Merge lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls into lp:~report-print-send-core-editors/report-print-send/7.0

Proposed by Lionel Sausin - Initiatives/Numérigraphe
Status: Merged
Approved by: Yannick Vaucher @ Camptocamp
Approved revision: 16
Merged at revision: 16
Proposed branch: lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls
Merge into: lp:~report-print-send-core-editors/report-print-send/7.0
Diff against target: 72 lines (+31/-11)
1 file modified
base_report_to_printer/printing_view.xml (+31/-11)
To merge this branch: bzr merge lp:~numerigraphe-team/report-print-send/7.0-improve-prefs-ui-ls
Reviewer Review Type Date Requested Status
Yannick Vaucher @ Camptocamp code review, no test Approve
Pedro Manuel Baeza code review Approve
Loïc Bellier - Numérigraphe Pending
Review via email: mp+219787@code.launchpad.net

Description of the change

Improve the preference views of base_report_to_printer:
Menus:
- move the printing settings to a menu of their own in the "settings" page (ie. move one level up)
- add a shortcut to the list of reports to the printing menu
Printing preferences:
- fix the preference view (the group was inserted inside another group, now it's cleanly inserted at the end)
- add the printing preferences to the user prefs ("preferences" in the menu on the far upper-right corner)

To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM.

Regards.

review: Approve (code review)
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

Thanks for the changes

LGTM

review: Approve (code review, no test)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'base_report_to_printer/printing_view.xml'
2--- base_report_to_printer/printing_view.xml 2013-10-01 13:04:19 +0000
3+++ base_report_to_printer/printing_view.xml 2014-05-16 07:25:13 +0000
4@@ -2,7 +2,7 @@
5 <openerp>
6 <data>
7
8- <menuitem name="Printing" id="menu_printing_main" parent="base.menu_config" groups="res_groups_printingprintoperator0"/>
9+ <menuitem name="Printing" id="menu_printing_main" parent="base.menu_administration" groups="res_groups_printingprintoperator0"/>
10
11 <record model="ir.ui.view" id="view_printing_printer_form">
12 <field name="name">printing.printer.form</field>
13@@ -76,12 +76,27 @@
14 <field name="model">res.users</field>
15 <field name="inherit_id" ref="base.view_users_form" />
16 <field name="arch" type="xml">
17- <field name="lang" position="after">
18- <group>
19- <field name="printing_action"/>
20- <field name="printing_printer_id"/>
21- </group>
22- </field>
23+ <page string="Preferences">
24+ <group name="printing">
25+ <field name="printing_action"/>
26+ <field name="printing_printer_id"/>
27+ </group>
28+ </page>
29+ </field>
30+ </record>
31+
32+ <!-- "My Preferences -->
33+ <record model="ir.ui.view" id="view_printing_users_prefs">
34+ <field name="name">res.users.form.printing</field>
35+ <field name="model">res.users</field>
36+ <field name="inherit_id" ref="base.view_users_form_simple_modif" />
37+ <field name="arch" type="xml">
38+ <footer position="before">
39+ <group string="Printing" name="printing">
40+ <field name="printing_action"/>
41+ <field name="printing_printer_id"/>
42+ </group>
43+ </footer>
44 </field>
45 </record>
46
47@@ -91,10 +106,11 @@
48 <field name="model">printing.report.xml.action</field>
49 <field name="arch" type="xml">
50 <form string="Report Printing Actions">
51- <field name="user_id"/>
52- <label string=""/>
53- <field name="action"/>
54- <field name="printer_id" select="1"/>
55+ <group col="2">
56+ <field name="user_id"/>
57+ <field name="action"/>
58+ <field name="printer_id" select="1"/>
59+ </group>
60 </form>
61 </field>
62 </record>
63@@ -110,5 +126,9 @@
64 </field>
65 </record>
66
67+ <!-- Add a shorcut to "Actions/Report" in the Printing menu -->
68+ <menuitem id="menu_printing_reports"
69+ parent="menu_printing_main"
70+ action="base.ir_action_report_xml"/>
71 </data>
72 </openerp>