Modification à apporter aux ChampionController.php (AdminBundle) - Githubissues

eric-chau / mvnerds

MIT License

0 stars 0 forks source link

Modification à apporter aux ChampionController.php (AdminBundle) #18

Closed eric-chau closed 12 years ago

eric-chau commented 12 years ago

Faire attention à l'alignement des clés valeurs dans les tableaux

<?php // incorrect ! array( 'form' => $form->createView(), 'champion' => $champion );

// Good !
array(
    'form'            => $form->createView(),
    'champion'  => $champion
);

?>

Traquer tous les TODO et les FIXME et les corriger
Faire des routes en français
Gérer l'accès à la variable de session et l'existence d'une variable 'comparison_list' au moins dans des méthodes privées du contrôleur (Peut-être commencer à penser à un service qui gère cela ? Car au final on aura bien un module de comparaison. A méditer)
Ne saute pas trop de ligne inutile, ça rallonge le fichier et ça devient pas très lisible au final

eric-chau commented 12 years ago

bon ben ça marche pas bien l'indentation sous github xDD donc mon exemple est foireux xDD