From 91aa4505331da18adbfefbd7e1fbdb6bd069c890 Mon Sep 17 00:00:00 2001 From: Jeremy Kersten Date: Fri, 8 Jan 2016 23:53:32 +0100 Subject: [PATCH] [FIX] website_slide: add domain on controller pdf_content for SEO Add domain on route to avoid sitemap to index this route if nothing to display. Return empty data if route called for a slide without datas. --- addons/website_slides/controllers/main.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/website_slides/controllers/main.py b/addons/website_slides/controllers/main.py index 70f70d5c6b7..6190bbc675b 100644 --- a/addons/website_slides/controllers/main.py +++ b/addons/website_slides/controllers/main.py @@ -160,10 +160,10 @@ class website_slides(http.Controller): self._set_viewed_slide(slide, 'slide') return request.website.render('website_slides.slide_detail_view', values) - @http.route('/slides/slide//pdf_content', type='http', auth="public", website=True) + @http.route('''/slides/slide//pdf_content''', type='http', auth="public", website=True) def slide_get_pdf_content(self, slide): response = werkzeug.wrappers.Response() - response.data = slide.datas.decode('base64') + response.data = slide.datas and slide.datas.decode('base64') or '' response.mimetype = 'application/pdf' return response