From a4a3b022eea07203038a2dd1e50a2a4b4052a268 Mon Sep 17 00:00:00 2001 From: Saurabh Date: Sat, 25 Jun 2016 10:41:03 +0530 Subject: [PATCH] [fix] return route from make_route of child class --- erpnext/setup/doctype/item_group/item_group.py | 2 ++ erpnext/stock/doctype/item/item.py | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/erpnext/setup/doctype/item_group/item_group.py b/erpnext/setup/doctype/item_group/item_group.py index 3484fe2e551..3feff6b5953 100644 --- a/erpnext/setup/doctype/item_group/item_group.py +++ b/erpnext/setup/doctype/item_group/item_group.py @@ -41,6 +41,8 @@ class ItemGroup(NestedSet, WebsiteGenerator): self.route = frappe.get_doc('Item Group', self.parent_item_group).route + '/' self.route += self.scrub(self.item_group_name) + + return self.route def after_rename(self, olddn, newdn, merge=False): NestedSet.after_rename(self, olddn, newdn, merge) diff --git a/erpnext/stock/doctype/item/item.py b/erpnext/stock/doctype/item/item.py index 686c5d887ec..d15673922ee 100644 --- a/erpnext/stock/doctype/item/item.py +++ b/erpnext/stock/doctype/item/item.py @@ -134,7 +134,7 @@ class Item(WebsiteGenerator): def make_route(self): if not self.route: - self.route = frappe.db.get_value('Item Group', self.item_group, 'route') + '/' + self.scrub(self.item_name) + return frappe.db.get_value('Item Group', self.item_group, 'route') + '/' + self.scrub(self.item_name) def get_parents(self, context): item_group, route = frappe.db.get_value('Item Group', self.item_group, ['name', 'route'])