Discussion :

[vivgar]

Bonjour,

J'ai un problème de redirection d'url après validation d'un formulaire en post dans un service que j'ai créé.

Je voudrais que l'on arrive sur une nouvelle page une fois le formulaire bien rempli mais ça ne marche pas.

Voici le service :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
 
<?php
 
namespace Europe\VoyageBundle\Services\Voyage;
 
use Doctrine\ORM\EntityManager;
use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\Routing\RouterInterface;
use Symfony\Component\Form\FormFactory;
 
class RechercheManager {
 
    private $em;
    private $req;
    private $ri;
    private $ff;
    private $form;
 
    public function __construct(EntityManager $em, RequestStack $req, RouterInterface $ri, FormFactory $ff)
    {
        $this->em=$em;
        $this->req=$req->getCurrentRequest();
        $this->ri=$ri;
        $this->ff=$ff;
    }
 
    public function setForm($formType, $recherche, $url=null)
    {
       $form=$this->ff->create($formType,$recherche);
 
       if ($this->req->getMethod() === 'POST')
       {
           $form->bind($this->req);
 
            if ($form->isValid())
            {
                $this->em->persist($recherche);
                $this->em->flush();
 
                if (!$url==null)
                $this->redirect($url, $recherche);
                //La méthode redirect est écrite ci dessous
            }
       }
 
       $this->form=$form;
    }
 
    public function getForm()
    {
        return $this->form->createView();
    }
 
    protected function redirect($url, $recherche)
    {
        $url=$this->ri->generate($url, array('id'=>$recherche->getId()));
 
        //L'url est bien généré mais c'est ceci qui ne fonctionne pas :
        return new RedirectResponse($url);
    }
 
}

Il y a bien insertion dans la base de donnée, mais je n'arrive pas à faire la redirection après. Comment pallier à ce problème?

[arnooo999]

Bonjour,

Ta méthode redirect renvoit un objet Response, pour autant je ne vois pas ou il serait utilisé, ta méthode setForm ne le renvoit rien, et de plus nous n'avons pas le code de ton controller, car c'est lui au final qui doit renvoyer cet objet Response.
A vrai dire je ne pense pas que ce soit dans un service qu'il faille faire une redirection.

Code :

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

1
2
3
4
5
6
7
8
9
 if ($form->isValid())
            {
                $this->em->persist($recherche);
                $this->em->flush();
 
                if (!$url==null)
                return $this->redirect($url, $recherche);
                //La méthode redirect est écrite ci dessous
            }

[vivgar]

En fait, voici le code actuel de mon controlleur ci-dessous :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
 
<?php
 
namespace Europe\VoyageBundle\Controller;
 
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 
use Europe\VoyageBundle\Entity\RechercheVoyage;
use Europe\VoyageBundle\Entity\Hotel;
 
use Europe\VoyageBundle\Form\RechercheVolHotelType;
use Europe\VoyageBundle\Form\RechercheHotelType;
use Europe\VoyageBundle\Form\RechercheVolType;
 
class DefaultController extends Controller
{
    public function indexAction()
    {
        $recherche=new RechercheVoyage($nbp=2);
        $form=$this->createForm(new RechercheVolHotelType(),$recherche);
    $em=$this->getDoctrine()->getManager();
 
        $req=$this->getRequest();
        if($req->getMethod() == 'POST')
        {
            $form->bind($req);
 
            if ($form->isValid())
            {
                $em->persist($recherche);
                $em->flush();
 
 
                return $this->redirect($this->generateUrl('europe_voyage_recherche_hotels', array('id'=>$recherche->getId(),'page'=>1)));
            }
        }
 
        return $this->render('EuropeVoyageBundle:Default:index.html.twig', array(
                'form'=>$form->createView(),
                ));
    }
 
    public function indexHotelAction()
    {
        $em=$this->getDoctrine()->getManager();
        $hr=$em->getRepository('EuropeVoyageBundle:Hotel');
 
        $rechercheHotel=new RechercheVoyage();
 
        $form=$this->createForm(new RechercheHotelType(), $rechercheHotel);
 
        $request=$this->getRequest();
            if ($request -> getMethod() === 'POST')
            {
                $form->bind($request);
 
                if($form->isValid())
                {
                    $em->persist($rechercheHotel);
                    $em->flush();
 
                    return $this->redirect($this->generateUrl('europe_voyage_recherche_hotels', array('id'=>$rechercheHotel->getId(),'page'=>1)));
                }
            }
 
        return $this->render('EuropeVoyageBundle:Default:indexHotel.html.twig', array(
                    'form'=>$form->createView(),
                    'hotels'=>$hr->findAll(),
                    'path'=>2,
                        ));
    }
 
    public function indexVolAction()
    {
        $em=$this->getDoctrine()->getManager();
 
        $recherche=new RechercheVoyage();
 
        $form=$this->createForm(new RechercheVolType(), $recherche);
 
        $request=$this->getRequest();
            if ($request -> getMethod() === 'POST')
            {
                $form->bind($request);
 
                if($form->isValid())
                {
                    $em->persist($recherche);
                    $em->flush();
 
                    return $this->redirect($this->generateUrl('europe_voyage_recherche_vols', array('id'=>$recherche->getId(),'page'=>1)));
                }
            }
 
        return $this->render('EuropeVoyageBundle:Default:indexVol.html.twig', array('form'=>$form->createView()));
    }
 
}

J'ai remarqué que les méthodes des controlleurs pouvaient devenir très longues et répétitives, et je croyais que cela était problématique j'ai eu l'idée de créer un service pour mettre en commun certaines actions, mais finalement, on m'a dit sur un autre forum que ce n'était pas vraiment utile, donc j'ai abandonné mon idée de service qui n'était peut être pas si bonne que ça, après tout, autant ne pas essayer de réinventer la roue (ce serait dommage avec sf quand même)

[Nico_F]

J'ai remarqué que les méthodes des controlleurs pouvaient devenir très longues et répétitives, et je croyais que cela était problématique j'ai eu l'idée de créer un service pour mettre en commun certaines actions, mais finalement, on m'a dit sur un autre forum que ce n'était pas vraiment utile, donc j'ai abandonné mon idée de service qui n'était peut être pas si bonne que ça, après tout, autant ne pas essayer de réinventer la roue (ce serait dommage avec sf quand même)

Le contrôleur doit effectivement contenir le minimum de code possible : il récupère une requête et renvoie une réponse : le reste doit être placé dans des services.
Par contre, une redirection qui correspond à une réponse doit elle apparaitre dans le contrôleur et non dans le service.

Concernant ton problème, es-tu sûr d'arriver jusqu'à la partie du code qui fait la redirection ? Si ce n'est pas le cas c'est en amont qu'il faut voir ou ça bloque (form non valide, problème lors du flush ...), si c'est le cas, alors il faudra creuser coté routing.

++