summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArnold Moyaux <arm@odoo.com>2017-12-11 10:47:42 +0100
committerArnold Moyaux <arm@odoo.com>2017-12-11 11:24:33 +0100
commit6d02f7d6bfa5c59495c44db35d39a2eeef59e395 (patch)
treeee9b07420e59ca9cbc9c75e705499116cd8cb571
parentf3623f1513f6f7aa31d495b41257708c6896f1c8 (diff)
[FIX] purchase: Only use purchase UOM on vendor bill
It was the purpose of this commit fea667a2433dfc2387daa7051989e840d30e18e0 However it uses purchase uom in any cases and not only vendor bill. This commit correct it by using it only if a purchase order is linked to the invoice.
-rw-r--r--addons/account/models/account_invoice.py3
-rw-r--r--addons/purchase/models/account_invoice.py7
2 files changed, 7 insertions, 3 deletions
diff --git a/addons/account/models/account_invoice.py b/addons/account/models/account_invoice.py
index 93951e68904..83a4c015179 100644
--- a/addons/account/models/account_invoice.py
+++ b/addons/account/models/account_invoice.py
@@ -1316,9 +1316,6 @@ class AccountInvoiceLine(models.Model):
self.price_unit = 0.0
domain['uom_id'] = []
else:
- # Use the purchase uom by default
- self.uom_id = self.product_id.uom_po_id
-
if part.lang:
product = self.product_id.with_context(lang=part.lang)
else:
diff --git a/addons/purchase/models/account_invoice.py b/addons/purchase/models/account_invoice.py
index 0aed9c09c26..43ffb17dddf 100644
--- a/addons/purchase/models/account_invoice.py
+++ b/addons/purchase/models/account_invoice.py
@@ -59,6 +59,13 @@ class AccountInvoice(models.Model):
data['account_id'] = account.id
return data
+ def _onchange_product_id(self):
+ domain = super(AccountInvoice, self)._onchange_product_id()
+ if self.purchase_id:
+ # Use the purchase uom by default
+ self.uom_id = self.product_id.uom_po_id
+ return domain
+
# Load all unsold PO lines
@api.onchange('purchase_id')
def purchase_order_change(self):