From 9efc11c8daa093dde11d82010d8a56828ed55996 Mon Sep 17 00:00:00 2001 From: Romain Derie Date: Tue, 4 Jun 2019 09:26:59 +0000 Subject: [PATCH] [FIX] website: redirect user MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Before this commit, user would be redirect to / instead of /my after signup. That error was detected with task-1829827 which has a tour ensuring user is correctly redirected to /my after signup on the checkout summary after payment. Since 0229ef4c4a39, the `web_login()` super call order has been changed when called from the `/web/signup` route. See the commit for more details but basically, before the fix, it went only through: 1. web.web_login 2. portal._login_redirect Where now, it goes through: 1. website.web_login 2. auth_signup.web_login 3. web.web_login 4.portal._login_redirect Thus, portal._login_redirect is not returning `/my` as redirect anymore. Indeed, website.web_login will ignore its super() and force redirect to `/` With this commit, website's user are redirected to /my as expected. Side effect: website's user will also be redirected to /my on login instead of `/`, which is also prefered. closes odoo/odoo#33865 Signed-off-by: Jérémy Kersten (jke) --- addons/website/controllers/main.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/website/controllers/main.py b/addons/website/controllers/main.py index 872cbd6dee2..639118cf73d 100644 --- a/addons/website/controllers/main.py +++ b/addons/website/controllers/main.py @@ -93,7 +93,7 @@ class Website(Home): if request.env['res.users'].browse(request.uid).has_group('base.group_user'): redirect = b'/web?' + request.httprequest.query_string else: - redirect = '/' + redirect = '/my' return http.redirect_with_hash(redirect) return response