8000 [VarDumper] Allow seamless use of Data clones · symfony/symfony@f7fa04b · GitHub
[go: up one dir, main page]
Skip to content

Commit f7fa04b

Browse files
nicolas-grekasnicolas-grekas
committed
[VarDumper] Allow seamless use of Data clones
1 parent e7a9f03 commit f7fa04b

38 files changed

+568
-420
lines changed

‎src/Symfony/Bundle/SecurityBundle/DataCollector/SecurityDataCollector.php

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
use Symfony\Component\Security\Http\Logout\LogoutUrlGenerator;
2121
use Symfony\Component\Security\Core\Authorization\AccessDecisionManagerInterface;
2222
use Symfony\Component\Security\Core\Authorization\TraceableAccessDecisionManager;
23+
use Symfony\Component\VarDumper\Caster\ClassStub;
2324
use Symfony\Component\VarDumper\Cloner\Data;
2425
use Symfony\Component\Security\Http\FirewallMapInterface;
2526
use Symfony\Bundle\SecurityBundle\Security\FirewallMap;
@@ -36,6 +37,7 @@ class SecurityDataCollector extends DataCollector
3637
private $logoutUrlGenerator;
3738
private $accessDecisionManager;
3839
private $firewallMap;
40+
private $hasVarDumper;
3941

