From 8fc7cb0117e6a7f4b31d7ab2d2788fbb3c5bb6a7 Mon Sep 17 00:00:00 2001 From: Mihir Kandoi Date: Mon, 22 Jun 2026 07:48:08 +0530 Subject: [PATCH] refactor(subcontracting): drop unused format arg in overconsumption message The "exceeds quantity available" throw in `validate_manufacture` passes a third positional arg (`item.transfer_qty`), but the message only has `{0}` and `{1}` placeholders, so `str.format` silently discards it. Remove the dead argument; no behaviour change. --- erpnext/controllers/subcontracting_inward_controller.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/erpnext/controllers/subcontracting_inward_controller.py b/erpnext/controllers/subcontracting_inward_controller.py index 5bdca5528e8..e4f1ffdacff 100644 --- a/erpnext/controllers/subcontracting_inward_controller.py +++ b/erpnext/controllers/subcontracting_inward_controller.py @@ -287,7 +287,7 @@ class SubcontractingInwardController: frappe.throw( _( "Row #{0}: Customer Provided Item {1} exceeds quantity available through Subcontracting Inward Order" - ).format(item.idx, get_link_to_form("Item", item.item_code), item.transfer_qty) + ).format(item.idx, get_link_to_form("Item", item.item_code)) ) elif item.s_warehouse != customer_warehouse: frappe.throw(