2
0

Merge PR #1820 into 16.0

Signed-off-by pedrobaeza
This commit is contained in:
OCA-git-bot 2024-02-16 08:36:27 +00:00
commit 39c092cca6
4 changed files with 67 additions and 6 deletions

View File

@ -66,6 +66,9 @@ class AccountAsset(models.Model):
"\nPurchase Value - Salvage Value.",
)
salvage_value = fields.Monetary(
compute="_compute_salvage_value",
store=True,
readonly=False,
states=READONLY_STATES,
help="The estimated value that an asset will realize upon "
"its sale at the end of its useful life.\n"
@ -282,6 +285,18 @@ class AccountAsset(models.Model):
asset.depreciation_line_ids.filtered("move_id")
)
def _get_salvage_value_profile(self):
self.ensure_one()
salvage_value = self.profile_id.salvage_value
if self.profile_id.salvage_type == "percent":
salvage_value = (salvage_value / 100) * self.purchase_value
return salvage_value
@api.depends("profile_id")
def _compute_salvage_value(self):
for asset in self:
asset.salvage_value = asset._get_salvage_value_profile()
@api.depends("purchase_value", "salvage_value", "method")
def _compute_depreciation_base(self):
for asset in self:
@ -425,13 +440,7 @@ class AccountAsset(models.Model):
@api.model_create_multi
def create(self, vals_list):
asset_ids = super().create(vals_list)
create_asset_from_move_line = self.env.context.get(
"create_asset_from_move_line"
)
for asset_id in asset_ids:
if create_asset_from_move_line:
# Trigger compute of depreciation_base
asset_id.salvage_value = 0.0
asset_id._create_first_asset_line()
return asset_ids

View File

@ -74,6 +74,15 @@ class AccountAssetProfile(models.Model):
check_company=True,
string="Asset Groups",
)
salvage_value = fields.Float(
digits="Account",
help="The estimated value that an asset will realize upon "
"its sale at the end of its useful life.\n"
"This value is used to determine the depreciation amounts.",
)
salvage_type = fields.Selection(
selection=[("fixed", "Fixed"), ("percent", "Percentage of Price")]
)
method = fields.Selection(
selection=lambda self: self._selection_method(),
string="Computation Method",

View File

@ -955,3 +955,30 @@ class TestAssetManagement(AccountTestInvoicingCommon):
last_line.create_move()
self.assertEqual(asset.value_residual, 0)
self.assertEqual(asset.state, "close")
def test_21_asset_profile_salvage_value(self):
"""Compute salvage value from asset profile."""
# Case percent
self.car5y.salvage_type = "percent"
self.car5y.salvage_value = 5
asset = self.asset_model.create(
{
"name": "test asset",
"profile_id": self.car5y.id,
"purchase_value": 1000,
"date_start": time.strftime("%Y-07-07"),
}
)
self.assertEqual(asset.salvage_value, 50)
# Case fixed amount
self.car5y.salvage_type = "fixed"
self.car5y.salvage_value = 5
asset = self.asset_model.create(
{
"name": "test asset",
"profile_id": self.car5y.id,
"purchase_value": 1000,
"date_start": time.strftime("%Y-07-07"),
}
)
self.assertEqual(asset.salvage_value, 5)

View File

@ -20,6 +20,22 @@
</div>
<group>
<group>
<label for="salvage_value" />
<div>
<field
name="salvage_value"
class="oe_inline"
nolabel="1"
/>
<span
class="o_form_label oe_inline"
attrs="{'invisible':[('salvage_type','!=','percent')]}"
>%</span>
</div>
<field
name="salvage_type"
attrs="{'required': [('salvage_value', '!=', 0.0)]}"
/>
<field name="company_id" invisible="1" />
<field name="group_ids" widget="many2many_tags" />
<field name="asset_product_item" />