Mohammed Shekha(Open ERP) has proposed merging 
lp:~openerp-dev/openobject-client-web/6.0-opw-572643-msh into 
lp:openobject-client-web.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-572643-msh/+merge/96973

Hello,

Fixed the issue of filter which opens listgroup instead of list view.

Demo :- Go to Warehouse -> Incoming Shipment -> Listview -> clear all filter -> 
now select extended filter "To Invoice"  and then "Available".

Result :- This will give traceback

Reason :- Here group_by_ctx has value '' which is appended in group_by in 
search.js which will converted in group_by_ctx as a [''] when we split 
group_by_ctx from eval_domain_filter of search.py, so in _view there is 
condition if there is group_by_ctx then we will open ListGroup otherwise 
ListView.

Hence changed the code in search.js.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-572643-msh/+merge/96973
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client-web/6.0-opw-572643-msh.
=== modified file 'addons/openerp/static/javascript/search.js'
--- addons/openerp/static/javascript/search.js	2011-07-27 11:03:26 +0000
+++ addons/openerp/static/javascript/search.js	2012-03-12 07:45:23 +0000
@@ -478,8 +478,10 @@
         jQuery('div.group-data button').each(function(){
             if (jQuery(this).hasClass('active')) {
                 var _grp = jQuery(this).next('input').attr('group_by_ctx');
-                if (jQuery.inArray(_grp, group_by) < 0) {
-                    group_by.push(_grp);
+                if (_grp){
+                	if (jQuery.inArray(_grp, group_by) < 0) {
+                    	group_by.push(_grp);
+                	}
                 }
             }
         });

_______________________________________________
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