Skip to content

[18.0][IMP] account_brand: compute account_id at move_line creation#282

Open
qgroulard wants to merge 1 commit intoOCA:18.0from
acsone:18.0-imp-account_brand_create_move_line-qgr
Open

[18.0][IMP] account_brand: compute account_id at move_line creation#282
qgroulard wants to merge 1 commit intoOCA:18.0from
acsone:18.0-imp-account_brand_create_move_line-qgr

Conversation

@qgroulard
Copy link
Copy Markdown
Contributor

This module has changed a lot between Odoo versions, especially in version 13.0 and 16.0 where the method _recompute_payment_terms_lines() has been added then removed.
I believe some features got lost on the way.

Today I don't understand how it is supposed to work anymore. For me:

  1. The account should be computed when a new move line is added, not only when the onchange triggers on the move.
  2. The onchange on the move should also trigger if the brand or the company changes (it was the case until migration to 13.0).

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @osi-scampbell, @sbejaoui,
some modules you are maintaining are being modified, check this out!

@qgroulard qgroulard force-pushed the 18.0-imp-account_brand_create_move_line-qgr branch from a91f747 to e16f543 Compare November 24, 2025 12:40
This module has changed a lot between Odoo versions, some features got lost on the way.
The account should be computed when a new move line is added, not only when the onchange triggers on the move.
The onchange on the move should trigger if the brand or the company changes.
@qgroulard qgroulard force-pushed the 18.0-imp-account_brand_create_move_line-qgr branch from e16f543 to b44202a Compare November 24, 2025 12:46
@github-actions
Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 29, 2026
Copy link
Copy Markdown
Member

@samirGuesmi samirGuesmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code review: LGTM

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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). 🤖

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants