zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 162 forks source link

[Article] Aperçu d'une réaction d'un article #190

Closed GerardPaligot closed 10 years ago

GerardPaligot commented 10 years ago

Scénario :

Piste pour le bug : La valeur du texte de la réaction n'est pas affichée dans le template de la prévisualisation, voire le texte n'est pas envoyé au template.

firm1 commented 10 years ago

Il suffit d'aller à la ligne suivante et modifier le code suivant :

{% if text %}
    {% include "forum/previsualization.part.html" %}
{% endif %}

par

{% if form.text %}
    {% with text=form.text %}
        {% include "forum/previsualization.part.html" %}
    {% endwith %}
{% endif %}
GerardPaligot commented 10 years ago

Ah effectivement. C'est plus jolie que de passer un text au template.

(Merci beaucoup pour toutes les revues sur les issues. @Ge0 voulait justement commencer à contribuer)

Ge0 commented 10 years ago

Je confirme les dires de @GerardPaligot, je me remets dans le bain doucement. Merci :)

SpaceFox commented 10 years ago

C'est plus joli, mais ça marche pas :(

Donc soit je passe un text au template comme c'est fait partout ailleurs, soit je laisse quelqu'un d'autre regarder pourquoi la sollution de @firm1 me crache :

AttributeError at /articles/message/nouveau 'BoundField' object has no attribute 'strip'

sur la ligne 6 de "templates/forum/previsualization.part.html" (c'est le |emarkdown qui provoque cette erreur, sans ça la prévisualisation marche - mais sans le markdown.

Dans tous les cas, je n'ai pas la moindre idée de comment écrire un test automatique qui couvre ce cas.

SpaceFox commented 10 years ago

Corrigé, PR, mergé --> Fermé