Development #52418
ne pas chercher à indexer les cellules marquées désactivées
Début:
27 mars 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
On reçoit des traces du genre :
Report at /page3/ failed to GET https://wcs.publik.love/api/cards/foobar/list/delcourt?orig=...&signature=3WNvWogtPLSO8w32rvyADB8CS/fLz7yjAioBrhMpZMA%3D (404) Request Method: GET Request URL: http://testserver/famille/
alors que si on va sur le /manage de la page, on voit que la cellule qui tente cette requête est marquée désactivée depuis plusieurs jours, et n'est plus affichée.
Dans ce cas je pense que index_site devrait passer son chemin ; typiquement le render_for_search d'une cellule pourrait juste renvoyer '' quand le système de validation a vu un problème.
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Fichier 0001-search-don-t-index-invalid-cells-52418.patch 0001-search-don-t-index-invalid-cells-52418.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Cool, merci Lauréline. Je valide !
... même si en dedans je pense qu'on pourrait écrire plus court sans détruire le sens, ainsi :
def is_visible(self, user=None, check_validity_info=True):
if check_validity_info and self.is_hidden_because_invalid():
return False
return element_is_visible(self, user=user)
Mis à jour par Lauréline Guérin il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1cfd7e90c4fcccb11c679d6b51b8fff0f8b272d5
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Thu Apr 1 16:36:03 2021 +0200
search: don't index invalid cells (#52418)
J'ai fait comme tu as dit :)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
search: don't index invalid cells (#52418)