diff options
| author | Romain Derie <rde@odoo.com> | 2018-04-11 11:44:39 +0200 |
|---|---|---|
| committer | Romain Derie <rde@odoo.com> | 2018-04-11 15:04:24 +0200 |
| commit | c09b89596e71b657d16fec15f6950e2e58afe42b (patch) | |
| tree | 83a5e4e1d746a4381f313a1f660fcfdc2e2c25d7 /addons/website/controllers/main.py | |
| parent | 78058259419157ae22a8fe485adafc82b6de7f3a (diff) | |
[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
Diffstat (limited to 'addons/website/controllers/main.py')
| -rw-r--r-- | addons/website/controllers/main.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/addons/website/controllers/main.py b/addons/website/controllers/main.py index 45940ac4cc4..49041b42844 100644 --- a/addons/website/controllers/main.py +++ b/addons/website/controllers/main.py @@ -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 |