diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[EventDispatcher] Fixed E_NOTICES with multiple eventnames per subscriber with mixed priorities

Browse Source
This commit is contained in:
Drak 2012-04-12 15:48:16 +05:45
parent 88353575e4
commit 57dd9147d9
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Symfony/Component/EventDispatcher/EventDispatcher.php
View File

@ -117,7 +117,7 @@ class EventDispatcher implements EventDispatcherInterface
if (is_string($params)) { if (is_string($params)) {
$this->addListener($eventName, array($subscriber, $params)); $this->addListener($eventName, array($subscriber, $params));
} else { } else {
$this->addListener($eventName, array($subscriber, $params[0]), $params[1]); $this->addListener($eventName, array($subscriber, $params[0]), isset($params[1]) ? $params[1] : 0);
} }
} }
} }

5
tests/Symfony/Tests/Component/EventDispatcher/EventDispatcherTest.php
View File

@ -241,6 +241,9 @@ class TestEventSubscriberWithPriorities implements EventSubscriberInterface
{ {
public static function getSubscribedEvents() public static function getSubscribedEvents()
{ {
return array('pre.foo' => array('preFoo', 10)); return array(
'pre.foo' => array('preFoo', 10),
'post.foo' => array('postFoo'),
);
} }
} }