[EventDispatcher] Fixed edge case not covered by tests that generated E_NOTICES

Drak · Drak · commit e199049581db · 2012-04-12T16:21:33.000+05:45
diff --git a/src/Symfony/Component/EventDispatcher/ContainerAwareEventDispatcher.php b/src/Symfony/Component/EventDispatcher/ContainerAwareEventDispatcher.php
@@ -141,7 +141,7 @@ public function addSubscriberService($serviceId, $class)
             if (is_string($params)) {
                 $this->listenerIds[$eventName][] = array($serviceId, $params, 0);
             } elseif (is_string($params[0])) {
-                $this->listenerIds[$eventName][] = array($serviceId, $params[0], $params[1]);
+                $this->listenerIds[$eventName][] = array($serviceId, $params[0], isset($params[1]) ? $params[1] : 0);
             } else {
                 foreach ($params as $listener) {
                     $this->listenerIds[$eventName][] = array($serviceId, $listener[0], isset($listener[1]) ? $listener[1] : 0);
diff --git a/src/Symfony/Component/EventDispatcher/Tests/ContainerAwareEventDispatcherTest.php b/src/Symfony/Component/EventDispatcher/Tests/ContainerAwareEventDispatcherTest.php
@@ -245,6 +245,8 @@ class SubscriberService implements EventSubscriberInterface
     static function getSubscribedEvents() {
         return array(
             'onEvent' => 'onEvent',
+            'onEvent' => array('onEvent', 10),
+            'onEvent' => array('onEvent'),
         );
     }
 

Original file line number	Diff line number	Diff line change
`@@ -245,6 +245,8 @@ class SubscriberService implements EventSubscriberInterface`
`245`	`245`	`static function getSubscribedEvents() {`
`246`	`246`	`return array(`
`247`	`247`	`'onEvent' => 'onEvent',`
	`248`	`+ 'onEvent' => array('onEvent', 10),`
	`249`	`+ 'onEvent' => array('onEvent'),`
`248`	`250`	`);`
`249`	`251`	`}`
`250`	`252`