Bug #41600
recherche: indexation d'un seul lien d'une cellule ayant des liens externes
0%
Description
Constaté sur l'exemple d'une cellule "Formulaires d'une categorie".
Test qui réproduit le souci (certainement mal placé) et une proposition de correction.
Fichiers
Révisions associées
search: index all cell's external links (#41600)
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 4 ans
- Fichier 0001-search-index-all-cell-s-external-links-41600.patch 0001-search-index-all-cell-s-external-links-41600.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le même nom utilisé pour l'indexation de la cellule et de ces liens fait qu'un seul lien va ressortir.
Mis à jour par Frédéric Péters il y a environ 4 ans
La description ne dit rien du problème, juste qu'il est constaté, et je lis le code mais il est compliqué par le fait qu'un truc s'appelait indexed_cell et maintenant indexed_link mais le code reste le même, et pareil pour indexed_cell (un autre?) qui devient indexed_cell_by_url ? Et complication encore dans le test; quelle importance à ce que "form title" vienne avant "a second form title" ?
Bref, il me semble que :
- indexed_cell = external_urls.get(indexed_cell.url)
+ indexed_cell = external_urls.get(link_data.get('url'))
« les liens externes sont individuellement indexés, ça se fait en itérant sur ce que renvoie cell.get_external_links_data(), si jamais deux liens pointent vers la même adresse ils doivent être indexés une seul fois, il y a un bug dans le code sur ce point qui fait qu'on ne récupérait pas comme adresse celle du lien mais celle précédemment indexée, et du coup tout le temps la même et pas d'indexation au-delà du premier élément. ».
Mis à jour par Frédéric Péters il y a environ 4 ans
En bonus,
- indexed_cell.indexed_text += ' ' + link_data.get('text') or ''
+ indexed_cell.indexed_text += ' ' + (link_data.get('text') or '')
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 4 ans
- Fichier 0001-search-index-all-cell-s-external-links-41600.patch 0001-search-index-all-cell-s-external-links-41600.patch ajouté
Frédéric Péters a écrit :
La description ne dit rien du problème, juste qu'il est constaté, et je lis le code mais il est compliqué par le fait qu'un truc s'appelait indexed_cell et maintenant indexed_link mais le code reste le même, et pareil pour indexed_cell (un autre?) qui devient indexed_cell_by_url ? Et complication encore dans le test; quelle importance à ce que "form title" vienne avant "a second form title" ?
Je renomme la variable car il ne me semble pas clair qu'au début de la boucle:
for link_data in cell.get_external_links_data():
# index external links
indexed_cell = external_urls.get(indexed_cell.url)
indexed_cell désigne l'index de la cellule et qu'à l'itération suivante il devienne l'index du lien.
Mais oui, la modification:
- indexed_cell = external_urls.get(indexed_cell.url)
+ indexed_cell = external_urls.get(link_data.get('url'))
suffit pour rendre les choses plus claires.
Et complication encore dans le test; quelle importance à ce que "form title" vienne avant "a second form title" ?
Aucune, en effet. Retirée.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
commit efc784d4e1409b2ebd84f11bc6ea91d12034a109 (origin/master)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Fri Apr 10 13:47:49 2020 +0200
search: index all cell's external links (#41600)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 4 ans
J'envoie le patch dans la hotfix aussi, sans contre-avis.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
search: index all cell's external links (#41600)