Contrôleur générique

Version imprimable

17/09/2012, 19h35
flutz

Contrôleur générique

Bonjour à tous.

J'effectue la même action au début de chaque méthode de l'ensemble des controllers afin de vérifier certaines si la session est active, serveur en maintenance, action autorisée... (si c'est le cas j'effectue une redirection vers une page spécifique) et aussi pour rafraichir quelques données de l'utilisateur.

Pour facilité la chose j'ai créé une classe qui étend Controller et qui effectue mes actions, ainsi l'ensemble de mes controllers étendent cette classe et n'ont qu'à appeler une méthode et gérer le retour.

Mais j'aimerais faire encore plus simple et éviter ces 2 lignes répétitives de code pour chaque controller, et savoir s'il est possible avec Symfony2 de ne pas écrire ce code mais qu'il soit exécuter automatiquement au début de chaque méthode (il m'est impossible de faire ça dans le constructeur car je ne peux pas récupérer le container, d'après ce que j'ai lu c'est parce qu'il est créé après le constructeur).

Merci d'avance pour vos réponses.
17/09/2012, 22h43
KzrData

Je te conseille de te renseigner sur les services.
17/09/2012, 23h05
flutz

Merci de ta réponse, je vais chercher de ce côté.:)
Je les utilise déjà un peu mais pas du tout pour ça (et c'est surtout copier coller du tuto de la fermeduweb).
18/09/2012, 00h49
Alcide_
Je ne connais pas bien Symfony mais je pencherais plutôt pour les Events.

Voici quelques éléments de réponse pour un équivalent de sf1 sfEvents::preExecute() dans sf2 :
- symfony2 doc -> events
- equivalent de preExecute
18/09/2012, 11h24
Muspil

Comme souvent la réponse est sur le site de symfony:

http://symfony.com/doc/current/cookb...r_filters.html

Il te faudra maitriser les events et les services mais en suivant le cookbook c'est très simple à mettre en place.
18/09/2012, 13h41
flutz

Merci beaucoup à vous 2, je regarderais ça ce soir:)
18/09/2012, 19h38
flutz
Merci à vous ça fonctionne nickel, le mini tutoriel http://www.tribulations.eu/articles/...-symfony2.html linker par Alcide_ est très bien fait et permet d'avoir ce que je voulais!:D

Derrière il suffit de créer un controller générique qui est appeler par l'ensemble des Controllers du projet et qui implémente la méthode preExecute (ou une autre suivant le nom que vous lui donné).

Petit truc en passant pour si certaines personnes tombent sur ce post :
Pour récupérer dans cette méthode preExecute le paramètre que vous donnez à votre controller :

Code:

$params = $request->get('_route_params');

Et ensuite vous récupérez le paramètre en faisant :
Code:

1 2 3 4 5 if (array_key_exists('monParam', $params)) { $monParam = $params['monParam']; }
Merci de votre aide!:ccool:

Ah par contre j'oubliais, dans l'exemple il n'y a pas la redirection, donc voici l'exemple pour faire ensuite une redirection :

Alors vous aller avoir besoin de cette classe :

Code:

use Symfony\Component\HttpKernel\Controller\ControllerResolver;

Elle permet de générer un callable (un controller quoi).

Et donc le code est tout simple (mais pas simple à trouver sur le net^^) :
Code:

1 2 3 4 5 6 7 8 // Création de l'instance du résolver $controllerResolver = new ControllerResolver(); // On récupère la request $request = new Request(); // On set dans la request la valeur _controller avec l'action que l'on souhaite exécuter (désolé c'est ici donc une redirection interne, on ne peut pas renvoyer de réponse avec l'event FilterControllerEvent, il faudrait sinon renvoyer une erreur et catcher l'event de l'erreur et setter la réponse dans ce nouvelle évent...) $request->attributes->set('_controller', 'Flz\LgcCoreBundle\Controller\MainController::homeAction'); // Et enfin on set le controller avec le nouveau créé à partir de la request que l'on vient de mettre à jour $event->setController($controllerResolver->getController($request));
J'espère que ce bout de code vous sera utile =).
Il y a sans doute une autre façon de faire mais je ne la connais pas.

Inconvénient majeur : impossibilité d'utiliser $this dans ce cas dans le controller:/

Au final après quelques heures passée là-dessus, la seule solution que j'ai trouvée (pas très propre à mon goût mais qui a le mérite de fonctionner) c'est :

Code:

1
2
3
4
5
6
7
8
 
services:
  controller.pre_execute_listener:
    class: Projet\MonBundle\Listener\MonListener
    tags:
      - { name: kernel.event_listener, event: kernel.controller, method: onKernelController }
      - { name: kernel.event_listener, event: kernel.response, method: onKernelResponse }
      - { name: kernel.event_listener, event: kernel.exception, method: onKernelException }

Code:

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
 
<?php
namespace Projet\MonBundle\Listener; 
 
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpKernel\HttpKernelInterface; 
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
use Symfony\Component\HttpFoundation\RedirectResponse;
use \Exception;
 
// Le nom de la classe est à votre discrétion
class ControllerListener 
{
	// Sur exception
	public function onKernelException(GetResponseForExceptionEvent $event)
	{
		// Recupération de l'exception
		$exception = $event->getException();
 
		if (get_class($exception) === 'Projet\MonBundle\Exception\MyException');
		{
			$response = new RedirectResponse('Votre action');
 
			$event->setResponse($response);
		}
	}
 
	// Après controller
	public function onKernelResponse(FilterResponseEvent $event)
	{
		// Faire quelque chose... (Ici on a accès à la response le problème ne se pose pas)
	}
 
	// Avant controller
	public function onKernelController(FilterControllerEvent $event) 	
	{
		// Récupération de l'event 	
		if(HttpKernelInterface::MASTER_REQUEST === $event->getRequestType()) 
		{
			// Récupération du controller    
			$_controller = $event->getController();
			if (isset($_controller[0]))
			{
				$controller = $_controller[0];
				// On vérifie que le controller implémente la méthode preExecute
				if(method_exists($controller, 'preExecute'))
				{
					$redirection = $controller->preExecute();
 
					if ($redirection !== null)
					{
						throw new MonException($redirection);
					}
				}
			}
		}
	}
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
<?php
 
namespace Projet\MonBundle\Controller;
 
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 
class MainController extends Controller
{
	/**
	 * Fonction à exécuter avant toute action.
	 */
	function preExecute()
	{
		// Faire quelque chose avant l'execution du controller et retourner une valeur qui permettra le test dans le listener
	}
}

Classe d'exception qui sert uniquement à être sur qu'on n'est pas sur une vrai exception

Code:

1
2
3
4
5
6
7
8
 
<?php
namespace Projet\MonBundle\Exception;
 
class MyException extends \Exception
{
 
}

Il ne reste plus qu'à étendre l'ensemble de vos Controller par la MainController et les actions tout sera fait automatiquement =)

Si vous avez des suggestions pour faire quelque chose de plus propre je suis preneur!
Je suis sur qu'il y a moyen d'éviter de passer par une exception mais comment...
Je suis malheureusement encore noob en php:cry: