Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 224a8e0

Browse files
nicolas-grekasnicolas-grekas
committed
[Form][EventDispatcher] Fix VarDumper usage related to perf regression
1 parent 84229f8 commit 224a8e0

File tree

7 files changed

+273
-244
lines changed

7 files changed

+273
-244
lines changed

‎src/Symfony/Component/EventDispatcher/Debug/TraceableEventDispatcher.php

Lines changed: 15 additions & 82 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@
1515
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
1616
use Symfony\Component\EventDispatcher\Event;
1717
use Symfony\Component\Stopwatch\Stopwatch;
18-
use Symfony\Component\VarDumper\Caster\ClassStub;
19-
use Symfony\Component\VarDumper\Cloner\VarCloner;
2018
use Psr\Log\LoggerInterface;
2119

2220
/**
@@ -34,7 +32,6 @@ class TraceableEventDispatcher implements TraceableEventDispatcherInterface
3432
private $called;
3533
private $dispatcher;
3634
private $wrappedListeners;
37-
private $cloner;
3835

3936
/**
4037
* Constructor.
@@ -50,9 +47,6 @@ public function __construct(EventDispatcherInterface $dispatcher, Stopwatch $sto
5047
$this->logger = $logger;
5148
$this->called = array();
5249
$this->wrappedListeners = array();
53-
if (class_exists(ClassStub::class)) {
54-
$this->cloner = new VarCloner();
55-
}
5650
}
5751

5852
/**
@@ -159,8 +153,7 @@ public function getCalledListeners()
159153
$called = array();
160154
foreach ($this->called as $eventName => $listeners) {
161155
foreach ($listeners as $listener) {
162-
$info = $this->getListenerInfo($listener->getWrappedListener(), $eventName);
163-
$called[$eventName.'.'.$info['pretty']] = $info;
156+
$called[$eventName.'.'.$listener->getPretty()] = $listener->getInfo($eventName);
164157
}
165158
}
166159

@@ -198,8 +191,10 @@ public function getNotCalledListeners()
198191
}
199192

200193
if (!$called) {
201-
$info = $this->getListenerInfo($listener, $eventName);
202-
$notCalled[$eventName.'.'.$info['pretty']] = $info;
194+
if (!$listener instanceof WrappedListener) {
195+
$listener = new WrappedListener($listener, null, $this->stopwatch, $this);
196+
}
197+
$notCalled[$eventName.'.'.$listener->getPretty()] = $listener->getInfo($eventName);
203198
}
204199
}
205200
}
@@ -245,12 +240,11 @@ protected function postDispatch($eventName, Event $event)
245240
private function preProcess($eventName)
246241
{
247242
foreach ($this->dispatcher->getListeners($eventName) as $listener) {
248-
$info = $this->getListenerInfo($listener, $eventName);
249-
$name = isset($info['class']) ? $info['class'] : $info['type'];
250-
$wrappedListener = new WrappedListener($listener, $name, $this->stopwatch, $this);
243+
$priority = $this->getListenerPriority($eventName, $listener);
244+
$wrappedListener = new WrappedListener($listener, null, $this->stopwatch, $this);
251245
$this->wrappedListeners[$eventName][] = $wrappedListener;
252246
$this->dispatcher->removeListener($eventName, $listener);
253-
$this->dispatcher->addListener($eventName, $wrappedListener, $info['priority']);
247+
$this->dispatcher->addListener($eventName, $wrappedListener, $priority);
254248
}
255249
}
256250

@@ -267,10 +261,13 @@ private function postProcess($eventName)
267261
$this->dispatcher->removeListener($eventName, $listener);
268262
$this->dispatcher->addListener($eventName, $listener->getWrappedListener(), $priority);
269263

270-
$info = $this->getListenerInfo($listener->getWrappedListener(), $eventName);
264+
if (null !== $this->logger) {
265+
$context = array('event' => $eventName, 'listener' => $listener->getPretty());
266+
}
267+
271268
if ($listener->wasCalled()) {
272269
if (null !== $this->logger) {
273-
$this->logger->debug('Notified event "{event}" to listener "{listener}".', array('event' => $eventName, 'listener' => $info['pretty']));
270+
$this->logger->debug('Notified event "{event}" to listener "{listener}".', $context);
274271
}
275272

276273
if (!isset($this->called[$eventName])) {
@@ -281,83 +278,19 @@ private function postProcess($eventName)
281278
}
282279

283280
if (null !== $this->logger && $skipped) {
284-
$this->logger->debug('Listener "{listener}" was not called for event "{event}".', array('listener' => $info['pretty'], 'event' => $eventName));
281+
$this->logger->debug('Listener "{listener}" was not called for event "{event}".', $context);
285282
}
286283

287284
if ($listener->stoppedPropagation()) {
288285
if (null !== $this->logger) {
289-
$this->logger->debug('Listener "{listener}" stopped propagation of the event "{event}".', array('listener' => $info['pretty'], 'event' => $eventName));
286+
$this->logger->debug('Listener "{listener}" stopped propagation of the event "{event}".', $context);
290287
}
291288

292289
$skipped = true;
293290
}
294291
}
295292
}
296293

297-
/**
298-
* Returns information about the listener.
299-
*
300-
* @param object $listener The listener
301-
* @param string $eventName The event name
302-
*
303-
* @return array Information about the listener
304-
*/
305-
private function getListenerInfo($listener, $eventName)
306-
{
307-
$info = array(
308-
'event' => $eventName,
309-
'priority' => $this->getListenerPriority($eventName, $listener),
310-
);
311-
if ($listener instanceof \Closure) {
312-
$info += array(
313-
'type' => 'Closure',
314-
'pretty' => 'closure',
315-
);
316-
} elseif (is_string($listener)) {
317-
try {
318-
$r = new \ReflectionFunction($listener);
319-
$file = $r->getFileName();
320-
$line = $r->getStartLine();
321-
} catch (\ReflectionException $e) {
322-
$file = null;
323-
$line = null;
324-
}
325-
$info += array(
326-
'type' => 'Function',
327-
'function' => $listener,
328-
'file' => $file,
329-
'line' => $line,
330-
'pretty' => $listener,
331-
);
332-
} elseif (is_array($listener) || (is_object($listener) && is_callable($listener))) {
333-
if (!is_array($listener)) {
334-
$listener = array($listener, '__invoke');
335-
}
336-
$class = is_object($listener[0]) ? get_class($listener[0]) : $listener[0];
337-
try {
338-
$r = new \ReflectionMethod($class, $listener[1]);
339-
$file = $r->getFileName();
340-
$line = $r->getStartLine();
341-
} catch (\ReflectionException $e) {
342-
$file = null;
343-
$line = null;
344-
}
345-
$info += array(
346-
'type' => 'Method',
347-
'class' => $class,
348-
'method' => $listener[1],
349-
'file' => $file,
350-
'line' => $line,
351-
'pretty' => $class.'::'.$listener[1],
352-
);
353-
}
354-
if (null !== $this->cloner) {
355-
$info['data'] = $this->cloner->cloneVar(array(new ClassStub($info['pretty'].'()', $listener)))->seek(0);
356-
}
357-
358-
return $info;
359-
}
360-
361294
private function sortListenersByPriority($a, $b)
362295
{
363296
if (is_int($a['priority']) && !is_int($b['priority'])) {

‎src/Symfony/Component/EventDispatcher/Debug/WrappedListener.php

Lines changed: 45 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
use Symfony\Component\Stopwatch\Stopwatch;
1515
use Symfony\Component\EventDispatcher\Event;
1616
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
17+
use Symfony\Component\VarDumper\Caster\ClassStub;
18+
use Symfony\Component\VarDumper\Cloner\VarCloner;
1719

1820
/**
1921
* @author Fabien Potencier <[email protected]>
@@ -26,6 +28,10 @@ class WrappedListener
2628
private $stoppedPropagation;
2729
private $stopwatch;
2830
private $dispatcher;
31+
private $pretty;
32+
private $data;
33+
34+
private static $cloner;
2935

3036
public function __construct($listener, $name, Stopwatch $stopwatch, EventDispatcherInterface $dispatcher = null)
3137
{
@@ -35,6 +41,26 @@ public function __construct($listener, $name, Stopwatch $stopwatch, EventDispatc
3541
$this->dispatcher = $dispatcher;
3642
$this->called = false;
3743
$this->stoppedPropagation = false;
44+
45+
if (is_array($listener)) {
46+
$this->name = is_object($listener[0]) ? get_class($listener[0]) : $listener[0];
47+
$this->pretty = $this->name.'::'.$listener[1];
48+
} elseif ($listener instanceof \Closure) {
49+
$this->pretty = $this->name = 'closure';
50+
} elseif (is_string($listener)) {
51+
$this->pretty = $this->name = $listener;
52+
} else {
53+
$this->name = get_class($listener);
54+
$this->pretty = $this->name.'::__invoke';
55+
}
56+
57+
if (null !== $name) {
58+
$this->name = $name;
59+
}
60+
61+
if (null === self::$cloner) {
62+
self::$cloner = class_exists(ClassStub::class) ? new VarCloner() : false;
63+
}
3864
}
3965

4066
public function getWrappedListener()
@@ -52,6 +78,25 @@ public function stoppedPropagation()
5278
return $this->stoppedPropagation;
5379
}
5480

81+
public function getPretty()
82+
{
83+
return $this->pretty;
84+
}
85+
86+
public function getInfo($eventName)
87+
{
88+
if (null === $this->data) {
89+
$this->data = false !== self::$cloner ? self::$cloner->cloneVar(array(new ClassStub($this->pretty.'()', $this->listener)))->seek(0) : $this->pretty;
90+
}
91+
92+
return array(
93+
'event' => $eventName,
94+
'priority' => null !== $this->dispatcher ? $this->dispatcher->getListenerPriority($eventName, $this->listener) : null,
95+
'pretty' => $this->pretty,
96+
'data' => $this->data,
97+
);
98+
}
99+
55100
public function __invoke(Event $event, $eventName, EventDispatcherInterface $dispatcher)
56101
{
57102
$this->called = true;

‎src/Symfony/Component/EventDispatcher/Tests/Debug/TraceableEventDispatcherTest.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,15 +96,17 @@ public function testGetCalledListeners()
9696
$tdispatcher->addListener('foo', $listener = function () {});
9797

9898
$listeners = $tdispatcher->getNotCalledListeners();
99+
$this->assertArrayHasKey('data', $listeners['foo.closure']);
99100
unset($listeners['foo.closure']['data']);
100101
$this->assertEquals(array(), $tdispatcher->getCalledListeners());
101-
$this->assertEquals(array('foo.closure' => array('event' => 'foo', 'type' => 'Closure', 'pretty' => 'closure', 'priority' => 0)), $listeners);
102+
$this->assertEquals(array('foo.closure' => array('event' => 'foo', 'pretty' => 'closure', 'priority' => 0)), $listeners);
102103

103104
$tdispatcher->dispatch('foo');
104105

105106
$listeners = $tdispatcher->getCalledListeners();
107+
$this->assertArrayHasKey('data', $listeners['foo.closure']);
106108
unset($listeners['foo.closure']['data']);
107-
$this->assertEquals(array('foo.closure' => array('event' => 'foo', 'type' => 'Closure', 'pretty' => 'closure', 'priority' => null)), $listeners);
109+
$this->assertEquals(array('foo.closure' => array('event' => 'foo', 'pretty' => 'closure', 'priority' => null)), $listeners);
108110
$this->assertEquals(array(), $tdispatcher->getNotCalledListeners());
109111
}
110112

0 commit comments

Comments
 (0)