fix: Sider and Patches

- Sider fixes
- Deleted patches that worked as per old schema
- Cleared instances of web fields linked to Item master
This commit is contained in:
marination
2021-05-17 21:27:42 +05:30
parent 897c08c8e7
commit 8c20f66905
5 changed files with 21 additions and 22 deletions

View File

@@ -203,7 +203,7 @@ class WebsiteItem(WebsiteGenerator):
# load variants # load variants
# also used in set_attribute_context # also used in set_attribute_context
context.variants = frappe.get_all("Item", context.variants = frappe.get_all("Item",
filters={"variant_of": self.name, "show_variant_in_website": 1}, filters={"variant_of": self.name, "published_in_website": 1},
order_by="name asc") order_by="name asc")
variant = frappe.form_dict.variant variant = frappe.form_dict.variant

View File

@@ -27,7 +27,7 @@ erpnext.ProductGrid = class {
html += me.get_image_html(item, title); html += me.get_image_html(item, title);
html += me.get_card_body_html(item, title, me.settings); html += me.get_card_body_html(item, title, me.settings);
html += `</div></div>`; html += `</div></div>`;
}) });
let $product_wrapper = this.products_section; let $product_wrapper = this.products_section;
$product_wrapper.append(html); $product_wrapper.append(html);
@@ -36,7 +36,7 @@ erpnext.ProductGrid = class {
get_image_html(item, title) { get_image_html(item, title) {
let image = item.website_image || item.image; let image = item.website_image || item.image;
if(image) { if (image) {
return ` return `
<div class="card-img-container"> <div class="card-img-container">
<a href="/${ item.route || '#' }" style="text-decoration: none;"> <a href="/${ item.route || '#' }" style="text-decoration: none;">
@@ -85,11 +85,11 @@ erpnext.ProductGrid = class {
<div class="product-title"> <div class="product-title">
${ title || '' } ${ title || '' }
`; `;
if (item.in_stock) { if (item.in_stock && settings.show_stock_availability) {
title_html += `<span class="indicator ${ item.in_stock } card-indicator"></span>`; title_html += `<span class="indicator ${ item.in_stock } card-indicator"></span>`;
} }
title_html += `</div></a>`; title_html += `</div></a>`;
return title_html return title_html;
} }
get_wishlist_icon(item) { get_wishlist_icon(item) {
@@ -145,4 +145,4 @@ erpnext.ProductGrid = class {
`; `;
} }
} }
} };

View File

@@ -27,7 +27,7 @@ erpnext.ProductList = class {
html += me.get_image_html(item, title); html += me.get_image_html(item, title);
html += me.get_row_body_html(item, title, me.settings); html += me.get_row_body_html(item, title, me.settings);
html += `</div>`; html += `</div>`;
}) });
let $product_wrapper = this.products_section; let $product_wrapper = this.products_section;
$product_wrapper.append(html); $product_wrapper.append(html);
@@ -36,7 +36,7 @@ erpnext.ProductList = class {
get_image_html(item, title) { get_image_html(item, title) {
let image = item.website_image || item.image; let image = item.website_image || item.image;
if(image) { if (image) {
return ` return `
<div class="col-2 border text-center rounded product-image" style="overflow: hidden; max-height: 200px;"> <div class="col-2 border text-center rounded product-image" style="overflow: hidden; max-height: 200px;">
<a class="product-link product-list-link" href="/${ item.route || '#' }"> <a class="product-link product-list-link" href="/${ item.route || '#' }">
@@ -74,7 +74,7 @@ erpnext.ProductList = class {
</a> </a>
`; `;
if (item.in_stock) { if (item.in_stock && settings.show_stock_availability) {
title_html += `<span class="indicator ${ item.in_stock } card-indicator"></span>`; title_html += `<span class="indicator ${ item.in_stock } card-indicator"></span>`;
} }
title_html += `</div>`; title_html += `</div>`;
@@ -92,7 +92,7 @@ erpnext.ProductList = class {
return title_html; return title_html;
} }
get_item_details(item, settings) { get_item_details(item) {
let details = ` let details = `
<p class="product-code"> <p class="product-code">
Item Code : ${ item.item_code } Item Code : ${ item.item_code }
@@ -104,7 +104,7 @@ erpnext.ProductList = class {
${ item.formatted_price || '' } ${ item.formatted_price || '' }
`; `;
if(item.formatted_mrp) { if (item.formatted_mrp) {
details += ` details += `
<small class="ml-1 text-muted"> <small class="ml-1 text-muted">
<s>${ item.formatted_mrp }</s> <s>${ item.formatted_mrp }</s>
@@ -155,4 +155,4 @@ erpnext.ProductList = class {
} }
} }
} };

View File

@@ -14,7 +14,7 @@ erpnext.ProductView = class {
} }
prepare_view_toggler() { prepare_view_toggler() {
if(!$("#list").length || !$("#image-view").length) { if (!$("#list").length || !$("#image-view").length) {
this.render_view_toggler(); this.render_view_toggler();
this.bind_view_toggler_actions(); this.bind_view_toggler_actions();
this.set_view_state(); this.set_view_state();
@@ -109,13 +109,13 @@ erpnext.ProductView = class {
attribute_filters: attribute_filters, attribute_filters: attribute_filters,
item_group: this.item_group, item_group: this.item_group,
start: filters.start || null start: filters.start || null
} };
} }
add_paging_section(settings) { add_paging_section(settings) {
$(".product-paging-area").remove(); $(".product-paging-area").remove();
if(this.products) { if (this.products) {
let paging_html = ` let paging_html = `
<div class="row product-paging-area mt-5"> <div class="row product-paging-area mt-5">
<div class="col-3"> <div class="col-3">
@@ -126,13 +126,13 @@ erpnext.ProductView = class {
let start = query_params.start ? cint(JSON.parse(query_params.start)) : 0; let start = query_params.start ? cint(JSON.parse(query_params.start)) : 0;
let page_length = settings.products_per_page || 0; let page_length = settings.products_per_page || 0;
if(start > 0) { if (start > 0) {
paging_html += ` paging_html += `
<button class="btn btn-default btn-prev" data-start="${ start - page_length }" style="float: left"> <button class="btn btn-default btn-prev" data-start="${ start - page_length }" style="float: left">
${ __("Prev") } ${ __("Prev") }
</button>`; </button>`;
} }
if(this.products.length > page_length || this.products.length == page_length) { if (this.products.length > page_length || this.products.length == page_length) {
paging_html += ` paging_html += `
<button class="btn btn-default btn-next" data-start="${ start + page_length }"> <button class="btn btn-default btn-next" data-start="${ start + page_length }">
${ __("Next") } ${ __("Next") }
@@ -171,7 +171,7 @@ erpnext.ProductView = class {
$("#products-grid-area").addClass("hidden"); $("#products-grid-area").addClass("hidden");
$("#products-list-area").removeClass("hidden"); $("#products-list-area").removeClass("hidden");
}) });
$("#image-view").click(function() { $("#image-view").click(function() {
let $btn = $(this); let $btn = $(this);
@@ -246,7 +246,7 @@ erpnext.ProductView = class {
} }
render_item_sub_categories(categories) { render_item_sub_categories(categories) {
if(categories) { if (categories) {
let sub_group_html = ` let sub_group_html = `
<div class="sub-category-container"> <div class="sub-category-container">
<div class="heading"> ${ __('Sub Categories') } </div> <div class="heading"> ${ __('Sub Categories') } </div>
@@ -262,10 +262,10 @@ erpnext.ProductView = class {
</div> </div>
</a> </a>
`; `;
}) });
sub_group_html += `</div>`; sub_group_html += `</div>`;
$("#product-listing").prepend(sub_group_html); $("#product-listing").prepend(sub_group_html);
} }
} }
} };

View File

@@ -149,7 +149,6 @@ def create_item_code(amazon_item_json, sku):
item.description = amazon_item_json.Product.AttributeSets.ItemAttributes.Title item.description = amazon_item_json.Product.AttributeSets.ItemAttributes.Title
item.brand = new_brand item.brand = new_brand
item.manufacturer = new_manufacturer item.manufacturer = new_manufacturer
item.web_long_description = amazon_item_json.Product.AttributeSets.ItemAttributes.Title
item.image = amazon_item_json.Product.AttributeSets.ItemAttributes.SmallImage.URL item.image = amazon_item_json.Product.AttributeSets.ItemAttributes.SmallImage.URL