Merge lp:~openerp-dev/openobject-addons/6.1-opw-575341-pso into lp:openobject-addons/6.1

[Priyesh (OpenERP)]

Hello,

It should be possible to search on the reference and EAN field via the search field of the POS.
For now, its only possible with product name.

Kindly review the branch and let me know your inputs on it.

Regards,
Priyesh

[Vinay Rana (OpenERP) (vra-openerp)]

As this is the improvement which priyesh propose for extra searching option from POS front end, here end user can search POS product code vice also.

the propose fixes is correct and we can search using product code.

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Confirmed on Trunk (means still existing and reproducible). A Merge Proposal for trunk was created to fix it. Here is the link to follow the MP on Launchpad https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-575341-port-sgo/+merge/137555 and be informed once it's been merged in trunk: ... If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6822. By Carlos Guerrero on 2012-06-01

[FIX] point_of_sale: Added search conditions for code and ean13 of the product in POS order search criteria(case:575341)

=== modified file 'point_of_sale/static/src/js/pos.js'
--- point_of_sale/static/src/js/pos.js	2012-05-05 16:09:02 +0000
+++ point_of_sale/static/src/js/pos.js	2012-06-01 10:55:22 +0000
@@ -69,7 +69,7 @@
                 }, this));
             }, this));
             $.when(this.fetch('pos.category', ['name', 'parent_id', 'child_id']),
-                this.fetch('product.product', ['name', 'list_price', 'pos_categ_id', 'taxes_id', 'product_image_small', 'ean13', 'id'], [['pos_categ_id', '!=', false]]),
+                this.fetch('product.product', ['name', 'list_price', 'pos_categ_id', 'taxes_id', 'product_image_small', 'code', 'ean13', 'id'], [['pos_categ_id', '!=', false]]),
                 this.fetch('product.packaging', ['product_id', 'ean']),
                 this.fetch('account.bank.statement', ['account_id', 'currency', 'journal_id', 'state', 'name'],
                     [['state', '=', 'open'], ['user_id', '=', this.session.uid]]),
@@ -1333,7 +1333,7 @@
                 s = $(this).val().toLowerCase();
                 if (s) {
                     m = products.filter( function(p) {
-                        return p.name.toLowerCase().indexOf(s) != -1;
+                        return (String(p.code).toLowerCase().indexOf(s) != -1) || (String(p.ean13).toLowerCase().indexOf(s) != -1) || (p.name.toLowerCase().indexOf(s) != -1);
                     });
                     $('.search-clear').fadeIn();
                 } else {