Feature auth

[victorweiss]

@Trait2Jauge Ah non horrible 😱

• T'as surchargé le framework en faisant étendre tous les controllers de BaseController. Ça peut se faire pour certaines raisons, mais ici tu veux juste récupérer le statut du User donc pas une raison valable.
• Tous les controllers doivent maintenant renvoyer la variable 'userStatus' ce qui n'est pas du tout pratique, et fait plein de code dupliqué

Là t'aurais du te douter que c'est pas la bonne façon de faire et me poser la question..

Pour dire un truc positif c'est que t'as abordé la notion d'héritage (extends) et de l'accès "protected".

Mais faut que tu trouves une autre solution pour régler les 2 points au dessus. Faut supprimer BaseController et supprimer tous les 'userStatus' => $this->getUserStatus()

Et il y a des conflicts apparement. Donc faut que tu fasses un git pull origin dev et que tu règles les conflicts en local, puis commit & push

[victorweiss]

@Trait2Jauge C'est mieux ! Mais dans le _header pourquoi ne pas faire tout simplement :

{% if app.user %}

😀

toujours dans le _header, pas besoin de checker if isVerified, de toute façon si c'est false il n'est pas sensé pouvoir être connecté

[Trait2Jauge]

Effectivement, c'est plus simple 😅

[victorweiss]

@Trait2Jauge Il restait des petits trucs (certains importants) si tu veux jeter un oeil aux derniers commits, mais sinon c'est mergé, c'est en ligne. Good job ! 👍😊