[MERGE] trunk

bzr revid: al@openerp.com-20131203192151-szbvhmanml37a9si

2 files changed, 9 insertions, 2 deletions

diff --git a/addons/web/static/src/js/core.js b/addons/web/static/src/js/core.js
index d62c04b0ecb..81e029f5f87 100644
--- a/addons/web/static/src/js/core.js
+++ b/addons/web/static/src/js/core.js
@@ -561,8 +561,8 @@ instance.web.TranslationDataBase.include({
 });
 
 /** Custom jQuery plugins */
+$.browser = $.browser || {};
 if(navigator.appVersion.indexOf("MSIE") !== -1) {
-    $.browser = $.browser || {};
     $.browser.msie = 1;
 }
 $.fn.getAttributes = function() {
diff --git a/addons/web/static/src/js/search.js b/addons/web/static/src/js/search.js
index fcfcfb08f8e..c3ef682bfcc 100644
--- a/addons/web/static/src/js/search.js
+++ b/addons/web/static/src/js/search.js
@@ -71,7 +71,10 @@ my.SearchQuery = B.Collection.extend({
     },
     add: function (values, options) {
         options = options || {};
-        if (!(values instanceof Array)) {
+
+        if (!values) {
+            values = [];
+        } else if (!(values instanceof Array)) {
             values = [values];
         }
 
@@ -87,6 +90,10 @@ my.SearchQuery = B.Collection.extend({
             }
             B.Collection.prototype.add.call(this, model, options);
         }, this);
+        // warning: in backbone 1.0+ add is supposed to return the added models,
+        // but here toggle may delegate to add and return its value directly.
+        // return value of neither seems actually used but should be tested
+        // before change, probably
         return this;
     },
     toggle: function (value, options) {