fix: description overwrite on qty change (backport #44606) (#44608)

* fix: description overwrite on qty change (#44606) (cherry picked from commit 9ad7962) # Conflicts: # erpnext/stock/doctype/material_request/material_request.js * chore: fix conflicts --------- Co-authored-by: rohitwaghchaure <[email protected]>
frappe · Dec 9, 2024 · a460bf9 · a460bf9
1 parent 30151da
commit a460bf9
Showing 1 changed file with 9 additions and 4 deletions.
diff --git 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;
 						}
 					});