Merge lp:~openerp-dev/openerp-web/7.0-opw-597026-chandni into lp:openerp-web/7.0

[Chandni Machchhar(OpenERP)]

Hello,

I have fixed the error "Uncaught TypeError: Cannot read property 'controller' of undefined". Error comes while clicking on graph view after click on line of tree view.

Steps to reproduce :

--> Active the debug mode.
--> Goto Menu Reporting > Lead Analysis > Edit Action
    Remove the context 'group_by_no_leaf':1
--> Now refresh window Remove the filter from search view so that lines will be displayed in tree view. Now click on line then click on Graph view. It will give you error.

This is because after clicking on line it will assume that form view should open but if form view is not defined then it will give you Error.

Thanks,
Chandni

Unmerged revisions

4028. By Chandni Machchhar(OpenERP) on 2013-09-12

[FIX] Error while clicking on graph view after click on line of tree view.

=== modified file 'addons/web/static/src/js/view_list.js'
--- addons/web/static/src/js/view_list.js	2013-06-10 07:10:36 +0000
+++ addons/web/static/src/js/view_list.js	2013-09-25 06:09:54 +0000
@@ -454,11 +454,13 @@
      * @param {String} [view="page"] the view type to switch to
      */
     select_record:function (index, view) {
-        view = view || index == null ? 'form' : 'form';
-        this.dataset.index = index;
-        _.delay(_.bind(function () {
-            this.do_switch_view(view);
-        }, this));
+        view = view || index == null ? 'form' : (this.ViewManager.views['form'] ? 'form': null);
+        if (view) {
+            this.dataset.index = index;
+            _.delay(_.bind(function () {
+                this.do_switch_view(view);
+            }, this));
+        }
     },
     do_show: function () {
         this._super();