Refactoring

[odolbeau]

Beaucoup de refactoring:

• le projet utilise maintenant composer
• les vendors ont été mis à jour
• les commandes ont été retravaillées pour utiliser le composant symfony/console
• les formulaires sont maintenant gérés par un FormExtension de manière à pouvoir accéder au services nécessaires.
• plus de mélange entre objet / repository. Plus d'objets d'ailleurs, puisqu'ils n'étaient pas utilisés ou presque. :)
• plus de confusions entre member et user. (quelques soucis à ce niveau là dans la création / l'update de drinks)
• utilisation d'Atoum pour l'écriture de tests unitaires / fonctionnels
• Reformatage du code pour respecter les standards PSR-1 / PSR-2 (certainement quelques oublis par endroit)
• Corrections de quelques bugs (édition d'un drink notamment)

Les tests couvrent un bon périmètre de l'application... mais pas tout, évidemment. Il pourrait être intéressant de les compléter, notamment pour être absolument certain que le comportement entre user / member est bien celui attendu. Le mail envoyé également n'est pas testé (et ne marche d'ailleurs sans doute plus avec tout ces changements). Je regarderais pour faire davantage de tests de ce côté là. :)

Bien entendu, si vous avez des remarques ou des suggestions, je suis ouvert. :)

[odolbeau]

L'utilisation de composer et des tests permet en grande partie d'éviter ce genre de problème. Le site fonctionne avec les versions de vendors fixées dans le composer.json. Pourquoi vouloir dégrader les versions de chaque vendors dans cette situation ? :(

[mikaelrandy]

La PR, sur le fond, me convient. Je trouve dommage que beaucoup de diff soient juste du changement du refacto de style de codage. Pour le reste, je n'ai pas tout validé ligne à ligne, mais je pense que le projet n'étant pas en prod, on peut merger, puis tester le fonctionnel du projet.

J'aimerais avoir le temps de rédiger quelques test fonctionnels ...