2
0
Commit Graph

68 Commits

Author SHA1 Message Date
Pedro M. Baeza
a16a456511 [FIX] account_asset_management: Avoid error
Steps to reproduce the problem:

* Go to assets view
* Group by profile
* Unfold a group and click on an asset
* Click on "Journal Entries" smart-button
* Go back to the asset list
* Click again on the same asset (or another).
* Click on "Journal Entries" smart-button

Current behavior:

Error saying "KeyError: 'profile_id'"

Expected behavior:

No error

The cause for this is that Odoo stores in the context the key `group_by` with the
value `profile_id` in the specified chain of steps. That context entry is used for
grouping records in the list, and system tries to group the journal entries also
by that field, which doesn't exists in the other model, and thus the error.

We avoided it copying the context to be passes and leaving out that entry.
2022-12-24 00:42:21 +01:00
ernestotejeda
7e9d38ef5e [MIG] account_asset_management: Migration to 13.0 2022-12-24 00:42:21 +01:00
ernestotejeda
bc3e0f8fa6 [IMP] account_asset_management: black, isort 2022-12-24 00:42:21 +01:00
Stéphane Valaeys
44c9368b2e Fixed dynamic build of the name of the view to be called when errors are found 2022-12-24 00:42:21 +01:00
Luc De Meyer
cd3bd6bbd2 [12.0][FIX]traceback when deviating FY 2022-12-24 00:42:21 +01:00
Pedro M. Baeza
ee1087d1b4 [MIG] account_asset_management: Handle account_asset_analytic migration
Only field renaming is needed.
2022-12-24 00:42:21 +01:00
Saran
181d7f2f83 [12.0][IMP] account_asset_management 2022-12-24 00:42:21 +01:00
Pedro M. Baeza
dab2a897b4 [MIG+FIX+IMP] account_asset_management: Rework migration scripts + make asset group m2m 2022-12-24 00:42:21 +01:00
Henrik Norlin
05e3fa1033 [IMP] removed type, parent_id, parent_path, child_ids from account.asset & account.asset.profile 2022-12-24 00:42:21 +01:00
Pedro M. Baeza
095a205ab1 [FIX] account_asset_management: Don't depend on account.fiscal.year created records
* There's no need of forcing to create such records if your fiscal years are regular ones.
* FY date range computation duplicates code that is already on core.
* DummyFY is a good idea, but was not used at all. Now it is.
2022-12-24 00:42:21 +01:00
Luc De Meyer
fabe61b412 [FIX][12.0]asset_management - fix date string compare 2022-12-24 00:42:21 +01:00
Pedro M. Baeza
66f3c3198a [FIX] account_asset_management: Invisible conditions must be independent from extra method_time values
If not, added methods won't hide these fields.
2022-12-24 00:42:21 +01:00
Pedro M. Baeza
68509d7386 [FIX] account_asset_management: Provide hook _compute_depreciation_amount_per_fiscal_year
This way, other method_time mechanism like localization ones is able to overwrite it.

You also got an error of variable `number` not set due to previous code. Although no
alternate implementation for a new method_time, now you don't have any error,
fallbacking to standard proportional repartition.
2022-12-24 00:42:21 +01:00
Bejaoui Souheil
8fa0de5226 Finish 12.0 migration of account_asset_management
* account_asset: Do not loop on all the lines to search for one linked asset

Before this change, the use of `mapped` on self did loop on all the move
lines that are included in self to get the assets, what could be very
costly for a simple write on a lot of move lines. As the goal is to raise
an error only if at least one move is linked to an asset, we break the
loop if the condition is fulfilled.

* performance improvement

* [RMV] - Remove useless dependency

In 12.0 account_fiscal_year is a standard feature no need to depend on oca
module account_fiscal_year
2022-12-24 00:42:21 +01:00
Henrik Norlin
886abfe610 [MIG] account_asset_management: Migration to 12.0 2022-12-24 00:42:21 +01:00
Stéphane Bidoul (ACSONE)
0cece3a4b7 [IMP] asset mgt: use exclude manifest key
This is now supported natively by Odoo 11.
2022-12-24 00:42:20 +01:00
Akim Juillerat
a7cdfdd14d [MIG] account_asset_management: Migration to 11.0 2022-12-24 00:42:20 +01:00
Luc De Meyer
7c76c17231 [10.0][MIG]account_asset_management suite refactoring + 10.0 port 2022-12-24 00:42:20 +01:00