From a460bf9433f3f567be6c4474551562193efa75c9 Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Mon, 9 Dec 2024 19:12:09 +0530 Subject: [PATCH] fix: description overwrite on qty change (backport #44606) (#44608) * fix: description overwrite on qty change (#44606) (cherry picked from commit 9ad79625e02772a6c72b5eacfd7ebf5f38706675) # Conflicts: # erpnext/stock/doctype/material_request/material_request.js * chore: fix conflicts --------- Co-authored-by: rohitwaghchaure --- .../doctype/material_request/material_request.js | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/erpnext/stock/doctype/material_request/material_request.js b/erpnext/stock/doctype/material_request/material_request.js index 20bfbef8f49..c32bd53af5e 100644 --- a/erpnext/stock/doctype/material_request/material_request.js +++ b/erpnext/stock/doctype/material_request/material_request.js @@ -226,11 +226,16 @@ frappe.ui.form.on('Material Request', { }, callback: function(r) { const d = item; - const allow_to_change_fields = ['actual_qty', 'projected_qty', 'min_order_qty', 'item_name', 'description', 'stock_uom', 'uom', 'conversion_factor', 'stock_qty']; - if(!r.exc) { - $.each(r.message, function(key, value) { - if(!d[key] || allow_to_change_fields.includes(key)) { + let allow_to_change_fields = ['actual_qty', 'projected_qty', 'min_order_qty', 'item_name', 'description', 'stock_uom', 'uom', 'conversion_factor', 'stock_qty']; + + if (overwrite_warehouse) { + allow_to_change_fields.push("description"); + } + + if (!r.exc) { + $.each(r.message, function (key, value) { + if (!d[key] || allow_to_change_fields.includes(key)) { d[key] = value; } });