Produits Entr'ouvert » w.c.s.

wip, le début, pour valider ma piste :

• 0001 : ajoute un nouvel objet Template qui mange du Django et de l'ezt et effectue la vérification syntaxique et le rendu
• 0002 : utilisation par tout ce qui est "compute", avec par ricochet prise en charge dans le widget

Dans 0001, j'ai posé Template dans qommon/template.py mais ça pourrait être ailleurs, j'avais juste pas trop d'inspiration à cet instant.

Il y a dans 0002 des ajouts dans les tests : là où on vérifiait la partie ezt, on ajoute des vérifications identiques en Django. Je n'ai pas déplacé le "compute" qui reste dans WorkflowStatusItem, pour éviter de rendre le patch trop intrusif et moins lisible. Ça pourrait être fait ensuite.

Étape suivante !

0001: aucune modif.
0002: j'avais oublié la gestion des "compute" dans le champ de signature des webservices, c'est ajouté. Avec un str() pour récupérer le résultat car sign_url n'aime pas l'unicode renvoyé par le render Django.
0003: utilisation dans les messages, mails, sms, commentaires dans le journal, etc.

Je pensais faire un truc un peu plus serré, mais modifier template_on_formdata a fait remonter pas mal de dépendances, que j'ai donc adaptées petit à petit. Mais ce patch 0003 reste encore lisible et cohérent à mon goût. Ca remplace clairement les appels ezt par l'utilisation de qommon.template.Template

Il y a de l'historique pas facile à gérer au niveau par exemple des JournalEvolutionPart, avec une détection du mode HTML si ça commence par "<", un encodage des variables dans ce cas, etc... Je n'ai pas réussi à m'en défaire, et donc si on doit utiliser du Django ici il faut jouer avec des {{truc|safe}}.

Plus dommage, et là encore j'ai pas de solution, c'est la même chose pour les mails. Comme les mails sont normalement du texte pur, on pourrait dire à Django de faire du "safe" systématique... mais d'abord ça n'existe pas dans Django (sauf à ajouter du "autoescape on" dans le template) et y'a un mic mac avec le ezt.FORMAT_RAW/FORMAT_URL et le processeur de contexte, que je n'ai pas réussi à déméler.

Malgré tout, on le voit dans les tests, ça marche bien. Sans doute faudrait-il poser ça sur une machine de dev que des bonnes volontées viendrait tester (rêvons).

(me reste à faire la partie 0004 variadic)

Rebasés sans dommage

Frédéric Péters a écrit :

scorie à retirer.
s/fail/failure/
str(self.value) ?

ok

Attention custom_ezt_email est utilisé dans le module auquotidien.

pour ça, j'ajoute, poussé en même temps, 0001-use-new-custom_template_email-system-19422.patch sur auquotidien

ok mais rien à faire ici.

En fait si, parce que Django a besoin d'une TypeError (et pas d'une AssertionError) pour comprendre quoi faire avec {{attachments.testfile.url}}, c-a-d aller chercher un attribut quand le getitem ne passe pas (sans ce raise, les tests plantent sur test_register_comment_attachment)

une fois on fait ça (template_on_formdata ) mais plus loin (template_on_context), pas.

Yep, petite optimisation ajoutée aussi.

Suite (et fin ? du moins pour la plomberie) dans le 0004 : on accepte la syntaxe Django pour les URLs. Cependant on ne gère pas, en Django, d'aspect magique comme avec ezt : le langage Django est suffisamment complet pour permettre à l'utilisateur s'indiquer s'il veut encoder, ajouter ou pas un /, etc.

• 0005 : import/export des destinataires de mail exprimés en django
• 0006 : Un dernier patch aussi pour ajouter des tests à des endroits où il n'y avait que les tests en ezt.

+        except TemplateSyntaxError, VariableDoesNotExist:

Manquent des parenthèses, et peut-être du coup un test pour vérifier que VariableDoesNotExist est bien attrapé; mais même sans le test supplémentaire, ok pour moi que ça soit poussé après l'ajout des parenthèses.

commit e693c2200e56372e3a75f7e714b3819c1fc177ee (HEAD -> master, origin/wip/19442-django-template, origin/master, origin/HEAD, wip/19442-django-template)
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Mon Nov 27 22:26:38 2017 +0100

    tests: add some Django/ezt templates tests (#19442)

commit 7809060474a74e8d72479fbed634dfd191788931
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Mon Nov 27 22:20:11 2017 +0100

    workflows: allow Django formatted recepient in sendmail action (#19442)

commit 3906f827cdcb4fd9abb31e173496c51123f384e9
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Mon Nov 27 22:05:14 2017 +0100

    misc: use Django/ezt templates in variadic URLs (#19442)

commit 18c9258e9faf8793bec49d1e15079164119fbbb7
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed Oct 18 00:02:29 2017 +0200

    use Django/ezt templates in messages (email, sms, journal) (#19442)

commit 7d92cf296e109c3ce92f255502f02f5190ecd844
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Sun Oct 15 02:48:32 2017 +0200

    use Django/ezt templates on computed expressions (#19442)

commit f626c45cbb51a7abf7bed66a36322ee5ca5ac184
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Sat Oct 14 18:26:39 2017 +0200

    add new Template system, authorizing Django and ezt syntaxes (#19442)

et sur auquotidien

commit fb7f1f78416772a1b8b16f7e89f13119e68f4553 (HEAD -> master, origin/master, origin/HEAD)
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Mon Nov 27 16:38:57 2017 +0100

    use new w.c.s. custom_template_email system (#19422)