Skip to content
Snippets Groups Projects
Commit 807947c9 authored by Larry Garfield's avatar Larry Garfield
Browse files

Clean up and document private property overrides.

parent 9fdd40f8
No related merge requests found
......@@ -34,21 +34,6 @@
*/
class DrupalKernel extends HttpKernel {
/**
* The event dispatcher used by this kernel.
*
* @var Symfony\Component\EventDispatcher\EventDispatcherInterface
*/
protected $dispatcher;
/**
* The controller resolver that will extract the controller from a Request.
*
* @var Symfony\Component\HttpKernel\Controller\ControllerResolverInterface
*/
protected $resolver;
/**
* Constructor.
*
......@@ -59,8 +44,6 @@ class DrupalKernel extends HttpKernel {
*/
public function __construct(EventDispatcherInterface $dispatcher, ControllerResolverInterface $resolver) {
parent::__construct($dispatcher, $resolver);
$this->dispatcher = $dispatcher;
$this->resolver = $resolver;
$this->matcher = new UrlMatcher();
$this->dispatcher->addSubscriber(new RouterListener($this->matcher));
......
......@@ -12,6 +12,7 @@
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\HttpKernel\Exception\MethodNotAllowedHttpException;
use Symfony\Component\HttpKernel\Log\LoggerInterface;
use Symfony\Component\Routing\Matcher\UrlMatcherInterface;
use Symfony\Component\Routing\Exception\ResourceNotFoundException;
use Symfony\Component\Routing\Exception\MethodNotFoundException;
......@@ -23,7 +24,22 @@
*/
class RouterListener extends SymfonyRouterListener {
/**
* The Matcher object for this listener.
*
* This property is private in the base class, so we have to hack around it.
*
* @var Symfony\Component\Router\Matcher\UrlMatcherInterface
*/
protected $urlMatcher;
/**
* The Logging object for this listener.
*
* This property is private in the base class, so we have to hack around it.
*
* @var Symfony\Component\HttpKernel\Log\LoggerInterface
*/
protected $logger;
public function __construct(UrlMatcherInterface $urlMatcher, LoggerInterface $logger = null) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment