Produits Entr'ouvert » Passerelle

Au lieu d'un file_data['text'] = six.text_type(link) final je l'aurais mis "au début" dans le

    331             file_data = {
    332                 'id': str(link.id),
                        'text': six.text_type(link),             <-- ici, hop, c'est tout joli
    333                 'numero_dossier': link.file_number,
    334                 'date_de_naissance': link.dob.isoformat(),
    335                 'err': 0,
    336             }

et dans le try qui cherche à faire le link.get_file je ferais juste :

   try:
       file_data['dossier'] = link.get_file(email=email)
   except:
       ...

ce qui rendra le else inutile.

Ensuite, ma petit lubie, pour accélérer les choses, j'ajouterais la possibilité d'un argument "?listonly=true" sur le endpoint "dossiers" ; qui renverrait alors juste la liste des dossiers liés, c-a-d comme sans faire le "link.get_file" qui va bouffer du temps pour rien.

(aussi, les tests montrent qu'il manque la migration sur Link)

Thomas Noël a écrit :

(aussi, les tests montrent qu'il manque la migration sur Link)

My bad pour aller vite je teste surtout via --no-migrations (surtout que --reuse-db / --create-db ne marche pas sur passerelle contrairement à toutes nos autres briques, je n'ai toujours pas trouvé pourquoi).

Thomas Noël a écrit :

Au lieu d'un file_data['text'] = six.text_type(link) final je l'aurais mis "au début" dans le

[...]

Je voulais, mais comme je met à jour display_name dans link.get_file() en cas de mise à jour je vais afficher le vieux label.

et dans le try qui cherche à faire le link.get_file je ferais juste :

[...]

ce qui rendra le else inutile.

Ensuite, ma petit lubie, pour accélérer les choses, j'ajouterais la possibilité d'un argument "?listonly=true" sur le endpoint "dossiers" ; qui renverrait alors juste la liste des dossiers liés, c-a-d comme sans faire le "link.get_file" qui va bouffer du temps pour rien.

J'attendrai une expression du besoin je pense.