From 7b0bef5bccf900100b7c463a0c3ae62f4db27064 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Wed, 27 Jun 2012 11:57:44 +0200 Subject: [PATCH] [FIX] incorrect handling of present-but-empty contexts, which seem to be rather common bzr revid: xmo@openerp.com-20120627095744-ijhf7p769pw4fqdg --- addons/web/static/src/js/search.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/web/static/src/js/search.js b/addons/web/static/src/js/search.js index 4faf9dabdd5..087c4c22294 100644 --- a/addons/web/static/src/js/search.js +++ b/addons/web/static/src/js/search.js @@ -1234,7 +1234,7 @@ instance.web.search.Field = instance.web.search.Input.extend( /** @lends instanc var self = this; // A field needs a context to send when active var context = this.attrs.context; - if (!context || !facet.values.length) { + if (_.isEmpty(context) || !facet.values.length) { return; } var contexts = facet.values.map(function (facetValue) { @@ -1533,7 +1533,7 @@ instance.web.search.ManyToOneField = instance.web.search.CharField.extend({ }, get_context: function (facet) { var values = facet.values; - if (!this.attrs.context && values.length === 1) { + if (_.isEmpty(this.attrs.context) && values.length === 1) { var c = {}; c['default_' + this.attrs.name] = values.at(0).get('value'); return c;