From b4e9c1da64a0b577585ce73eeed635f1ed5f2c84 Mon Sep 17 00:00:00 2001 From: Anand Doshi Date: Mon, 19 Oct 2015 12:57:19 +0530 Subject: [PATCH] [fix] thumbnail generation for item --- erpnext/stock/doctype/item/item.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/erpnext/stock/doctype/item/item.py b/erpnext/stock/doctype/item/item.py index 41329f3b4c..faae960343 100644 --- a/erpnext/stock/doctype/item/item.py +++ b/erpnext/stock/doctype/item/item.py @@ -82,6 +82,9 @@ class Item(WebsiteGenerator): def make_thumbnail(self): """Make a thumbnail of `website_image`""" + if not self.is_new() and self.website_image != frappe.db.get_value(self.doctype, self.name, "website_image"): + self.thumbnail = None + if self.website_image and not self.thumbnail: file_doc = frappe.get_doc("File", { "file_url": self.website_image, @@ -89,6 +92,7 @@ class Item(WebsiteGenerator): "attached_to_name": self.name }) + # for CSV import if not file_doc: file_doc = frappe.get_doc({ "doctype": "File", @@ -98,7 +102,10 @@ class Item(WebsiteGenerator): }).insert() if file_doc: - self.thumbnail = file_doc.make_thumbnail() + if not file_doc.thumbnail_url: + file_doc.make_thumbnail() + + self.thumbnail = file_doc.thumbnail_url def get_context(self, context): context.parent_groups = get_parent_item_groups(self.item_group) + \