Discussion :

[romuluslepunk]

Bonjour,

J'aimerai sécuriser une entité en empêchant sa suppression en cas de certaines conditions.
J'ai donc pensé à un événement preRemove afin d'être sûre que cette vérification s'effectue.

Ma question est simple : comment stopper toutes actions depuis une classe paramétré comme service en effectuant une redirection vers une route ?
La fonction redirect() étant une méthode de Controller et non d'un service particulier, je sèche un peu.

Merci d'avance

[Tsilefy]

La couche service ne doit pas être couplée avec HTTP, une redirection dans un service est donc une mauvaise idée. Un service ne doit même pas être au courant que les routes existent ou qu'il est dans une application web.

La méthode de ta classe service doit lancer une exception ou retourner false ou un code d'erreur, et c'est à ton controlleur de lancer la redirection lorsqu'il reçoit cette exception/erreur/code.

[romuluslepunk]

Bonjour,

Je comprend le principe et j'aurais fait comme cela si c'est je faisait appel au service depuis un contrôleur.
Mais dans ce cas, c'est le kernel qui fait appel au service via les événement de Doctrine (preRemove).

Dans mon controleur qui supprime l'entité, je fais mes test.
Mais j'aimerais sécuriser pour être sûre que l'entité ne sera pas supprimé en cas de cascade par exemple (ou un oubli autre part dans le code)

Je peut retourner une exception, mais celà affichera une belle erreur 500 et c'est ce que j'aimerai éviter

[Tsilefy]

Dans ce cas, injecte les routes dans ta classe service (dans le constructeur), et ensuite utilise la classe RedirectResponse dans ta méthode pour faire la redirection.

[romuluslepunk]

Bonjour,

Si je retourne un objet RedirectResponse, il n'y pas de redirection (logique vu que le kernel n'attend rien de l'événement).

J'ai trouvé une solution bis :
Je détache l'entité dans l'événement preRemove :

Code :

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

1
2
3
4
public function preRemove(LifecycleEventArgs $args)
	{
	$args->getEntityManager()->detach($args->getEntity());
	}

De cette manière, elle n'est pas effacée.

Concernant la redirection, je contourne en utilisant les flashbag:
- 1 flashbag dans preRemove si l'entité est détaché pour prévenir l'utilisateur
- 1 flashbag dans postRemove pour confirmer la supression.

Merci pour ton aide.

[Tsilefy]

Pas retourner un RedirectResponse, mais en créer un puis utiliser sa méthode send();

Mais c'est de toute façon une sale solution, comme je le disais au début. Ta solution est plus propre.