Discussion :

[merou19]

Bonjour à tous,

Dans mon projet en Symfony 5.4, je vois que pour mes entités, des fonctions add et remove ont été générées automatiquement dans les repository.

Exemple pour mon entité user :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
    public function add(User $entity, bool $flush = false): void
    {
        $this->getEntityManager()->persist($entity);
 
        if ($flush) {
            $this->getEntityManager()->flush();
        }
    }
 
    public function remove(User $entity, bool $flush = false): void
    {
        $this->getEntityManager()->remove($entity);
 
        if ($flush) {
            $this->getEntityManager()->flush();
        }
    }

J'ai 2 entités, user et reservation, sans relation entre elles.
Quand un user fait une réservation, j'écris le mail du user dans l'entité réservation.

J'ai le code suivant dans un controller, pour gérer les users qui changent de mail, et ainsi impacter les réservations :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    //... actions sur entité user
    $entitymanager->persist($user);
 
	// si le mail a changé, modifier le mail des reservations
	$ancien_mail = $email_user;
	$nouveau_mail = $user->getEmail();
	if ($ancien_mail != $nouveau_mail) {
		$tab_reservation = $reservationRepository->SelectReservationMail($ancien_mail);
 
		if (count($tab_reservation) > 0) {
			foreach ($tab_reservation as $element_reservation) {
				$id_reservation = $element_reservation->getId();
				$entite_reservation = $reservationRepository->SelectReservation($id_reservation);
				$entite_reservation->setEmail($nouveau_mail);
				$entitymanager->persist($entite_reservation);
			}
		}
	}
 
	$entitymanager->flush();

Ce code fonctionne, ma question concerne les persist et le flush.
Dans mes repository, j'ai la fonction add. Si je modifie mon controller pour l'utiliser, est ce qu'il faut que je flush à chaque fois mon user et mes réservations dans ma boucle ?
Ou comment, selon les bonnes pratiques, en passant par les repository, je peux faire les persist et qu'un flush final ?

Merci d'avance pour votre aide !

[Pytet]

J'ai 2 entités, user et reservation, sans relation entre elles.

Bonjour,

Tu sembles avoir un problème de conception à ce niveau : si les réservations sont liées aux utilisateurs, tu dois avoir une relation entre tes entités (utilises le maker pour t'aider à créer la relation).

La duplication des données en bdd est également une mauvaise pratique : l'adresse mail de l'utilisateur ne devrait pas être renseignée dans la table reservation.
En créant correctement la relation entre tes entités, tu n'auras pas ce problème de duplication et tu n'auras donc pas besoin de modifier l'adresse mail dans la table reservation en cas de changement.

Concernant la fonction add :
- soit tu définies le 2e paramètre $flush avec la valeur true pour exécuter un flush lors de l'appel de cette fonction (à éviter dans une boucle)
- soit tu ne définies pas le 2e paramètre qui est par défaut à false, il faudra donc faire un flush final

[merou19]

Merci beaucoup pour tes explications et indications !

Techniquement, je vois le principe, ok, il vaut mieux faire un flush final plutôt que dans chaque passage dans ma boucle.

En effet, j'ai peut être un problème de conception, voilà ce que j'avais en tête à cette époque :
Sur mon site, les visiteurs peuvent s'enregistrer, ce qui crée une ligne dans la table User.
Quand ils créent une réservation, je fige les infos, dont le mail, dans la table Reservation, comme ça je conserve les données pour la facturation.
Et le visiteur enregistré a un écran pour supprimer son user s'il le souhaite. Et pour être conforme au RGPD, j'ai une fonction qui supprime les users inactifs depuis un an.
Ce principe me permet donc de garder les données de facturation, qui légalement doivent être conservées plus longtemps.
Enfin, pour le RGPD, c'est ce que j'avais compris à l'époque, j'espère que je n'ai pas tout fait de travers...