c2c-oerpscenario team mailing list archive

[Phong Nguyen-Thanh <726294@xxxxxxxxxxxxxxxxxx>]

I'm making fool of myself by changing the wrong line of code
Here is the current code

                            <group colspan="2" col="2">
                                  <group col="2" colspan="2" attrs="{'invisible':[('uom_type','!=','smaller')]}">
                                        <field name="factor"/>
                                        <label colspan="2" string="  e.g: 1 * (reference unit) = ratio * (this unit)"/>
                                   </group>
                                   <group col="2" colspan="2" attrs="{'invisible':[('uom_type','!=','bigger')]}">
                                        <field name="factor_inv"/>
                                        <label colspan="2"  string="  e.g: 1 * (this unit) = ratio * (reference unit)"/>
                                   </group>
                            </group>

what it should be is (swap the factor and factor_inv lines)

                            <group colspan="2" col="2">
                                  <group col="2" colspan="2" attrs="{'invisible':[('uom_type','!=','smaller')]}">
                                        <field name="factor_inv"/>
                                        <label colspan="2" string="  e.g: 1 * (reference unit) = ratio * (this unit)"/>
                                   </group>
                                   <group col="2" colspan="2" attrs="{'invisible':[('uom_type','!=','bigger')]}">
                                        <field name="factor"/>
                                        <label colspan="2"  string="  e.g: 1 * (this unit) = ratio * (reference unit)"/>
                                   </group>
                            </group>

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/726294

Title:
  inconsistence in form/tree of product.uom

Status in OpenERP Modules (addons):
  Opinion

Bug description:
  The form view product.uom.form is not correct/consistent in regards to
  product.uom.tree. In the product.uom.tree view, the field "factor" is
  used and correctly shown. In product.uom.form, "factor" and
  "factor_inv" is used depending on whether the unit is bigger or
  smaller than the reference price, together with notes explaining the
  usage, but I think the notes/the code is not correct: it should be the
  opposite. This is very confused, for people new to OpenERP (when using
  the form view to add/edit)

                              <group colspan="2" col="2">
                                    <group col="2" colspan="2" attrs="{'invisible':[('uom_type','!=','smaller')]}">
                                          <field name="factor"/>
                                          <label colspan="2" string="  e.g: 1 * (reference unit) = ratio * (this unit)"/>
                                     </group>
                                     <group col="2" colspan="2" attrs="{'invisible':[('uom_type','!=','bigger')]}">
                                          <field name="factor_inv"/>
                                          <label colspan="2" string="  e.g: 1 * (this unit) = ratio * (reference unit)"/>
                                     </group>
                              </group>