Discussion :

[Laurentq1970]

Bonjour,

Je ne m'y prends peut-être pas comme il le faut, mais je souhaite faire un test sur l'existance d'enregistrement dans une entité avant l'action des Controllers.

J'ai donc pensé au Listener, mais je ne vois pas comment faire l'appel au Repository.

Voici le code de mon Listener:

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
<?php
 
namespace Glsr\GestockBundle\Listener;
 
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
 
class GlsrControllerListener
{
    public function onKernelController(FilterControllerEvent $event)
    {
        // nous récupérons l'objet controller depuis l'évènement reçu
        $controller = $event->getController();
        var_dump($controller);
 
        // vérifie que l'objet Company n'est pas vide
        $etm = $controller->getDoctrine()->getManager();
        $repoCompany = $etm->getRepository('GlsrGestockBundle:Company');
        $company = $repoCompany->findAll();
        if (empty($company)) {
            // On définit un message flash
            $controller->get('session')->getFlashBag()->add('info', 'Il faut renseigner les informations de la société');
            // On redirige vers la page d'ajout d'information de la société
            $controller->redirect($controller->generateUrl('glstock_company_add'));
        }
...

Et voici le message d'erreur :

FatalErrorException: Error: Call to a member function getDoctrine() on a non-object in G:\wamp\www\Symfony-2\src\Glsr\GestockBundle\Listener\GlsrControllerListener.php line 16

[Laurentq1970]

Bonjour VynkaVynka,

Je n'avais pas due formuler ma question de la bonne manière. Je n'avais pas mis mon post à jour, mais à force de recherches, j'ai réussi à trouver la solution pour l'entityManager.

Inclusion dans le service :

Code :

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

1
2
3
4
5
6
services:
    kernel.listener.glsr_controller:
        class: Glsr\GestockBundle\Listener\GlsrControllerListener
        arguments: [ @doctrine.orm.entity_manager, @service_container ]
        tags:
            - { name: kernel.event_listener, event: kernel.controller, method: onKernelController }

Puis la classe :

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
<?php
 
namespace Glsr\GestockBundle\Listener;
 
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
use Doctrine\ORM\EntityManager;
use Symfony\Component\DependencyInjection\ContainerInterface;
 
use Glsr\GestockBundle\Entity\Company;
 
class GlsrControllerListener
{
    private $etm;
    private $container;
 
 
    public function __construct(EntityManager $etm, ContainerInterface $container)
    {
        $this->etm       = $etm;
        $this->container = $container;
    }
 
    public function onKernelController(FilterControllerEvent $event)
    {
         // vérifie que l'objet Company n'est pas vide
        $repoCompany = $this->etm->getRepository('GlsrGestockBundle:Company');
        $company = $repoCompany->findAll();
 
        if (empty($company)) {
            // On définit un message flash
            $this->container->get('session')->getFlashBag()->add('info', 'Il faut renseigner les informations de la société');
            // On redirige vers la page d'ajout d'information de la société
            $event->getRequest()->attributes->add(array('_controller', 'Glsr\GestockBundle\Controller\Settings::addCompany'));
        }
    }
}

Maintenant reste à régler la redirection. En fait, je souhaite vérifier que certaines informations soient renseignées, pour laisser l'utilisateur continuer. Si ces infos ce sont pas présentent, il est rediriger vers la page d'ajout de ces infos.

Donc, je pense qu'un listener est une bonne solution. Mais reste à savoir si le choix de l'évènement FilterControllerEvent est le bon choix. En fait, le code ci-dessus me renvoie sur la page demandée en première requête. Aucune redirection n'est faite, et le FlashBag est affiché plusieurs fois, comme s'il y avait une boucle sur l’événement.

J'ai essayé $event->setController(), mais impossible de trouver comment déclarer le controller de redirection souhaité. J'ai beaucoup de mal à trouver des information sur l'utilisation de cette méthode, et je n'ai que des erreurs sur le fait que mes déclarations ne sont pas "callable"...

[VynkaVynka]

Pourquoi vouloir utiliser un listener ? Si des informations sont manquantes sur une entité et que la validation a bien été configurée, la méthode isValid() ne laisse pas passer l'utilisateur, il suffit alors de configurer la redirection dans ce cas. Si ça ne concerne pas une entité, le controller peut encore faire son travail : un test si l'information est manquante puis une redirection le cas échéant est une bonne solution.
Si j'ai mal cerné ce que tu dois faire, et qu'utiliser le Listener n'est pas un choix mais une nécessité dans ton cas, j'ai bien peur de ne pas pouvoir t'aider car je n'ai jamais eu à faire ceci. Cependant, je continue de penser qu'il est plus judicieux de faire le traitement dans le controller.

[Laurentq1970]

En fait, je souhaite que ça fonctionne un peu comme un système d'installation. Obliger l'utilisateur à remplir les infos, sinon, le reste ne fonctionne pas.

Je m'explique :
je débute le développement de cette application. Certaines informations, stockées dans différentes entités sont nécessaires au fonctionnement de toute l'appli.

Il faut donc qu'elles soient renseignées. Je voulais faire un test où que l'on soit et que l'on aille dans l'appli. Comme s'il y avait un contrôle de ces infos avant chaque traitement de requêtes.

Plutôt que de répéter ce test à chaque controller, je pensais passer par un listener.

J'espère être clair.

Continuant mes recherches, je suis tombé sur un code sur un autre site.

Je l'ai adapté :

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
<?php
 
namespace Glsr\GestockBundle\Listener;
 
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
use Doctrine\ORM\EntityManager;
use Symfony\Component\DependencyInjection\ContainerInterface;
 
use Symfony\Bundle\FrameworkBundle\Routing\Router;
use Symfony\Component\HttpFoundation\RedirectResponse;
 
class GlsrControllerListener
{
    private $etm;
    private $container;
    protected $router;
    protected $redirect;
 
 
    public function __construct(EntityManager $etm, ContainerInterface $container, Router $router)
    {
        $this->etm       = $etm;
        $this->container = $container;
        $this->router    = $router;
    }
 
    public function onKernelController(FilterControllerEvent $event)
    {
        // vérifie que l'objet Company n'est pas vide
        $repoCompany = $this->etm->getRepository('GlsrGestockBundle:Company');
        $company = $repoCompany->findAll();
 
        if (empty($company)) {
            // On définit un message flash
            $this->container->get('session')->getFlashBag()->add('info', 'Il faut renseigner les informations de la société');
            // On crée la route vers la page d'ajout d'information de la société
            $this->redirect = 'glstock_company_add';
        }       
    }
 
    public function onKernelResponse(FilterResponseEvent $event)
    {
        // On redirige vers la page d'ajout d'information de la société
        if (NULL !== $this->redirect) {
            $url = $this->router->generate($this->redirect);
            $event->setResponse(new RedirectResponse($url));
        }        
    }
}

Ajouté le tag au service:

Code :

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

1
2
3
4
5
6
    kernel.listener.glsr_controller:
        class: Glsr\GestockBundle\Listener\GlsrControllerListener
        arguments: [ @doctrine.orm.entity_manager, @service_container, @router ]
        tags:
            - { name: kernel.event_listener, event: kernel.controller, method: onKernelController }
            - { name: kernel.event_listener, event: kernel.response, method: onKernelResponse }

Mais je suis confronté à une erreur :

Cette page Web présente une boucle de redirection.

Même en testant le StatusCode, je ne trouve pas la solution:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
    public function onKernelResponse(FilterResponseEvent $event)
    {
        if ($event->getResponse()->getStatusCode() === 302) {
            return;
        }
        // On redirige vers la page d'ajout d'information de la société
        if (NULL !== $this->redirect) {
            $url = $this->router->generate($this->redirect);
            $event->setResponse(new RedirectResponse($url));
        }        
    }

Mais, enfin, l'url est correctement modifié

[VynkaVynka]

Pour la boucle de redirection, ce peut être une erreur au niveau des rôles. Par exemple (exemple donné dans la doc SF) : tu veux accéder au login, mais toutes les URL sont protégées et accessibles qu'au rôle admin. Tu es donc redirigé sur la page de login car tu n'est pas admin, et ainsi de suite.
Une autre raison possible, à vue de nez et sans garantie (mais je suppose que ce soit possible) : ton Listener redirige vers un controller. Sauf que vu qu'il redirige vers un controller, il repasse dans lui même, car son but est d'écouter chaque requête ! D'où une boucle de redirections.

Pour la solution à ton problème, pourquoi ne pas faire un controller parent dont héritent les autres controller, et qui s'occupe d'avoir le traitement centralisé ? Si le controller est instancié à chaque fois, il suffit de mettre ton traitement dans la méthode __construct() de ta classe parente. Sinon... C'est plus problématique car il faudrait faire un appel de la méthode en début de chaque autre méthode des controller fils.

Désolé, je ne peux malheureusement pas t'éclairer plus que ça.

[Laurentq1970]

J'ai trouvé ma solution !!!

A force de tester, je me rends compte qu'écouter le Controller n'est pas la meilleure solution. Je me suis penché sur la requête, elle-même. Pour bloquer la redirection, je teste la route de la requête, et si elle corresponds à la route que je souhaite rediriger l'utilisateur, je sors de la fonction onKernel. Et pour la réponse (function onKernelResponse), vu que la redirection ($this->redirect) est nulle, pas d'instruction supplémentaire...

Voici mon code final.

Pour le service :

Code :

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

1
2
3
4
5
6
    kernel.listener.glsr_request:
        class: Glsr\GestockBundle\Listener\GlsrRequestListener
        arguments: [ @doctrine.orm.entity_manager, @service_container, @router ]
        tags:
            - { name: kernel.event_listener, event: kernel.request, method: onKernelRequest }
            - { name: kernel.event_listener, event: kernel.response, method: onKernelResponse }

Et pour le listener:

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
<?php
 
namespace Glsr\GestockBundle\Listener;
 
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
use Doctrine\ORM\EntityManager;
use Symfony\Component\DependencyInjection\ContainerInterface;
 
use Symfony\Bundle\FrameworkBundle\Routing\Router;
use Symfony\Component\HttpFoundation\RedirectResponse;
 
class GlsrRequestListener
{
    private $etm;
    private $container;
    protected $router;
    protected $redirect = null;
 
 
    public function __construct(EntityManager $etm, ContainerInterface $container, Router $router)
    {
        $this->etm       = $etm;
        $this->container = $container;
        $this->router    = $router;
    }
 
    public function onKernelRequest(GetResponseEvent $event)
    {
        // Si la route en cours est celle que l'on veut attribuer, 
        // on sort de la fonction
        if ($event->getRequest()->attributes->get('_route') === 'glstock_company_add') {
            return;
        }
        // vérifie que l'objet Company n'est pas vide
        $repoCompany = $this->etm->getRepository('GlsrGestockBundle:Company');
        $company = $repoCompany->findAll();
 
        if (empty($company)) {
            // On définit un message flash
            $this->container->get('session')->getFlashBag()->add('info', 'Il faut renseigner les informations de la société');
            // On crée la route vers la page d'ajout d'information de la société
            $this->redirect = 'glstock_company_add';
        }
 
    }
 
    public function onKernelResponse(FilterResponseEvent $event)
    {
        // On redirige vers la page d'ajout d'information de la société
        if (NULL !== $this->redirect) {
            $url = $this->router->generate($this->redirect);
            $event->setResponse(new RedirectResponse($url));
        }        
    }
}

Merci pour ta participation VynkaVynka