I think this is incorrect. It can lead to a weird situation where it looks like assets have lived beyond their lifetime, when really they've essentially been decommissioned before the next milestone year has begun. Let's fix this to make the output more consistent for users.
One way we could implement this is to change Asset::decommission() so that it will use either the asset's max_decommission_year() or the year argument, depending on which is lower.
I think this is incorrect. It can lead to a weird situation where it looks like assets have lived beyond their lifetime, when really they've essentially been decommissioned before the next milestone year has begun. Let's fix this to make the output more consistent for users.
One way we could implement this is to change
Asset::decommission()so that it will use either the asset'smax_decommission_year()or theyearargument, depending on which is lower.