← Back to team overview

openerp-expert-accounting team mailing list archive

[Bug 381910] Re: account_asset: incorrect calculation of asset depreciation amount

 

Hello, I have decided to publish the account_asset module at current
stage.

Now it is in branch:
lp:~grzechu.g/openobject-addons/extra-5.0_account_asset

Current functionality:
    * Depreciation methods:
        - Straight-Line
        - Declining-Balance
        - Sum of Year Digits
        - Units of Production - this method can be used for individual depreciation schedule.
        - Progressive
    * Default accounts and other parameters for method types and categories
    * Method Parameters:
	- Starting interval (period)
	- Number of depreciation intervals
	- Number of intervals per year
	- Progressive factor for Declining-Balance method
	- Salvage Value
	- Life Quantity for Unit of Production method
        - Number of intervals calculated before current system (for continuing depreciation from other system)
    * Functionality:
	- Defining the asset in invoice when purchasing
	- Adjusting the asset value in purchasing Refund
	- Periodical entering of units of production
	- Periodical asset depreciation calculation
	- Sales invoice stops the depreciation and make final postings
        - Tracking changes in Asset History
    * Wizards:
	- Initial Values for continuing depreciation from previous system or other purpose
	- Revaluation
	- Abandonment
	- Method parameters changing
	- Suppressing and Resuming depreciation
	- Localisation changing
     * Postings direct and indirect
You can find quite big users guide in module description.

Known wishes not implemented:
- Calculating depreciation from given date (currently depreciation are calculated for whole periods/intervals only)
- Depreciation rounding
- Automatic tax depreciation starting (you can start tax depreciation manually with or without Initial Values wizard).
Above wishes are not used in my country and I don't think I can understand the needs correctly. I think these features should be developed by someone feeling better that nuances. 
- Import wizard
This feature is needed as many system introductions will have to continue previous depreciations. But I am blind yet in importing. So you are invited to continue development.
- Multicompany
I can implement it for 6.0

Everyone is welcome to test and evaluate the module. According to your
response I will fix bugs or make very small and safe improvements. If
there is no critical response I will merge module back to extra-addons
5.0 in a week. I didn't keep any backward compatibility but I don't
think it is problem as original module was unusable in any production
introduction.

All the best
Grzegorz Grzelak

-- 
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: In Progress

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