Corrige le texte au survol des tutoriels non publiés (fix #2879)

[Situphen]

Q	R
Correction de bugs ?	oui
Nouvelle Fonctionnalité ?	non
Tickets (issues) concernés	#2879

Corrige le texte au survol des tutoriels non publiés

QA :

• Sur un item d'un tutoriel publié, au survol de la date, vérifier que la date s'affiche bien dans une infobulle ;
• Sur un item d'un tutoriel non publié, au survol de la date, vérifier qu'aucune date n'est affiché et qu'aucun "None" n'est affiché.

[pierre-24]

Rapport de QA: pas OK, j'ai "publié" même sur les tutoriels qui ne le sont pas

[Situphen]

Corrigé

[pierre-24]

Rapport de QA: ok pour moi, mais il faut bien garder en tête que manage.py load_fixtures type=tutorial ne donne pas de pubdate aux tutoriels. Du coup, quand on les génère par là, on a des résultats qui sont pas conforme ("en rédaction" à chaque fois). Deux solutions:

1. On considère qu'un "vrai" tuto a une pubdate et qu'on s'en fout (auquel cas je merge)
2. On change pour le "vrai" test qui est {{ if tutorial.on_line }} (vivement la ZEP-12)

[Situphen]

Oui mais si on utilise {{ if tutorial.on_line }}, on aura un "Publié None dans ..." en local donc ça ne le fera pas !

[pierre-24]

Ah, mais moi je suis d'avis de faire ça:

{% captureas tutorial_state %}
    {{ if tutorial.on_line }}
        {% trans "Publié" %} 
        {% if tutorial.pubdate %}
            {{ tutorial.pubdate|format_date }}
        {% endif %}
    {% else %}
        {% trans "En rédaction" %}
    {% endif %}
{% endcaptureas %}

Au moins c'est correct ;)

[Situphen]

Ça me va, mais est-ce que ça vaut vraiment le coup ? Un tutoriel publié à toujours une date de publication en production !

[pierre-24]

Comme tu le sens :)

[Situphen]

@SpaceFox : Tu en penses quoi ? Est-ce que ça vaut le coup d'utiliser le code de Pierre étant donné qu'en production un tutoriel publié devrait avoir toujours une date de publication ?

[SpaceFox]

étant donné qu'en production un tutoriel publié devrait avoir toujours une date de publication ?

Corrigé. C'est ça qu'il faut faire.

[Situphen]

Bon bah on est bon pour le merge je pense ?

Il faudra juste penser à corriger la fonction python manage.py load_fixtures pour qu'elle mette des dates de publication !

[pierre-24]

Vendu :)