Projet

Général

Profil

Bug #36706

astregs: mettre en cache le nom de l'association

Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
05 octobre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pour éviter les soucis décrits dans #36086.


Fichiers

0001-astregs-store-linked-association-label-36706.patch (4,34 ko) 0001-astregs-store-linked-association-label-36706.patch Serghei Mihai (congés, retour 15/05), 06 octobre 2019 19:21
0001-astregs-store-linked-association-label-36706.patch (5,91 ko) 0001-astregs-store-linked-association-label-36706.patch Serghei Mihai (congés, retour 15/05), 07 octobre 2019 16:10

Révisions associées

Révision e153f5c6 (diff)
Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 4 ans

astregs: store linked association label (#36706)

Historique

#2

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 4 ans

#3

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

J'aurai plutôt écrit ça comme cela :

def get_label(self):
   # on essaie d'obtenir le label
   ...
       self.association_label = label
       self.save()
   return self.association_label

i.e. on utilise le cache que si nécessaire, pas tout le temps et il manque le log dans le cas où ça n'est pas possible, il manque aussi un try/catch autour du call je pense.

#5

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit e153f5c6c863e6dedb558b5b0e6881f19fd216d3 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Sun Oct 6 14:54:02 2019 +0200

    astregs: store linked association label (#36706)
#7

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

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

Formats disponibles : Atom PDF