From 7a11966992475a1b4cccc95e4dd88e4ccd73afe4 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Thu, 13 Apr 2017 20:05:15 +0200
Subject: [PATCH] Better views for product.ul

Track partner_id on stock.picking
Improve stock.quant.package view
---
 product_usability/product_view.xml | 72 ++++++++++++++++++++++++++++++
 stock_usability/stock.py           |  2 +
 stock_usability/stock_view.xml     | 10 +++++
 3 files changed, 84 insertions(+)

diff --git a/product_usability/product_view.xml b/product_usability/product_view.xml
index 97b7358..fd5ac90 100644
--- a/product_usability/product_view.xml
+++ b/product_usability/product_view.xml
@@ -125,6 +125,78 @@
     </field>
 </record>
 
+<record id="product_ul_form_view" model="ir.ui.view">
+    <field name="name">usability.product.ul.form</field>
+    <field name="model">product.ul</field>
+    <field name="inherit_id" ref="product.product_ul_form_view"/>
+    <field name="arch" type="xml">
+        <!-- Useusally, the size of packaging is written in cm,
+        so I propose to use that UoM for size of packaging.
+        Nothing is written in the native product module about this -->
+        <field name="height" position="replace">
+            <label for="height"/>
+            <div name="height">
+                <field name="height" class="oe_inline"/>
+                <label string=" cm" class="oe_inline"/>
+            </div>
+        </field>
+        <field name="width" position="replace">
+            <label for="width"/>
+            <div name="width">
+                <field name="width" class="oe_inline"/>
+                <label string=" cm" class="oe_inline"/>
+            </div>
+        </field>
+        <field name="length" position="replace">
+            <label for="length"/>
+            <div name="length">
+                <field name="length" class="oe_inline"/>
+                <label string=" cm" class="oe_inline"/>
+            </div>
+        </field>
+        <field name="weight" position="replace">
+            <label for="weight"/>
+            <div name="weight">
+                <field name="weight" class="oe_inline"/>
+                <label string=" kg" class="oe_inline"/>
+            </div>
+        </field>
+    </field>
+</record>
+
+<record id="product_ul_tree" model="ir.ui.view">
+    <field name="name">usability.product.ul.tree</field>
+    <field name="model">product.ul</field>
+    <field name="inherit_id" ref="product.product_ul_tree"/>
+    <field name="arch" type="xml">
+        <field name="type" position="after">
+            <!-- not so important, but we have room in the tree view, so let's use it -->
+            <field name="height"/>
+            <field name="width"/>
+            <field name="length"/>
+            <field name="weight"/>
+        </field>
+    </field>
+</record>
+
+<!-- Add missing search view on product.ul -->
+<!-- It doesn't work: when you activate the group by type, it gives a JS error
+TypeError: key.indexOf is not a function
+-->
+<!--
+<record id="product_ul_search" model="ir.ui.view">
+    <field name="name">usability.product.ul.search</field>
+    <field name="model">product.ul</field>
+    <field name="arch" type="xml">
+        <search string="Search Logistic Units">
+            <field name="name"/>
+            <group string="Group By" name="groupby">
+                <filter name="type_groupby" string="Type" context="{'group_by': 'type'}"/>
+            </group>
+        </search>
+    </field>
+</record>
+-->
 
 </data>
 </openerp>
diff --git a/stock_usability/stock.py b/stock_usability/stock.py
index 6172a25..398c15d 100644
--- a/stock_usability/stock.py
+++ b/stock_usability/stock.py
@@ -38,6 +38,8 @@ class StockPicking(models.Model):
     # In the stock module: _order = "priority desc, date asc, id desc"
     # The problem is date asc
 
+    partner_id = fields.Many2one(track_visibility='onchange')
+
 
 class StockLocation(models.Model):
     _inherit = 'stock.location'
diff --git a/stock_usability/stock_view.xml b/stock_usability/stock_view.xml
index 1dda15b..7386202 100644
--- a/stock_usability/stock_view.xml
+++ b/stock_usability/stock_view.xml
@@ -256,6 +256,16 @@ in the quantities for his reception -->
     </field>
 </record>
 
+<record id="view_quant_package_tree" model="ir.ui.view">
+    <field name="name">stock.usability.quant.package.tree</field>
+    <field name="model">stock.quant.package</field>
+    <field name="inherit_id" ref="stock.view_quant_package_tree"/>
+    <field name="arch" type="xml">
+        <field name="company_id" position="attributes">
+            <attribute name="groups">base.group_multi_company</attribute>
+        </field>
+    </field>
+</record>
 
 <record id="view_move_tree_receipt_picking" model="ir.ui.view">
     <field name="name">stock.usability.stock.move.tree2</field>