Projet

Général

Profil

Development #29763

ne pas faire une redirection si l'URL de redirection est finalement vide

Ajouté par Thomas Noël il y a plus de 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
15 janvier 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Il est possible d'avoir un template dans une redirection, par exemple {{mentions_legales_url}} ... si le résultat est vide, il faudrait ne finalement pas faire la redirection.


Fichiers

Révisions associées

Révision fb718c6c (diff)
Ajouté par Thomas Noël il y a environ 5 ans

public: do not redirect if template rendering is empty (#29763)

Also, on template error, raise a 404 with a debug message.

Historique

#1

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Assigné à mis à Thomas Noël
#2

Mis à jour par Thomas Noël il y a plus de 5 ans

#3

Mis à jour par Thomas Noël il y a environ 5 ans

Juste pour préciser le cas d'usage : on a une configuration de combo "générique", utilisé par x communes. On fait une page combo "Mentions légales" avec texte par défaut.

Mais certaines communes veulent que la page "mentions légales" renvoie vers une page déjà existante ailleurs.

On pose alors sur la page "Mentions légales" une redirection vers "{{mentions_legales_url}}" qui ne sera effective que si cette variables existe, sur ces communes.

#4

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

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

"can't compute" plutôt que "can't calculate". (et perso je n'aurais pas indenté le bloc "if redirect_url").

#5

Mis à jour par Thomas Noël il y a environ 5 ans

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

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

"can't compute" plutôt que "can't calculate". (et perso je n'aurais pas indenté le bloc "if redirect_url").

Poussé avec ces corrections

commit fb718c6c47a65cf1008bfa9adde6b24de707b1bb
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Jan 15 11:59:09 2019 +0100

    public: do not redirect if template rendering is empty (#29763)

    Also, on template error, raise a 404 with a debug message.

#6

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

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

Formats disponibles : Atom PDF