Skip to content

[19.0][MIG] sale_product_set: Migration to 19.0#4283

Open
CRogos wants to merge 76 commits intoOCA:19.0from
c4a8-odoo:copilot/migrate-sale-product-set-18-0-to-19-0
Open

[19.0][MIG] sale_product_set: Migration to 19.0#4283
CRogos wants to merge 76 commits intoOCA:19.0from
c4a8-odoo:copilot/migrate-sale-product-set-18-0-to-19-0

Conversation

@CRogos
Copy link
Copy Markdown
Contributor

@CRogos CRogos commented Apr 16, 2026

supersede: #3941

@CRogos CRogos force-pushed the copilot/migrate-sale-product-set-18-0-to-19-0 branch 3 times, most recently from f4efb91 to 7802b2c Compare April 17, 2026 11:12
@CRogos CRogos marked this pull request as ready for review April 17, 2026 11:21
@CRogos CRogos force-pushed the copilot/migrate-sale-product-set-18-0-to-19-0 branch 3 times, most recently from 72ce950 to ff86f7e Compare April 20, 2026 12:30
@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot migration sale_product_set

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Apr 29, 2026
@OCA-git-bot
Copy link
Copy Markdown
Contributor

The migration issue (#3914) has not been updated to reference the current pull request because a previous pull request (#3941) is not closed.
Perhaps you should check that there is no duplicate work.
CC @mmrondon

@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot migration sale_product_set

Copy link
Copy Markdown
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

LGTM

@rousseldenis
Copy link
Copy Markdown
Contributor

@BhaveshHeliconia Could you review? Thanks

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.

I got an error while testing.

@BhaveshHeliconia
Copy link
Copy Markdown
Contributor

S00026.mp4

francesco-ooops and others added 18 commits April 29, 2026 14:37
Currently translated at 33.3% (12 of 36 strings)

Translation: sale-workflow-14.0/sale-workflow-14.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-14-0/sale-workflow-14-0-sale_product_set/it/
If the user plays too much with sequences on the product set, it's possible to have some negative sequences.
But when the set is applied on the SO negative sequences are added to max_sequence. As the sequence is < 0 it adds the product 'in the middle' of the SO instead of let grouped every product's set.
Currently translated at 36.1% (13 of 36 strings)

Translation: sale-workflow-15.0/sale-workflow-15.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_product_set/it/
Currently translated at 58.3% (21 of 36 strings)

Translation: sale-workflow-15.0/sale-workflow-15.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_product_set/sl/
[UPD] Update sale_product_set.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/

Translated using Weblate (Spanish)

Currently translated at 100.0% (23 of 23 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/es/

[UPD] README.rst
Translated using Weblate (Italian)

Currently translated at 100.0% (23 of 23 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/it/

[UPD] Update sale_product_set.pot

[BOT] post-merge updates

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/

Translated using Weblate (Italian)

Currently translated at 100.0% (26 of 26 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/it/

Translated using Weblate (Spanish)

Currently translated at 100.0% (26 of 26 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/es/
… product_set

The reason to move this logic is that there are other modules that extend
product_set as for example stock_product_set but at the same time make use
of the transient model to define a wizard. For this reason it is better to
have the logic available in product_set module and avoid duplicating code or
inheriting from sale_product_set and what this implies in their respective
dependencies with the only need to extend the transient model.
In addition, the transient model is renamed to make it clearer to identify
that it is this type of model.

TT48100

[UPD] Update sale_product_set.pot

[BOT] post-merge updates

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_product_set/
Currently translated at 100.0% (25 of 25 strings)

Translation: sale-workflow-17.0/sale-workflow-17.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-17-0/sale-workflow-17-0-sale_product_set/de/
Currently translated at 100.0% (25 of 25 strings)

Translation: sale-workflow-17.0/sale-workflow-17.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-17-0/sale-workflow-17-0-sale_product_set/sv/
Currently translated at 100.0% (25 of 25 strings)

Translation: sale-workflow-17.0/sale-workflow-17.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-17-0/sale-workflow-17-0-sale_product_set/it/
[MIG] sale_product_set: add migration to 18.0 for syncing product set state
Currently translated at 100.0% (25 of 25 strings)

Translation: sale-workflow-18.0/sale-workflow-18.0-sale_product_set
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-18-0/sale-workflow-18-0-sale_product_set/tr/
@CRogos CRogos force-pushed the copilot/migrate-sale-product-set-18-0-to-19-0 branch from ff86f7e to 02f3969 Compare April 29, 2026 14:37
@CRogos
Copy link
Copy Markdown
Contributor Author

CRogos commented Apr 29, 2026

@BhaveshHeliconia could you retest. I think this was related to a 0.0 unit_price.

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.

I’ve retested it, but I’m still getting the same error on Runbot.
I also tested it locally, and it works fine there without any error.

@MohamedOsman7
Copy link
Copy Markdown
Contributor

If i add a set, then change the Quantity of the products, the amount will be set to 0:
image

@CRogos CRogos force-pushed the copilot/migrate-sale-product-set-18-0-to-19-0 branch from 02f3969 to cfaafe8 Compare April 30, 2026 20:22
@CRogos CRogos force-pushed the copilot/migrate-sale-product-set-18-0-to-19-0 branch from cfaafe8 to cb46f3d Compare April 30, 2026 20:31
@CRogos
Copy link
Copy Markdown
Contributor Author

CRogos commented Apr 30, 2026

@MohamedOsman7 The quantity issue was related to a not correctly set uom_id.
@BhaveshHeliconia could you also retest... for me it is now working on runboat.

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.