Merge pull request #332 from savoirfairelinux/1679-structured-data

Fix #1679 - Add structured data
erudit · Jul 26, 2016 · 121d514 · 121d514
2 parents a15f8fb + b58530e
commit 121d514
Show file tree

Hide file tree

Showing 18 changed files with 513 additions and 41 deletions.
diff --git a/eruditorg/apps/public/journal/urls.py b/eruditorg/apps/public/journal/urls.py
@@ -49,7 +49,4 @@
 
     # Compatibility URLs
     url('^', include(urls_compat.urlpatterns)),
-
-    # Journal catchall
-    url(r'^(?P<code>[\w-]+)/$', views.JournalDetailView.as_view(), name='journal_detail'),
 ]
diff --git a/eruditorg/templates/base.html b/eruditorg/templates/base.html
@@ -46,6 +46,9 @@
     {% block meta_tags %}
     {% endblock meta_tags %}
 
+    {% block structured_data %}
+    {% endblock structured_data %}
+
     {% block stylesheets %}
     {% endblock stylesheets %}
 

diff --git a/eruditorg/templates/public/base.html b/eruditorg/templates/public/base.html
@@ -4,6 +4,23 @@
 {% load rules %}
 {% load base_urls_tags %}
 
+{% block structured_data %}
+<script type="application/ld+json">
+{
+  "@context" : "http://schema.org",
+  "@type" : "Organization",
+  "name" : "Érudit",
+  "url" : "{{ request.scheme }}://{{ request.site.domain }}",
+  "logo" : "{{ request.scheme }}://{{ request.site.domain }}{% static 'img/logo-erudit.png' %}",
+  "sameAs" : [
+    "https://www.facebook.com/%C3%89rudit-141469165320/",
+    "https://twitter.com/eruditorg",
+    "https://github.com/erudit/",
+    "https://www.linkedin.com/company/3628951"]
+}
+</script>
+{% endblock structured_data %}
+
 {% block stylesheets %}
 {% if not WEBPACK_DEV_SERVER_URL %}
 <link rel="stylesheet" type="text/css" href="{% static 'css/PublicApp.css' %}" />

diff --git a/eruditorg/templates/public/home.html b/eruditorg/templates/public/home.html
@@ -5,14 +5,44 @@
 
 {% block page_title %}{% trans "Accueil" %}{% endblock page_title %}
 
-{% block data_controller %}public:home{% endblock data_controller %}
-
-{% block body_class %}{{ block.super }} home{% endblock %}
+{% block structured_data %}
+{{ block.super }}
+<script type="application/ld+json">
+{
+  "@context" : "http://schema.org",
+  "@type" : "WebSite",
+  "name" : "Érudit",
+  "url" : "{{ request.scheme }}://{{ request.site.domain }}",
+  "potentialAction": {
+    "@type": "SearchAction",
+    "target": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:search:results' %}?basic_search_term={search_term_string}",
+    "query-input": "required name=search_term_string"
+  }
+}
+</script>
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "BreadcrumbList",
+  "itemListElement": [{
+    "@type": "ListItem",
+    "position": 1,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}",
+      "name": "Érudit"
+    }
+  }]
+}
+</script>
+{% endblock structured_data %}
 
 {% block extrahead %}
 <link rel="stylesheet" href="{% static 'css/videojs.css' %}" />
 {% endblock extrahead %}
 
+{% block data_controller %}public:home{% endblock data_controller %}
+{% block body_class %}{{ block.super }} home{% endblock %}
+
 {% block content %}
 {% trans "inconnu" as i18n_unknown %}
 <div id="homepage-content">

diff --git a/eruditorg/templates/public/journal/article_detail.html b/eruditorg/templates/public/journal/article_detail.html
@@ -1,4 +1,4 @@
-{% extends "public/journal/journal_base.html" %}
+{% extends "public/base.html" %}
 {% load i18n staticfiles %}
 {% load public_journal_tags %}
 {% load cache %}
@@ -65,20 +65,69 @@
 {% endcache %}
 {% endblock meta_tags %}
 
+{% block structured_data %}
+{{ block.super }}
+{% cache FOREVER_TTL "public_article_detail_structured_data" article.id LANGUAGE_CODE %}
+{% include "public/journal/partials/article_structured_data.html" %}
+{% endcache %}
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "BreadcrumbList",
+  "itemListElement": [{
+    "@type": "ListItem",
+    "position": 1,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}",
+      "name": "Érudit"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 2,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_list' %}",
+      "name": "{% trans 'Revues' %}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 3,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_detail' article.issue.journal.code %}",
+      "name": "{{ article.issue.journal.name }}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 4,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:issue_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier %}",
+      "name": "{{ article.issue.volume_title_with_pages }}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 5,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:article_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier article.localidentifier %}",
+      "name": "{{ article.title }}"
+    }
+  }]
+}
+</script>
+{% endblock structured_data %}
+
 {% block body_class %}public static-header{% endblock %}
 
 {% block header_class %}static{% endblock %}
 
 {% block breadcrumb %}{{ block.super }}
- / <a href="{% url 'public:journal:journal_detail' article.issue.journal.code %}">{{ article.issue.journal.name }}</a>
- / <a href="{% url 'public:journal:issue_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier %}">
+  / <a href="{% url 'public:journal:journal_detail' article.issue.journal.code %}">{{ article.issue.journal.name }}</a>
+  / <a href="{% url 'public:journal:issue_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier %}">
    {% if article.issue.erudit_object.html_theme %}
    {{ article.issue.erudit_object.html_theme|safe|truncatechars:50 }}
    {% else %}
    {{ article.issue.volume_title_with_pages }}
    {% endif %}
-   </a>
- / <a>{{ article.title|truncatechars:50 }}</a>
+  </a>
+  / <a href="{% url 'public:journal:article_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier article.localidentifier %}">{{ article.title|truncatechars:50 }}</a>
 {% endblock %}
 
 {% block content %}

diff --git a/eruditorg/templates/public/journal/article_summary.html b/eruditorg/templates/public/journal/article_summary.html
@@ -1,4 +1,4 @@
-{% extends "public/journal/journal_base.html" %}
+{% extends "public/base.html" %}
 {% load i18n staticfiles %}
 {% load public_journal_tags %}
 {% load cache %}
@@ -63,20 +63,62 @@
 {% endcache %}
 {% endblock meta_tags %}
 
+{% block structured_data %}
+{{ block.super }}
+{% cache FOREVER_TTL "public_article_summary_structured_data" article.id LANGUAGE_CODE %}
+{% include "public/journal/partials/article_structured_data.html" %}
+{% endcache %}
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "BreadcrumbList",
+  "itemListElement": [{
+    "@type": "ListItem",
+    "position": 1,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}",
+      "name": "Érudit"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 2,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_list' %}",
+      "name": "{% trans 'Revues' %}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 3,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_detail' article.issue.journal.code %}",
+      "name": "{{ article.issue.journal.name }}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 5,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:article_summary' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier article.localidentifier %}",
+      "name": "{{ article.title }}"
+    }
+  }]
+}
+</script>
+{% endblock structured_data %}
+
 {% block body_class %}public static-header{% endblock %}
 
 {% block header_class %}static{% endblock %}
 
 {% block breadcrumb %}{{ block.super }}
- / <a href="{% url 'public:journal:journal_detail' article.issue.journal.code %}">{{ article.issue.journal.name }}</a>
- / <a href="{% url 'public:journal:issue_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier %}">
+  / <a href="{% url 'public:journal:journal_detail' article.issue.journal.code %}">{{ article.issue.journal.name }}</a>
+  / <a href="{% url 'public:journal:issue_detail' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier %}">
    {% if article.issue.erudit_object.html_theme %}
    {{ article.issue.erudit_object.html_theme|safe|truncatechars:50 }}
    {% else %}
    {{ article.issue.volume_title_with_pages }}
    {% endif %}
-   </a>
- / <a>{% blocktrans with title=article.title|truncatechars:50 %}{{ title }} - Résumé{% endblocktrans %}</a>
+  </a>
+  / <a href="{% url 'public:journal:article_summary' article.issue.journal.code article.issue.volume_slug article.issue.localidentifier article.localidentifier %}">{% blocktrans with title=article.title|truncatechars:50 %}{{ title }} - Résumé{% endblocktrans %}</a>
 {% endblock %}
 
 {% block content %}

diff --git a/eruditorg/templates/public/journal/issue_detail.html b/eruditorg/templates/public/journal/issue_detail.html
@@ -1,4 +1,4 @@
-{% extends "public/journal/journal_base.html" %}
+{% extends "public/base.html" %}
 {% load i18n %}
 {% load staticfiles %}
 {% load cache %}
@@ -7,10 +7,7 @@
 
 {% block zone_title %}{{ journal.name }}{% endblock %}
 
-{% block page_title %}
-{% if issue.erudit_object.html_theme %}{{ issue.erudit_object.html_theme|safe }}. {% endif %}
-{{ issue.volume_title_with_pages }}
-{% endblock page_title %}
+{% block page_title %}{% if issue.erudit_object.html_theme %}{{ issue.erudit_object.html_theme|safe }}. {% endif %}{{ issue.volume_title_with_pages }} - {{ issue.journal.name }}{% endblock page_title %}
 
 {% block meta_tags %}
 <!-- Facebook / Open Graph -->
@@ -28,15 +25,55 @@
 <meta name="twitter:title" content="{{ issue.erudit_object.html_theme|safe }} – Érudit">
 {% endblock meta_tags %}
 
+{% block structured_data %}
+{{ block.super }}
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "BreadcrumbList",
+  "itemListElement": [{
+    "@type": "ListItem",
+    "position": 1,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}",
+      "name": "Érudit"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 2,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_list' %}",
+      "name": "{% trans 'Revues' %}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 3,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_detail' issue.journal.code %}",
+      "name": "{{ issue.journal.name }}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 4,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:issue_detail' issue.journal.code issue.volume_slug issue.localidentifier %}",
+      "name": "{{ issue.volume_title_with_pages }}"
+    }
+  }]
+}
+</script>
+{% endblock structured_data %}
+
 {% block breadcrumb %}{{ block.super }}
-/
-<a href="{% url 'public:journal:issue_detail' issue.journal.code issue.volume_slug issue.localidentifier %}">
-  {% if issue.erudit_object.html_theme %}
-  {{ issue.erudit_object.html_theme|safe|truncatechars:50 }}
-  {% else %}
-  {{ issue.volume_title_with_pages }}
-  {% endif %}
-</a>
+  / <a href="{% url 'public:journal:journal_list' %}">{% trans "Revues" %}</a>
+  / <a href="{% url 'public:journal:journal_detail' journal.code %}">{{ issue.journal.name }}</a>
+  / <a href="{% url 'public:journal:issue_detail' issue.journal.code issue.volume_slug issue.localidentifier %}">
+    {% if issue.erudit_object.html_theme %}
+    {{ issue.erudit_object.html_theme|safe|truncatechars:50 }}
+    {% else %}
+    {{ issue.volume_title_with_pages }}
+    {% endif %}
+  </a>
 {% endblock %}
 
 {% block content %}

diff --git a/eruditorg/templates/public/journal/journal_authors_list.html b/eruditorg/templates/public/journal/journal_authors_list.html
@@ -2,7 +2,52 @@
 {% load i18n %}
 {% load public_journal_tags %}
 
-{% block title %}{% trans "Index des auteurs" %}{% endblock title %}
+{% block title %}{% trans "Index des auteurs" %} - {{ journal.name }}{% endblock title %}
+
+{% block structured_data %}
+{{ block.super }}
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "BreadcrumbList",
+  "itemListElement": [{
+    "@type": "ListItem",
+    "position": 1,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}",
+      "name": "Érudit"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 2,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_list' %}",
+      "name": "{% trans 'Revues' %}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 3,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_detail' journal.code %}",
+      "name": "{{ article.issue.journal.name }}"
+    }
+  },{
+    "@type": "ListItem",
+    "position": 4,
+    "item": {
+      "@id": "{{ request.scheme }}://{{ request.site.domain }}{% url 'public:journal:journal_authors_list' journal.code %}",
+      "name": "{% trans 'Auteurs' %}"
+    }
+  }]
+}
+</script>
+{% endblock structured_data %}
+
+{% block breadcrumb %}
+  {{ block.super }} / <a href="{% url 'public:journal:journal_list' %}">{% trans "Revues" %}</a>
+  / <a href="{% url 'public:journal:journal_detail' journal.code %}">{{ journal.name }}</a>
+  / <a href="{% url 'public:journal:journal_authors_list' journal.code %}">{% trans "Auteurs" %}</a>
+{% endblock %}
 
 {% block content %}
 <div class="container">

diff --git a/eruditorg/templates/public/journal/journal_base.html b/eruditorg/templates/public/journal/journal_base.html