[FIX] website: keep query strings when redirecting 'website.' page
Urls like `/page/website.XXX` are redirected to `/page/XXX`. Before this commit, the redirection would not keep query strings: `/page/website.XXX?rde=1` would redirect to `/page/XXX`. Now, we keep the query strings: `/page/website.XXX?rde=1` will redirect to `/page/XXX?rde=1` Closes #24140 Courtesy of @MTantin
This commit is contained in:
@@ -76,7 +76,10 @@ class Website(openerp.addons.web.controllers.main.Home):
|
||||
}
|
||||
# /page/website.XXX --> /page/XXX
|
||||
if page.startswith('website.'):
|
||||
return request.redirect('/page/' + page[8:], code=301)
|
||||
url = '/page/' + page[8:]
|
||||
if request.httprequest.query_string:
|
||||
url += '?' + request.httprequest.query_string
|
||||
return request.redirect(url, code=301)
|
||||
elif '.' not in page:
|
||||
page = 'website.%s' % page
|
||||
|
||||
|
||||
Reference in New Issue
Block a user