From 145d2314495587cacfa7dd3a7b7436dbb0a25101 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Tue, 12 Jan 2021 10:50:11 +0100
Subject: [PATCH] [MIG] base_report_to_printer: Migration to v14

---
 base_report_to_printer/__manifest__.py           |  2 +-
 base_report_to_printer/data/printing_data.xml    |  4 ++--
 base_report_to_printer/i18n/am.po                |  1 -
 .../i18n/base_report_to_printer.pot              |  1 -
 base_report_to_printer/i18n/bg.po                |  1 -
 base_report_to_printer/i18n/ca.po                |  1 -
 base_report_to_printer/i18n/de.po                |  1 -
 base_report_to_printer/i18n/el_GR.po             |  1 -
 base_report_to_printer/i18n/es.po                |  1 -
 base_report_to_printer/i18n/es_AR.po             |  6 +++---
 base_report_to_printer/i18n/es_ES.po             |  1 -
 base_report_to_printer/i18n/fi.po                |  1 -
 base_report_to_printer/i18n/fr.po                |  1 -
 base_report_to_printer/i18n/gl.po                |  1 -
 base_report_to_printer/i18n/hr.po                |  1 -
 base_report_to_printer/i18n/hr_HR.po             |  1 -
 base_report_to_printer/i18n/it.po                |  1 -
 base_report_to_printer/i18n/nl.po                |  1 -
 base_report_to_printer/i18n/nl_NL.po             |  1 -
 base_report_to_printer/i18n/pt.po                |  1 -
 base_report_to_printer/i18n/pt_BR.po             |  1 -
 base_report_to_printer/i18n/pt_PT.po             |  1 -
 base_report_to_printer/i18n/sl.po                |  1 -
 base_report_to_printer/i18n/tr.po                |  1 -
 base_report_to_printer/i18n/zh_CN.po             |  1 -
 .../models/ir_actions_report.py                  |  4 +---
 .../models/printing_printer.py                   |  4 ++--
 base_report_to_printer/models/printing_server.py |  5 +++--
 base_report_to_printer/security/security.xml     | 10 +++++++++-
 .../tests/test_printing_server.py                | 13 -------------
 .../views/ir_actions_report.xml                  |  4 ++--
 .../views/printing_printer.xml                   |  5 +++++
 base_report_to_printer/views/res_users.xml       | 10 +++++-----
 .../printing_printer_update_wizard_view.xml      | 16 +++++++---------
 34 files changed, 40 insertions(+), 65 deletions(-)

diff --git a/base_report_to_printer/__manifest__.py b/base_report_to_printer/__manifest__.py
index de84530..5f521f5 100644
--- a/base_report_to_printer/__manifest__.py
+++ b/base_report_to_printer/__manifest__.py
@@ -7,7 +7,7 @@
 
 {
     "name": "Report to printer",
-    "version": "13.0.1.2.0",
+    "version": "14.0.1.0.0",
     "category": "Generic Modules/Base",
     "author": "Agile Business Group & Domsense, Pegueroles SCP, NaN,"
     " LasLabs, Camptocamp, Odoo Community Association (OCA),"
diff --git a/base_report_to_printer/data/printing_data.xml b/base_report_to_printer/data/printing_data.xml
index 99785c5..c008aa5 100644
--- a/base_report_to_printer/data/printing_data.xml
+++ b/base_report_to_printer/data/printing_data.xml
@@ -20,12 +20,12 @@
     </record>
     <record forcecreate="True" id="ir_cron_update_printers" model="ir.cron">
         <field name="name">Update Printers Jobs</field>
-        <field eval="True" name="active" />
+        <field name="active" eval="True" />
         <field name="user_id" ref="base.user_root" />
         <field name="interval_number">1</field>
         <field name="interval_type">minutes</field>
         <field name="numbercall">-1</field>
-        <field eval="False" name="doall" />
+        <field name="doall" eval="False" />
         <field name="model_id" ref="base_report_to_printer.model_printing_server" />
         <field name="state">code</field>
         <field name="code">model.action_update_jobs()</field>
diff --git a/base_report_to_printer/i18n/am.po b/base_report_to_printer/i18n/am.po
index 5f7573f..875028e 100644
--- a/base_report_to_printer/i18n/am.po
+++ b/base_report_to_printer/i18n/am.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/base_report_to_printer.pot b/base_report_to_printer/i18n/base_report_to_printer.pot
index 48f6769..a82d82d 100644
--- a/base_report_to_printer/i18n/base_report_to_printer.pot
+++ b/base_report_to_printer/i18n/base_report_to_printer.pot
@@ -35,7 +35,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/bg.po b/base_report_to_printer/i18n/bg.po
index 159c683..c40bdc6 100644
--- a/base_report_to_printer/i18n/bg.po
+++ b/base_report_to_printer/i18n/bg.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/ca.po b/base_report_to_printer/i18n/ca.po
index 5c31d3d..b386c99 100644
--- a/base_report_to_printer/i18n/ca.po
+++ b/base_report_to_printer/i18n/ca.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/de.po b/base_report_to_printer/i18n/de.po
index 5fe45b5..450131b 100644
--- a/base_report_to_printer/i18n/de.po
+++ b/base_report_to_printer/i18n/de.po
@@ -41,7 +41,6 @@ msgstr "Aktionen"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr "Aktiv"
diff --git a/base_report_to_printer/i18n/el_GR.po b/base_report_to_printer/i18n/el_GR.po
index 67c4c8a..52b1fb7 100644
--- a/base_report_to_printer/i18n/el_GR.po
+++ b/base_report_to_printer/i18n/el_GR.po
@@ -41,7 +41,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/es.po b/base_report_to_printer/i18n/es.po
index 0301592..a3a05ae 100644
--- a/base_report_to_printer/i18n/es.po
+++ b/base_report_to_printer/i18n/es.po
@@ -41,7 +41,6 @@ msgstr "Acciones"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr "Activo"
diff --git a/base_report_to_printer/i18n/es_AR.po b/base_report_to_printer/i18n/es_AR.po
index 1601d8e..b215ee9 100644
--- a/base_report_to_printer/i18n/es_AR.po
+++ b/base_report_to_printer/i18n/es_AR.po
@@ -38,7 +38,6 @@ msgstr "Acciones"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr "Activo"
@@ -276,8 +275,8 @@ msgstr "Error enviando el documento a la impresora "
 #: code:addons/base_report_to_printer/models/printing_server.py:0
 #, python-format
 msgid ""
-"Failed to connect to the CUPS server on %s:%s. Check that the CUPS server is "
-"running and that you can reach it from the Odoo server."
+"Failed to connect to the CUPS server on %s:%s. Check that the CUPS server is"
+" running and that you can reach it from the Odoo server."
 msgstr ""
 "Ha fallado la conexión al servidor CUPS %s:%s. Comprueba que el servidor "
 "CUPS está funcionando y que el servidor de Odoo puede comunicarse con él."
@@ -595,6 +594,7 @@ msgstr "Motivo del estado actual del trabajo."
 #. module: base_report_to_printer
 #. openerp-web
 #: code:addons/base_report_to_printer/static/src/js/qweb_action_manager.js:0
+#: code:addons/base_report_to_printer/static/src/js/qweb_action_manager.js:0
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_report_xml_action__report_id
 #, python-format
 msgid "Report"
diff --git a/base_report_to_printer/i18n/es_ES.po b/base_report_to_printer/i18n/es_ES.po
index f30a515..ce572df 100644
--- a/base_report_to_printer/i18n/es_ES.po
+++ b/base_report_to_printer/i18n/es_ES.po
@@ -41,7 +41,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/fi.po b/base_report_to_printer/i18n/fi.po
index 89ee05d..31354d2 100644
--- a/base_report_to_printer/i18n/fi.po
+++ b/base_report_to_printer/i18n/fi.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/fr.po b/base_report_to_printer/i18n/fr.po
index 597a76e..497dcb3 100644
--- a/base_report_to_printer/i18n/fr.po
+++ b/base_report_to_printer/i18n/fr.po
@@ -40,7 +40,6 @@ msgstr "Actions"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/gl.po b/base_report_to_printer/i18n/gl.po
index 001f39e..2d23ced 100644
--- a/base_report_to_printer/i18n/gl.po
+++ b/base_report_to_printer/i18n/gl.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/hr.po b/base_report_to_printer/i18n/hr.po
index a805af0..d07e3e8 100644
--- a/base_report_to_printer/i18n/hr.po
+++ b/base_report_to_printer/i18n/hr.po
@@ -41,7 +41,6 @@ msgstr "Akcije"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr "Aktivno"
diff --git a/base_report_to_printer/i18n/hr_HR.po b/base_report_to_printer/i18n/hr_HR.po
index c13d8a9..f9e9e2c 100644
--- a/base_report_to_printer/i18n/hr_HR.po
+++ b/base_report_to_printer/i18n/hr_HR.po
@@ -42,7 +42,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/it.po b/base_report_to_printer/i18n/it.po
index c017319..390b121 100644
--- a/base_report_to_printer/i18n/it.po
+++ b/base_report_to_printer/i18n/it.po
@@ -40,7 +40,6 @@ msgstr "Azioni"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/nl.po b/base_report_to_printer/i18n/nl.po
index 22ad75e..8a59c4c 100644
--- a/base_report_to_printer/i18n/nl.po
+++ b/base_report_to_printer/i18n/nl.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/nl_NL.po b/base_report_to_printer/i18n/nl_NL.po
index 5539bff..bbe5ada 100644
--- a/base_report_to_printer/i18n/nl_NL.po
+++ b/base_report_to_printer/i18n/nl_NL.po
@@ -41,7 +41,6 @@ msgstr "Acties"
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr "Actief"
diff --git a/base_report_to_printer/i18n/pt.po b/base_report_to_printer/i18n/pt.po
index 19e15fe..539e40e 100644
--- a/base_report_to_printer/i18n/pt.po
+++ b/base_report_to_printer/i18n/pt.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/pt_BR.po b/base_report_to_printer/i18n/pt_BR.po
index 0c717b8..6822905 100644
--- a/base_report_to_printer/i18n/pt_BR.po
+++ b/base_report_to_printer/i18n/pt_BR.po
@@ -41,7 +41,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/pt_PT.po b/base_report_to_printer/i18n/pt_PT.po
index 3423c0d..562dd50 100644
--- a/base_report_to_printer/i18n/pt_PT.po
+++ b/base_report_to_printer/i18n/pt_PT.po
@@ -41,7 +41,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/sl.po b/base_report_to_printer/i18n/sl.po
index 5d1f5c2..e20dc31 100644
--- a/base_report_to_printer/i18n/sl.po
+++ b/base_report_to_printer/i18n/sl.po
@@ -41,7 +41,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/tr.po b/base_report_to_printer/i18n/tr.po
index cb38908..2668bed 100644
--- a/base_report_to_printer/i18n/tr.po
+++ b/base_report_to_printer/i18n/tr.po
@@ -40,7 +40,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/i18n/zh_CN.po b/base_report_to_printer/i18n/zh_CN.po
index 78fd1a6..6032d2f 100644
--- a/base_report_to_printer/i18n/zh_CN.po
+++ b/base_report_to_printer/i18n/zh_CN.po
@@ -41,7 +41,6 @@ msgstr ""
 
 #. module: base_report_to_printer
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_job__active
-#: model:ir.model.fields,field_description:base_report_to_printer.field_printing_printer__active
 #: model:ir.model.fields,field_description:base_report_to_printer.field_printing_server__active
 msgid "Active"
 msgstr ""
diff --git a/base_report_to_printer/models/ir_actions_report.py b/base_report_to_printer/models/ir_actions_report.py
index fddc2d4..0d0bf63 100644
--- a/base_report_to_printer/models/ir_actions_report.py
+++ b/base_report_to_printer/models/ir_actions_report.py
@@ -135,9 +135,7 @@ class IrActionsReport(models.Model):
         If the action configured on the report is server, it prints the
         generated document as well.
         """
-        document, doc_format = super(IrActionsReport, self).render_qweb_pdf(
-            res_ids=res_ids, data=data
-        )
+        document, doc_format = super().render_qweb_pdf(res_ids=res_ids, data=data)
 
         behaviour = self.behaviour()
         printer = behaviour.pop("printer", None)
diff --git a/base_report_to_printer/models/printing_printer.py b/base_report_to_printer/models/printing_printer.py
index 690fc90..e16bbc3 100644
--- a/base_report_to_printer/models/printing_printer.py
+++ b/base_report_to_printer/models/printing_printer.py
@@ -169,8 +169,8 @@ class PrintingPrinter(models.Model):
         options = self.print_options(report=report, **print_opts)
 
         _logger.debug(
-            "Sending job to CUPS printer %s on %s"
-            % (self.system_name, self.server_id.address)
+            "Sending job to CUPS printer %s on %s with options %s"
+            % (self.system_name, self.server_id.address, options)
         )
         connection.printFile(self.system_name, file_name, file_name, options=options)
         _logger.info(
diff --git a/base_report_to_printer/models/printing_server.py b/base_report_to_printer/models/printing_server.py
index ad84029..e38117e 100644
--- a/base_report_to_printer/models/printing_server.py
+++ b/base_report_to_printer/models/printing_server.py
@@ -61,7 +61,7 @@ class PrintingServer(models.Model):
             servers = self.search(domain)
 
         res = True
-        for server in servers.with_context(active_test=False):
+        for server in servers:
             connection = server._open_connection(raise_on_error=raise_on_error)
             if not connection:
                 server.printer_ids.write({"status": "server-error"})
@@ -71,7 +71,8 @@ class PrintingServer(models.Model):
             # Update Printers
             printers = connection.getPrinters()
             existing_printers = {
-                printer.system_name: printer for printer in server.printer_ids
+                printer.system_name: printer
+                for printer in server.with_context(active_test=False).printer_ids
             }
             updated_printers = []
             for name, printer_info in printers.items():
diff --git a/base_report_to_printer/security/security.xml b/base_report_to_printer/security/security.xml
index e354f61..ebda2e8 100644
--- a/base_report_to_printer/security/security.xml
+++ b/base_report_to_printer/security/security.xml
@@ -103,7 +103,6 @@
         <field eval="0" name="perm_write" />
         <field eval="0" name="perm_create" />
     </record>
-    <!--Ported from printing_tray -->
     <record id="access_printing_tray_all" model="ir.model.access">
         <field name="name">Printing Tray User</field>
         <field name="model_id" ref="model_printing_tray" />
@@ -122,4 +121,13 @@
         <field eval="1" name="perm_write" />
         <field eval="1" name="perm_create" />
     </record>
+    <record id="access_printing_printer_update_wizard" model="ir.model.access">
+        <field name="name">Update printer wizard</field>
+        <field name="model_id" ref="model_printing_printer_update_wizard" />
+        <field name="group_id" ref="printing_group_manager" />
+        <field eval="1" name="perm_read" />
+        <field eval="1" name="perm_unlink" />
+        <field eval="1" name="perm_write" />
+        <field eval="1" name="perm_create" />
+    </record>
 </odoo>
diff --git a/base_report_to_printer/tests/test_printing_server.py b/base_report_to_printer/tests/test_printing_server.py
index a48f263..f7c444e 100644
--- a/base_report_to_printer/tests/test_printing_server.py
+++ b/base_report_to_printer/tests/test_printing_server.py
@@ -89,19 +89,6 @@ class TestPrintingServer(TransactionCase):
         self.Model.action_update_printers()
         self.assertEqual("unavailable", rec_id.status)
 
-    @mock.patch("%s.cups" % model)
-    def test_update_archived_printers(self, cups):
-        """ It should update status even if printer is archived """
-        rec_id = self.new_printer()
-        rec_id.toggle_active()
-        self.server.refresh()
-        cups.Connection().getPrinters().get.return_value = False
-        self.Model.action_update_printers()
-        self.assertEqual(
-            "unavailable",
-            rec_id.status,
-        )
-
     @mock.patch("%s.cups" % model)
     def test_update_jobs_cron(self, cups):
         """ It should get all jobs from CUPS server """
diff --git a/base_report_to_printer/views/ir_actions_report.xml b/base_report_to_printer/views/ir_actions_report.xml
index 85f3474..b250fdf 100644
--- a/base_report_to_printer/views/ir_actions_report.xml
+++ b/base_report_to_printer/views/ir_actions_report.xml
@@ -5,7 +5,7 @@
         <field name="model">ir.actions.report</field>
         <field name="inherit_id" ref="base.act_report_xml_view" />
         <field name="arch" type="xml">
-            <xpath expr="//page[@name='security']" position="before">
+            <page name="security" position="before">
                 <page string="Print" name="print">
                     <group>
                         <field name="property_printing_action_id" />
@@ -15,7 +15,7 @@
                     <separator string="Specific actions per user" />
                     <field name="printing_action_ids" />
                 </page>
-            </xpath>
+            </page>
         </field>
     </record>
 </odoo>
diff --git a/base_report_to_printer/views/printing_printer.xml b/base_report_to_printer/views/printing_printer.xml
index 852280d..27e5210 100644
--- a/base_report_to_printer/views/printing_printer.xml
+++ b/base_report_to_printer/views/printing_printer.xml
@@ -113,6 +113,11 @@
                 <field name="system_name" />
                 <field name="location" />
                 <field name="status" />
+                <filter
+                    string="Archived"
+                    name="inactive"
+                    domain="[('active', '=', False)]"
+                />
             </search>
         </field>
     </record>
diff --git a/base_report_to_printer/views/res_users.xml b/base_report_to_printer/views/res_users.xml
index 48a65e9..1470f62 100644
--- a/base_report_to_printer/views/res_users.xml
+++ b/base_report_to_printer/views/res_users.xml
@@ -5,12 +5,12 @@
         <field name="model">res.users</field>
         <field name="inherit_id" ref="base.view_users_form" />
         <field name="arch" type="xml">
-            <xpath expr="//group[@name='preferences']/ancestor::page" position="inside">
+            <group name="preferences" position="after">
                 <group string="Printing" name="printing">
                     <field name="printing_action" />
                     <field name="printing_printer_id" options="{'no_create': True}" />
                 </group>
-            </xpath>
+            </group>
         </field>
     </record>
     <record model="ir.ui.view" id="view_users_form_simple_modif">
@@ -18,8 +18,8 @@
         <field name="model">res.users</field>
         <field name="inherit_id" ref="base.view_users_form_simple_modif" />
         <field name="arch" type="xml">
-            <footer position="before">
-                <group string="Printing" name="printing">
+            <group name="preferences" position="inside">
+                <group name="printing">
                     <field name="printing_action" readonly="0" />
                     <field
                         name="printing_printer_id"
@@ -27,7 +27,7 @@
                         options="{'no_create': True}"
                     />
                 </group>
-            </footer>
+            </group>
         </field>
     </record>
 </odoo>
diff --git a/base_report_to_printer/wizards/printing_printer_update_wizard_view.xml b/base_report_to_printer/wizards/printing_printer_update_wizard_view.xml
index 136ee41..ba19d61 100644
--- a/base_report_to_printer/wizards/printing_printer_update_wizard_view.xml
+++ b/base_report_to_printer/wizards/printing_printer_update_wizard_view.xml
@@ -5,27 +5,25 @@
         <field name="model">printing.printer.update.wizard</field>
         <field name="arch" type="xml">
             <form string="Update Printers from CUPS">
-                <label
-                    string="This process will create all missing printers from the current CUPS server."
-                    colspan="2"
-                    for=""
-                />
+                <div name="introduction">
+                    <p>
+                    This process will create all missing printers from the current CUPS server."
+                    </p>
+                </div>
                 <footer>
                     <button
                         name="action_ok"
                         string="Ok"
                         type="object"
-                        class="oe_highlight"
+                        class="btn-primary"
                     />
-                    or
-                    <button string="Cancel" class="oe_link" special="cancel" />
+                    <button string="Cancel" class="btn-default" special="cancel" />
                 </footer>
             </form>
         </field>
     </record>
     <record id="action_printer_update_wizard" model="ir.actions.act_window">
         <field name="name">Update Printers from CUPS</field>
-        <field name="type">ir.actions.act_window</field>
         <field name="res_model">printing.printer.update.wizard</field>
         <field name="view_mode">form</field>
         <field name="target">new</field>