[19.0][MIG] account_asset_management: Migration to 19 by acysos · Pull Request #2206 · OCA/account-financial-tools

acysos · 2026-01-25T22:05:02Z

Migration to 19.0

add asset management modules asset mgt update redo synch asset mgt with recent V7 changes

[UPD] add places arg in assertAlmostEqual tests [UPD] flake clean

…ding page

…an't be modified if a move is linked with a depriciation line [IMP][account_asset_management] Define FIELDS_AFFETCS_ASSET_MOVE as a set directly

…iods for asset posting

…nalytic account

* Fix compute methods dependencies and small optimizations * Rename demo file to test and move it into the right folder

This is now supported natively by Odoo 11.

…sets

… multiple assets in invoice

* 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

…ar, account_analytic_id without domain

Currently translated at 40.1% (79 of 197 strings) Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_asset_management/pt_BR/

Currently translated at 100.0% (195 of 195 strings) Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_asset_management/fr/

Currently translated at 91.8% (179 of 195 strings) Translation: account-financial-tools-12.0/account-financial-tools-12.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-12-0/account-financial-tools-12-0-account_asset_management/es/

…ount_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.

…ent from extra method_time values If not, added methods won't hide these fields.

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_asset_management/

Currently translated at 100.0% (255 of 255 strings) Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_asset_management/tr/

Currently translated at 100.0% (255 of 255 strings) Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_asset_management/it/

Currently translated at 100.0% (255 of 255 strings) Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_asset_management/sv/

Currently translated at 100.0% (255 of 255 strings) Translation: account-financial-tools-18.0/account-financial-tools-18.0-account_asset_management Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-18-0/account-financial-tools-18-0-account_asset_management/ca/

…) to tests to reduce logs

acysos · 2026-01-26T09:34:30Z

Hello,

codevoc/project give red, but the report don't show nothing, only the percentages.
Can some one help us?

Thank you

pedrobaeza · 2026-01-27T09:00:22Z

/ocabot migration account_asset_management

You can see the coverage in https://app.codecov.io/gh/OCA/account-financial-tools/pull/2206?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=OCA, but that check is not enforced and can be red.

acysos · 2026-01-27T11:37:25Z

You can see the coverage in https://app.codecov.io/gh/OCA/account-financial-tools/pull/2206?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=checks&utm_campaign=pr+comments&utm_term=OCA, but that check is not enforced and can be red.

Hi, I saw the report. In other instances, it highlights the uncovered lines, but this time it isn't displaying anything specific. It only reports a coverage drop with no further details, and everything shows as 0.

pedrobaeza · 2026-01-27T11:46:12Z

Uhm, I see, can you do git commit --all --amend and then git push -f to trigger another build?

[MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0 [MIG] account_asset_management: Migration to 19.0

acysos · 2026-01-27T15:35:33Z

It's the same issue, the report is blank.

pedrobaeza · 2026-01-27T15:38:27Z

Definitively it should be a failure in Codecov or the CI triggers. Not sure if @sbidoul knows the internal of this.

acysos · 2026-01-27T15:51:33Z

I'm checking this PR OCA/account-financial-reporting#1413 and Codecov has the same issue. It seems to be a general problem.

Honeyxilia · 2026-02-04T09:07:14Z

Hello - we migrated from the v12.0 to v16.0 a BI report on Assets, which adds a view to see the revenue related to several assets from within Odoo c:

The BI report was only added after the v13.0 migration of the account_asset_management module, which was then migrated subsequently without it.
As such, we thought it was a shame it wasn't available in more recent versions, and we ported it to 16.0

Would you be interested in porting it to v19 within this PR ?
#2212

florian-dacosta · 2026-02-04T14:33:06Z

Hello @acysos Could you take a look on this one please ? #2160

lepistone and others added 30 commits January 25, 2026 20:25

move account_asset_management* out of __unported__

7d9f486

add asset management modules asset mgt update redo synch asset mgt with recent V7 changes

[FIX] test floats with assertAlmostEqual

24b35a5

[UPD] add places arg in assertAlmostEqual tests [UPD] flake clean

make *asset* installable again

c5f7fbf

[FIX] Inherited model is a transient model

00af399

[IMP] account_asset_management: use reload_on_button instead of reloa…

cb87b85

…ding page

[IMP][account_asset_management] Define a list of move's fields that c…

cb9074d

…an't be modified if a move is linked with a depriciation line [IMP][account_asset_management] Define FIELDS_AFFETCS_ASSET_MOVE as a set directly

[IMP] account_asset_management: code cosmetics

464e325

account_asset_management: group invoice lines fix

45fe9c4

analytic enhancement

15f8fcd

[ADD][account_asset_management] Add check on account move line

540de02

[FIX][account_asset_management] Add items on affects fields list

e43a433

[FIX] account_asset_management: do not propose special and closed per…

0b9e612

…iods for asset posting

[FIX][account_asset_management] Avoid to select closed or cancelled a…

f3c2ac8

…nalytic account

[UPD] prefix versions with 8.0

aa13841

[10.0][MIG]account_asset_management suite refactoring + 10.0 port

c7c7cf2

[FIX+IMP] account_asset_management: 2 things:

4d9eafc

* Fix compute methods dependencies and small optimizations * Rename demo file to test and move it into the right folder

[FIX] account_asset_management: Test data + onchange

a85f6b8

[MIG] account_asset_management: Migration to 11.0

9155fa0

[IMP] asset mgt: use exclude manifest key

6051adb

This is now supported natively by Odoo 11.

[MIG] account_asset_management: Migration to 12.0

ad401c8

account_asset_management: Add unit test for invoices with multiple as…

71d5185

…sets

Included PR OCA#828 Fix computation of depreciation lines when having…

1870567

… multiple assets in invoice

[12.0][FIX] account_asset_management: date_range -> account_fiscal_ye…

7214050

…ar, account_analytic_id without domain

[FIX] account_asset_management: Invisible conditions must be independ…

1185f78

…ent from extra method_time values If not, added methods won't hide these fields.

[FIX][12.0]asset_management - fix date string compare

0d3d08a

oca-ci and others added 13 commits January 25, 2026 20:25

[UPD] Update account_asset_management.pot

06aa765

[BOT] post-merge updates

696557d

[IMP] account_asset_management: Add @mute_logger("odoo.models.unlink"…

0b9853f

…) to tests to reduce logs

[BOT] post-merge updates

c7d1e8e

[IMP] account_asset_management: pre-commit auto fixes

57385e9

acysos mentioned this pull request Jan 25, 2026

Migration to version 19.0 #2155

Open

34 tasks

acysos force-pushed the 19.0-mig-account_asset_management branch 3 times, most recently from f699c80 to 9d1dc49 Compare January 25, 2026 22:54

pedrobaeza changed the title ~~19.0 mig account asset management~~ [19.0][MIG] account_asset_management: Migration to 19 Jan 27, 2026

OCA-git-bot added this to the 19.0 milestone Jan 27, 2026

acysos force-pushed the 19.0-mig-account_asset_management branch from 9d1dc49 to 78bea05 Compare January 27, 2026 15:00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[19.0][MIG] account_asset_management: Migration to 19#2206

[19.0][MIG] account_asset_management: Migration to 19#2206
acysos wants to merge 207 commits intoOCA:19.0from
acysos:19.0-mig-account_asset_management

acysos commented Jan 25, 2026

Uh oh!

acysos commented Jan 26, 2026

Uh oh!

pedrobaeza commented Jan 27, 2026

Uh oh!

acysos commented Jan 27, 2026

Uh oh!

pedrobaeza commented Jan 27, 2026

Uh oh!

acysos commented Jan 27, 2026

Uh oh!

pedrobaeza commented Jan 27, 2026

Uh oh!

acysos commented Jan 27, 2026

Uh oh!

Honeyxilia commented Feb 4, 2026 •

edited

Loading

Uh oh!

florian-dacosta commented Feb 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Uh oh!

Conversation

acysos commented Jan 25, 2026

Uh oh!

acysos commented Jan 26, 2026

Uh oh!

pedrobaeza commented Jan 27, 2026

Uh oh!

acysos commented Jan 27, 2026

Uh oh!

pedrobaeza commented Jan 27, 2026

Uh oh!

acysos commented Jan 27, 2026

Uh oh!

pedrobaeza commented Jan 27, 2026

Uh oh!

acysos commented Jan 27, 2026

Uh oh!

Honeyxilia commented Feb 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

florian-dacosta commented Feb 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Honeyxilia commented Feb 4, 2026 •

edited

Loading