[18.0][MIG] pos_product_multi_barcode: Migration to version 18.0#1500
[18.0][MIG] pos_product_multi_barcode: Migration to version 18.0#1500OCA-git-bot merged 16 commits intoOCA:18.0from
Conversation
…roducts by multiple barcodes in search bar of POS.
Currently translated at 100.0% (3 of 3 strings) Translation: pos-16.0/pos-16.0-pos_product_multi_barcode Translate-URL: https://translation.odoo-community.org/projects/pos-16-0/pos-16-0-pos_product_multi_barcode/it/
If a `barcode` is already in `product_by_barcode`, do not override it, as Odoo does with `product_by_id`. This can lead to incoherent data if something changes after product is first loaded and then reloads (if limited loading is enabled), or it is modified in `product_by_id` and not in `product_by_barcode`.
[DON'T MERGE] test-requirements.txt
cristina-hidalgo-tecnativa
force-pushed
the
18.0-mig-pos_product_multi_barcode
branch
from
d125db2 to
4201f77
Compare
|
/ocabot migration pos_product_multi_barcode |
pedrobaeza
left a comment
pedrobaeza
left a comment
There was a problem hiding this comment.
I have tried with https://addons.mozilla.org/en-US/firefox/addon/barcode-scanner-emulator/ to scan this product with the second barcode:
and it's not working:
Can you please check?
cristina-hidalgo-tecnativa
force-pushed
the
18.0-mig-pos_product_multi_barcode
branch
from
4201f77 to
7a6f0dc
Compare
I guess now It's fixed, could you please review it again? |
cristina-hidalgo-tecnativa
force-pushed
the
18.0-mig-pos_product_multi_barcode
branch
from
7a6f0dc to
e1ae6e8
Compare
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at b2f8e97. Thanks a lot for contributing to OCA. ❤️ |
11 participants
PosDBwas removed in Odoo 18 and the POS data loading API changed.pos_session.pyis removed._load_pos_data_fieldsis now overridden directly onproduct.product, replacing the removed_loader_params_product_productdb.esm.jsis updated to patchDataServiceOptionsandProductProductinstead of the removedPosDB. Additional barcodes are indexed for scanner lookup viasetup()and exposed to text search viasearchString, fixing the \n-based replacement that no longer works.@Tecnativa TT59146
@christian-ramos-tecnativa @pedrobaeza