nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-06-03 20:29:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(pos): escape item data on pos item selector (backport #55503) (#55508)

Browse Source 
Co-authored-by: Diptanil Saha <diptanil@frappe.io>
fix(pos): escape item data on pos item selector (#55503)
This commit is contained in:
mergify[bot]
2026-06-01 18:27:30 +05:30
committed by GitHub
parent fdf5439ece
commit 5393c93675
1 changed files with 41 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

62
erpnext/selling/page/point_of_sale/pos_item_selector.js
View File

@@ -112,17 +112,37 @@ erpnext.PointOfSale.ItemSelector = class {
render_item_list_column_header() {
return `<div class="list-column">
<div class="column-name">Name</div>
<div class="column-price">Price</div>
<div class="column-uom">UOM</div>
<div class="column-qty-available">Quantity Available</div>
<div class="column-name">${__("Name")}</div>
<div class="column-price">${__("Price")}</div>
<div class="column-uom">${__("UOM")}</div>
<div class="column-qty-available">${__("Quantity Available")}</div>
</div>`;
}
get_item_html(item) {
const me = this;
// eslint-disable-next-line no-unused-vars
const { item_image, serial_no, batch_no, barcode, actual_qty, uom, price_list_rate } = item;
function sanitize_item_data(item) {
return Object.fromEntries(
Object.entries(item).map(([key, value]) => [
key,
typeof value === "string" ? frappe.utils.escape_html(value) : value,
])
);
}
const sanitize_item = sanitize_item_data(item);
const {
item_code,
stock_uom,
item_name,
item_image,
serial_no,
batch_no,
barcode,
actual_qty,
uom,
price_list_rate,
} = sanitize_item;
const precision = flt(price_list_rate, 2) % 1 != 0 ? 2 : 0;
let indicator_color;
let qty_to_display = actual_qty;
@@ -149,29 +169,29 @@ erpnext.PointOfSale.ItemSelector = class {
<img
onerror="cur_pos.item_selector.handle_broken_image(this)"
class="item-img" src="${item_image}"
alt="${item.item_name}"
alt="${item_name}"
>
</div>`;
} else {
return `<div class="item-qty-pill">
<span class="indicator-pill whitespace-nowrap ${indicator_color}">${qty_to_display}</span>
</div>
<div class="item-display abbr">${frappe.get_abbr(item.item_name)}</div>`;
<div class="item-display abbr">${frappe.get_abbr(item_name)}</div>`;
}
}
return `<div class="item-wrapper"
data-item-code="${escape(item.item_code)}" data-serial-no="${escape(serial_no)}"
data-batch-no="${escape(batch_no)}" data-uom="${escape(uom)}"
data-rate="${escape(price_list_rate || 0)}"
data-stock-uom="${escape(item.stock_uom)}"
title="${item.item_name}">
data-item-code="${item_code}" data-serial-no="${serial_no}"
data-batch-no="${batch_no}" data-uom="${uom}"
data-rate="${price_list_rate || 0}"
data-stock-uom="${stock_uom}"
title="${item_name}">
${get_item_image_html()}
<div class="item-detail">
<div class="item-name">
${!me.hide_images ? frappe.ellipsis(item.item_name, 18) : item.item_name}
${!me.hide_images ? frappe.ellipsis(item_name, 18) : item_name}
</div>
${
!me.hide_images
@@ -189,7 +209,7 @@ erpnext.PointOfSale.ItemSelector = class {
}
handle_broken_image($img) {
const item_abbr = $($img).attr("alt");
const item_abbr = frappe.utils.escape_html($($img).attr("alt"));
$($img).parent().replaceWith(`<div class="item-display abbr">${item_abbr}</div>`);
}
@@ -244,7 +264,7 @@ erpnext.PointOfSale.ItemSelector = class {
set_item_selector_filter_label(value) {
const $filter_label = this.$component.find(".label");
$filter_label.html(value ? __(value) : __("All Items"));
$filter_label.html(value ? frappe.utils.escape_html(__(value)) : __("All Items"));
}
hide_open_link_btn() {
@@ -329,12 +349,12 @@ erpnext.PointOfSale.ItemSelector = class {
this.$component.on("click", ".item-wrapper", function () {
const $item = $(this);
const item_code = unescape($item.attr("data-item-code"));
let batch_no = unescape($item.attr("data-batch-no"));
let serial_no = unescape($item.attr("data-serial-no"));
let uom = unescape($item.attr("data-uom"));
let rate = unescape($item.attr("data-rate"));
let stock_uom = unescape($item.attr("data-stock-uom"));
const item_code = $item.attr("data-item-code");
let batch_no = $item.attr("data-batch-no");
let serial_no = $item.attr("data-serial-no");
let uom = $item.attr("data-uom");
let rate = $item.attr("data-rate");
let stock_uom = $item.attr("data-stock-uom");
// escape(undefined) returns "undefined" then unescape returns "undefined"
batch_no = batch_no === "undefined" ? undefined : batch_no;