From 0ad34228e20934c2784bd11f6dec06cfcd8094c4 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Thu, 31 Jan 2013 14:51:25 +0100 Subject: [PATCH] [FIX] small issue with adding an attribute called 'id' to a backbone model triggers backbone's thinking that the model is 'not new' and trying to sync it with a remote on e.g. removal bzr revid: xmo@openerp.com-20130131135125-1x7czib7q0kafok8 --- 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 65df912bdb0..8391f7a8ca5 100644 --- a/addons/web/static/src/js/search.js +++ b/addons/web/static/src/js/search.js @@ -1598,7 +1598,7 @@ instance.web.search.CustomFilters = instance.web.search.Input.extend({ get_groupby: function () { return [filter.context]; }, get_domain: function () { return filter.domain; } }, - id: filter['id'], + _id: filter['id'], is_custom_filter: true, values: [{label: filter.name, value: null}] }; @@ -1645,7 +1645,7 @@ instance.web.search.CustomFilters = instance.web.search.Input.extend({ }, toggle_filter: function (filter, preventSearch) { var current = this.view.query.find(function (facet) { - return facet.get('id') === filter.id; + return facet.get('_id') === filter.id; }); if (current) { this.view.query.remove(current);