Projet

Général

Profil

Development #65846

uniformiser le rendu des liens vers les formulaires

Ajouté par Serghei Mihai il y a presque 2 ans. Mis à jour il y a plus d'un an.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Les cellules "Démarches d'une categorie", "Lien vers une démarche", y compris au sein de la cellule "Liste de liens" utilisent chacune son template pour le rendu du lien vers le formulaire.
On devrait uniformiser cela, converger vers le template proposé dans #62074.


Fichiers

Révisions associées

Révision 964461e2 (diff)
Ajouté par Serghei Mihai il y a plus d'un an

wcs: uniformize forms links rendering (#65846)

Historique

#2

Mis à jour par Serghei Mihai il y a plus d'un an

Au passage ajout de tryauth et cancelurl à la cellule "Lien vers une démarche".

#3

Mis à jour par Lauréline Guérin il y a plus d'un an

{% page_absolute_url cell.page as cancel_url %}

@register.simple_tag(takes_context=True)
def page_absolute_url(context, page):
    return context['request'].build_absolute_uri(page.get_online_url())

Si la page a un subslug, ça ne va pas donner la bonne url il me semble

#4

Mis à jour par Serghei Mihai il y a plus d'un an

  • Statut changé de Solution proposée à En cours
  • Assigné à mis à Serghei Mihai

En effet.

#5

Mis à jour par Serghei Mihai il y a plus d'un an

Et il y a déjà absolute_uri dans le contexte qui pointe correctement vers la page.

#6

Mis à jour par Serghei Mihai il y a plus d'un an

Et une correction sur la description pour desactiver l'autoescape.

#7

Mis à jour par Lauréline Guérin il y a plus d'un an

Et une correction sur la description pour desactiver l'autoescape.

il y a déjà un |safe dans le template, pourquoi ajouter un mark_safe ?

#8

Mis à jour par Serghei Mihai il y a plus d'un an

Le résultat de {% firstof ... sera un texte escaped: https://docs.djangoproject.com/en/3.2/ref/templates/builtins/#firstof et donc |safe perd son sens.

#9

Mis à jour par Lauréline Guérin il y a plus d'un an

ok, tu peux ajouter un test du coup ?

#11

Mis à jour par Lauréline Guérin il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée
#12

Mis à jour par Serghei Mihai il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)
commit 964461e27e039ce3fe3225a5cdc6272837fec727 (HEAD -> main, origin/main, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jun 21 10:14:55 2022 +0200

    wcs: uniformize forms links rendering (#65846)
#13

Mis à jour par Transition automatique il y a plus d'un an

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

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF