Merge lp:~openerp-dev/openerp-web/trunk-fix-manage-view into lp:openerp-web

[Thibault Delavallée (OpenERP)]

[FIX] view editor: unable to select a different view

[Thibault Delavallée (OpenERP) (tde-openerp)]

Hello,

Seems correct; however I will backport it to saas-3.

review: Approve

[Thibault Delavallée (OpenERP) (tde-openerp)]

(note: set as rejected because there isn't any other suitable option)

[Luis Gondelles (luis-gondelles)]

FUCK YOU ALL.... REMOVE ME FROM DISTRIBUTION LIST!!!!!

Sent from my iPad

> On 17/02/2014, at 07:31, Thibault Delavallée (OpenERP) <email address hidden> wrote:
>
> The proposal to merge lp:~openerp-dev/openerp-web/trunk-fix-manage-view into lp:openerp-web has been updated.
>
> Status: Needs review => Work in progress
>
> For more details, see:
> https://code.launchpad.net/~openerp-dev/openerp-web/trunk-fix-manage-view/+merge/206680
> --
> https://code.launchpad.net/~openerp-dev/openerp-web/trunk-fix-manage-view/+merge/206680
> You are subscribed to branch lp:openerp-web.

Unmerged revisions

3917. By Vidhin Mehta (OpenERP) on 2014-01-21

[FIX]manage view.

=== modified file 'addons/web_view_editor/static/src/js/view_editor.js'
--- addons/web_view_editor/static/src/js/view_editor.js	2013-04-05 08:52:24 +0000
+++ addons/web_view_editor/static/src/js/view_editor.js	2014-02-17 10:40:50 +0000
@@ -73,12 +73,10 @@
         $.when(this.action_manager.do_action(action)).done(function() {
             var viewmanager = self.action_manager.inner_widget;
             var controller = viewmanager.views[viewmanager.active_view].controller;
-            controller.on('view_loaded', self, function(){
-                $(controller.groups).bind({
-                    'selected': function(e, ids, records) {
+            $(controller.groups).bind({
+                'selected': function (e, ids, records, deselected) {
                         self.main_view_id = ids[0];
-                    }
-                });
+                }
             });
         });
     },