nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-27 08:54:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #49522 from KerollesFathy/clear-custodian-when-asset-take-backed

Browse Source 
fix: clear asset custodian when asset take back from employee without assign to another employee
This commit is contained in:
Khushi Rawat
2025-10-02 22:16:35 +05:30
committed by GitHub
parent b91fb6b280 4bc76be130
commit 09bcff13ad
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
erpnext/assets/doctype/asset_movement/asset_movement.py
View File

@@ -142,10 +142,18 @@ class AssetMovement(Document):
def update_asset_location_and_custodian(self, asset_id, location, employee): def update_asset_location_and_custodian(self, asset_id, location, employee):
asset = frappe.get_doc("Asset", asset_id) asset = frappe.get_doc("Asset", asset_id)
updates = {}
if employee and employee != asset.custodian: if employee and employee != asset.custodian:
frappe.db.set_value("Asset", asset_id, "custodian", employee) updates["custodian"] = employee
elif not employee and asset.custodian:
updates["custodian"] = ""
if location and location != asset.location: if location and location != asset.location:
frappe.db.set_value("Asset", asset_id, "location", location) updates["location"] = location
if updates:
frappe.db.set_value("Asset", asset_id, updates)
def log_asset_activity(self, asset_id, location, employee): def log_asset_activity(self, asset_id, location, employee):
if location and employee: if location and employee: