Ajoute des champs DCMI aux pages des contenus - Rends le site compatible avec Zotero #4392

rezemika · 2017-07-03T23:29:25Z

Q	R
Type de modification	nouvelle fonctionnalité
Ticket(s) (issue(s)) concerné(s)	#4261

QA

Installer l'extension Firefox Zotero ou un équivalent.
Utiliser le bouton idoine pour enregistrer dans Zotero divers contenus du site. Vérifier le formatage des informations pour :
- chaque type de contenu ;
- un contenu avec un seul auteur, deux et plus de deux auteurs ;
- un contenu avec et un contenu sans description ;
- un contenu avec une source externe.

Voici le formatage que j'ai utilisé pour les champs DCMI, à modifier au besoin. La liste des auteurs est de type "Auteur 1, Auteur 2 et Auteur 3". Celle des tags est de type "Tag 1 ; Tag 2 ; Tag 3".

DC.publisher = "Zeste de Savoir"
DC.creator = [liste des auteurs du contenu]
DC.type = "text"
DC.title = [titre du contenu]
DC.abstract = [titre et description du contenu si disponnible]
DC.subject = [liste des catégories et des tags du contenu]
DC.date = [date de publication ou de dernière mise à jour du contenu]
DC.format = "text/html"
DC.source = [source du contenu si disponnible]
DC.language = "fr"
DC.rights = [licence du contenu]

PS : J'ai eu quelques soucis lors du merge. J'espère que ça ne posera pas de problème ici.

Ça fonctionne !
Code relu et approuvé !

gllmc

Ça semble marcher sans problème de mon côté. Il y a quelques machins que j'améliorerais bien et des erreurs PEP-8 à corriger. :)

gllmc · 2017-07-04T12:36:12Z

templates/tutorialv2/view/content_online.html

-
+{% block DCMI_cards %}
+    <link rel="schema.DC" href="http://purl.org/dc/elements/1.1/" />
+    <meta name="DC.publisher" lang="fr" content="{{app.site.name}}" />


Plutôt {{ app.site.litteral_name }} pour un rendu plus correct ("Zeste de Savoir" au lieu de "ZesteDeSavoir").

gllmc · 2017-07-04T12:40:22Z

templates/base.html

@@ -721,7 +725,7 @@ <h2 class="subtitle" {% if schema %}itemprop="description"{% endif %}>{{ headlin
                    {% endif %}
                    {% if app.site.social.googleplus %}
                        <li><a href="{{app.site.social.googleplus}}" title="Retrouvez-nous sur Google+" class="btn ico-after google-plus light btn-google-plus btn-holder"></a></li>
-		    {% endif %}
+            {% endif %}


Je n'ai pas vraiment compris cette modification, mais j'ai une indentation incorrecte sur ta branche :

Actuellement :

{% if app.site.social.googleplus %} <li><a href="{{app.site.social.googleplus}}" title="Retrouvez-nous sur Google+" class="btn ico-after google-plus light btn-google-plus btn-holder"></a></li> {% endif %}

Correct :

{% if app.site.social.googleplus %} <li><a href="{{app.site.social.googleplus}}" title="Retrouvez-nous sur Google+" class="btn ico-after google-plus light btn-google-plus btn-holder"></a></li> {% endif %}

Ah oui en effet, ça doit être une fausse manip', désolé.

gllmc · 2017-07-04T12:42:36Z

zds/utils/templatetags/joinby.py

+
+
+@register.simple_tag
+def joinby(values, separator=", ", same_final_separator=False):


PEP-8 : il faudrait des guillemets simples.

gllmc · 2017-07-04T12:42:51Z

zds/utils/templatetags/joinby.py

+    by a comma.
+    """
+    if not values:
+        return ""


PEP-8 : il faudrait des guillemets simples.

gllmc · 2017-07-04T12:44:07Z

zds/utils/templatetags/joinby.py

+    if same_final_separator:
+        final_sep = separator
+    else:
+        final_sep = unicode(_(" et "))


PEP-8 : il faudrait des guillemets simples. Je n'ai pas vraiment compris pourquoi tu as utilisé unicode.

Pour le unicode, c'est parce que j'avais une erreur étrange à la ligne d'après quand je concatenais final_sep avec l'autre string.

Voilà ce que j'obtenais : TypeError: coercing to Unicode: need string or buffer, __proxy__ found. Je n'ai pu résoudre ça qu'en rajoutant un unicode(), j'ignores pourquoi.

Dans ce cas, essaye de rajouter sous l'encodage la ligne from __future__ import unicode_literals, je pense que ça devrait marcher. ;)

Update : J'avais bêtement oublié de mettre à jour mon virtualenv avec les derniers paquets utilisés par ZdS. Je l'ai mis à jour et ça marche correctement sans nouvelle importation. :)

gllmc · 2017-07-04T12:44:48Z

zds/utils/templatetags/joinby.py

+        final_sep = separator
+    else:
+        final_sep = unicode(_(" et "))
+    return separator.join(map(str, values[:len(values)-1])) + (


PEP-8 : il faut des espaces autour des opérateur mathématiques. ;)

gllmc · 2017-07-04T12:51:56Z

zds/utils/templatetags/joinby.py

+    if not values:
+        return ""
+    if len(values) == 1:
+        return values[0].__str__()


- return values[0].__str__() + return str(values[0])

gllmc · 2017-07-04T12:52:57Z

zds/utils/templatetags/joinby.py

+    else:
+        final_sep = unicode(_(" et "))
+    return separator.join(map(str, values[:len(values)-1])) + (
+        final_sep + values.last().__str__()


- values.last().__str__() + str(values.last())

gllmc · 2017-07-04T12:55:24Z

templates/tutorialv2/view/content_online.html

+    <meta name="DC.type" content="text" />
+    <meta name="DC.title" content="{{ content.title }}" />
+    <meta name="DC.abstract" content="{{ content.type.title }} : {{ content.title }}{% if content.description %} - {{ content.description }}{% endif %}" />
+    <meta name="DC.subject" lang="fr" content="{% joinby content.subcategory.all %}{% if content.tags.all %} - {% joinby content.tags.all separator=' ; ' same_final_separator=True %}{% endif %}" />


Pour ces deux lignes, j'utiliserais plutôt le tiret – au lieu de -. :)

rezemika · 2017-07-06T16:22:00Z

Changements effectués, désolé pour le délai. :)

Je suis passé par Github directement, d'où le triple commit.

gllmc

Il y a juste un oubli qui a provoqué l'échec de Travis. Sinon, tout est OK.

Un rebase est requis par contre :

git fetch upstream
git rebase upstream/dev

gllmc · 2017-07-09T20:24:45Z

zds/utils/templatetags/joinby.py

+    if same_final_separator:
+        final_sep = separator
+    else:
+        final_sep = _(" et ")


L'erreur de Travis vient d'ici, il faut des guillemets simples. :)

Ah ? Il me semblait que c'était pythonique de mettre des guillemets doubles pour des mots. Je refais un commit.

@rezemika : en fait, on met en règle générale des guillemets simples. On met des guillemets doubles quand il y a une apostrophe dans la chaîne (pour éviter d'avoir à utiliser un antislash). ;)

Ah ok. De mon côté j'ai toujous mis des guillemets doubles pour les mots ou les phrases (genre "Hello world !") et des simples pour le reste (les séparateurs pour les split, ce genre de choses). My bad.

Ce n'est pas vraiment un problème, ça. La règle que j'ai citée est celle qui a été choisie pour ce dépôt, mais PEP-8 est beaucoup plus permissif (d'où le fait qu'une règle ait été décidée afin de garder une certaine cohérence dans l'utilisation des guillemets).

Ah je vois. Merci de cette précision. :)

coveralls · 2017-07-09T20:46:32Z

Coverage decreased (-15.6%) to 73.496% when pulling 74018ac on rezemika:zds-rezemika into c708844 on zestedesavoir:dev.

coveralls · 2017-08-02T11:44:27Z

Coverage decreased (-15.8%) to 73.496% when pulling 840cdb5 on rezemika:zds-rezemika into 82c2e6f on zestedesavoir:dev.

pierre-24 · 2017-08-02T11:53:28Z

Ah, y'a un test à réparer, ici.

pierre-24 · 2017-08-04T13:25:58Z

templates/tutorialv2/view/content_online.html

+{% block DCMI_cards %}
+    <link rel="schema.DC" href="http://purl.org/dc/elements/1.1/" />
+    <meta name="DC.publisher" lang="fr" content="{{ app.site.litteral_name }}" />
+    <meta name="DC.creator" content="{% joinby content.authors.all %}" />


La gestion des auteurs est un peu particulière : pour que ça fonctionne, ça devrait probablement être joinby content|displayable_authors:True. Ça provient du fait que les auteurs peuvent varier en fonction des mises à jour d'un tuto.

Plus d'infos ici

pierre-24 · 2017-08-04T13:26:55Z

zds/utils/templatetags/joinby.py

@@ -0,0 +1,28 @@
+# coding: utf-8


Si tu ajoute un templatetag, il faut le documenter ici ;)

coveralls · 2017-08-04T15:19:53Z

Coverage decreased (-0.02%) to 89.261% when pulling 050d9c4 on rezemika:zds-rezemika into 82c2e6f on zestedesavoir:dev.

coveralls · 2017-08-04T15:29:31Z

Coverage decreased (-0.02%) to 89.261% when pulling f8950e5 on rezemika:zds-rezemika into fcf26ba on zestedesavoir:dev.

pierre-24 · 2017-08-04T15:40:34Z

Ben ça me semble fonctionner pas mal :)

…ble avec Zotero (#4392) * Ajoute des champs DCMI aux pages des contenus (#4261) * Corrige le commit précédent pour les champs DCMI (#4392) * Corrige le commit précédent pour les champs DCMI (#4392) * Corrige le commit précédent pour les champs DCMI (#4392) * Remplace les guillemets simples par des guillemets doubles * Ajoute la documentation du tag 'joinby' * N'affiche que les auteurs de la version courante * Corrige une variable dans le template

Ajoute des champs DCMI aux pages des contenus (zestedesavoir#4261)

e1e2a84

rezemika force-pushed the zds-rezemika branch from 7e0d339 to e1e2a84 Compare July 4, 2017 12:51

gllmc suggested changes Jul 4, 2017

View reviewed changes

rezemika added 3 commits July 6, 2017 18:09

Corrige le commit précédent pour les champs DCMI (zestedesavoir#4392)

40fcd6b

Corrige le commit précédent pour les champs DCMI (zestedesavoir#4392)

4064127

Corrige le commit précédent pour les champs DCMI (zestedesavoir#4392)

950821b

gllmc reviewed Jul 9, 2017

View reviewed changes

Remplace les guillemets simples par des guillemets doubles

74018ac

rezemika added a commit to rezemika/zds-site that referenced this pull request Jul 11, 2017

Corrige le commit précédent pour les champs DCMI (zestedesavoir#4392)

ebe244b

rezemika added a commit to rezemika/zds-site that referenced this pull request Jul 11, 2017

Corrige le commit précédent pour les champs DCMI (zestedesavoir#4392)

1075f7c

rezemika added a commit to rezemika/zds-site that referenced this pull request Jul 11, 2017

Corrige le commit précédent pour les champs DCMI (zestedesavoir#4392)

ee2d1c6

Merge branch 'dev' into zds-rezemika

840cdb5

pierre-24 reviewed Aug 4, 2017

View reviewed changes

pierre-24 suggested changes Aug 4, 2017

View reviewed changes

zds/utils/templatetags/joinby.py

@@ -0,0 +1,28 @@

# coding: utf-8

Copy link

Member

pierre-24 Aug 4, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si tu ajoute un templatetag, il faut le documenter ici ;)

rezemika added 3 commits August 4, 2017 15:59

Ajoute la documentation du tag 'joinby'

ee2dd8e

N'affiche que les auteurs de la version courante

88d2051

Corrige une variable dans le template

050d9c4

Merge branch 'dev' into zds-rezemika

f8950e5

gllmc approved these changes Aug 4, 2017

View reviewed changes

pierre-24 approved these changes Aug 4, 2017

View reviewed changes

pierre-24 merged commit a7ef93e into zestedesavoir:dev Aug 4, 2017

pierre-24 added this to the Version de développement milestone Aug 4, 2017

pierre-24 added the C-Front Concerne l'interface du site label Aug 4, 2017

pierre-24 added the S-Évolution label Aug 4, 2017

rezemika deleted the zds-rezemika branch January 8, 2018 20:13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ajoute des champs DCMI aux pages des contenus - Rends le site compatible avec Zotero #4392

Ajoute des champs DCMI aux pages des contenus - Rends le site compatible avec Zotero #4392

rezemika commented Jul 3, 2017 •

edited by pierre-24

Loading

gllmc left a comment

gllmc Jul 4, 2017

gllmc Jul 4, 2017

rezemika Jul 4, 2017

gllmc Jul 4, 2017

gllmc Jul 4, 2017

gllmc Jul 4, 2017

rezemika Jul 4, 2017

gllmc Jul 4, 2017

rezemika Jul 4, 2017

gllmc Jul 4, 2017

gllmc Jul 4, 2017

gllmc Jul 4, 2017

gllmc Jul 4, 2017

rezemika commented Jul 6, 2017 •

edited

Loading

gllmc left a comment

gllmc Jul 9, 2017

rezemika Jul 9, 2017

gllmc Jul 9, 2017

rezemika Jul 9, 2017

gllmc Jul 9, 2017

rezemika Jul 9, 2017

coveralls commented Jul 9, 2017 •

edited

Loading

coveralls commented Aug 2, 2017

pierre-24 commented Aug 2, 2017

pierre-24 Aug 4, 2017

pierre-24 Aug 4, 2017

pierre-24 Aug 4, 2017

coveralls commented Aug 4, 2017 •

edited

Loading

coveralls commented Aug 4, 2017 •

edited

Loading

pierre-24 commented Aug 4, 2017



		@register.simple_tag
		def joinby(values, separator=", ", same_final_separator=False):

Ajoute des champs DCMI aux pages des contenus - Rends le site compatible avec Zotero #4392

Ajoute des champs DCMI aux pages des contenus - Rends le site compatible avec Zotero #4392

Conversation

rezemika commented Jul 3, 2017 • edited by pierre-24 Loading

QA

gllmc left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rezemika commented Jul 6, 2017 • edited Loading

gllmc left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

coveralls commented Jul 9, 2017 • edited Loading

coveralls commented Aug 2, 2017

pierre-24 commented Aug 2, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

coveralls commented Aug 4, 2017 • edited Loading

coveralls commented Aug 4, 2017 • edited Loading

pierre-24 commented Aug 4, 2017

rezemika commented Jul 3, 2017 •

edited by pierre-24

Loading

rezemika commented Jul 6, 2017 •

edited

Loading

coveralls commented Jul 9, 2017 •

edited

Loading

coveralls commented Aug 4, 2017 •

edited

Loading

coveralls commented Aug 4, 2017 •

edited

Loading