diff options
| author | Jeremy Kersten <jke@odoo.com> | 2016-01-08 23:53:32 +0100 |
|---|---|---|
| committer | Jeremy Kersten <jke@odoo.com> | 2016-01-08 23:55:44 +0100 |
| commit | 91aa4505331da18adbfefbd7e1fbdb6bd069c890 (patch) | |
| tree | 28fcb7502100facfa3d8205816b19ad81ee2ae64 /addons/website_slides | |
| parent | ce3f7970686c5658c3fd72a67d5ee06ce61849d1 (diff) | |
[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.
Diffstat (limited to 'addons/website_slides')
| -rw-r--r-- | addons/website_slides/controllers/main.py | 4 |
1 files 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/<model("slide.slide"):slide>/pdf_content', type='http', auth="public", website=True) + @http.route('''/slides/slide/<model("slide.slide", "[('datas', '!=', False), ('slide_type', '=', 'presentation')]"):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 |