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
=== modified file 'base_report_to_printer/printing_view.xml'
--- base_report_to_printer/printing_view.xml 2013-10-01 13:04:19 +0000
+++ base_report_to_printer/printing_view.xml 2014-05-16 07:25:13 +0000
@@ -2,7 +2,7 @@
2<openerp>2<openerp>
3 <data>3 <data>
44
5 <menuitem name="Printing" id="menu_printing_main" parent="base.menu_config" groups="res_groups_printingprintoperator0"/>5 <menuitem name="Printing" id="menu_printing_main" parent="base.menu_administration" groups="res_groups_printingprintoperator0"/>
66
7 <record model="ir.ui.view" id="view_printing_printer_form">7 <record model="ir.ui.view" id="view_printing_printer_form">
8 <field name="name">printing.printer.form</field>8 <field name="name">printing.printer.form</field>
@@ -76,12 +76,27 @@
76 <field name="model">res.users</field>76 <field name="model">res.users</field>
77 <field name="inherit_id" ref="base.view_users_form" />77 <field name="inherit_id" ref="base.view_users_form" />
78 <field name="arch" type="xml">78 <field name="arch" type="xml">
79 <field name="lang" position="after">79 <page string="Preferences">
80 <group>80 <group name="printing">
81 <field name="printing_action"/>81 <field name="printing_action"/>
82 <field name="printing_printer_id"/>82 <field name="printing_printer_id"/>
83 </group>83 </group>
84 </field>84 </page>
85 </field>
86 </record>
87
88 <!-- "My Preferences -->
89 <record model="ir.ui.view" id="view_printing_users_prefs">
90 <field name="name">res.users.form.printing</field>
91 <field name="model">res.users</field>
92 <field name="inherit_id" ref="base.view_users_form_simple_modif" />
93 <field name="arch" type="xml">
94 <footer position="before">
95 <group string="Printing" name="printing">
96 <field name="printing_action"/>
97 <field name="printing_printer_id"/>
98 </group>
99 </footer>
85 </field>100 </field>
86 </record>101 </record>
87102
@@ -91,10 +106,11 @@
91 <field name="model">printing.report.xml.action</field>106 <field name="model">printing.report.xml.action</field>
92 <field name="arch" type="xml">107 <field name="arch" type="xml">
93 <form string="Report Printing Actions">108 <form string="Report Printing Actions">
94 <field name="user_id"/>109 <group col="2">
95 <label string=""/>110 <field name="user_id"/>
96 <field name="action"/>111 <field name="action"/>
97 <field name="printer_id" select="1"/>112 <field name="printer_id" select="1"/>
113 </group>
98 </form>114 </form>
99 </field>115 </field>
100 </record>116 </record>
@@ -110,5 +126,9 @@
110 </field>126 </field>
111 </record>127 </record>
112128
129 <!-- Add a shorcut to "Actions/Report" in the Printing menu -->
130 <menuitem id="menu_printing_reports"
131 parent="menu_printing_main"
132 action="base.ir_action_report_xml"/>
113 </data>133 </data>
114</openerp>134</openerp>