From 515372d8dc4429f72ce06f5f840b3b59224a9c77 Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@openerp.com>
Date: Thu, 26 Jul 2012 18:16:05 +0200
Subject: [PATCH] [FIX] correct Login widget to send event instead of creating
 a child client action

bzr revid: chs@openerp.com-20120726161605-wtwi5uzau37dc9iq
---
 addons/web/static/src/js/chrome.js | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/addons/web/static/src/js/chrome.js b/addons/web/static/src/js/chrome.js
index e72ee5f2a93..a1759397e21 100644
--- a/addons/web/static/src/js/chrome.js
+++ b/addons/web/static/src/js/chrome.js
@@ -584,7 +584,7 @@ instance.web.Login =  instance.web.Widget.extend({
                     localStorage.setItem('last_password_login_success', '');
                 }
             }
-            self.do_action("login_sucessful");
+            self.trigger('login_successful');
         },function () {
             self.$(".oe_login_pane").fadeIn("fast");
             self.$element.addClass("oe_login_invalid");
@@ -593,16 +593,6 @@ instance.web.Login =  instance.web.Widget.extend({
 });
 instance.web.client_actions.add("login", "instance.web.Login");
 
-instance.web.LoginSuccessful =  instance.web.Widget.extend({
-    init: function(parent) {
-        this._super(parent);
-    },
-    start: function() {
-        this.getParent().getParent().show_application();
-    },
-});
-instance.web.client_actions.add("login_sucessful", "instance.web.LoginSuccessful");
-
 instance.web.Menu =  instance.web.Widget.extend({
     template: 'Menu',
     init: function() {
@@ -979,10 +969,9 @@ instance.web.WebClient = instance.web.Client.extend({
         };
     },
     show_login: function() {
-        var self = this;
-        self.$('.oe_topbar').hide();
-        self.action_manager.do_action("login");
-        //self.login.appendTo(self.$element);
+        this.$('.oe_topbar').hide();
+        this.action_manager.do_action("login");
+        this.action_manager.inner_widget.on('login_successful', this, this.show_application);
     },
     show_application: function() {
         var self = this;