Merge lp:~openerp-dev/openobject-addons/6.0-opw-57629-ado into lp:openobject-addons/6.0
Proposed by
Amit Dodiya (OpenERP)
Status: | Needs review |
---|---|
Proposed branch: | lp:~openerp-dev/openobject-addons/6.0-opw-57629-ado |
Merge into: | lp:openobject-addons/6.0 |
Diff against target: |
64 lines (+10/-5) 2 files modified
mrp_repair/mrp_repair.py (+8/-3) mrp_repair/mrp_repair_view.xml (+2/-2) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/6.0-opw-57629-ado |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Amit Dodiya (OpenERP) (community) | Needs Resubmitting | ||
Naresh(OpenERP) (community) | Needs Fixing | ||
Review via email: mp+87919@code.launchpad.net |
Description of the change
Hello,
[FIX] UOM is not taken into consideration neighter for subtotal nor for total while creating a repair order"
1). Create a mrp repair order
2). Add 3 order line, 1 for centimer, 1 for meter and 1 for kilometer of wood product.
3). Change the UOM of meter to cm / meter to km.
Uom change is not affecting to total & Subtotal.
Thanks,
Amit
To post a comment you must log in.
Unmerged revisions
- 5289. By Amit Dodiya<email address hidden>
-
[FIX] UOM is not taken into consideration neighter for subtotal nor for total while creating a repair order line
Hello Amit,
There is a scope to improve here...
+ if not uom: obj.uom_ id.id or False
+ uom = product_obj.uom_id and product_
- result[ 'product_ uom'] = product_obj.uom_id and product_ obj.uom_ id.id or False 'product_ uom'] = uom
+ result[
then no need of this change too update( {'price_ unit': price, 'price_subtotal': price*product_ uom_qty} ) update( {'price_ unit': price, 'price_subtotal': price*product_ uom_qty, 'product_uom': uom})
- result.
19 -
20 + result.
Thanks,