From b37cda9752aaece836f68ee7da5616b452a99d1b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Iv=C3=A0n=20Todorovich?= <ivan.todorovich@gmail.com>
Date: Wed, 15 Feb 2023 12:25:32 -0300
Subject: [PATCH] [IMP] delivery_line_sale_line_position: pre-commit execution

---
 .../report/report_deliveryslip.xml            | 69 +++++++++----------
 .../views/stock_move.xml                      | 25 +++----
 .../views/stock_picking.xml                   | 22 +++---
 3 files changed, 54 insertions(+), 62 deletions(-)

diff --git a/delivery_line_sale_line_position/report/report_deliveryslip.xml b/delivery_line_sale_line_position/report/report_deliveryslip.xml
index b01335a..1627fc3 100644
--- a/delivery_line_sale_line_position/report/report_deliveryslip.xml
+++ b/delivery_line_sale_line_position/report/report_deliveryslip.xml
@@ -1,52 +1,45 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
-  <template id="report_delivery_document" inherit_id="stock.report_delivery_document">
-
-    <!-- Changes for when the picking is not yet done -->
-
-    <xpath expr="//table[@name='stock_move_table']/thead" position="before">
-      <t
+    <template id="report_delivery_document" inherit_id="stock.report_delivery_document">
+        <!-- Changes for when the picking is not yet done -->
+        <xpath expr="//table[@name='stock_move_table']/thead" position="before">
+            <t
                 t-set="has_line_position"
                 t-value="any(o.move_lines.filtered(lambda x: x.position_sale_line and x.product_uom_qty))"
             />
-    </xpath>
-
-    <xpath expr="//th[@name='th_sm_product']" position="before">
-      <th t-if="has_line_position" name="th_sm_pos"><strong>Pos</strong></th>
-    </xpath>
-    <!-- The star in the selector may seem not very specific, but it is done -->
-    <!-- in purpose so it works with some other module that change this report. -->
-    <!-- Looking at you stock_picking_group_by_partner_by_carrier -->
-    <xpath expr="//table[@name='stock_move_table']/tbody/tr/*[1]" position="before">
-      <td t-if="has_line_position">
-        <span t-field="move.position_sale_line" />
-      </td>
-    </xpath>
-
-
-    <!-- Changes for when the picking is done -->
-
-    <xpath expr="//table[@name='stock_move_line_table']/thead" position="before">
-      <t
+        </xpath>
+        <xpath expr="//th[@name='th_sm_product']" position="before">
+            <th t-if="has_line_position" name="th_sm_pos"><strong>Pos</strong></th>
+        </xpath>
+        <!-- The star in the selector may seem not very specific, but it is done -->
+        <!-- in purpose so it works with some other module that change this report. -->
+        <!-- Looking at you stock_picking_group_by_partner_by_carrier -->
+        <xpath expr="//table[@name='stock_move_table']/tbody/tr/*[1]" position="before">
+            <td t-if="has_line_position">
+                <span t-field="move.position_sale_line" />
+            </td>
+        </xpath>
+        <!-- Changes for when the picking is done -->
+        <xpath expr="//table[@name='stock_move_line_table']/thead" position="before">
+            <t
                 t-set="has_line_position"
                 t-value="any(o.move_line_ids.mapped('position_sale_line'))"
             />
-    </xpath>
+        </xpath>
+        <xpath expr="//th[@name='th_sml_product']" position="before">
+            <th t-if="has_line_position" name="th_sml_pos"><strong>Pos</strong></th>
+        </xpath>
+    </template>
 
-    <xpath expr="//th[@name='th_sml_product']" position="before">
-      <th t-if="has_line_position" name="th_sml_pos"><strong>Pos</strong></th>
-    </xpath>
-  </template>
-
-  <template
+    <template
         id="stock_report_delivery_has_serial_move_line"
         inherit_id="stock.stock_report_delivery_has_serial_move_line"
     >
-    <xpath expr="//span[@t-field='move_line.product_id']/.." position="before">
-      <td t-if="has_line_position">
-        <span t-field="move_line.move_id.position_sale_line" />
-      </td>
-    </xpath>
+        <xpath expr="//span[@t-field='move_line.product_id']/.." position="before">
+            <td t-if="has_line_position">
+                <span t-field="move_line.move_id.position_sale_line" />
+            </td>
+        </xpath>
+    </template>
 
-  </template>
 </odoo>
diff --git a/delivery_line_sale_line_position/views/stock_move.xml b/delivery_line_sale_line_position/views/stock_move.xml
index 8a8dfd2..4423303 100644
--- a/delivery_line_sale_line_position/views/stock_move.xml
+++ b/delivery_line_sale_line_position/views/stock_move.xml
@@ -1,17 +1,18 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
 
-  <record id="view_stock_move_line_detailed_operation_tree" model="ir.ui.view">
-    <field name="name">stock.move.line.operations.tree</field>
-    <field name="model">stock.move.line</field>
-    <field name="inherit_id" ref="stock.view_stock_move_line_detailed_operation_tree" />
-    <field name="arch" type="xml">
-
-      <xpath expr="//field[@name='product_id']" position="before">
-        <field name="position_sale_line" optional="hide" />
-      </xpath>
-
-    </field>
-  </record>
+    <record id="view_stock_move_line_detailed_operation_tree" model="ir.ui.view">
+        <field name="name">stock.move.line.operations.tree</field>
+        <field name="model">stock.move.line</field>
+        <field
+            name="inherit_id"
+            ref="stock.view_stock_move_line_detailed_operation_tree"
+        />
+        <field name="arch" type="xml">
+            <xpath expr="//field[@name='product_id']" position="before">
+                <field name="position_sale_line" optional="hide" />
+            </xpath>
+        </field>
+    </record>
 
 </odoo>
diff --git a/delivery_line_sale_line_position/views/stock_picking.xml b/delivery_line_sale_line_position/views/stock_picking.xml
index 5bf97e9..614c12f 100644
--- a/delivery_line_sale_line_position/views/stock_picking.xml
+++ b/delivery_line_sale_line_position/views/stock_picking.xml
@@ -1,20 +1,18 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
 
-  <record id="view_picking_form" model="ir.ui.view">
-    <field name="name">stock.view.picking.form</field>
-    <field name="model">stock.picking</field>
-    <field name="inherit_id" ref="stock.view_picking_form" />
-    <field name="arch" type="xml">
-
-      <xpath
+    <record id="view_picking_form" model="ir.ui.view">
+        <field name="name">stock.view.picking.form</field>
+        <field name="model">stock.picking</field>
+        <field name="inherit_id" ref="stock.view_picking_form" />
+        <field name="arch" type="xml">
+            <xpath
                 expr="//field[@name='move_ids_without_package']/tree/field[@name='product_id']"
                 position="before"
             >
-        <field name="position_sale_line" optional="hide" />
-      </xpath>
-
-    </field>
-  </record>
+                <field name="position_sale_line" optional="hide" />
+            </xpath>
+        </field>
+    </record>
 
 </odoo>