Asset should be takes date from the invoice date rather than current date

[Ana Juaristi Olalde]

This one could be one requirement only in our country:

Creating the account move the taken data is:
http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/view/head:/account_asset/account_asset.py line 358

 depreciation_date = line.asset_id.prorata and line.asset_id.purchase_date or time.strftime('%Y-%m-%d')

This is the purchase data or the moment where user pushes the wizard button to calculate asset account_moves

In our country it should be simply:
depreciation_date = line.depreciation_date

That is the account movement data is the one set in account.asset.depreciation.line

[Ana Juaristi Olalde (ajuaristio)]

That is... the required changes would be:
http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/view/head:/account_asset/account_asset_invoice.py line 51

                vals = {
                    'name': line.name,
                    'code': line.invoice_id.number or False,
                    'category_id': line.asset_category_id.id,
                    'purchase_value': line.price_subtotal,
                    'period_id': line.invoice_id.period_id.id,
                    'partner_id': line.invoice_id.partner_id.id,
                    'company_id': line.invoice_id.company_id.id,
                    'currency_id': line.invoice_id.currency_id.id,
                }

ADD field:
                    'purchase_date' : line.invoice_id.date_invoice,

-----

On same point the default values of asset should be taken from asset category instead of being hardcoded on asset defaults:

                    'method': line.asset_category_id.method,
                    'method_number': line.asset_category_id.method_number,
                    'method_time': line.asset_category_id.method_time,
                    'method_period': line.asset_category_id.method_period,
                    'method_progress_factor': line.asset_category_id.method_progress_factor,
                    'method_end': line.asset_category_id.method_end,
                    'prorata': line.asset_category_id.prorata,

And... I think that are all little bugs I found in account_asset modules. Wishing it will be helpful:

Ana

[Amit Parik (amit-parik)]

Hello Ana,

I am totally agree with you, the date should be takes the purchase date (invoice date) instead of current date.

Currently it takes the current date. I have checked your suggestion and it works fine while setting
 'purchase_date' : line.invoice_id.date_invoice,

on asset_create method.

Thanks for reporting and valuable solution!

[Amit Parik (amit-parik)]

Hello Ana,

Sorry I missed to explain...your 2nd point.

To be continued....

I have checked your another point. Yes, you are right. The default values of asset should be taken from asset category i.e method, method_number etc... But currently it is working correct as you said, The assets entry created form invoice takes all this values based on asset category, It's not hard-corded.

So I am confirming this issue only for date issue.

Thanks and correct me If I am worng!

[Amit Parik (amit-parik)]

Hello Ana,

Thanks for the reporting!

This issue has been fixed on lp:~openerp-dev/openobject-addons/trunk-bug-935564-amp with following revision no and id.
rev no : 6679
6679 <email address hidden>

Thank you!

[Cristian Salamea (ovnicraft)]

Hello Amit, it was confirmed in 6.1 too ?

[Fabien (Open ERP) (fp-tinyerp)]

fixed in trunk.
Thanks for the good contribution.