From c016078f7b837e261cb8820ea709c4351d296074 Mon Sep 17 00:00:00 2001 From: David Monjoie Date: Wed, 18 Nov 2015 10:57:50 +0100 Subject: [PATCH] [FIX] website: no login_success on oauth redirect It seems like nobody other than me ever had issues with this line since its introduction in January 2015, so I just swapped the conditions, since this param is seemingly always present, unless you are using oauth with redirect. --- 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 587ac962846..e99d443f54c 100644 --- a/addons/website/controllers/main.py +++ b/addons/website/controllers/main.py @@ -53,7 +53,7 @@ class Website(openerp.addons.web.controllers.main.Home): @http.route(website=True, auth="public") def web_login(self, redirect=None, *args, **kw): r = super(Website, self).web_login(redirect=redirect, *args, **kw) - if request.params['login_success'] and not redirect: + if not redirect and request.params['login_success']: if request.registry['res.users'].has_group(request.cr, request.uid, 'base.group_user'): redirect = '/web?' + request.httprequest.query_string else: