Merge lp:~guerrerocarlos/openobject-addons/point_of_sale_6.1_barcode__merge into lp:openobject-addons/6.1

[Carlos Guerrero]

Solves 'OpenERP 6.1 POS: lookup by barcode (EAN13 or Reference)'[1] bug.

* [FIX] point_of_sale: Added product search by EAN13 or Reference in the POS

[1] https://bugs.launchpad.net/openobject-addons/+bug/949664

[Cristian Salamea (ovnicraft)]

Hello thanks for fix it, so i have a question, is possible get on if for both evaluations ?

Regards,

[Carlos Guerrero (guerrerocarlos)]

Yes but then another if would be needed for selecting the right field
to apply the "toLowerCase" and "indexOf(s)", so I think that with two
separated "if" like it is right now, is the most efficient.

On Fri, Mar 23, 2012 at 4:27 PM, Cristian Salamea (Gnuthink)
<email address hidden> wrote:
> Hello thanks for fix it, so i have a question, is possible get on if for both evaluations ?
>
> Regards,
> --
> https://code.launchpad.net/~guerrerocarlos/openobject-addons/point_of_sale_6.1_barcode__merge/+merge/99129
> You are the owner of lp:~guerrerocarlos/openobject-addons/point_of_sale_6.1_barcode__merge.

--
Atte:
Carlos A. Guerrero M.
http://www.carlosguerrero.com

[lp:~guerrerocarlos/openobject-addons/point_of_sale_6.1_barcode__merge]

6703. By Carlos Guerrero on 2012-03-23

[FIX] Validation for searching of all three fields: "name", "ean13" and "code"

6704. By Carlos Guerrero on 2012-03-24

[IMP] point_of_sale: lookup by barcode in the same if

[Carlos Guerrero (guerrerocarlos)]

Now all the evaluations are in one same "if" as requested and has been proved to work as well.

[lp:~guerrerocarlos/openobject-addons/point_of_sale_6.1_barcode__merge]

6705. By Carlos Guerrero on 2012-03-24

[FIX] point_of_sale: Add product to order automatically with <Enter> from barcode reader

[Carlos Guerrero (guerrerocarlos)]

New improvement to the code, now the "if" has been removed at all, and the product is automatically added to the order when using a bar code reader.

[lp:~guerrerocarlos/openobject-addons/point_of_sale_6.1_barcode__merge]

6706. By Carlos Guerrero on 2012-03-24

[IMP] point_of_sale: Automatically clear the search box after <Enter> from bar code reader

[Nhomar - Vauxoo (nhomar)]

Hello.

Please core team, can you merge this improvement please.

Regards.

review: Approve

[Roberto López López (bergensk)]

review: Approve

[Selami Demir (sdemir)]

Hi,

It is work now.

Thank you.

review: Approve

[Olivier Dony (Odoo) (odo-openerp)]

Hello Carlos,

Well done, your change looks clean and simple enough to me. However I'm not an expert in 6.1 web modules, so I'm not sure if this is the best way to handle the <ENTER> event to autoselect a product.
I'll request a second review by someone from the web team.

Thanks for your good contribution!

review: Abstain

[Antony Lesuisse (OpenERP) (al-openerp)]

Could you rebase your patch on the latest 6.1 ?

[Carlos Guerrero (guerrerocarlos)]

There seems to be a merge in
http://bazaar.launchpad.net/~openerp/openobject-addons/6.1/revision/6726

But it is a very different amount of code, and doesn't include the "Reference" product field, but seems to be trying to solve the same issue.

Could someone enlight me about this?

Where can I see the "discussion" of that other [MERGE] ?

[Lluis Gili Belmonte (tictacbum)]

Hi, I am working on same issue, there's any update about this?
I tried applying Carlos patch, but sometimes OpenERP can't find products, and I have to close and reopen PoS to make it work.

Sometimes it also shows a warning message repeated several times (overlapped):

"Warningclose
The product could not be recognized. Please contact an employee."

Unmerged revisions

6706. By Carlos Guerrero on 2012-03-24

[IMP] point_of_sale: Automatically clear the search box after <Enter> from bar code reader

6705. By Carlos Guerrero on 2012-03-24

[FIX] point_of_sale: Add product to order automatically with <Enter> from barcode reader

6704. By Carlos Guerrero on 2012-03-24

[IMP] point_of_sale: lookup by barcode in the same if

6703. By Carlos Guerrero on 2012-03-23

[FIX] Validation for searching of all three fields: "name", "ean13" and "code"

6702. By Carlos Guerrero on 2012-03-23

[FIX] point_of_sale: Added product search by EAN13 or Reference in the POS

=== modified file 'point_of_sale/static/src/js/pos.js'
--- point_of_sale/static/src/js/pos.js	2012-02-24 14:49:39 +0000
+++ point_of_sale/static/src/js/pos.js	2012-03-24 22:24:18 +0000
@@ -85,7 +85,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'], [['pos_categ_id', '!=', 'false']]),
+                this.fetch('product.product', ['name','code','ean13', 'list_price', 'pos_categ_id', 'taxes_id', 'product_image_small'], [['pos_categ_id', '!=', 'false']]),
                 this.fetch('account.bank.statement', ['account_id', 'currency', 'journal_id', 'state', 'name'],
                     [['state', '=', 'open'], ['user_id', '=', this.session.uid]]),
                 this.fetch('account.journal', ['auto_cash', 'check_dtls', 'currency', 'name', 'type']),
@@ -1259,19 +1259,27 @@
             });
             (this.shop.get('products')).reset(products);
             var self = this;
-            $('.searchbox input').keyup(function() {
-                var m, s;
+            $('.searchbox input').keyup(function(event) {
+                var m, s, myproduct;
                 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 {
                     m = products;
                     $('.search-clear').fadeOut();
                 }
-                return (self.shop.get('products')).reset(m);
+                if (m.length == 1 && event.keyCode == 13) {
+			myproduct = self.shop.get('products').get(m[0]);
+                  	self.shop.get('selectedOrder').addProduct(myproduct);
+                	$(this).val('');
+                    	m = products;
+                }
+		return (self.shop.get('products')).reset(m);
+
+
             });
             return $('.search-clear').click( function() {
                 (this.shop.get('products')).reset(products);