From 704607db5a8b2b193a3cd6b47f17148e334bd9a6 Mon Sep 17 00:00:00 2001
From: Christophe Matthieu <chm@openerp.com>
Date: Tue, 3 Dec 2013 18:31:46 +0100
Subject: [PATCH] [FIX] website: signin and logout redirect url

bzr revid: chm@openerp.com-20131203173146-7om0xl9b7ubb5cqc
---
 addons/website/models/website.py           | 4 ++++
 addons/website/views/website_templates.xml | 4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/addons/website/models/website.py b/addons/website/models/website.py
index 2f58cb206bd..962a4bf5479 100644
--- a/addons/website/models/website.py
+++ b/addons/website/models/website.py
@@ -77,6 +77,9 @@ def urlplus(url, params):
         for k, v in params.iteritems()
     ))
 
+def quote_plus(value):
+    return urllib.quote_plus(value.encode('utf-8') if isinstance(value, unicode) else str(value))
+
 class website(osv.osv):
     def _get_menu_website(self, cr, uid, ids, context=None):
         # IF a menu is changed, update all websites
@@ -226,6 +229,7 @@ class website(osv.osv):
             slug=slug,
             res_company=request.website.company_id,
             user_id=user.browse(cr, uid, uid),
+            quote_plus=quote_plus,
         )
 
         context.update(
diff --git a/addons/website/views/website_templates.xml b/addons/website/views/website_templates.xml
index 05bb3a8d018..ddde76e0cf3 100644
--- a/addons/website/views/website_templates.xml
+++ b/addons/website/views/website_templates.xml
@@ -138,7 +138,7 @@
                                                     <ul class="dropdown-menu js_usermenu" role="menu">
                                                         <li><a href="/web" role="menuitem">Administration</a></li>
                                                         <li class="divider"/>
-                                                        <li><a t-attf-href="/web/session/logout?redirect=#{ url_for('') }" role="menuitem">Logout</a></li>
+                                                        <li><a t-attf-href="/web/session/logout?redirect=#{ quote_plus(url_for('', keep_query='*')) }" role="menuitem">Logout</a></li>
                                                     </ul>
                                                 </li>
                                             </ul>
@@ -227,7 +227,7 @@
         <template id="show_sign_in" inherit_option_id="website.layout" inherit_id="website.layout" name="Show Sign In">
             <xpath expr="//ul[@id='top_menu']" position="inside">
                 <li t-if="user_id.id == website.public_user.id">
-                    <a t-attf-href="/web#redirect=#{ url_for('') }">
+                    <a t-attf-href="/web#redirect=#{ quote_plus(url_for('', keep_query='*')) }">
                         <span class="badge">Sign in</span>
                     </a>
                 </li>