Issue 5329 - Added standard social media meta data to website-defaults

includes/meta/social.tmpl

-{# open graph #}
+{#
+ # This file is part of website-defaults
+ # Copyright (C) 2016-2017 eyeo GmbH
+ #
+ # website-defaults is free software: you can redistribute it and/or
+ # modify it under the terms of the GNU General Public License as published by
+ # the Free Software Foundation, either version 3 of the License, or
+ # (at your option) any later version.
+ #
+ # website-defaults is distributed in the hope that it will be useful,
+ # but WITHOUT ANY WARRANTY; without even the implied warranty of
+ # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ # GNU General Public License for more details.
+ #
+ # You should have received a copy of the GNU General Public License
+ # along with website-defaults.  If not, see <http://www.gnu.org/licenses/>.
+ #}
 
-{% if host and page %}
-  <meta property="og:url" content="{{ host }}/{{ page }}">

[juliandoucette, 2017/07/05 21:47:25]

Note: I created an issue to provide us with a better way to do this (instead of
using a global `host`)

[ire, 2017/07/07 16:05:06]

Acknowledged.

-{% endif %}
-{% if og_type %}
-  <meta property="og:type" content="{{ og_type }}">
-{% endif %}
-{% if og_site_name %}
-  <meta property="og:site_name" content="{{ og_site_name }}">
-{% endif %}
-{% if og_title %}
-  <meta property="og:title" content="{{ og_title }}">
-{% endif %}
-{% if og_description %}
-  <meta property="og:descirption" content="{{ og_description }}">

[ire, 2017/07/07 16:05:06]

Typo in "og:descirption"

[juliandoucette, 2017/07/07 20:05:20]

Done.

Good eye.

[ire, 2017/07/10 07:50:57]

Thanks :)

-{% endif %}
-{% if og_image %}
-  <meta property="og:image" content="{{ og_image }}">
-{% else if og_site_image %}
-  <meta property="og:image" content="{{ og_site_image }}">
-{% endif %}
-<meta property="og:locale" content="{{ locale }}">

[juliandoucette, 2017/07/05 21:47:25]

Note: I'm not sure if the accepted locales are the same as the locales we
provide.

[ire, 2017/07/07 16:05:06]

Should this be wrapped in an if statement? Or is locale a required variable?

[juliandoucette, 2017/07/07 20:05:20]

locale is always set. A default must be set in settings.ini.

[ire, 2017/07/10 07:50:57]

Acknowledged.

-{% for alternate_locale in locales %}
+{% macro meta_tag(property, content, defaultContent=None) -%}
+  {% if content %}
+    <meta property="{{ property }}" content="{{ content }}">
+  {% elif defaultContent %}
+    <meta property="{{ property }}" content="{{ defaultContent }}">
+  {% endif %}
+{%- endmacro %}
+
+{% set social_media_meta = [
+  ('og:url', og_url, get_canonical_url(page)),
+  ('og:type', og_type, 'website'),
+  ('og:site_name', og_site_name, get_string('site-name', 'global') if has_string('site-name', 'global') else None),
+  ('og:title', og_title, title | translate('page-title', 'Meta data') if title else None),
+  ('og:description', og_description, description | translate('page-description', 'Meta data') if description else None),
+  ('og:image', og_image, image),
+  ('og:image:alt', og_image_alt, image_alt),
+  ('og:locale', og_locale, locale | to_og_locale),
+  ('fb:app_id', facebook_id, None),
+  ('twitter:site', twitter_site, '@eyeo'),
+  ('twitter:card', twitter_card, 'summary_large_image' if og_image or twitter_image or image else 'summary'),
+  ('twitter:image', twitter_image, None),
+  ('twitter:image:alt', twitter_image_alt, None),
+  ('p:domain_verify', pinterest_id, None),
+] %}
+
+{% for property, content, defaultContent in social_media_meta %}
+  {{ meta_tag(property, content, defaultContent) }}
+{% endfor %}
+
+{% for alternate_locale in available_locales %}
   {% if alternate_locale != locale %}
-    <meta property="og:locale:alternate" content="{{ alternate_locale }}">
+    {{ meta_tag(('og:locale:alternate', alternate_locale | to_og_locale)) }}
   {% endif %}
 {% endfor %}
-
-{# twitter #}
-
-{% if twitter_site %}
-  <meta name="twitter:site" content="@{{ twitter_site }}">
-{% endif %}
-{% if twitter_card %}
-  <meta name="twitter:card" content="{{ twitter_card }}">
-{% else if og_image or og_site_image %}
-  <meta name="twitter:card" content="summary_large_image">
-{% else %}
-  <meta name="twitter:card" content="summary">
-{% endif %}
-{% if twitter_image_alt %}
-  <meta name="twitter:image:alt" content="{{ twitter_image_alt }}">
-{% endif %}