Projet

Général

Profil

Development #24296

Paramètre de config pour noter les templatetags à charger par défaut

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
05 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Parce qu'attendre Django 1.11 (/authentic) n'est pas nécessairement une option, je verrais bien un paramètre (dans site-options.cfg) dans lequel pourrait se paramétrer la liste des modules à rendre disponible pour templatetags (ajout de {% load ...).


Fichiers


Demandes liées

Lié à w.c.s. - Development #21579: ajouter {% load qommon %} systématiquement (django 1.8)Fermé01 février 2018

Actions

Révisions associées

Révision 996749aa (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

general: load some templatetags libraries by default (#24296)

Révision a0dfbf80 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

templates: restore {% load qommon %} in pure django template (#24296)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Lié à Development #21579: ajouter {% load qommon %} systématiquement (django 1.8) ajouté
#2

Mis à jour par Thomas Noël il y a presque 6 ans

Et le mettre absolument partout à "qommon" pour que tout le monde ait la même doc ? Ça me semble assez tordu... Dans #21579 je pensais à un "load qommon" systématique si django < 1.11, ça me semblait plus facile à gérer... non ?

#3

Mis à jour par Frédéric Péters il y a presque 6 ans

J'imaginais des {% load %} pour qommon + tout ce qui serait posé dans site-options.cfg.

#4

Mis à jour par Thomas Noël il y a presque 6 ans

Mais si y'a rien dans site-options.cfg, on load quand même qommon ?

ie ce ticket, c'est une évolution après #21579 ?

#5

Mis à jour par Frédéric Péters il y a presque 6 ans

En fait j'avais comme commentaire dans #21579 que je préférais ne rien bricoler pour 1.8 et attendre 1.11, mais c'était dans l'idée d"arriver à quelque chose rapidement. Bref ici, je couvre #21579 et celui-ci.

#6

Mis à jour par Frédéric Péters il y a presque 6 ans

(patch vaguement fait dans le train, pas testé à fond)

#7

Mis à jour par Thomas Noël il y a presque 6 ans

C'est comme ça que j'imaginais l'affaire, donc ça me va bien.

Partant, j'en étais à me demander si on ne poserait pas i18n et humanize par défaut également.

Dans les tests, peut-être en ajouter un avec un template qui commence par un truc genre {% load tz %}, histoire de vérifier que ça ne gène pas.

Ah sinon il y aussi un load qommon à retirer dans tests/templates/qommon/forms/widgets/select--test.html

Et pour finir de contextualiser un peu ce ticket, la config des templatetags, est-ce dans l'idée de "suppléer" à l'usage des scripts ? Ou autre besoin ?

#8

Mis à jour par Frédéric Péters il y a presque 6 ans

Et pour finir de contextualiser un peu ce ticket, la config des templatetags, est-ce dans l'idée de "suppléer" à l'usage des scripts ? Ou autre besoin ?

Oui c'est mon idée de dégager les scripts; remplacés au niveau des conditions par la possibilité de templatetags et au niveau des champs "commentaire" par la possibilité d'overrider le template de rendu (#24294). (ce sont les deux usages massifs de scripts par imio).

#9

Mis à jour par Frédéric Péters il y a presque 6 ans

Sans module supplémentaire chargé par défaut mais avec un test pour assurer qu'un load supplémentaire fonctionne (en utilisant i18n).

#10

Mis à jour par Thomas Noël il y a presque 6 ans

Ack

#11

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 996749aace3493121f750c24ce5afc4da87ed3e2
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jun 5 18:03:02 2018 +0200

    general: load some templatetags libraries by default (#24296)
#12

Mis à jour par Frédéric Péters il y a presque 6 ans

J'avais noté puis oublié que je n'avais pas exécuté les tests, résultat une erreur.

Les {% load ... %} automatiques sont pour les endroits exposés à l'usager, conditions et gabarits, pour les templates internes, comme widget.html, on envoie directement dans Django, il n'y a pas de déclaration automatique (on verra quand on pourra changer la mécanique, en 1.11).

commit a0dfbf80466c6500bb22fd5d2fb3ae20fce83149
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jun 6 21:38:04 2018 +0200

    templates: restore {% load qommon %} in pure django template (#24296)
#13

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF