Merge pull request #16527 from nabinhait/bom-fix
fix: Default value for 'raw materials rate based on' in BOM
diff --git a/erpnext/manufacturing/doctype/bom/bom.json b/erpnext/manufacturing/doctype/bom/bom.json
index 0cf7dc4..79c883a 100644
--- a/erpnext/manufacturing/doctype/bom/bom.json
+++ b/erpnext/manufacturing/doctype/bom/bom.json
@@ -642,6 +642,7 @@
"bold": 0,
"collapsible": 0,
"columns": 0,
+ "default": "Valuation Rate",
"fieldname": "rm_cost_as_per",
"fieldtype": "Select",
"hidden": 0,
@@ -1977,7 +1978,7 @@
"issingle": 0,
"istable": 0,
"max_attachments": 0,
- "modified": "2018-12-13 17:45:44.843197",
+ "modified": "2019-01-30 16:39:34.353721",
"modified_by": "Administrator",
"module": "Manufacturing",
"name": "BOM",
diff --git a/erpnext/manufacturing/doctype/bom/bom.py b/erpnext/manufacturing/doctype/bom/bom.py
index 2317634..ba3b7192 100644
--- a/erpnext/manufacturing/doctype/bom/bom.py
+++ b/erpnext/manufacturing/doctype/bom/bom.py
@@ -163,6 +163,8 @@
def get_rm_rate(self, arg):
""" Get raw material rate as per selected method, if bom exists takes bom cost """
rate = 0
+ if not self.rm_cost_as_per:
+ self.rm_cost_as_per = "Valuation Rate"
if arg.get('scrap_items'):
rate = self.get_valuation_rate(arg)