Development #45841
Réutiliser une cellule pointant sur une URL qui a été désactivée depuis 2 jours.
0%
Description
- Créer un cellule Lien avec une URL invalide (404)
- La cellule affiche l'URL (en rouge)
- Attendre 2 jours, ou modifier la date en base :
> update data_validityinfo set invalid_since = '2020-08-09 17:27:02.111872+02';
- La cellule n'affiche plus l'URL (et est listée dans les cellules invalides)
- Modifier l'URL de la cellule avec une URL valide
- Observer que l'URL a bien été modifiée en base (malgré l'avertissement "L’adresse ne semble pas exister")
> select url from data_linkcell;
- La cellule n'affiche toujours pas l'URL
Idem avec une cellule JSON : le texte ne s'affiche pas.
> select url from data_jsoncell;
Demandes liées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
La cellule affiche l'URL (en rouge)
Tu parles toujours du backoffice, ici ? Pour moi il y a message "L’adresse ne semble pas exister" en rouge mais pas d'URL affichée en rouge.
Cette étape ne m'est pas claire.
Plus loin, c'est le souhait de résolution de ce ticket que je ne fais qu'entrapercevoir, tu peux compléter la description par le comportement que tu souhaiterais ?
Et conseil aussi, ne pas introduire en dernière phrase un sujet moins développpé, qui ajoute de la confusion.
~~
Bref, trop vaguement de ce ticket j'obtiens un "il y a un truc avec la validité qui marche pas tip top".
Mis à jour par Frédéric Péters il y a plus de 3 ans
Aussi, parce que je vois maintenant la ligne jenkins. Quand tu comptes enchainer un ticket et son patch, assigne-toi le ticket, ça permet au public de ne pas s'y intéresser et d'attendre le patch.
Mis à jour par Frédéric Péters il y a plus de 3 ans
Et comme je lis maintenant le patch, je t'aurais conseillé une description de ticket de ce style :
« si une cellule est marquée invalide et qu'on la modifie, il faudrait considérer que cette modification est là pour la rendre valide, et retirer l'info d'invalidité »
Mais c'est une sur-intreprétation du patch, qui ne fait pas ça, qui fait "on laisse l'info d'invalidité mais plus tard quand is_visible() sera interrogé refaire l'évaluation si la cellule a depuis été modifiée".
Ce qui marche dans un test qui fait cet appel à is_visible() mais en vrai, qui appelle(ra) is_visible() ?
Mis à jour par Nicolas Roche il y a plus de 3 ans
La cellule affiche l'URL (en rouge)
J'ai oublié de préciser qu'il s'agissait de l'affichage de la cellule en frontoffice
Comportement que tu souhaiterais ?
Je pensais que si l'on modifiait l'URL pointée par la cellule avec une URL fonctionnelle,
alors la cellule serait à nouveau affichée.
Dans les faits (ticket lié) j'ai mis très longtemps avant de me décider à supprimer ma cellule pour la re-créer,
ce que devait m'indiquer le message "L’adresse ne semble pas exister" et que je n'ai pas compris.
« si une cellule est marquée invalide et qu'on la modifie, il faudrait considérer que cette modification est là pour la rendre valide, et retirer l'info d'invalidité »
Oui, c'est tout à fait ça (que j'aurais du écrire).
Cela dit, en testant avec cellules Liens je réalise que l'on ne peut plus Enregistrer la cellule une fois qu'elle a été désactivée (le bouton Enregistrer est Grisé). Finalement ce serait plus simple d'étendre ce comportement aux cellules JSON (ce qui aurait peut-être dû être l'objet du ticket).
qui appelle(ra) is_visible() ?
combo/public/views.py::publish_page()
Au prochain affichage en frontoffice, la cellule "semble" notée comme valide.
J'ai soumis ma branche surtout parce que j'essayais d'écrire un test.
Mais ce n'est pas concluant.
(comme je ne suis pas certain de bien comprendre tous les tenants et les aboutissants, liés à #38009 à priori, je préfère laisser la main).
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Nouveau à Rejeté
Je vais rejeter ce ticket qui parle de cellule lien mais lie un ticket où il est question de user_nameid dans une url de cellule json et dont la description m'est toujours confuse.
Et je créerai un ticket une fois que je serai sûr d'avoir compris le problème. (le test dans la branche valide la modification au code dans la branche mais je n'en capte pas le sens et je ne suis pas sûr que ma proposition de description rapidement écrite ci-dessus soit vraiment la bonne).
Mis à jour par Frédéric Péters il y a plus de 3 ans
https://perso.entrouvert.org/~fred/tmp/45841.ogv est une capture de ce que je lis des étapes et tout m'a l'air d'aller correctement.
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Lié à Development #45842: l'évaluation de validité des cellules json et fil atom/rss se fait uniquement lors du rendu ajouté