From db2e2105ab02cbd7bda366c55f3aa744d9e91d49 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Fri, 19 Jun 2026 14:37:32 +0530 Subject: [PATCH] fix: Use get_value instead of get_doc Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> --- erpnext/projects/doctype/project/project.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/erpnext/projects/doctype/project/project.py b/erpnext/projects/doctype/project/project.py index 211fcfc8afe..5f82b2257c4 100644 --- a/erpnext/projects/doctype/project/project.py +++ b/erpnext/projects/doctype/project/project.py @@ -252,17 +252,17 @@ class Project(Document): if not self.sales_order: return - sales_order = frappe.get_doc("Sales Order", self.sales_order) - if sales_order.project and sales_order.project != self.name: + existing_project = frappe.db.get_value("Sales Order", self.sales_order, "project") + if existing_project and existing_project != self.name: frappe.msgprint( _("Sales Order {0} is already linked to Project {1}, skipping the link.").format( - self.sales_order, sales_order.project + self.sales_order, existing_project ), alert=True, ) return - sales_order.db_set("project", self.name) + frappe.db.set_value("Sales Order", self.sales_order, "project", self.name) def on_trash(self): frappe.db.set_value("Sales Order", {"project": self.name}, "project", "")