4042
/**
4143
* Constructor.
@@ -53,6 +55,7 @@ public function __construct(TokenStorageInterface $tokenStorage = null, RoleHier
5355
$this->logoutUrlGenerator = $logoutUrlGenerator;
5456
$this->accessDecisionManager = $accessDecisionManager;
5557
$this->firewallMap = $firewallMap;
58+
$this->hasVarDumper = class_exists(ClassStub::class);
5659
}
5760

5861
/**
@@ -109,28 +112,23 @@ public function collect(Request $request, Response $response, \Exception $except
109112
$this->data = array(
110113
'enabled' => true,
111114
'authenticated' => $token->isAuthenticated(),
112-
'token' => $this->cloneVar($token),
113-
'token_class' => get_class($token),
115+
'token' => $token,
116+
'token_class' => $this->hasVarDumper ? new ClassStub(get_class($token)) : get_class($token),
114117
'logout_url' => $logoutUrl,
115118
'user' => $token->getUsername(),
116-
'roles' => $this->cloneVar(array_map(function (RoleInterface $role) { return $role->getRole(); }, $assignedRoles)),
117-
'inherited_roles' => $this->cloneVar(array_map(function (RoleInterface $role) { return $role->getRole(); }, $inheritedRoles)),
119+
'roles' => array_map(function (RoleInterface $role) { return $role->getRole(); }, $assignedRoles),
120+
'inherited_roles' => array_map(function (RoleInterface $role) { return $role->getRole(); }, $inheritedRoles),
118121
'supports_role_hierarchy' => null !== $this->roleHierarchy,
119122
);
120123
}
121124

122125
// collect voters and access decision manager information
123126
if ($this->accessDecisionManager instanceof TraceableAccessDecisionManager) {
124-
$this->data['access_decision_log'] = array_map(function ($decision) {
125-
$decision['object'] = $this-> 105FA cloneVar($decision['object']);
126-
127-
return $decision;
128-
}, $this->accessDecisionManager->getDecisionLog());
129-
127+
$this->data['access_decision_log'] = $this->accessDecisionManager->getDecisionLog();
130128
$this->data['voter_strategy'] = $this->accessDecisionManager->getStrategy();
131129

132130
foreach ($this->accessDecisionManager->getVoters() as $voter) {
133-
$this->data['voters'][] = get_class($voter);
131+
$this->data['voters'][] = $this->hasVarDumper ? new ClassStub(get_class($voter)) : get_class($voter);
134132
}
135133
} else {
136134
$this->data['access_decision_log'] = array();
@@ -155,10 +153,12 @@ public function collect(Request $request, Response $response, \Exception $except
155153
'access_denied_handler' => $firewallConfig->getAccessDeniedHandler(),
156154
'access_denied_url' => $firewallConfig->getAccessDeniedUrl(),
157155
'user_checker' => $firewallConfig->getUserChecker(),
158-
'listeners' => $this->cloneVar($firewallConfig->getListeners()),
156+
'listeners' => $firewallConfig->getListeners(),
159157
);
160158
}
161159
}
160+
161+
$this->data = $this->cloneVar($this->data);
162162
}
163163

164164
/**

‎src/Symfony/Bundle/SecurityBundle/Tests/DataCollector/SecurityDataCollectorTest.php

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -66,14 +66,10 @@ public function testCollectAuthenticationTokenAndRoles(array $roles, array $norm
6666

6767
$this->assertTrue($collector->isEnabled());
6868
$this->assertTrue($collector->isAuthenticated());
69-
$this->assertSame('Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken', $collector->getTokenClass());
69+
$this->assertSame('Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken', $collector->get E15D TokenClass()->getValue());
7070
$this->assertTrue($collector->supportsRoleHierarchy());
71-
$this->assertSame($normalizedRoles, $collector->getRoles()->getRawData()[1]);
72-
if ($inheritedRoles) {
73-
$this->assertSame($inheritedRoles, $collector->getInheritedRoles()->getRawData()[1]);
74-
} else {
75-
$this->assertSame($inheritedRoles, $collector->getInheritedRoles()->getRawData()[0][0]);
76-
}
71+
$this->assertSame($normalizedRoles, $collector->getRoles()->getValue(true));
72+
$this->assertSame($inheritedRoles, $collector->getInheritedRoles()->getValue(true));
7773
$this->assertSame('hhamon', $collector->getUser());
7874
}
7975

@@ -107,7 +103,7 @@ public function testGetFirewall()
107103
$this->assertSame($firewallConfig->getAccessDeniedHandler(), $collected['access_denied_handler']);
108104
$this->assertSame($firewallConfig->getAccessDeniedUrl(), $collected['access_denied_url']);
109105
$this->assertSame($firewallConfig->getUserChecker(), $collected['user_checker']);
110-
$this->assertSame($firewallConfig->getListeners(), $collected['listeners']->getRawData()[0][0]);
106+
$this->assertSame($firewallConfig->getListeners(), $collected['listeners']->getValue());
111107
}
112108

113109
public function testGetFirewallReturnsNull()

‎src/Symfony/Bundle/SecurityBundle/composer.json

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
"php": ">=5.5.9",
2020
"symfony/security": "~3.3",
2121
"symfony/dependency-injection": "~3.3",
22-
"symfony/http-kernel": "~3.2",
22+
"symfony/http-kernel": "~3.3",
2323
"symfony/polyfill-php70": "~1.0"
2424
},
2525
"require-dev": {
@@ -37,12 +37,15 @@
3737
"symfony/twig-bridge": "~2.8|~3.0",
3838
"symfony/process": "~2.8|~3.0",
3939
"symfony/validator": "^3.2.5",
40-
"symfony/var-dumper": "~3.2",
40+
"symfony/var-dumper": "~3.3",
4141
"symfony/yaml": "~2.8|~3.0",
4242
"symfony/expression-language": "~2.8|~3.0",
4343
"doctrine/doctrine-bundle": "~1.4",
4444
"twig/twig": "~1.28|~2.0"
4545
},
46+
"conflict": {
47+
"symfony/var-dumper": "<3.3"
48+
},
4649
"suggest": {
4750
"symfony/security-acl": "For using the ACL functionality of this bundle"
4851
},

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/cache.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@
9090
<div class="metric">
9191
<span class="value">
9292
{% if key == 'time' %}
93-
{{ '%0.2f'|format(1000*value) }} ms
93+
{{ '%0.2f'|format(1000*value.value) }} ms
9494
{% else %}
9595
{{ value }}
9696
{% endif %}

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/config.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,7 @@
267267
{% for name in collector.bundles|keys|sort %}
268268
<tr>
269269
<th scope="row" class="font-normal">{{ name }}</th>
270-
<td class="font-normal">{{ collector.bundles[name] }}</td>
270+
<td class="font-normal">{{ profiler_dump(collector.bundles[name]) }}</td>
271271
</tr>
272272
{% endfor %}
273273
</tbody>

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/events.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@
7575

7676
<tr>
7777
<td class="text-right">{{ listener.priority|default('-') }}</td>
78-
<td class="font-normal">{{ profiler_dump(listener.data) }}</td>
78+
<td class="font-normal">{{ profiler_dump(listener.stub) }}</td>
7979
</tr>
8080

8181
{% if loop.last %}

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/logger.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@
182182
<a class="btn btn-link text-small sf-toggle" data-toggle-selector="#{{ context_id }}" data-toggle-alt-content="Hide trace">Show trace</a>
183183

184184
<div id="{{ context_id }}" class="context sf-toggle-content sf-toggle-hidden">
185-
{{ profiler_dump(log.context.seek('exception').seek('\0Exception\0trace'), maxDepth=2) }}
185+
{{ profiler_dump(log.context.exception['\0Exception\0trace'], maxDepth=2) }}
186186
</div>
187187
</span>
188188
{% elseif log.context is defined and log.context is not empty %}

‎src/Symfony/Bundle/WebProfilerBundle/Twig/WebProfilerExtension.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,9 @@ public function dumpLog(\Twig_Environment $env, $message, Data $context)
9898
}
9999

100100
$replacements = array();
101-
foreach ($context->getRawData()[1] as $k => $v) {
102-
$v = '{'.twig_escape_filter($env, $k).'}';
103-
$replacements['&quot;'.$v.'&quot;'] = $replacements[$v] = $this->dumpData($env, $context->seek($k));
101+
foreach ($context as $k => $v) {
102+
$k = '{'.twig_escape_filter($env, $k).'}';
103+
$replacements['&quot;'.$k.'&quot;'] = $replacements[$k] = $this->dumpData($env, $v);
104104
}
105105

106106
return '<span class="dump-inline">'.strtr($message, $replacements).'</span>';

‎src/Symfony/Bundle/WebProfilerBundle/composer.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
"symfony/routing": "~2.8|~3.0",
2323
"symfony/twig-bridge": "~2.8|~3.0",
2424
"twig/twig": "~1.28|~2.0",
25-
"symfony/var-dumper": "~3.2"
25+
"symfony/var-dumper": "~3.3"
2626
},
2727
"require-dev": {
2828
"symfony/config": "~2.8|~3.0",
@@ -31,7 +31,8 @@
3131
"symfony/stopwatch": "~2.8|~3.0"
3232
},
 3BFD
3333
"conflict": {
34-
"symfony/event-dispatcher": "<3.2"
34+
"symfony/event-dispatcher": "<3.2",
35+
"symfony/var-dumper": "<3.3"
3536
},
3637
"autoload": {
3738
"psr-4": { "Symfony\\Bundle\\WebProfilerBundle\\": "" },

‎src/Symfony/Component/Cache/DataCollector/CacheDataCollector.php

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -45,20 +45,13 @@ public function collect(Request $request, Response $response, \Exception $except
4545
$empty = array('calls' => array(), 'config' => array(), 'options' => array(), 'statistics' => array());
4646
$this->data = array('instances' => $empty, 'total' => $empty);
4747
foreach ($this->instances as $name => $instance) {
48-
$calls = $instance->getCalls();
49-
foreach ($calls as $call) {
50-
if (isset($call->result)) {
51-
$call->result = $this->cloneVar($call->result);
52-
}
53-
if (isset($call->argument)) {
54-
$call->argument = $this->cloneVar($call->argument);
55-
}
56-
}
57-
$this->data['instances']['calls'][$name] = $calls;
48+
$this->data['instances']['calls'][$name] = $instance->getCalls();
5849
}
5950

6051
$this->data['instances']['statistics'] = $this->calculateStatistics();
6152
$this->data['total']['statistics'] = $this->calculateTotalStatistics();
53+
54+
$this->data = $this->cloneVar($this->data);
6255
}
6356

6457
/**
@@ -133,7 +126,7 @@ private function calculateStatistics()
133126
$statistics[$name]['misses'] += $count - $call->misses;
134127
} elseif ('hasItem' === $call->name) {
135128
$statistics[$name]['reads'] += 1;
136-
if (false === $call->result->getRawData()[0][0]) {
129+
if (false === $call->result) {
137130
$statistics[$name]['misses'] += 1;
138131
} else {
139132
$statistics[$name]['hits'] += 1;

‎src/Symfony/Component/Cache/composer.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@
3131
"doctrine/dbal": "~2.4",
3232
"predis/predis": "~1.0"
3333
},
34+
"conflict": {
35+
"symfony/var-dumper": "<3.3"
36+
},
3437
"suggest": {
3538
"symfony/polyfill-apcu": "For using ApcuAdapter on HHVM"
3639
},

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ class WrappedListener
2929
private $stopwatch;
3030
private $dispatcher;
3131
private $pretty;
32-
private $data;
32+
private $stub;
3333

3434
private static $cloner;
3535

@@ -91,15 +91,15 @@ public function getPretty()
9191

9292
public function getInfo($eventName)
9393
{
94-
if (null === $this->data) {
95-
$this->data = false !== self::$cloner ? self::$cloner->cloneVar(array(new ClassStub($this->pretty.'()', $this->listener)))->seek(0) : $this->pretty;
94+
if (null === $this->stub) {
95+
$this->stub = false === self::$cloner ? $this->pretty.'()' : new ClassStub($this->pretty.'()', $this->listener);
9696
}
9797

9898
return array(
9999
'event' => $eventName,
100100
'priority' => null !== $this->dispatcher ? $this->dispatcher->getListenerPriority($eventName, $this->listener) : null,
101101
'pretty' => $this->pretty,
102-
'data' => $this->data,
102+
'stub' => $this->stub,
103103
);
104104
}
105105

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

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

9999
$listeners = $tdispatcher->getNotCalledListeners();
100-
$this->assertArrayHasKey('data', $listeners['foo.closure']);
101-
unset($listeners['foo.closure']['data']);
100+
$this->assertArrayHasKey('stub', $listeners['foo.closure']);
101+
unset($listeners['foo.closure']['stub']);
102102
$this->assertEquals(array(), $tdispatcher->getCalledListeners());
103103
$this->assertEquals(array('foo.closure' => array('event' => 'foo', 'pretty' => 'closure', 'priority' => 0)), $listeners);
104104

105105
$tdispatcher->dispatch('foo');
106106

107107
$listeners = $tdispatcher->getCalledListeners();
108-
$this->assertArrayHasKey('data', $listeners['foo.closure']);
109-
unset($listeners['foo.closure']['data']);
108+
$this->assertArrayHasKey('stub', $listeners['foo.closure']);
109+
unset($listeners['foo.closure']['stub']);
110110
$this->assertEquals(array('foo.closure' => array('event' => 'foo', 'pretty' => 'closure', 'priority' => null)), $listeners);
111111
$this->assertEquals(array(), $tdispatcher->getNotCalledListeners());
112112
}

0 commit comments

Comments
 (0)
0