From 6a5ee406d81513e20baf1bdd610ca56c8348d6ec Mon Sep 17 00:00:00 2001
From: David Beguin <dbe@odoo.com>
Date: Tue, 18 Aug 2020 09:59:22 +0000
Subject: [PATCH] [IMP] website_event_*: clean sitemap

From now, you need to explicitely add sitemap=True if you want your controller
into the sitemap.  It's the default value, but if you forgot it, it will raise
a Warning on runbot. It will avoid wrong controller in sitemap and duplicate
(empty) content.  From now, if your model contains a field website_id, the
modelConverter for sitemap will automatically add the domain:
"[('website_id', 'in', (False, current_website_id))]"

It avoid redundant declaration and ugly url in redirect/rewrite view.

Task ID: 2318597

closes odoo/odoo#56114

X-original-commit: aabd31d1523fb6ca68336cc66a6e2f074173f396
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Signed-off-by: Christophe Monniez (moc) <moc@odoo.com>
---
 addons/website_event_meet/controllers/community.py            | 2 +-
 addons/website_event_track_exhibitor/controllers/exhibitor.py | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/addons/website_event_meet/controllers/community.py b/addons/website_event_meet/controllers/community.py
index 291a6017dfc..6a3d17e3043 100644
--- a/addons/website_event_meet/controllers/community.py
+++ b/addons/website_event_meet/controllers/community.py
@@ -114,7 +114,7 @@ class WebsiteEventMeetController(EventCommunityController):
     # ROOM PAGE VIEW
     # ------------------------------------------------------------
 
-    @http.route("/event/<model('event.event'):event>/meeting_room/<model('event.meeting.room'):meeting_room>",
+    @http.route('''/event/<model('event.event', "[('community_menu', '=', True)]"):event>/meeting_room/<model("event.meeting.room","[('event_id','=',event.id)]"):meeting_room>''',
                 type="http", auth="public", website=True, sitemap=True)
     def event_meeting_room_page(self, event, meeting_room, **post):
         """Display the meeting room frontend view.
diff --git a/addons/website_event_track_exhibitor/controllers/exhibitor.py b/addons/website_event_track_exhibitor/controllers/exhibitor.py
index 4f96c90e2da..08fb71862cf 100644
--- a/addons/website_event_track_exhibitor/controllers/exhibitor.py
+++ b/addons/website_event_track_exhibitor/controllers/exhibitor.py
@@ -108,7 +108,8 @@ class ExhibitorController(EventTrackController):
     # FRONTEND FORM
     # ------------------------------------------------------------
 
-    @http.route(['/event/<model("event.event"):event>/exhibitor/<model("event.sponsor"):sponsor>'], type='http', auth="public", website=True, sitemap=False)
+    @http.route(['''/event/<model("event.event", "[('exhibitor_menu', '=', True)]"):event>/exhibitor/<model("event.sponsor", "[('event_id', '=', event.id)]"):sponsor>'''],
+                type='http', auth="public", website=True, sitemap=True)
     def event_exhibitor(self, event, sponsor, **options):
         if not event.can_access_from_current_website():
             raise NotFound()