Correction des supressions et utilisation des request

src/AppBundle/Controller/RealisateurController.php

@@ -51,16 +51,14 @@ class RealisateurController extends Controller
 	/**
 	 * @Route("/real/modifier/{id}", name="realisateur_modifier")
 	 */
-	public function modifierAction(Request $request, $id)
+	public function modifierAction(Request $request, Realisateur $realisateur)
 	{
-		$em = $this->getDoctrine()->getManager();
-		$repo = $em->getRepository('AppBundle:Realisateur');
-		$realisateur = $repo->find($id);
 		$form = $this->createForm(RealisateurType::class, $realisateur);
 		$form->handleRequest($request);
 
 		if ($form->isSubmitted() && $form->isValid())
 		{
+            $em = $this->getDoctrine()->getManager();
             $realisateur = $form->getData();
 			$em->flush();
 			$this->addFlash('success', 'Le réalisateur a été modifié');
@@ -76,18 +74,20 @@ class RealisateurController extends Controller
 	/**
 	 * @Route("/real/supprimer/{id}", name="realisateur_supprimer")
 	 */
-	public function supprimerAction($id)
-	{
-	    $em = $this->getDoctrine()->getManager();
-	    $repo = $em->getRepository('AppBundle:Realisateur');
-        $realisateur = $repo->find($id);
-
+	public function supprimerAction(Request $request, Realisateur $realisateur)
+    {
         $form = $this->get('form.factory')->create();
+        $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid())
         {
+            $em = $this->getDoctrine()->getManager();
+            foreach ($realisateur->getFilms() as $film)
+            {
+                $film->setRealisateur(null);
+            }
             $em->remove($realisateur);
             $em->flush();
-            $this->addFlash('success', "Le réalisateur \"$realisateur\" a bien été supprimé.");
+            $this->addFlash('success', 'Le réalisateur '.$realisateur->getNomComplet().' a bien été supprimé.');
             return $this->redirectToRoute('realisateur_liste');
         }
 

src/AppBundle/Controller/SecurityController.php

@@ -54,20 +54,14 @@ class SecurityController extends Controller
     /**
      * @Route("/admin/edituser/{id}", name="admin_edituser")
      */
-    public function editUserAction(Request $request, UserPasswordEncoderInterface $encoder, $id)
+    public function editUserAction(Request $request, UserPasswordEncoderInterface $encoder, User $user)
     {
-        $em = $this->getDoctrine()->getManager();
-        $repo = $em->getRepository('AppBundle:User');
-        $user = $repo->find($id);
-
         $form = $this->createForm(UserEditType::class, $user);
-
         $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid())
         {
+            $em = $this->getDoctrine()->getManager();
             $user = $form->getData();
-            //$encoded = $encoder->encodePassword($user, $user->getPassword());
-            //$user->setPassword($encoded);
             $em->flush();
             $this->addFlash('success', 'L\'utilisateur a été modifié');
             return $this->redirectToRoute('admin_index');
@@ -94,16 +88,15 @@ class SecurityController extends Controller
     /**
      * @Route("/admin/deluser/{id}", name="admin_deluser")
      */
-    public function delUserAction (Request $request, $id)
+    public function delUserAction (Request $request, User $user)
     {
-        $em = $this->getDoctrine()->getManager();
-        $repo = $em->getRepository('AppBundle:User');
-        $user = $repo->find($id);
         $form = $this->get('form.factory')->create();
+        $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid()) {
+            $em = $this->getDoctrine()->getManager();
             $em->remove($user);
             $em->flush();
-            $this->addFlash("L'utilisateur a bien été supprimé");
+            $this->addFlash("success", "L'utilisateur a bien été supprimé");
             return $this->redirectToRoute('admin_index');
         }
         return $this->render('@App/security/supprimeruser.html.twig', array (

src/AppBundle/Controller/VideothequeController.php

@@ -52,8 +52,6 @@ class VideothequeController extends Controller
 			return $this->redirectToRoute('videotheque_liste');
 		}
 
-
-
 		return $this->render('@App/videotheque/ajouter.html.twig', array(
 			'form'	=>	$form->createView(),
 			'film'	=>	$film
@@ -63,16 +61,14 @@ class VideothequeController extends Controller
 	/**
 	 * @Route("/modifier/{id}", name="videotheque_modifier")
 	 */
-	public function modifierAction(Request $request, $id)
+	public function modifierAction(Request $request, Film $film)
 	{
-		$em = $this->getDoctrine()->getManager();
-		$repo = $em->getRepository('AppBundle:Film');
-		$film = $repo->find($id);
 		$form = $this->createForm(FilmType::class, $film);
 		$form->handleRequest($request);
 
 		if ($form->isSubmitted() && $form->isValid())
 		{
+            $em = $this->getDoctrine()->getManager();
 			$film = $form->getData();
 			$em->flush();
 			$this->addFlash('success', 'Le film a été modifié');
@@ -88,18 +84,16 @@ class VideothequeController extends Controller
 	/**
 	 * @Route("/supprimer/{id}", name="videotheque_supprimer")
 	 */
-	public function supprimerAction($id)
+	public function supprimerAction(Request $request, Film $film)
 	{
-	    $em = $this->getDoctrine()->getManager();
-	    $repo = $em->getRepository('AppBundle:Film');
-        $film = $repo->find($id);
-
         $form = $this->get('form.factory')->create();
+        $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid())
         {
+            $em = $this->getDoctrine()->getManager();
             $em->remove($film);
             $em->flush();
-            $this->addFlash('success', "Le film \"$film\" a bien été supprimé.");
+            $this->addFlash('success', 'Le film '.$film->getTitre().' a bien été supprimé.');
             return $this->redirectToRoute('videotheque_liste');
         }
 

src/AppBundle/Resources/views/security/supprimeruser.html.twig

@@ -1,4 +1,4 @@
-{% extends "@App/base.html.twig" %}
+{% extends "base.html.twig" %}
 
 {% block title %}Supprimer un utilisateur{% endblock %}
 {% block titre %}Supprimer un utilisateur{% endblock %}
@@ -7,7 +7,7 @@
     <h2>Confirmation ?</h2>
     <p>Êtes vous certain de vouloir supprimer l'utilisateur "{{ user.username }}" ?</p>
     <form action="{{ path('admin_deluser', {'id': user.id}) }}" method="post">
-        <a href="{{ path('admin_index' }}" class="btn btn-secondary">
+        <a href="{{ path('admin_index') }}" class="btn btn-secondary">
             <i class="fas fa-chevron-circle-left"></i>
             Retour à la liste des utilisateurs
         </a>

src/AppBundle/Resources/views/videotheque/supprimer.html.twig

@@ -7,7 +7,7 @@
     <h2>Confirmation ?</h2>
     <p>Êtes vous certain de vouloir supprimer le film "{{ film.titre }}" ?</p>
     <form action="{{ path('videotheque_supprimer', {'id': film.id}) }}" method="post">
-        <a href="{{ path('videotheque_liste', {'id': film.id}) }}" class="btn btn-default">
+        <a href="{{ path('videotheque_liste') }}" class="btn btn-secondary">
             <i class="fas fa-chevron-circle-left"></i>
             Retour à la liste des films
         </a>