Radiation globale
1 fil de conversation non résolu
Fusionnées
a demandé de fusionner hsilvant/cgscop_partner:12.0-RV-Radiation-globale vers 12.0-evo-202003
1 fil de conversation non résolu
Propose de cloturer toutes les périodes d'adhésion en une seule transaction
Tuleap # 77
Rapports de requête de fusion
Activité
Je merge la branche mais il faudra qu'on reparle des règles de codage. Pour info :
Je dirais même plus, ce code est pas très optimisé :
- les search() utilisent beaucoup de ressources et tu en fais dans les deux fonctions pour chercher à peu près la même chose, il vaudrait mieux n'en faire qu'un et passer les id à fermer directement à la fonction plutôt que le type_id
- tu passes en paramètres des fonctions des infos qui sont déjà dans le contexte, vu que tu appelles ta fonction avec period.member_out_period(xxx) toute la period est dans le contexte passé à la fonction (que tu récupères dans la variable self) donc pas besoin de les passer en plus en paramètre
mentioned in commit 8b5b9dda
Veuillez vous inscrire ou vous connecter pour répondre