From 664de510ad8e29951bd7a19e8e5b4cdd25f6214a Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@openerp.com>
Date: Fri, 13 Jan 2012 12:17:36 +0100
Subject: [PATCH 1/3] [IMP] edi: remove debug in edi url

bzr revid: chs@openerp.com-20120113111736-w542872mwozrhbpp
---
 addons/edi/models/edi.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/addons/edi/models/edi.py b/addons/edi/models/edi.py
index c573115618e..589922ff178 100644
--- a/addons/edi/models/edi.py
+++ b/addons/edi/models/edi.py
@@ -2,7 +2,7 @@
 ##############################################################################
 #
 #    OpenERP, Open Source Business Applications
-#    Copyright (c) 2011 OpenERP S.A. <http://openerp.com>
+#    Copyright (c) 2011-2012 OpenERP S.A. <http://openerp.com>
 #
 #    This program is free software: you can redistribute it and/or modify
 #    it under the terms of the GNU Affero General Public License as
@@ -37,7 +37,7 @@ from tools.translate import _
 from tools.safe_eval import safe_eval as eval
 
 EXTERNAL_ID_PATTERN = re.compile(r'^([^.:]+)(?::([^.]+))?\.(\S+)$')
-EDI_VIEW_WEB_URL = '%s/edi/view?debug=1&db=%s&token=%s'
+EDI_VIEW_WEB_URL = '%s/edi/view?db=%s&token=%s'
 EDI_PROTOCOL_VERSION = 1 # arbitrary ever-increasing version number
 EDI_GENERATOR = 'OpenERP ' + release.major_version
 EDI_GENERATOR_VERSION = release.version_info
@@ -682,4 +682,4 @@ class EDIMixin(object):
 
         return record_id
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
\ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

From fb2e5f04c00d492b6edfab0c73b230ab6c0ef591 Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@openerp.com>
Date: Fri, 13 Jan 2012 12:20:34 +0100
Subject: [PATCH 2/3] [FIX] edi: correct design

bzr revid: chs@openerp.com-20120113112034-zr1v1qsvwn6ya0x7
---
 addons/edi/static/src/js/edi.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/addons/edi/static/src/js/edi.js b/addons/edi/static/src/js/edi.js
index 73bcfa267e5..352177156e1 100644
--- a/addons/edi/static/src/js/edi.js
+++ b/addons/edi/static/src/js/edi.js
@@ -109,7 +109,7 @@ openerp.edi.EdiView = openerp.web.Widget.extend({
 
 openerp.edi.edi_view = function (db, token) {
     openerp.connection.bind().then(function () {
-        new openerp.edi.EdiView(null,db,token).appendTo($("body"));
+        new openerp.edi.EdiView(null,db,token).appendTo($("body").addClass('openerp'));
     });
 }
 
@@ -172,7 +172,7 @@ openerp.edi.EdiImport = openerp.web.Widget.extend({
 
 openerp.edi.edi_import = function (url) {
     openerp.connection.bind().then(function () {
-        new openerp.edi.EdiImport(null,url).appendTo($("body"));
+        new openerp.edi.EdiImport(null,url).appendTo($("body").addClass('openerp'));
     });
 }
 

From db1ffb561b704d76a6ee80faf94fbce906df82e5 Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@openerp.com>
Date: Fri, 13 Jan 2012 12:21:21 +0100
Subject: [PATCH 3/3] [FIX] edi: ask login before import

bzr revid: chs@openerp.com-20120113112121-5tlnb2he9sw8cbe0
---
 addons/edi/static/src/js/edi.js | 47 +++++++++++++++++++++++----------
 1 file changed, 33 insertions(+), 14 deletions(-)

diff --git a/addons/edi/static/src/js/edi.js b/addons/edi/static/src/js/edi.js
index 352177156e1..d62a4c3b780 100644
--- a/addons/edi/static/src/js/edi.js
+++ b/addons/edi/static/src/js/edi.js
@@ -114,30 +114,49 @@ openerp.edi.edi_view = function (db, token) {
 }
 
 openerp.edi.EdiImport = openerp.web.Widget.extend({
-    init: function(parent,url) {
-        this._super();
-        this.url = url;
-        var params = {};
+    template: "EdiImport",
 
-        this.template = "EdiImport";
-        this.session = openerp.connection;
-        this.login = new openerp.web.Login(this);
-        this.header = new openerp.web.Header(this);
+    init: function(parent, url) {
+        this._super(parent);
+        this.url = url;
     },
     start: function() {
-        // TODO fix by explicitly asking login if needed
-        //this.session.on_session_invalid.add_last(this.do_ask_login);
-        this.header.appendTo($("#oe_header"));
-        this.login.appendTo($('#oe_login'));
+        if (!this.session.session_is_valid()) {
+            this.show_login();
+            this.session.on_session_valid.add({
+                callback: this.proxy('show_import'),
+                unique: true,
+            });
+        } else {
+            this.show_import();
+        }
+    },
+
+    show_import: function() {
+        this.destroy_content();
         this.do_import();
     },
+
+    show_login: function() {
+        this.destroy_content();
+        this.login = new openerp.web.Login(this);
+        this.login.appendTo(this.$element);
+    },
+
+    destroy_content: function() {
+        _.each(_.clone(this.widget_children), function(el) {
+            el.stop();
+        });
+        this.$element.children().remove();
+    },
+
     do_import: function() {
         this.rpc('/edi/import_edi_url', {url: this.url}, this.on_imported, this.on_imported_error);
     },
     on_imported: function(response) {
         if ('action' in response) {
             this.rpc("/web/session/save_session_action", {the_action: response.action}, function(key) {
-                window.location = "/web/webclient/home?debug=1&s_action="+encodeURIComponent(key);
+                window.location = "/web/webclient/home#sa="+encodeURIComponent(key);
             });
         }
         else {
@@ -147,7 +166,7 @@ openerp.edi.EdiImport = openerp.web.Widget.extend({
                 buttons: {
                     Ok: function() {
                         $(this).dialog("close");
-                        window.location = "/web/webclient/home";
+                        window.location = "/";
                     }
                 }
             }).html('The document has been successfully imported!');