Skip to content

[16.0][FIX] purchase_order_ubl: skip section and note lines during UBL generation - #1427

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16-fix-purchase_order_ubl-filter-po-note-lines
Oct 16, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:16.0from
acsone:16-fix-purchase_order_ubl-filter-po-note-lines

Conversation

@nicolas-delbovier-acsone

@nicolas-delbovier-acsone nicolas-delbovier-acsone commented Oct 12, 2026 •

Copy link
Copy Markdown

Note (display_type = 'line_note') and Section (display_type = 'line_section')
lines on purchase orders do not contain products or units of measure.

Including them when building UBL XML causes a UserError due to oline.product_uom
evaluating to False.

Reproduce the error

Create a PO with a "note" or "section" line

Create the PO The error
image image

@OCA-git-bot OCA-git-bot added series:16.0 mod:purchase_order_ubl Module purchase_order_ubl labels Oct 12, 2026
@nicolas-delbovier-acsone
nicolas-delbovier-acsone force-pushed the 16-fix-purchase_order_ubl-filter-po-note-lines branch from 0fa9766 to 1f225ae Compare October 13, 2026 06:35

@simahawk simahawk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

to be fwd port to 18 I guess

@simahawk

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1427-by-simahawk-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit dd066ac into OCA:16.0 Oct 16, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@lmignon
lmignon deleted the 16-fix-purchase_order_ubl-filter-po-note-lines branch October 20, 2026 15:27
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