nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-16 18:24:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #57091 from aerele/fix/so-qty-company-warehouse

Browse Source 
fix(stock): show qty (company) and qty (warehouse) in sales transactions
This commit is contained in:
Mihir Kandoi
2026-07-13 21:03:07 +05:30
committed by GitHub
parent c0147e5a2e 4e5e1f6596
commit b23224b3d6
3 changed files with 42 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
erpnext/public/js/utils/sales_common.js
View File

@@ -284,19 +284,17 @@ erpnext.sales_common = {
set_actual_qty(doc, cdt, cdn) { set_actual_qty(doc, cdt, cdn) {
let row = locals[cdt][cdn]; let row = locals[cdt][cdn];
let sales_doctypes = ["Sales Invoice", "Delivery Note", "Sales Order"]; let sales_doctypes = ["Sales Invoice", "Delivery Note", "Sales Order", "Quotation"];
if (row.item_code && row.warehouse && sales_doctypes.includes(doc.doctype)) { if (row.item_code && row.warehouse && sales_doctypes.includes(doc.doctype)) {
frappe.call({ return this.frm.call({
method: "erpnext.stock.get_item_details.get_bin_details", method: "erpnext.stock.get_item_details.get_bin_details",
child: row,
args: { args: {
item_code: row.item_code, item_code: row.item_code,
warehouse: row.warehouse, warehouse: row.warehouse,
}, company: doc.company,
callback(r) { include_child_warehouses: true,
if (r.message) {
frappe.model.set_value(cdt, cdn, "actual_qty", r.message.actual_qty);
}
}, },
}); });
} }

7
erpnext/stock/get_item_details.py
View File

@@ -323,10 +323,9 @@ def update_bin_details(ctx: frappe._dict, out: frappe._dict, doc):
out.update(get_bin_details(ctx.item_code, ctx.from_warehouse)) out.update(get_bin_details(ctx.item_code, ctx.from_warehouse))
elif out.get("warehouse"): elif out.get("warehouse"):
company = ctx.company if (doc and doc.get("doctype") == "Purchase Order") else None bin_details = get_bin_details(
ctx.item_code, out.warehouse, ctx.company, include_child_warehouses=True
# calculate company_total_stock only for po )
bin_details = get_bin_details(ctx.item_code, out.warehouse, company, include_child_warehouses=True)
out.update(bin_details) out.update(bin_details)

34
erpnext/stock/tests/test_get_item_details.py
View File

@@ -28,6 +28,40 @@ class TestGetItemDetail(ERPNextTestSuite):
details = get_item_details(args) details = get_item_details(args)
self.assertEqual(details.get("price_list_rate"), 100) self.assertEqual(details.get("price_list_rate"), 100)
def test_bin_details_for_selling_doctypes(self):
from erpnext.stock.doctype.item.test_item import make_item
from erpnext.stock.doctype.purchase_receipt.test_purchase_receipt import make_purchase_receipt
item_code = make_item(properties={"is_stock_item": 1}).name
make_purchase_receipt(item_code=item_code, warehouse="_Test Warehouse - _TC", qty=100, rate=100)
make_purchase_receipt(item_code=item_code, warehouse="_Test Warehouse 1 - _TC", qty=50, rate=100)
args = frappe._dict(
{
"item_code": item_code,
"warehouse": "_Test Warehouse - _TC",
"company": "_Test Company",
"customer": "_Test Customer",
"currency": "INR",
"conversion_rate": 1.0,
"price_list": "_Test Price List",
"price_list_currency": "INR",
"plc_conversion_rate": 1.0,
"transaction_date": None,
"name": None,
"ignore_pricing_rule": 1,
"qty": 1,
}
)
for doctype in ("Sales Order", "Quotation", "Sales Invoice", "Delivery Note", "Purchase Order"):
with self.subTest(doctype=doctype):
details = get_item_details(args.copy().update({"doctype": doctype}))
self.assertEqual(details.get("actual_qty"), 100)
self.assertEqual(details.get("company_total_stock"), 150)
# making this test in get_item_details test file as feat/fix is present in that method # making this test in get_item_details test file as feat/fix is present in that method
def test_fetch_price_from_list_rate_on_doc_save(self): def test_fetch_price_from_list_rate_on_doc_save(self):
# create item # create item