diff --git a/erpnext/manufacturing/page/bom_comparison_tool/bom_comparison_tool.js b/erpnext/manufacturing/page/bom_comparison_tool/bom_comparison_tool.js
index fcb7e884ecc..753fb4e896d 100644
--- a/erpnext/manufacturing/page/bom_comparison_tool/bom_comparison_tool.js
+++ b/erpnext/manufacturing/page/bom_comparison_tool/bom_comparison_tool.js
@@ -96,8 +96,8 @@ erpnext.BOMComparisonTool = class BOMComparisonTool {
return `
| ${frappe.meta.get_label(doctype, fieldname)} |
- ${value1} |
- ${value2} |
+ ${frappe.utils.escape_html(cstr(value1))} |
+ ${frappe.utils.escape_html(cstr(value2))} |
`;
})
@@ -138,13 +138,17 @@ erpnext.BOMComparisonTool = class BOMComparisonTool {
.map((change, i) => {
let [fieldname, value1, value2] = change;
let th =
- i === 0 ? `${item_code} | ` : "";
+ i === 0
+ ? `${frappe.utils.escape_html(
+ cstr(item_code)
+ )} | `
+ : "";
return `
${th}
| ${frappe.meta.get_label(child_doctype, fieldname)} |
- ${value1} |
- ${value2} |
+ ${frappe.utils.escape_html(cstr(value1))} |
+ ${frappe.utils.escape_html(cstr(value2))} |
`;
})
@@ -177,7 +181,9 @@ erpnext.BOMComparisonTool = class BOMComparisonTool {
let html = rows
.map((row) => {
let [, doc] = row;
- let cells = fields.map((df) => `${doc[df.fieldname]} | `).join("");
+ let cells = fields
+ .map((df) => `${frappe.utils.escape_html(cstr(doc[df.fieldname]))} | `)
+ .join("");
return `${cells}
`;
})
.join("");