[FIX] base: trans_implied_ids recursive computation - #1306
Conversation
1. Install `sale` module 2. In the Odoo shell execute: `env["res.groups"].get_groups_by_application()` *Actual result* `[..., (ir.module.category(52,), 'boolean', res.groups(22, 21, 20), (100, 'Other')) , ...]` *Expected result* `[..., (ir.module.category(52,), 'selection', res.groups(20, 21, 22), (5, 'Sales')) , ...]` *Additional info* The result of `get_groups_by_application` is correct if the computation of `res.groups.trans_implied_ids` is called executing: `env["res.groups"].search([])._compute_trans_implied()`
|
@thomaspaulb here you go, let me know what you think! |
|
This PR has the |
|
@thomaspaulb can this be merged? thanks! |
|
/ocabot merge patch |
|
not sure if I have rights here |
|
This PR looks fantastic, let's merge it! |
I believe that if you have the green check ✔️ in Thanks for merging! |
|
@thomaspaulb can you please relaunch merge command? thanks! |
|
/ocabot merge patch |
|
This PR looks fantastic, let's merge it! |
|
@gurneyalex any idea why the bot is not completing the merge? thanks! |
|
@OCA/backport-maintainers could this be tried again? |
|
/ocabot merge patch |
|
What a great day to merge this nice PR. Let's do it! |
|
ocabot is not working in OCB. You have to manually merge. |
Steps
salemoduleenv["res.groups"].get_groups_by_application()Actual result
[..., (ir.module.category(52,), 'boolean', res.groups(22, 21, 20), (100, 'Other')) , ...]Expected result
[..., (ir.module.category(52,), 'selection', res.groups(20, 21, 22), (5, 'Sales')) , ...]Additional info
The result of
get_groups_by_applicationis correct if the computation ofres.groups.trans_implied_idsis called executing:env["res.groups"].search([])._compute_trans_implied()Moved here from OCA/server-tools#3320.
I tried to add a test in
salemodule but didn't manage to add a test that would fail without this change.This is already present in
15.0and following versions of Odoo thanks to odoo@34d6f87.