From ee41e55343166f88cd105fd608a3792d0fb84bb0 Mon Sep 17 00:00:00 2001
From: Mihir Kandoi <kandoimihir@gmail.com>
Date: Mon, 17 Feb 2025 18:05:58 +0530
Subject: [PATCH] perf: patch

(cherry picked from commit a41024813b22fea5d737be8fab50224c2b909cbc)
---
 .../recalculate_amount_difference_field.py    | 81 +++++++++++++------
 1 file changed, 57 insertions(+), 24 deletions(-)

diff --git a/erpnext/stock/doctype/purchase_receipt_item/patches/recalculate_amount_difference_field.py b/erpnext/stock/doctype/purchase_receipt_item/patches/recalculate_amount_difference_field.py
index e2d8bedaa66..9def5aa8d3b 100644
--- a/erpnext/stock/doctype/purchase_receipt_item/patches/recalculate_amount_difference_field.py
+++ b/erpnext/stock/doctype/purchase_receipt_item/patches/recalculate_amount_difference_field.py
@@ -1,37 +1,70 @@
 import frappe
+from frappe.query_builder.functions import Sum
 from frappe.utils import flt
 
-from erpnext.stock.doctype.purchase_receipt.purchase_receipt import (
-	adjust_incoming_rate_for_pr,
-	get_billed_qty_against_purchase_receipt,
-)
+from erpnext.accounts.utils import get_fiscal_year
+from erpnext.stock.doctype.purchase_receipt.purchase_receipt import adjust_incoming_rate_for_pr
 
 
 def execute():
+	fiscal_year_dates = get_fiscal_year(frappe.utils.datetime.date.today())
 	table = frappe.qb.DocType("Purchase Receipt Item")
+	parent = frappe.qb.DocType("Purchase Receipt")
 	query = (
 		frappe.qb.from_(table)
-		.select(table.parent)
-		.distinct()
-		.where((table.amount_difference_with_purchase_invoice > 0) & (table.docstatus == 1))
+		.join(parent)
+		.on(table.parent == parent.name)
+		.select(
+			table.parent,
+			table.name,
+			table.amount,
+			table.billed_amt,
+			table.amount_difference_with_purchase_invoice,
+			table.rate,
+			table.qty,
+			parent.conversion_rate,
+		)
+		.where(
+			(table.amount_difference_with_purchase_invoice != 0)
+			& (table.docstatus == 1)
+			& (parent.posting_date.between(fiscal_year_dates[1], fiscal_year_dates[2]))
+		)
 	)
-	pr_names = [item.parent for item in query.run(as_dict=True)]
+	result = query.run(as_dict=True)
 
-	for pr_name in pr_names:
-		pr_doc = frappe.get_doc("Purchase Receipt", pr_name)
-		for item in pr_doc.items:
-			adjusted_amt = 0.0
-			item_wise_billed_qty = get_billed_qty_against_purchase_receipt(pr_doc)
+	item_wise_billed_qty = get_billed_qty_against_purchase_receipt([item.name for item in result])
 
-			if (
-				item.billed_amt is not None
-				and item.amount is not None
-				and item_wise_billed_qty.get(item.name)
-			):
-				adjusted_amt = (
-					flt(item.billed_amt / item_wise_billed_qty.get(item.name)) - flt(item.rate)
-				) * item.qty
+	for item in result:
+		adjusted_amt = 0.0
 
-			adjusted_amt = flt(adjusted_amt * flt(pr_doc.conversion_rate), item.precision("amount"))
-			item.db_set("amount_difference_with_purchase_invoice", adjusted_amt, update_modified=False)
-		adjust_incoming_rate_for_pr(pr_doc)
+		if item.billed_amt is not None and item.amount is not None and item_wise_billed_qty.get(item.name):
+			adjusted_amt = (
+				flt(item.billed_amt / item_wise_billed_qty.get(item.name)) - flt(item.rate)
+			) * item.qty
+		adjusted_amt = flt(
+			adjusted_amt * flt(item.conversion_rate), frappe.get_precision("Purchase Receipt Item", "amount")
+		)
+
+		if adjusted_amt != item.amount_difference_with_purchase_invoice:
+			frappe.db.set_value(
+				"Purchase Receipt Item",
+				item.name,
+				"amount_difference_with_purchase_invoice",
+				adjusted_amt,
+				update_modified=False,
+			)
+			adjust_incoming_rate_for_pr(frappe.get_doc("Purchase Receipt", item.parent))
+
+
+def get_billed_qty_against_purchase_receipt(pr_names):
+	table = frappe.qb.DocType("Purchase Invoice Item")
+	query = (
+		frappe.qb.from_(table)
+		.select(table.pr_detail, Sum(table.qty).as_("qty"))
+		.where((table.pr_detail.isin(pr_names)) & (table.docstatus == 1))
+	)
+	invoice_data = query.run(as_list=1)
+
+	if not invoice_data:
+		return frappe._dict()
+	return frappe._dict(invoice_data)