nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-13 10:11:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

[fix] copy item variant values on save and make non no_copy fields as not editable. fixes frappe/erpnext#4253 (#8837)

Browse Source
This commit is contained in:
Rushabh Mehta
2017-05-16 07:42:44 +05:30
committed by Nabin Hait
parent 4fcd8a6db3
commit 939db36ad4
2 changed files with 25 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
erpnext/stock/doctype/item/item.js
View File

@@ -65,7 +65,8 @@ frappe.ui.form.on("Item", {
frm.page.set_inner_btn_group_as_primary(__("Make")); frm.page.set_inner_btn_group_as_primary(__("Make"));
} }
if (frm.doc.variant_of) { if (frm.doc.variant_of) {
frm.set_intro(__("This Item is a Variant of {0} (Template). Attributes will be copied over from the template unless 'No Copy' is set", [frm.doc.variant_of]), true); frm.set_intro(__("This Item is a Variant of {0} (Template). Attributes will be copied over from the template unless 'No Copy' is set",
[frm.doc.variant_of]), true);
} }
if (frappe.defaults.get_default("item_naming_by")!="Naming Series" || frm.doc.variant_of) { if (frappe.defaults.get_default("item_naming_by")!="Naming Series" || frm.doc.variant_of) {
@@ -96,6 +97,8 @@ frappe.ui.form.on("Item", {
} }
frappe.set_route('Form', 'Item', new_item.name); frappe.set_route('Form', 'Item', new_item.name);
}); });
frm.trigger('make_variant_fields_read_only');
}, },
validate: function(frm){ validate: function(frm){
@@ -106,6 +109,16 @@ frappe.ui.form.on("Item", {
refresh_field("image_view"); refresh_field("image_view");
}, },
make_variant_fields_read_only: function(frm) {
if(frm.doc.variant_of) {
frm.meta.fields.forEach(function(df) {
if (!df.no_copy) {
frm.toggle_enable(df.fieldname, false);
}
});
}
},
is_fixed_asset: function(frm) { is_fixed_asset: function(frm) {
if (frm.doc.is_fixed_asset) { if (frm.doc.is_fixed_asset) {
frm.set_value("is_stock_item", 0); frm.set_value("is_stock_item", 0);

13
erpnext/stock/doctype/item/item.py
View File

@@ -86,6 +86,7 @@ class Item(WebsiteGenerator):
self.validate_has_variants() self.validate_has_variants()
self.validate_attributes() self.validate_attributes()
self.validate_variant_attributes() self.validate_variant_attributes()
self.copy_variant_attributes()
self.validate_website_image() self.validate_website_image()
self.make_thumbnail() self.make_thumbnail()
self.validate_fixed_asset() self.validate_fixed_asset()
@@ -616,7 +617,9 @@ class Item(WebsiteGenerator):
template_item.save() template_item.save()
def update_variants(self): def update_variants(self):
if self.has_variants and not self.flags.dont_update_variants: if self.flags.dont_update_variants:
return
if self.has_variants:
updated = [] updated = []
variants = frappe.db.get_all("Item", fields=["item_code"], filters={"variant_of": self.name }) variants = frappe.db.get_all("Item", fields=["item_code"], filters={"variant_of": self.name })
for d in variants: for d in variants:
@@ -625,7 +628,7 @@ class Item(WebsiteGenerator):
variant.save() variant.save()
updated.append(d.item_code) updated.append(d.item_code)
if updated: if updated:
frappe.msgprint(_("Item Variants {0} updated").format(", ".join(updated))) frappe.msgprint(_("Item Variants {0} updated").format(", ".join(updated)))
def validate_has_variants(self): def validate_has_variants(self):
if not self.has_variants and frappe.db.get_value("Item", self.name, "has_variants"): if not self.has_variants and frappe.db.get_value("Item", self.name, "has_variants"):
@@ -670,6 +673,12 @@ class Item(WebsiteGenerator):
validate_item_variant_attributes(self, args) validate_item_variant_attributes(self, args)
def copy_variant_attributes(self):
'''Copy attributes from template (if they have been changed before saving)'''
if self.variant_of:
template = frappe.get_doc('Item', self.variant_of)
copy_attributes_to_variant(template, self)
def get_timeline_data(doctype, name): def get_timeline_data(doctype, name):
'''returns timeline data based on stock ledger entry''' '''returns timeline data based on stock ledger entry'''
out = {} out = {}