Skip to content

[18.0][FIX] product_pricelist_revision: Fix date range in duplicate wizard test#2291

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-FIX-product_pricelist_revision
May 8, 2026
Merged

[18.0][FIX] product_pricelist_revision: Fix date range in duplicate wizard test#2291
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-FIX-product_pricelist_revision

Conversation

@CarlosRoca13
Copy link
Copy Markdown
Contributor

Odoo 18 enforces a strict date_end > date_start constraint on pricelist items. The test was using the same datetime for both fields, causing a ValidationError.

See https://github.com/odoo/odoo/blob/36a7cadbfb9007ebddd660b7709e4bef58e96c2d/addons/product/models/product_pricelist_item.py#L307

cc @Tecnativa TT62281

ping @pedrobaeza @victoralmau

…test

Odoo 18 enforces a strict date_end > date_start constraint on pricelist items. The test was using the same datetime for both fields, causing a ValidationError.

See https://github.com/odoo/odoo/blob/36a7cadbfb9007ebddd660b7709e4bef58e96c2d/addons/product/models/product_pricelist_item.py#L307
@OCA-git-bot OCA-git-bot added series:18.0 mod:product_pricelist_revision Module product_pricelist_revision labels May 8, 2026
@pedrobaeza pedrobaeza added this to the 18.0 milestone May 8, 2026
@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-2291-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f5541a1 into OCA:18.0 May 8, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-FIX-product_pricelist_revision branch May 8, 2026 07:05
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