Projet

Général

Profil

Bug #20975

indexation vs cellule non configurée

Ajouté par Frédéric Péters il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

  File "/usr/lib/python2.7/dist-packages/combo/apps/search/management/commands/update_index.py", line 51, in collect_external_links
    for link_data in cell.get_external_links_data():
  File "/usr/lib/python2.7/dist-packages/combo/apps/wcs/models.py", line 398, in get_external_links_data
    formdefs = self.get_data({'synchronous': True})
  File "/usr/lib/python2.7/dist-packages/combo/apps/wcs/models.py", line 181, in get_data
    self.api_url,
  File "/usr/lib/python2.7/dist-packages/combo/apps/wcs/models.py", line 351, in api_url
    return '/api/categories/%s/formdefs/' % self.category_reference.split(':')[1]
IndexError: list index out of range

Fichiers

0001-wcs-don-t-crash-indexing-an-unconfigured-category-ce.patch (861 octets) 0001-wcs-don-t-crash-indexing-an-unconfigured-category-ce.patch Frédéric Péters, 03 janvier 2018 22:37

Révisions associées

Révision 7ab32ab4 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

wcs: don't crash indexing an unconfigured category cell (#20975)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Fichier 0001-wcs-don-t-crash-indexing-an-unconfigured-category-ce.patch ajouté
  • Statut changé de Nouveau à En cours
  • Patch proposed changé de Non à Oui
#3

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Fichier 0001-wcs-don-t-crash-indexing-an-unconfigured-category-ce.patch supprimé
#4

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

Nope, c'est un return [] qu'il faut.

#5

Mis à jour par Frédéric Péters il y a plus de 6 ans

Comme la fonction est un générateur (via le yield), ce n'est pas le cas.

def t():
    return

for v in t():
    print v

→

  File "/tmp/t.py", line 4, in <module>
    for v in t():
TypeError: 'NoneType' object is not iterable

alors que

def t():
    return
    yield 1

for v in t():
    print v

s'exécute sans erreur.

Mais

def t():
    return []
    yield 1

for v in t():
    print v

→

  File "/tmp/t.py", line 3
    yield 1
SyntaxError: 'return' with argument inside generator
#6

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

Arf, le yield n'était pas visible dans le patch : ack.

#7

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 7ab32ab4f1118fe3e9b111d56af20196b8e926d4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jan 3 21:37:06 2018 +0100

    wcs: don't crash indexing an unconfigured category cell (#20975)
#8

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

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

Formats disponibles : Atom PDF