Projet

Général

Profil

Development #87046

inspect d'une demande avec beaucoup de choses dans form_workflow_form

Ajouté par Lauréline Guérin il y a 2 mois. Mis à jour il y a 2 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 février 2024
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

cf #https://dev.entrouvert.org/issues/87001, la même action a été jouée une fois puis rejouée 27 fois.

class LazyFormDataWorkflowForms:
    ...

    def inspect_keys(self):
        for part in self._formdata.iter_evolution_parts(WorkflowFormEvolutionPart):
            if part.varname and part.data:
                yield part.varname

ici le varname qualification_service est renvoyé 28 fois, résultat get_flat_keys passe 28 fois sur chacune des 28 items form_workflow_form_qualification_service, et la page inspect timeout.

On pourrait peut-être dans la méthode flatten de get_flat_keys:

        def flatten(base, depth=10):
            if not depth:
                return
            item = self[base]
            flat_keys[base] = item  <- ici

poser un return si flat_keys[base] est défini et non null (car plus bas il est setté à None avant de rappeler flatten sur les enfants)

En en plus, parce que ça ne mange pas de pain, dédoublonner les varnames dans LazyFormDataWorkflowForms.inspect_keys

Révisions associées

Révision b15bef6d (diff)
Ajouté par Lauréline Guérin il y a 2 mois

misc: get_flat_keys, check sub_keys unicity (#87046)

Révision 5609d89d (diff)
Ajouté par Lauréline Guérin il y a 2 mois

misc: deduplicate varnames in inspect_keys (#87046)

Historique

#2

Mis à jour par Robot Gitea il y a 2 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Lauréline Guérin

Lauréline Guérin (lguerin) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Robot Gitea il y a 2 mois

  • Statut changé de En cours à Solution proposée
#4

Mis à jour par Robot Gitea il y a 2 mois

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

Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Robot Gitea il y a 2 mois

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

Lauréline Guérin (lguerin) a mergé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Transition automatique il y a 2 mois

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

Mis à jour par Transition automatique il y a environ 17 heures

Automatic expiration

Formats disponibles : Atom PDF