[REF] stock: removal of TODO statements

bzr revid: qdp-launchpad@openerp.com-20140425142939-atjre88c9zvl5r93

1 files changed, 8 insertions, 2 deletions

diff --git a/addons/stock/static/src/js/widgets.js b/addons/stock/static/src/js/widgets.js
index b7753cf1432..6a770a777b3 100644
--- a/addons/stock/static/src/js/widgets.js
+++ b/addons/stock/static/src/js/widgets.js
@@ -452,8 +452,14 @@ function openerp_picking_widgets(instance){
             $(window).trigger('hashchange');
         },
         search_picking: function(barcode){
-            //TODO don't crash if a not supported char is given
-            var re = RegExp("([0-9]+):.*?"+barcode.toUpperCase(),"gi");
+            try {
+                var re = RegExp("([0-9]+):.*?"+barcode.toUpperCase(),"gi");
+            }
+            catch(e) {
+                //avoid crash if a not supported char is given (like '\' or ')')
+	        return [];
+            }
+            
             var results = [];
             for(var i = 0; i < 100; i++){
                 r = re.exec(this.picking_search_string);