search

mission-apprentissage / cerfa

Générateur de contrat publique en apprentissage
https://contrat.apprentissage.beta.gouv.fr
MIT License
4 stars 0 forks source link

SONAR - Variables should not be shadowed (42) #794

Closed JulienBurgade closed 2 years ago

JulienBurgade commented 2 years ago

Certaines variables ont été nommées à l'identique de variables qui ont une portée supérieure. Ces variables "cachent" les variables de portée supérieure, qui ne sont plus accessible dans la portion de code concernée.

Il faut renommer ces variables pour les différencier et s'assurer que c'est bien la bonne variable qui est utilisée.

JulienBurgade commented 2 years ago

Certains fix simples ont été faits (20 fixs). D'autres plus complexe impliquent des risques de régressions (15 fixs concernés) -> risqué à ce stade, mis de côté pour le moment.

Seuls les fixs simples vont être mergés.