[ADD] fix_compute_trans_implied_groups

[SirPyTech]

This fix should be in Odoo's 14.0 branch but it's no more maintained, so we have to create a module.
This isn't needed in 16.0 thanks to odoo/odoo@34d6f87.

I haven't added a test in this module because the only way I know to reproduce this bug is to install the sale module and it makes no sense to add such a dependency in this module.

Let me know what you think!

[GSLabIt]

LGTM

[HekkiMelody]

LGTM.

Ideally this fix should be added to OCB, but I understand that not everyone uses it.

[francesco-ooops]

@OCA/tools-maintainers wdyt?

[thomaspaulb]

@SirPyTech For the unit test what you could do is to create a separate _test module which has the test and also depends on sale.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[thomaspaulb]

Also, can you explain your reason for not using OCB?

[SirPyTech]

Thanks @thomaspaulb for having a look!
I'll try to move this fix to OCB and then maybe I'll be able to add a proper test in sale module 🚀
I didn't use OCB at first because I thought our customer wasn't using it.

[SirPyTech]

Moved to OCA/OCB#1306

[SirPyTech]

Reopening to allow non-OCB users to enjoy this fix

[thomaspaulb]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-3320-by-thomaspaulb-bump-nobump, awaiting test results.

[OCA-git-bot]

@thomaspaulb your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3320-by-thomaspaulb-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[SirPyTech]

/ocabot merge nobump

Thanks @thomaspaulb for trying to merge!
The module letsencrypt is failing a test only in the OCB execution (see https://github.com/OCA/server-tools/actions/runs/21340400922/job/61418864454#step:8:1115), it is probably temporary, please try again 🙏

[thomaspaulb]

/ocabot merge nobump

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3320-by-thomaspaulb-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 57b21e9. Thanks a lot for contributing to OCA. ❤️