From 323d8eaccdb213e12cdbdbde3fe8e8b24323c4ea Mon Sep 17 00:00:00 2001 From: ravibharathi656 Date: Wed, 8 Oct 2025 08:43:42 +0530 Subject: [PATCH] fix(asset movement): clear custodian if not present --- erpnext/assets/doctype/asset_movement/asset_movement.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/erpnext/assets/doctype/asset_movement/asset_movement.py b/erpnext/assets/doctype/asset_movement/asset_movement.py index d63492846c9..25a06e463bb 100644 --- a/erpnext/assets/doctype/asset_movement/asset_movement.py +++ b/erpnext/assets/doctype/asset_movement/asset_movement.py @@ -5,7 +5,7 @@ import frappe from frappe import _ from frappe.model.document import Document -from frappe.utils import get_link_to_form +from frappe.utils import cstr, get_link_to_form from erpnext.assets.doctype.asset_activity.asset_activity import add_asset_activity @@ -143,8 +143,8 @@ class AssetMovement(Document): def update_asset_location_and_custodian(self, asset_id, location, employee): asset = frappe.get_doc("Asset", asset_id) - if employee and employee != asset.custodian: - frappe.db.set_value("Asset", asset_id, "custodian", employee) + if cstr(employee) != asset.custodian: + frappe.db.set_value("Asset", asset_id, "custodian", cstr(employee)) if location and location != asset.location: frappe.db.set_value("Asset", asset_id, "location", location)