Skip to content
Extraits de code Groupes Projets
  • Maïeul's avatar
    98db2b61
    remove: appel à un pipeline inutile et piégeux · 98db2b61
    Maïeul a rédigé
    1. L'ordre des traitements par défaut correspond à l'ordre stockés en base, hors
       lors de la configuration des traitements, on appel déjà le pipeline
    2. La structure du tableau passé ne correspond pas aux autres appels au
       pipeline. Normalement c'est les traitements + la description des
       options + la description du traitement, là c'est juste les
       traitements + **les valeurs** des options. Donc ca peut
       potentiellement faire planter des pipelines qui ajouterait des
       options à un traitement
    3. Le seul cas où cela pouvait éventuellement influencer c'est lorsqu'on
       avait ajouté ces lignes il y a x années ... cela permettait que les
       anciennes configs soient automatiquement ajustées et de passer
       l'envoi d'email après l'enregistrement. Deux solutions
         a. On fait un script de migration pour les rares cas des
         formulaires dont les traitements n'auraient pas été édités depuis.
         b. Tant pis, on considère que cela ne vaut pas la peine de perdre
         de l'énergie à un script de migration, vu que la seule plus value
         de ce truc c'était de permettre d'avoir un lien vers la réponse
         côté privé dans l'email.
    
    J'ai tendance à opter pour b.
    98db2b61
    Historique
    remove: appel à un pipeline inutile et piégeux
    Maïeul a rédigé
    1. L'ordre des traitements par défaut correspond à l'ordre stockés en base, hors
       lors de la configuration des traitements, on appel déjà le pipeline
    2. La structure du tableau passé ne correspond pas aux autres appels au
       pipeline. Normalement c'est les traitements + la description des
       options + la description du traitement, là c'est juste les
       traitements + **les valeurs** des options. Donc ca peut
       potentiellement faire planter des pipelines qui ajouterait des
       options à un traitement
    3. Le seul cas où cela pouvait éventuellement influencer c'est lorsqu'on
       avait ajouté ces lignes il y a x années ... cela permettait que les
       anciennes configs soient automatiquement ajustées et de passer
       l'envoi d'email après l'enregistrement. Deux solutions
         a. On fait un script de migration pour les rares cas des
         formulaires dont les traitements n'auraient pas été édités depuis.
         b. Tant pis, on considère que cela ne vaut pas la peine de perdre
         de l'énergie à un script de migration, vu que la seule plus value
         de ce truc c'était de permettre d'avoir un lien vers la réponse
         côté privé dans l'email.
    
    J'ai tendance à opter pour b.