From 1fa1ec850c2548efeee1dc19b8e16100a7c7de46 Mon Sep 17 00:00:00 2001
From: qsm-odoo <qsm@odoo.com>
Date: Thu, 21 Jan 2016 11:03:07 +0100
Subject: [PATCH] [IMP] stock_landed_costs: Broken layout on stock.landed.cost
 form view

---
 .../stock_landed_costs_view.xml               | 115 ++++++++----------
 1 file changed, 52 insertions(+), 63 deletions(-)

diff --git a/addons/stock_landed_costs/stock_landed_costs_view.xml b/addons/stock_landed_costs/stock_landed_costs_view.xml
index b4a7e6c4861..a07f41caaa4 100644
--- a/addons/stock_landed_costs/stock_landed_costs_view.xml
+++ b/addons/stock_landed_costs/stock_landed_costs_view.xml
@@ -30,74 +30,63 @@
                                 <field name="account_move_id"/>
                             </group>
                         </group>
-                        <group>
-                            <separator string="Cost Lines" colspan="4"/>
-                            <group>
-                                <field name="cost_lines" nolabel="1">
-                                    <form string="Cost Lines">
-                                        <group>
-                                            <group>
-                                                <field name="product_id"
-                                                    on_change="onchange_product_id(product_id)" domain="[('landed_cost_ok', '=', True)]" context="{'default_landed_cost_ok': True, 'form_view_ref':'stock_landed_costs.view_stock_landed_cost_type_form'}"/>
-                                                <field name="price_unit"/>
-                                            </group>
-                                            <group>
-                                                <field name="split_method"/>
-                                                <field name="account_id"/>
-                                            </group>
-                                        </group>
-                                        <label for="name"/>
-                                        <field name="name"/>
-                                    </form>
-                                    <tree string="Cost Lines" editable="bottom">
-                                        <field name="product_id" on_change="onchange_product_id(product_id)" domain="[('landed_cost_ok', '=', True)]" context="{'default_landed_cost_ok': True, 'form_view_ref':'stock_landed_costs.view_stock_landed_cost_type_form'}"/>
-                                        <field name="name"/>
-                                        <field name="account_id"/>
-                                        <field name="split_method"/>
+
+                        <separator string="Cost Lines"/>
+                        <field name="cost_lines">
+                            <form string="Cost Lines">
+                                <group>
+                                    <group>
+                                        <field name="product_id"
+                                            on_change="onchange_product_id(product_id)" domain="[('landed_cost_ok', '=', True)]" context="{'default_landed_cost_ok': True, 'form_view_ref':'stock_landed_costs.view_stock_landed_cost_type_form'}"/>
                                         <field name="price_unit"/>
-                                    </tree>
-                                </field>
-                            </group>
-                        </group>
+                                    </group>
+                                    <group>
+                                        <field name="split_method"/>
+                                        <field name="account_id"/>
+                                    </group>
+                                </group>
+                                <label for="name"/>
+                                <field name="name"/>
+                            </form>
+                            <tree string="Cost Lines" editable="bottom">
+                                <field name="product_id" on_change="onchange_product_id(product_id)" domain="[('landed_cost_ok', '=', True)]" context="{'default_landed_cost_ok': True, 'form_view_ref':'stock_landed_costs.view_stock_landed_cost_type_form'}"/>
+                                <field name="name"/>
+                                <field name="account_id"/>
+                                <field name="split_method"/>
+                                <field name="price_unit"/>
+                            </tree>
+                        </field>
                         <group class="oe_subtotal_footer oe_right">
                             <field name="amount_total" widget="monetary"/>
+                            <button colspan="2" name="compute_landed_cost" string="Compute" type="object" class="oe_right btn-default" states='draft'/>
                         </group>
-                        <group>
-                            <group>
-                                <separator string="Valuation Adjustments"/>
-                            </group>
-                            <group states='draft'>
-                                <button name="compute_landed_cost" string="Compute" type="object" class="oe_right"/>
-                            </group>
-                        </group>
-                        <group>
-                            <group>
-                                <field name="valuation_adjustment_lines" nolabel="1">
-                                    <form string="Valuation Adjustments">
-                                        <group>
-                                            <group>
-                                                <field name="product_id"/>
-                                                <field name="quantity"/>
-                                            </group>
-                                            <group>
-                                                <field name="former_cost"/>
-                                                <field name="additional_landed_cost"/>
-                                            </group>
-                                        </group>
-                                    </form>
-                                    <tree string="Valuation Adjustments" editable="bottom" create="0">
-                                        <field name="cost_line_id" readonly="1"/>
-                                        <field name="product_id" readonly="1"/>
-                                        <field name="weight" readonly="1"/>
-                                        <field name="volume" readonly="1"/>
-                                        <field name="quantity" readonly="1"/>
-                                        <field name="former_cost_per_unit" readonly="1" widget="monetary"/>
-                                        <field name="former_cost" readonly="1"/>
+
+                        <div class="oe_clear"/>
+                        <separator string="Valuation Adjustments"/>
+                        <field name="valuation_adjustment_lines">
+                            <form string="Valuation Adjustments">
+                                <group>
+                                    <group>
+                                        <field name="product_id"/>
+                                        <field name="quantity"/>
+                                    </group>
+                                    <group>
+                                        <field name="former_cost"/>
                                         <field name="additional_landed_cost"/>
-                                    </tree>
-                                </field>
-                            </group>
-                        </group>
+                                    </group>
+                                </group>
+                            </form>
+                            <tree string="Valuation Adjustments" editable="bottom" create="0">
+                                <field name="cost_line_id" readonly="1"/>
+                                <field name="product_id" readonly="1"/>
+                                <field name="weight" readonly="1"/>
+                                <field name="volume" readonly="1"/>
+                                <field name="quantity" readonly="1"/>
+                                <field name="former_cost_per_unit" readonly="1" widget="monetary"/>
+                                <field name="former_cost" readonly="1"/>
+                                <field name="additional_landed_cost"/>
+                            </tree>
+                        </field>
                     </sheet>
                     <div class="oe_chatter">
                         <field name="message_follower_ids" widget="mail_followers"/>