[FIX] website_forum: redirect /forum/x/question/y simplify

Don't try to keep extra params and co. Keep it simple...

Else we should pop from request.params `forum` and `post` keys because
now they are converted as query param with the slug format:

/forum/help-1/question-1?forum=forum.forum(1,)&question=forum.post(1,)

closes odoo/odoo#156910

X-original-commit: b62bd391ccf754da9411ca5e8bff9cfafeed2a66
Signed-off-by: Jérémy Kersten <jke@odoo.com>
This commit is contained in:
Jeremy Kersten
2024-03-07 19:06:54 +00:00
parent 8c13a26cd3
commit 8da2d4a092
@@ -11,7 +11,6 @@ import werkzeug.wrappers
from odoo import _, http, tools
from odoo.addons.http_routing.models.ir_http import slug
from odoo.addons.portal.controllers.portal import _build_url_w_params
from odoo.addons.website.models.ir_http import sitemap_qs2dom
from odoo.addons.website_profile.controllers.main import WebsiteProfile
from odoo.exceptions import AccessError, UserError
@@ -281,7 +280,7 @@ class WebsiteForum(WebsiteProfile):
type='http', auth="public", website=True, sitemap=False)
def old_question(self, forum, question, **post):
# Compatibility pre-v14
return request.redirect(_build_url_w_params("/forum/%s/%s" % (slug(forum), slug(question)), request.params), code=301)
return request.redirect("/forum/%s/%s" % (slug(forum), slug(question)), code=301)
def sitemap_forum_post(env, rule, qs):
ForumPost = env['forum.post']