creation de l'entite User #7

Merged
melvin-leveque merged 6 commits from feat/creation-entite into main 2026-03-17 12:06:42 +01:00
Collaborator

peux tu me relire stp ?

peux tu me relire stp ?
maurine added 1 commit 2026-03-13 12:21:58 +01:00
maurine requested review from melvin-leveque 2026-03-13 12:21:58 +01:00
melvin-leveque requested changes 2026-03-13 12:27:20 +01:00
@@ -0,0 +16,4 @@
parent::__construct($registry, User::class);
}
// /**

!! Code commenté !!
Il faut enlever l'exemple et créer les bonnes méthodes associé si besoin !

!! Code commenté !! Il faut enlever l'exemple et créer les bonnes méthodes associé si besoin !
Author
Collaborator

oui t'as raison !!! (>_<)

oui t'as raison !!! (>_<)
melvin-leveque marked this conversation as resolved
maurine added 3 commits 2026-03-16 13:25:49 +01:00
melvin-leveque added 3 commits 2026-03-16 13:47:19 +01:00
melvin-leveque requested changes 2026-03-17 11:35:50 +01:00
@@ -15,2 +15,4 @@
class UserController extends AbstractController
{
// TODO : UserPasswordHasherInterface

TODO : UserPasswordHasheInterface ?
Cela signifie ajouter une route pour changer de mot de passe ? Dans ce cas pourquoi ne pas la mettre dans SecurityController ?

Sinon pourquoi intégrer cette interface ?

TODO : UserPasswordHasheInterface ? Cela signifie ajouter une route pour changer de mot de passe ? Dans ce cas pourquoi ne pas la mettre dans SecurityController ? Sinon pourquoi intégrer cette interface ?
Author
Collaborator

je voulais que le SecurityController ne serve qu'à contrôler les entrées et sorties. Dans ma tête, le "UserPasswordHasheInterface" devait être dans le UserController, parce qu'il servira à l'adhérent dans son espace KAZ à modifier son mot de passe

je voulais que le SecurityController ne serve qu'à contrôler les entrées et sorties. Dans ma tête, le "UserPasswordHasheInterface" devait être dans le UserController, parce qu'il servira à l'adhérent dans son espace KAZ à modifier son mot de passe
melvin-leveque marked this conversation as resolved
@@ -38,3 +51,3 @@
#[ORM\Column(length: 255)]
private ?string $nextcloudQuota = null;
private ?string $nextcloudQuota = '1G';

Pourquoi 1GB et pas null comme les autres ?

Pourquoi 1GB et pas null comme les autres ?
Author
Collaborator

parce que je me suis trompée ^^ je devais le mettre sur emailQuota

parce que je me suis trompée ^^ je devais le mettre sur emailQuota
melvin-leveque marked this conversation as resolved
@@ -73,2 +96,3 @@
public function getUserIdentifier(): string
{
return $this->mail;
return (string) $this->email;

On a pas besoin de caster l'email en String, le setter du mail prend une String donc on aura forcément une variable de ce type.
return $this->email; fonctionnera très bien

On a pas besoin de caster l'email en String, le setter du mail prend une String donc on aura forcément une variable de ce type. `return $this->email;` fonctionnera très bien
Author
Collaborator

je pense que ça s'est généré avec mon symfony console make:user

je pense que ça s'est généré avec mon symfony console make:user
melvin-leveque marked this conversation as resolved
@@ -79,1 +106,3 @@
$this->mail = $mail;
$roles = $this->roles;
// guarantee every user at least has ROLE_USER
$roles[] = 'ROLE_USER';

Il ne faut pas mettre la valeur par défaut dans un getter.
Avec le code actuel, si on créer un utilisateur avec le rôle ROLE_USER alors on récupérera ['ROLE_USER', 'ROLE_USER'] en appelant cette méthode.

Il ne faut pas mettre la valeur par défaut dans un getter. Avec le code actuel, si on créer un utilisateur avec le rôle ROLE_USER alors on récupérera ['ROLE_USER', 'ROLE_USER'] en appelant cette méthode.
Author
Collaborator

pareil que l'email, c'est le make:user qui a généré, je n'ai pas vérifié derrière (c'est d'ailleurs pour ça que les commentaires sont en anglais)

pareil que l'email, c'est le make:user qui a généré, je n'ai pas vérifié derrière (c'est d'ailleurs pour ça que les commentaires sont en anglais)
melvin-leveque marked this conversation as resolved
Author
Collaborator

Ma réponse à tes commentaires

Ma réponse à tes commentaires
melvin-leveque merged commit f72c99f56a into main 2026-03-17 12:06:42 +01:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: melvin-leveque/interface-kaznautes#7