Skip to content

[18.0][FIX] purchase_allowed_product: avoid parent in line form#3082

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
fsmw:18.0-fix-purchase-allowed-product-parent-context
Jun 8, 2026
Merged

[18.0][FIX] purchase_allowed_product: avoid parent in line form#3082
OCA-git-bot merged 1 commit into
OCA:18.0from
fsmw:18.0-fix-purchase-allowed-product-parent-context

Conversation

@fsmw
Copy link
Copy Markdown
Contributor

@fsmw fsmw commented May 30, 2026

This fixes the standalone form crash in when the field context is evaluated without an x2many parent scope.

Cause

  • the inherited standalone form view used in the context
  • standalone move line forms do not define, so the web client raised
    Solution
  • replace references with in the standalone move line form view
  • add a regression test asserting the merged form architecture uses and no longer references

Refs #3070

@OCA-git-bot OCA-git-bot added series:18.0 mod:purchase_allowed_product Module purchase_allowed_product labels May 30, 2026
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

OK for changing parent to move_id, but please don't add a test that is not adding value.

@pedrobaeza pedrobaeza added this to the 18.0 milestone Jun 1, 2026
@fsmw fsmw force-pushed the 18.0-fix-purchase-allowed-product-parent-context branch from 8d23dba to 7a466b8 Compare June 3, 2026 23:02
@fsmw
Copy link
Copy Markdown
Contributor Author

fsmw commented Jun 3, 2026

@pedrobaeza thanks for the review — done. Dropped the new test in 7a466b8; only the XML fix remains. Could you re-review when you have a moment?

Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

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

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-3082-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 58f6549 into OCA:18.0 Jun 8, 2026
9 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

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