From edd3f5da1ce9bd2ca9bd46707e94ad37348073bc Mon Sep 17 00:00:00 2001 From: KerollesFathy Date: Wed, 10 Sep 2025 17:57:30 +0000 Subject: [PATCH 1/2] fix: correct typo in asset movement purpose validation (cherry picked from commit 56da3bd2e4d92f57e2bbf6562051aeedc17239ac) --- erpnext/assets/doctype/asset_movement/asset_movement.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/erpnext/assets/doctype/asset_movement/asset_movement.py b/erpnext/assets/doctype/asset_movement/asset_movement.py index db4e7510670..5514c6a512a 100644 --- a/erpnext/assets/doctype/asset_movement/asset_movement.py +++ b/erpnext/assets/doctype/asset_movement/asset_movement.py @@ -79,7 +79,7 @@ class AssetMovement(Document): ) def validate_employee(self, d): - if self.purpose == "Tranfer and Issue": + if self.purpose == "Transfer and Issue": if not d.from_employee: frappe.throw(_("From Employee is required while issuing Asset {0}").format(d.asset)) From 97765ce8bd7fc31269259db817b101d6638cad2e Mon Sep 17 00:00:00 2001 From: KerollesFathy Date: Wed, 10 Sep 2025 18:00:44 +0000 Subject: [PATCH 2/2] fix: correct grammatical errors in asset movement validation messages (cherry picked from commit 5f083d55b5aa255e6fb963be8828779750335347) --- erpnext/assets/doctype/asset_movement/asset_movement.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/erpnext/assets/doctype/asset_movement/asset_movement.py b/erpnext/assets/doctype/asset_movement/asset_movement.py index 5514c6a512a..611f2c44cfd 100644 --- a/erpnext/assets/doctype/asset_movement/asset_movement.py +++ b/erpnext/assets/doctype/asset_movement/asset_movement.py @@ -61,7 +61,7 @@ class AssetMovement(Document): if d.source_location: if current_location != d.source_location: frappe.throw( - _("Asset {0} does not belongs to the location {1}").format(d.asset, d.source_location) + _("Asset {0} does not belong to the location {1}").format(d.asset, d.source_location) ) else: d.source_location = current_location @@ -75,7 +75,7 @@ class AssetMovement(Document): frappe.throw(_("Target Location is required while receiving Asset {0}").format(d.asset)) if d.to_employee and frappe.db.get_value("Employee", d.to_employee, "company") != self.company: frappe.throw( - _("Employee {0} does not belongs to the company {1}").format(d.to_employee, self.company) + _("Employee {0} does not belong to the company {1}").format(d.to_employee, self.company) ) def validate_employee(self, d): @@ -88,7 +88,7 @@ class AssetMovement(Document): if current_custodian != d.from_employee: frappe.throw( - _("Asset {0} does not belongs to the custodian {1}").format(d.asset, d.from_employee) + _("Asset {0} does not belong to the custodian {1}").format(d.asset, d.from_employee) ) if not d.to_employee: @@ -96,7 +96,7 @@ class AssetMovement(Document): if d.to_employee and frappe.db.get_value("Employee", d.to_employee, "company") != self.company: frappe.throw( - _("Employee {0} does not belongs to the company {1}").format(d.to_employee, self.company) + _("Employee {0} does not belong to the company {1}").format(d.to_employee, self.company) ) def on_submit(self):