Skip to content

[16.0][FIX] sale_order_report_zero_qty: handle section/note lines#899

Open
deeniiz wants to merge 1 commit into
16.0from
16.0-fix-sale_order_report_zero_qty-section_note_lines
Open

[16.0][FIX] sale_order_report_zero_qty: handle section/note lines#899
deeniiz wants to merge 1 commit into
16.0from
16.0-fix-sale_order_report_zero_qty-section_note_lines

Conversation

@deeniiz
Copy link
Copy Markdown
Collaborator

@deeniiz deeniiz commented May 4, 2026

Section and note lines (display_type) have no product_uom, so product_uom.rounding evaluates to 0.0 and float_is_zero raises "precision_rounding must be positive". Skip the qty check for these lines so they always render in the report.

Section and note lines (display_type) have no product_uom, so
product_uom.rounding evaluates to 0.0 and float_is_zero raises
"precision_rounding must be positive". Skip the qty check for these
lines so they always render in the report.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.79%. Comparing base (5352fd7) to head (12f28c4).

Additional details and impacted files
@@           Coverage Diff           @@
##             16.0     #899   +/-   ##
=======================================
  Coverage   43.79%   43.79%           
=======================================
  Files         325      325           
  Lines        6464     6464           
  Branches     1011     1011           
=======================================
  Hits         2831     2831           
  Misses       3612     3612           
  Partials       21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@deeniiz deeniiz force-pushed the 16.0-fix-sale_order_report_zero_qty-section_note_lines branch from 9f69807 to 12f28c4 Compare May 4, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant