Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Sider and indexing (minor)

Browse Source
This commit is contained in:
marination 2021-02-19 13:14:37 +05:30
parent 025574d9b5
commit b38339c979
3 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
erpnext/e_commerce/doctype/website_item/website_item.py
View File

@ -8,7 +8,6 @@ import json
import itertools import itertools
from frappe import _ from frappe import _
from frappe.website.render import clear_cache
from frappe.website.website_generator import WebsiteGenerator from frappe.website.website_generator import WebsiteGenerator
from frappe.utils import cstr, random_string, cint, flt from frappe.utils import cstr, random_string, cint, flt
@ -359,3 +358,7 @@ def make_website_item(doc):
website_item.save() website_item.save()
return [website_item.name, website_item.web_item_name] return [website_item.name, website_item.web_item_name]
def on_doctype_update():
# since route is a Text column, it needs a length for indexing
frappe.db.add_index("Website Item", ["route(500)"])

2
erpnext/portal/doctype/homepage/homepage.js
View File

@ -22,7 +22,7 @@ frappe.ui.form.on('Homepage', {
frappe.ui.form.on('Homepage Featured Product', { frappe.ui.form.on('Homepage Featured Product', {
view: function(frm, cdt, cdn) { view: function(frm, cdt, cdn) {
var child= locals[cdt][cdn] var child= locals[cdt][cdn];
if (child.item_code && child.route) { if (child.item_code && child.route) {
window.open('/' + child.route, '_blank'); window.open('/' + child.route, '_blank');
} }

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

@ -408,7 +408,6 @@ class Item(Document):
if self.published_in_website: if self.published_in_website:
invalidate_cache_for_item(self) invalidate_cache_for_item(self)
clear_cache(self.route)
frappe.db.set_value("Item", new_name, "item_code", new_name) frappe.db.set_value("Item", new_name, "item_code", new_name)
@ -998,10 +997,6 @@ def update_variants(variants, template, publish_progress=True):
if publish_progress: if publish_progress:
frappe.publish_progress(count / total * 100, title=_("Updating Variants...")) frappe.publish_progress(count / total * 100, title=_("Updating Variants..."))
def on_doctype_update():
# since route is a Text column, it needs a length for indexing
frappe.db.add_index("Item", ["route(500)"])
@erpnext.allow_regional @erpnext.allow_regional
def set_item_tax_from_hsn_code(item): def set_item_tax_from_hsn_code(item):
pass pass