Skip to content

Commit b02bd28

Browse files
committed
[FIX] website: fix twitter image format and use it
Twitter image was wrongly computed, appending /300x300 after the unique parameters. Twitter image was always overwrite by `website_meta_og_img` or `opengraph_meta['og:image']` closes odoo#93953 X-original-commit: ffdfa26 Signed-off-by: Jérémy Kersten <[email protected]>
1 parent 39f619d commit b02bd28

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

addons/website/models/mixins.py

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -45,21 +45,22 @@ def _default_website_meta(self):
4545
title = (request.website or company).name
4646
if 'name' in self:
4747
title = '%s | %s' % (self.name, title)
48+
4849
img_field = 'social_default_image' if request.website.has_social_default_image else 'logo'
49-
img = request.website.image_url(request.website, img_field)
50+
5051
# Default meta for OpenGraph
5152
default_opengraph = {
5253
'og:type': 'website',
5354
'og:title': title,
5455
'og:site_name': company.name,
5556
'og:url': url_join(request.httprequest.url_root, url_for(request.httprequest.path)),
56-
'og:image': img,
57+
'og:image': request.website.image_url(request.website, img_field),
5758
}
5859
# Default meta for Twitter
5960
default_twitter = {
6061
'twitter:card': 'summary_large_image',
6162
'twitter:title': title,
62-
'twitter:image': img + '/300x300',
63+
'twitter:image': request.website.image_url(request.website, img_field, size='300x300'),
6364
}
6465
if company.social_twitter:
6566
default_twitter['twitter:site'] = "@%s" % company.social_twitter.split('/')[-1]
@@ -87,11 +88,8 @@ def get_website_meta(self):
8788
if self.website_meta_description:
8889
opengraph_meta['og:description'] = self.website_meta_description
8990
twitter_meta['twitter:description'] = self.website_meta_description
90-
meta_image = self.website_meta_og_img or opengraph_meta['og:image']
91-
if meta_image.startswith('/'):
92-
meta_image = "%s%s" % (root_url, meta_image)
93-
opengraph_meta['og:image'] = meta_image
94-
twitter_meta['twitter:image'] = meta_image
91+
opengraph_meta['og:image'] = url_join(root_url, url_for(self.website_meta_og_img or opengraph_meta['og:image']))
92+
twitter_meta['twitter:image'] = url_join(root_url, url_for(self.website_meta_og_img or twitter_meta['twitter:image']))
9593
return {
9694
'opengraph_meta': opengraph_meta,
9795
'twitter_meta': twitter_meta,

0 commit comments

Comments
 (0)