schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: invalid logic for cache hit

Browse Source 
if there are no inventory dimensions then `[]` gets treated as a miss.
This commit is contained in:
Ankush Menat
2025-06-04 17:13:41 +05:30
parent a36daec939
commit 8e17054b67
2 changed files with 17 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
erpnext/stock/doctype/inventory_dimension/inventory_dimension.py
View File

@@ -364,28 +364,21 @@ def get_evaluated_inventory_dimension(doc, sl_dict, parent_doc=None):
return filter_dimensions
@request_cache
def get_document_wise_inventory_dimensions(doctype) -> dict:
if not hasattr(frappe.local, "document_wise_inventory_dimensions"):
frappe.local.document_wise_inventory_dimensions = {}
if not frappe.local.document_wise_inventory_dimensions.get(doctype):
dimensions = frappe.get_all(
"Inventory Dimension",
fields=[
"name",
"source_fieldname",
"condition",
"target_fieldname",
"type_of_transaction",
"fetch_from_parent",
],
filters={"disabled": 0},
or_filters={"document_type": doctype, "apply_to_all_doctypes": 1},
)
frappe.local.document_wise_inventory_dimensions[doctype] = dimensions
return frappe.local.document_wise_inventory_dimensions[doctype]
return frappe.get_all(
"Inventory Dimension",
fields=[
"name",
"source_fieldname",
"condition",
"target_fieldname",
"type_of_transaction",
"fetch_from_parent",
],
filters={"disabled": 0},
or_filters={"document_type": doctype, "apply_to_all_doctypes": 1},
)
@frappe.whitelist()

5
erpnext/stock/doctype/inventory_dimension/test_inventory_dimension.py
View File

@@ -87,7 +87,7 @@ class TestInventoryDimension(IntegrationTestCase):
self.assertFalse(custom_field)
def test_inventory_dimension(self):
frappe.local.document_wise_inventory_dimensions = {}
frappe.clear_cache(doctype="Inventory Dimension")
warehouse = "Shelf Warehouse - _TC"
item_code = "_Test Item"
@@ -159,7 +159,8 @@ class TestInventoryDimension(IntegrationTestCase):
self.assertRaises(DoNotChangeError, inv_dim1.save)
def test_inventory_dimension_for_purchase_receipt_and_delivery_note(self):
frappe.local.document_wise_inventory_dimensions = {}
frappe.clear_cache(doctype="Inventory Dimension")
frappe.clear_cache("Inventory Dimension")
inv_dimension = create_inventory_dimension(
reference_document="Rack", dimension_name="Rack", apply_to_all_doctypes=1