schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: allow both custodian and location while creating asset (#36263)

Browse Source
This commit is contained in:
Anand Baburajan
2023-07-24 15:51:16 +05:30
committed by GitHub
parent b849f6c4d6
commit 2b47f5815e
1 changed files with 12 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -62,11 +62,12 @@ class AssetMovement(Document):
frappe.throw(_("Source and Target Location cannot be same")) frappe.throw(_("Source and Target Location cannot be same"))
if self.purpose == "Receipt": if self.purpose == "Receipt":
if not (d.source_location or d.from_employee) and not (d.target_location or d.to_employee): if not (d.source_location) and not (d.target_location or d.to_employee):
frappe.throw( frappe.throw(
_("Target Location or To Employee is required while receiving Asset {0}").format(d.asset) _("Target Location or To Employee is required while receiving Asset {0}").format(d.asset)
) )
elif d.from_employee and not d.target_location: elif d.source_location:
if d.from_employee and not d.target_location:
frappe.throw( frappe.throw(
_("Target Location is required while receiving Asset {0} from an employee").format(d.asset) _("Target Location is required while receiving Asset {0} from an employee").format(d.asset)
) )