summaryrefslogtreecommitdiff
path: root/addons
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 /addons
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.
Diffstat (limited to 'addons')
-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):