Merge pull request #40975 from rohitwaghchaure/fixed-donot-validate-for-lcv
fix: do not validate batch qty for LCV
diff --git a/erpnext/selling/doctype/sales_order/sales_order.js b/erpnext/selling/doctype/sales_order/sales_order.js
index 69bbae3..99d9799 100644
--- a/erpnext/selling/doctype/sales_order/sales_order.js
+++ b/erpnext/selling/doctype/sales_order/sales_order.js
@@ -288,6 +288,7 @@
label: __("Items to Reserve"),
allow_bulk_edit: false,
cannot_add_rows: true,
+ cannot_delete_rows: true,
data: [],
fields: [
{
@@ -356,7 +357,7 @@
],
primary_action_label: __("Reserve Stock"),
primary_action: () => {
- var data = { items: dialog.fields_dict.items.grid.data };
+ var data = { items: dialog.fields_dict.items.grid.get_selected_children() };
if (data.items && data.items.length > 0) {
frappe.call({
@@ -373,9 +374,11 @@
frm.reload_doc();
},
});
- }
- dialog.hide();
+ dialog.hide();
+ } else {
+ frappe.msgprint(__("Please select items to reserve."));
+ }
},
});
@@ -390,6 +393,7 @@
if (unreserved_qty > 0) {
dialog.fields_dict.items.df.data.push({
+ __checked: 1,
sales_order_item: item.name,
item_code: item.item_code,
warehouse: item.warehouse,
@@ -414,6 +418,7 @@
label: __("Reserved Stock"),
allow_bulk_edit: false,
cannot_add_rows: true,
+ cannot_delete_rows: true,
in_place_edit: true,
data: [],
fields: [
@@ -457,7 +462,7 @@
],
primary_action_label: __("Unreserve Stock"),
primary_action: () => {
- var data = { sr_entries: dialog.fields_dict.sr_entries.grid.data };
+ var data = { sr_entries: dialog.fields_dict.sr_entries.grid.get_selected_children() };
if (data.sr_entries && data.sr_entries.length > 0) {
frappe.call({
@@ -473,9 +478,11 @@
frm.reload_doc();
},
});
- }
- dialog.hide();
+ dialog.hide();
+ } else {
+ frappe.msgprint(__("Please select items to unreserve."));
+ }
},
});
diff --git a/erpnext/subcontracting/doctype/subcontracting_receipt/subcontracting_receipt.py b/erpnext/subcontracting/doctype/subcontracting_receipt/subcontracting_receipt.py
index 49b7b0f..e20eb27 100644
--- a/erpnext/subcontracting/doctype/subcontracting_receipt/subcontracting_receipt.py
+++ b/erpnext/subcontracting/doctype/subcontracting_receipt/subcontracting_receipt.py
@@ -498,8 +498,6 @@
return process_gl_map(gl_entries)
def make_item_gl_entries(self, gl_entries, warehouse_account=None):
- stock_rbnb = self.get_company_default("stock_received_but_not_billed")
-
warehouse_with_no_account = []
for item in self.items:
@@ -517,31 +515,41 @@
"stock_value_difference",
)
- warehouse_account_name = warehouse_account[item.warehouse]["account"]
- warehouse_account_currency = warehouse_account[item.warehouse]["account_currency"]
+ accepted_warehouse_account = warehouse_account[item.warehouse]["account"]
supplier_warehouse_account = warehouse_account.get(self.supplier_warehouse, {}).get(
"account"
)
- supplier_warehouse_account_currency = warehouse_account.get(
- self.supplier_warehouse, {}
- ).get("account_currency")
remarks = self.get("remarks") or _("Accounting Entry for Stock")
- # FG Warehouse Account (Debit)
+ # Accepted Warehouse Account (Debit)
self.add_gl_entry(
gl_entries=gl_entries,
- account=warehouse_account_name,
+ account=accepted_warehouse_account,
cost_center=item.cost_center,
debit=stock_value_diff,
credit=0.0,
remarks=remarks,
- against_account=stock_rbnb,
- account_currency=warehouse_account_currency,
+ against_account=item.expense_account,
+ account_currency=get_account_currency(accepted_warehouse_account),
+ project=item.project,
+ item=item,
+ )
+ # Expense Account (Credit)
+ self.add_gl_entry(
+ gl_entries=gl_entries,
+ account=item.expense_account,
+ cost_center=item.cost_center,
+ debit=0.0,
+ credit=stock_value_diff,
+ remarks=remarks,
+ against_account=accepted_warehouse_account,
+ account_currency=get_account_currency(item.expense_account),
+ project=item.project,
item=item,
)
- # Supplier Warehouse Account (Credit)
- if flt(item.rm_supp_cost) and warehouse_account.get(self.supplier_warehouse):
+ if flt(item.rm_supp_cost) and supplier_warehouse_account:
+ # Supplier Warehouse Account (Credit)
self.add_gl_entry(
gl_entries=gl_entries,
account=supplier_warehouse_account,
@@ -549,40 +557,66 @@
debit=0.0,
credit=flt(item.rm_supp_cost),
remarks=remarks,
- against_account=warehouse_account_name,
- account_currency=supplier_warehouse_account_currency,
+ against_account=item.expense_account,
+ account_currency=get_account_currency(supplier_warehouse_account),
+ project=item.project,
item=item,
)
-
- # Expense Account (Credit)
- if flt(item.service_cost_per_qty):
+ # Expense Account (Debit)
self.add_gl_entry(
gl_entries=gl_entries,
account=item.expense_account,
cost_center=item.cost_center,
- debit=0.0,
- credit=flt(item.service_cost_per_qty) * flt(item.qty),
+ debit=flt(item.rm_supp_cost),
+ credit=0.0,
remarks=remarks,
- against_account=warehouse_account_name,
+ against_account=supplier_warehouse_account,
account_currency=get_account_currency(item.expense_account),
+ project=item.project,
item=item,
)
- # Loss Account (Credit)
- divisional_loss = flt(item.amount - stock_value_diff, item.precision("amount"))
+ # Expense Account (Debit)
+ if item.additional_cost_per_qty:
+ self.add_gl_entry(
+ gl_entries=gl_entries,
+ account=item.expense_account,
+ cost_center=self.cost_center or self.get_company_default("cost_center"),
+ debit=item.qty * item.additional_cost_per_qty,
+ credit=0.0,
+ remarks=remarks,
+ against_account=None,
+ account_currency=get_account_currency(item.expense_account),
+ )
- if divisional_loss:
- loss_account = item.expense_account
+ if divisional_loss := flt(item.amount - stock_value_diff, item.precision("amount")):
+ loss_account = self.get_company_default(
+ "stock_adjustment_account", ignore_validation=True
+ )
+ # Loss Account (Credit)
self.add_gl_entry(
gl_entries=gl_entries,
account=loss_account,
cost_center=item.cost_center,
+ debit=0.0,
+ credit=divisional_loss,
+ remarks=remarks,
+ against_account=item.expense_account,
+ account_currency=get_account_currency(loss_account),
+ project=item.project,
+ item=item,
+ )
+ # Expense Account (Debit)
+ self.add_gl_entry(
+ gl_entries=gl_entries,
+ account=item.expense_account,
+ cost_center=item.cost_center,
debit=divisional_loss,
credit=0.0,
remarks=remarks,
- against_account=warehouse_account_name,
- account_currency=get_account_currency(loss_account),
+ against_account=loss_account,
+ account_currency=get_account_currency(item.expense_account),
project=item.project,
item=item,
)
@@ -592,7 +626,6 @@
):
warehouse_with_no_account.append(item.warehouse)
- # Additional Costs Expense Accounts (Credit)
for row in self.additional_costs:
credit_amount = (
flt(row.base_amount)
@@ -600,6 +633,7 @@
else flt(row.amount)
)
+ # Additional Cost Expense Account (Credit)
self.add_gl_entry(
gl_entries=gl_entries,
account=row.expense_account,
@@ -608,6 +642,7 @@
credit=credit_amount,
remarks=remarks,
against_account=None,
+ account_currency=get_account_currency(row.expense_account),
)
if warehouse_with_no_account:
diff --git a/erpnext/subcontracting/doctype/subcontracting_receipt/test_subcontracting_receipt.py b/erpnext/subcontracting/doctype/subcontracting_receipt/test_subcontracting_receipt.py
index 564096f..996a990 100644
--- a/erpnext/subcontracting/doctype/subcontracting_receipt/test_subcontracting_receipt.py
+++ b/erpnext/subcontracting/doctype/subcontracting_receipt/test_subcontracting_receipt.py
@@ -10,6 +10,7 @@
import erpnext
from erpnext.accounts.doctype.account.test_account import get_inventory_account
+from erpnext.accounts.utils import get_company_default
from erpnext.controllers.sales_and_purchase_return import make_return_doc
from erpnext.controllers.tests.test_subcontracting_controller import (
get_rm_items,
@@ -351,26 +352,15 @@
self.assertEqual(cint(erpnext.is_perpetual_inventory_enabled(scr.company)), 1)
gl_entries = get_gl_entries("Subcontracting Receipt", scr.name)
-
self.assertTrue(gl_entries)
fg_warehouse_ac = get_inventory_account(scr.company, scr.items[0].warehouse)
- supplier_warehouse_ac = get_inventory_account(scr.company, scr.supplier_warehouse)
expense_account = scr.items[0].expense_account
-
- if fg_warehouse_ac == supplier_warehouse_ac:
- expected_values = {
- fg_warehouse_ac: [2100.0, 1000.0], # FG Amount (D), RM Cost (C)
- expense_account: [0.0, 1000.0], # Service Cost (C)
- additional_costs_expense_account: [0.0, 100.0], # Additional Cost (C)
- }
- else:
- expected_values = {
- fg_warehouse_ac: [2100.0, 0.0], # FG Amount (D)
- supplier_warehouse_ac: [0.0, 1000.0], # RM Cost (C)
- expense_account: [0.0, 1000.0], # Service Cost (C)
- additional_costs_expense_account: [0.0, 100.0], # Additional Cost (C)
- }
+ expected_values = {
+ fg_warehouse_ac: [2100.0, 1000],
+ expense_account: [1100, 2100],
+ additional_costs_expense_account: [0.0, 100.0],
+ }
for gle in gl_entries:
self.assertEqual(expected_values[gle.account][0], gle.debit)
@@ -381,6 +371,53 @@
self.assertTrue(get_gl_entries("Subcontracting Receipt", scr.name))
frappe.db.set_single_value("Stock Settings", "use_serial_batch_fields", 1)
+ @change_settings("Stock Settings", {"use_serial_batch_fields": 0})
+ def test_subcontracting_receipt_with_zero_service_cost(self):
+ warehouse = "Stores - TCP1"
+ service_items = [
+ {
+ "warehouse": warehouse,
+ "item_code": "Subcontracted Service Item 7",
+ "qty": 10,
+ "rate": 0,
+ "fg_item": "Subcontracted Item SA7",
+ "fg_item_qty": 10,
+ },
+ ]
+ sco = get_subcontracting_order(
+ company="_Test Company with perpetual inventory",
+ warehouse=warehouse,
+ supplier_warehouse="Work In Progress - TCP1",
+ service_items=service_items,
+ )
+ rm_items = get_rm_items(sco.supplied_items)
+ itemwise_details = make_stock_in_entry(rm_items=rm_items)
+ make_stock_transfer_entry(
+ sco_no=sco.name,
+ rm_items=rm_items,
+ itemwise_details=copy.deepcopy(itemwise_details),
+ )
+ scr = make_subcontracting_receipt(sco.name)
+ scr.save()
+ scr.submit()
+
+ gl_entries = get_gl_entries("Subcontracting Receipt", scr.name)
+ self.assertTrue(gl_entries)
+
+ fg_warehouse_ac = get_inventory_account(scr.company, scr.items[0].warehouse)
+ expense_account = scr.items[0].expense_account
+ expected_values = {
+ fg_warehouse_ac: [1000, 1000],
+ expense_account: [1000, 1000],
+ }
+
+ for gle in gl_entries:
+ self.assertEqual(expected_values[gle.account][0], gle.debit)
+ self.assertEqual(expected_values[gle.account][1], gle.credit)
+
+ scr.reload()
+ scr.cancel()
+
def test_supplied_items_consumed_qty(self):
# Set Backflush Based On as "Material Transferred for Subcontracting" to transfer RM's more than the required qty
set_backflush_based_on("Material Transferred for Subcontract")