mirror of
https://github.com/frappe/erpnext.git
synced 2026-05-31 10:49:09 +00:00
fix: Only allow posting of Depreciation Entries if the Asset is in the Submitted or Partially Depreciated state
This commit is contained in:
@@ -34,6 +34,8 @@ def make_depreciation_entry(asset_name, date=None):
|
|||||||
date = today()
|
date = today()
|
||||||
|
|
||||||
asset = frappe.get_doc("Asset", asset_name)
|
asset = frappe.get_doc("Asset", asset_name)
|
||||||
|
validate_asset(asset)
|
||||||
|
|
||||||
fixed_asset_account, accumulated_depreciation_account, depreciation_expense_account = \
|
fixed_asset_account, accumulated_depreciation_account, depreciation_expense_account = \
|
||||||
get_depreciation_accounts(asset)
|
get_depreciation_accounts(asset)
|
||||||
|
|
||||||
@@ -101,6 +103,10 @@ def make_depreciation_entry(asset_name, date=None):
|
|||||||
|
|
||||||
return asset
|
return asset
|
||||||
|
|
||||||
|
def validate_asset(asset):
|
||||||
|
if asset.status not in ['Submitted', 'Partially Depreciated']:
|
||||||
|
frappe.throw(_("Cannot depreciate {0} Asset").format(asset.status))
|
||||||
|
|
||||||
def get_depreciation_accounts(asset):
|
def get_depreciation_accounts(asset):
|
||||||
fixed_asset_account = accumulated_depreciation_account = depreciation_expense_account = None
|
fixed_asset_account = accumulated_depreciation_account = depreciation_expense_account = None
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user