Thursday, May 12, 2016
2 changes
Resolved issues and error corrections
This update corrects how database query placeholders are written in the Point of Sale reporting code and shared test utilities. It helps prevent potential errors when reports or tests run, improving reliability without changing user-facing workflows.
Original PR description
Description of the issue/feature this PR addresses: Addresses the use of %d as a placeholder within the point_of_sale module's "pos_order_report" report and openerp/tests' common.py. Per psycopg's documentation[1], "the variables placeholder **must always be a %s**, even if a different placeholder (such as a %d for integers or %f for floats) may look more appropriate." [1] http://initd.org/psycopg/docs/usage.html#passing-parameters-to-sql-queries See, too, commit ed7567d3f55b12181693ae27c0019aad849b7a80 Current behavior before PR: Untested, but the use of %d should lead to an error. Desired behavior after PR is merged: Compliant type ## I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Cody K
Belgian VAT reporting was updated around manual adjustments for VAT grids 61 and 62. The change removes a duplicate adjustment wizard from Enterprise because that functionality is already available in the community edition, helping avoid overlap and confusion.