Bug #24681
Passer les url d'assets absolues en mode "skeleton"
0%
Description
Quand une page combo redirige vers un formulaire W.C.S, les images intégrées aux cellules "texte" n'apparaissent pas, car elles sont servies en url relatives. Or la redirection vers le serveur de démarches provoque une 404 sur ces images/assets.
Fichiers
Révisions associées
misc: use absolute URIs for links in text cell in skeleton mode (#24681)
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0002-misc-use-absolute-URIs-for-links-in-text-cell-in-ske.patch 0002-misc-use-absolute-URIs-for-links-in-text-cell-in-ske.patch ajouté
- Fichier 0001-misc-use-absolute-URIs-for-text-cell-images-in-skele.patch 0001-misc-use-absolute-URIs-for-text-cell-images-in-skele.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Un commit pour les images, un autre pour les liens dans les cellules; au début je ne comptais pas trop faire ce second, me disant qu'on avait peut-être des utilisations légitimes de liens relatifs (je pensais à un href="/logout" ajouté avec l'idée de déconnexion + rester sur la même brique), mais j'ai regardé le SaaS de prod sans trouver d'exemple, mais en trouvant par contre des liens relatifs, qui bénéficieront de ce second commit.
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
On a pas mal de cell.text|safe dans publik-base-theme, il faudra sans doute les adapter ?
Peut-être aussi, dans combo, adapter déjà tests/templates-1/combo/cells/foobar/text-cell.html (remplacer "cell.text" par "text") ?
Mis à jour par Frédéric Péters il y a presque 5 ans
On a pas mal de cell.text|safe dans publik-base-theme, il faudra sans doute les adapter ?
Vite fait regardé, c'est jamais en pied de page, donc pas vraiment important. (mais oui ça pourra/it)
Peut-être aussi, dans combo, adapter déjà tests/templates-1/combo/cells/foobar/text-cell.html (remplacer "cell.text" par "text") ?
Ou pas.
En fait, ce que je me dis, c'est que la méthode pourra(it) être changée pour faire mark_safe(), et plus tard alors quand on modifiera on pourra faire d'une pierre deux coups.
C'est un peu chiant à faire dans ce patch parce que ça demande derrière de rebaser #17360, je le ferai dans celui-là.
(je ne comptais pas ce ticket dans ce cycle de toute façon).
Mis à jour par Thomas Noël il y a presque 5 ans
Frédéric Péters a écrit :
On a pas mal de cell.text|safe dans publik-base-theme, il faudra sans doute les adapter ?
Vite fait regardé, c'est jamais en pied de page, donc pas vraiment important. (mais oui ça pourra/it)
Yep yep ... c'est juste dans l'idée que tous les text-cell.html (dont un jour un pourrait servir de modèle pour tout autre chose) soient synchro avec le nouveau contexte.
Peut-être aussi, dans combo, adapter déjà tests/templates-1/combo/cells/foobar/text-cell.html (remplacer "cell.text" par "text") ?
Ou pas.
En fait, ce que je me dis, c'est que la méthode pourra(it) être changée pour faire mark_safe(), et plus tard alors quand on modifiera on pourra faire d'une pierre deux coups.
C'est un peu chiant à faire dans ce patch parce que ça demande derrière de rebaser #17360, je le ferai dans celui-là.
Ok pour tout ça ; merci.
(je ne comptais pas ce ticket dans ce cycle de toute façon).
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f80bac14e0c13074ab9110dfc40bd0db48efb20b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun May 5 13:13:11 2019 +0200
misc: use absolute URIs for links in text cell in skeleton mode (#24681)
commit 9727fde513a6f5c071ef8a24cd5a02af4ba8ca7d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun May 5 12:32:39 2019 +0200
misc: use absolute URIs for text cell images in skeleton mode (#24681)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: use absolute URIs for text cell images in skeleton mode (#24681)