make precommit green again

pms/report/invoice.xml

@@ -1,6 +1,10 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-    <template id="report_invoice_document" inherit_id="account.report_invoice_document">
+    <template
+        id="report_invoice_document"
+        inherit_id="account.report_invoice_document"
+        priority="99999"
+    >
         <xpath expr="//address" position="before">
             <t t-if="o.pms_property_id">
                 <t t-set="information_block">
@@ -13,6 +17,9 @@
                 </t>
             </t>
         </xpath>
+<!-- fixme Dangerous use of "replace". Only replace as a last resort.
+     Try position="attributes", position="move" or t-if="False" first - [xml-dangerous-qweb-replace-low-priority]
+-->
         <xpath expr="//div[@class='page']//h2" position="replace">
             <h2>
                 <t t-if="not (env.context.get('proforma', False) or is_pro_forma)">

pms/tests/test_pms_booking_engine.py

@@ -1002,8 +1002,7 @@ class TestPmsBookingEngine(TestPms):
                 "channel_type_id": self.sale_channel_direct1.id,
             }
         )
-        lines_availability_test_double = booking_engine.availability_results.filtered(
+        booking_engine.availability_results.filtered(
             lambda r: r.room_type_id.id == self.test_room_type_double.id
         )
-        lines_availability_test_double
         self.assertTrue(False)

pms_l10n_es/data/pms_data.xml

@@ -43,7 +43,9 @@ else:
         </record>
 
         <!-- aeat document type mapped -->
+        <!-- disabled because this is an override of an existe record in this file -->
         <!-- pylint: disable=redundant-modulename-xml -->
+        <!-- oca-hooks:disable=xml-redundant-module-name -->
         <record id="pms_l10n_es.document_type_dni" model="res.partner.id_category">
             <field name="aeat_identification_type">02</field>
         </record>