Vaibhav (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/trunk-bug-928280-vda into lp:openerp-web.

Requested reviews:
  OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
  Bug #928280 in OpenERP Web: "web rev 2124: "Got non-search view after asking 
for a search view: type search, arch root form""
  https://bugs.launchpad.net/openerp-web/+bug/928280

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-928280-vda/+merge/93542

hidden searchview should not through such error.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-928280-vda/+merge/93542
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-928280-vda.
=== modified file 'addons/web/static/src/js/search.js'
--- addons/web/static/src/js/search.js	2012-02-14 14:29:54 +0000
+++ addons/web/static/src/js/search.js	2012-02-17 09:08:17 +0000
@@ -140,8 +140,9 @@
     },
     on_loaded: function(data) {
         this.fields_view = data.fields_view;
-        if (data.fields_view.type !== 'search' ||
-            data.fields_view.arch.tag !== 'search') {
+        
+        if (!this.hidden && (data.fields_view.type !== 'search' ||
+            data.fields_view.arch.tag !== 'search')) {
                 throw new Error(_.str.sprintf(
                     "Got non-search view after asking for a search view: type %s, arch root %s",
                     data.fields_view.type, data.fields_view.arch.tag));

_______________________________________________
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