openerp-expert-accounting team mailing list archive
-
openerp-expert-accounting team
-
Mailing list archive
-
Message #00515
[Bug 381910] Re: account_asset: incorrect calculation of asset depreciation amount
Hello, all my improvements account_asset are merged back to stable_5.0-extra-addons. So I close this bug and relevant blueprint. Farther development is of course welcome and needed.
Known issue: Duplicating of asset works bad.
By the way: I think modules should have not just field of author but
some list of authors to expand it during development.
Thank you
** Changed in: openobject-addons
Status: In Progress => Fix Released
--
account_asset: incorrect calculation of asset depreciation amount
https://bugs.launchpad.net/bugs/381910
You received this bug notification because you are a member of OpenERP
Accounting Experts, which is a direct subscriber.
Status in OpenObject Addons Modules: Fix Released
Bug description:
The function Financial Management/Periodical Processing/Compute assets calculate asset depreciation amount incorrectly: instead of subtracting already-depreciated amounts before calculating amount for 2nd, 3rd etc months, the algorithm adds already-depreciated amounts to asset cost.
For example,
Asset 1 costs $1000
Depreciation method is progressive - 15% per period
For first period the algorithm create an account move in $150.
For second period the algorithm creates a move in $172.5 (10% from $1000 + $150)
After short investigation I've found a bug in the file account_asset.py in line 118:
is
114 for move in property.asset_id.entry_ids:
115 total += move.debit-move.credit
116 for move in property.entry_asset_ids:
117 if move.account_id == property.account_asset_ids:
118 total += move.debit-move.credit
should be
114 for move in property.asset_id.entry_ids:
115 total += move.debit-move.credit
116 for move in property.entry_asset_ids:
117 if move.account_id == property.account_asset_ids:
118 total -= move.debit-move.credit
Regards,
Michael