[FIX]product_multiple_barcodes: check only the limited products

ventor-tech / merp

mERP Warehouse free addons to simplify works with scanner

28 stars 40 forks source link

Closed bizzappdev closed 4 years ago

bizzappdev commented 4 years ago

Old Version: it was looping over all the products and checking/grouping all products and its barcode & barcode ids-->name

problem: it was having a problem when we have a long list of products it was slowing down the system.

New version: make a list of the current product's(the product which has been changed) barcode. and checking existing product/barcode for that.

Also fixed some test cases which was failing because on not null constraints of product_id in product.barcode.multi

okuryan commented 4 years ago

Wow! Thank you for the contribution! I review and it looks good!

bizzappdev commented 4 years ago

thank you so much for prompt action.

ploegvde commented 4 years ago

With complements from @odooexperts