Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-581723-cpa into lp:openerp-web/6.1.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-581723-cpa/+merge/133642

Hello,

Remove group field from advanced filter.

Demo : Settings > Users > users
1) Add Advanced Filter.
2) In advanced filter field selection display groups field.

Observed: Select one group field and search will gives an error.
Expected: In advanced filter field selection should not display groups field 
like gtk-client.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-581723-cpa/+merge/133642
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-581723-cpa.
=== modified file 'addons/web/static/src/js/search.js'
--- addons/web/static/src/js/search.js	2012-06-29 12:18:05 +0000
+++ addons/web/static/src/js/search.js	2012-11-09 10:43:04 +0000
@@ -1228,7 +1228,7 @@
         this._super(parent);
         this.fields = _(fields).chain()
             .map(function(val, key) {return _.extend({}, val, {'name': key}); })
-            .filter(function(field){return typeof field.store === 'undefined' || field.store || field.fnct_search})
+            .filter(function(field){return (typeof field.store === 'undefined' || field.store || field.fnct_search) && (field.selectable)})
             .sortBy(function(field) {return field.string;})
             .value();
         this.attrs = {_: _, fields: this.fields, selected: null};

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to