Skip to content

Commit 117d7f1

Browse files
Never rely on dynamic properties
1 parent 3916560 commit 117d7f1

File tree

3 files changed

+8
-1
lines changed

3 files changed

+8
-1
lines changed

Tests/ContainerBuilderTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,6 @@ public function testGetServiceIds()
253253
{
254254
$builder = new ContainerBuilder();
255255
$builder->register('foo', 'stdClass');
256-
$builder->bar = $bar = new \stdClass();
257256
$builder->register('bar', 'stdClass');
258257
$this->assertEquals(
259258
[

Tests/Fixtures/StdClassDecorator.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44

55
final class StdClassDecorator
66
{
7+
public $foo;
8+
79
public function __construct(\stdClass $foo)
810
{
911
$this->foo = $foo;

Tests/Fixtures/includes/classes.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,8 @@ public static function create($config)
142142

143143
class FoobarCircular
144144
{
145+
public $foo;
146+
145147
public function __construct(FooCircular $foo)
146148
{
147149
$this->foo = $foo;
@@ -150,6 +152,8 @@ public function __construct(FooCircular $foo)
150152

151153
class FooCircular
152154
{
155+
public $bar;
156+
153157
public function __construct(BarCircular $bar)
154158
{
155159
$this->bar = $bar;
@@ -158,6 +162,8 @@ public function __construct(BarCircular $bar)
158162

159163
class BarCircular
160164
{
165+
public $foobar;
166+
161167
public function addFoobar(FoobarCircular $foobar)
162168
{
163169
$this->foobar = $foobar;

0 commit comments

Comments
 (0)