Bhumi Thakkar (Open ERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-574625-bth 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-574625-bth/+merge/109993

Hello,

Using browser's back button to navigate back to a opportunities screen in graph 
or list view results in error.

1. Go to Sales => Opportunities, 
2. Open in Graph View or List View.
3. Select any other menu
4. Click on the browser's back button.

Observed: Error is coming in dialog box: "Cannot read property 'view_id' of 
undefined"
Extected: "Open in Graph View or List View"

put condition if fields_view do not get from controller then return false;
if fields_view get then view_id get from fields_view.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-574625-bth/+merge/109993
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-574625-bth.
=== modified file 'addons/web/static/src/js/views.js'
--- addons/web/static/src/js/views.js	2012-06-12 14:19:43 +0000
+++ addons/web/static/src/js/views.js	2012-06-13 07:38:27 +0000
@@ -661,7 +661,7 @@
             self.shortcut_check(self.views[view_type]);
 
             self.$element.find('.oe-view-manager-logs:first').addClass('oe-folded').removeClass('oe-has-more').css('display','none').find('ul').empty();
-
+            if(!self.views[self.active_view].controller.fields_view) return false;
             var controller = self.views[self.active_view].controller,
                 fvg = controller.fields_view,
                 view_id = (fvg && fvg.view_id) || '--';
@@ -699,7 +699,7 @@
                     return self.on_mode_switch(state.view_type, true);
                 })
             );
-        } 
+        }
 
         $.when(defs).then(function() {
             self.views[self.active_view].controller.do_load_state(state, warm);
@@ -1200,7 +1200,7 @@
      *
      * @param {String} view view type to switch to
      */
-    do_switch_view: function(view) { 
+    do_switch_view: function(view) {
     },
     /**
      * Cancels the switch to the current view, switches to the previous one

_______________________________________________
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