Skip to content

Commit

Permalink
[Config] Fix for signatures of typed properties
Browse files Browse the repository at this point in the history
  • Loading branch information
tvandervorm authored and fabpot committed Jul 15, 2019
1 parent 0825ea7 commit bad2a2c
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
Expand Up @@ -140,7 +140,7 @@ private function generateSignature(\ReflectionClass $class)

foreach ($class->getProperties(\ReflectionProperty::IS_PUBLIC | \ReflectionProperty::IS_PROTECTED) as $p) {
yield $p->getDocComment().$p;
yield print_r($defaults[$p->name], true);
yield print_r($defaults[$p->name] ?? null, true);
}
}

Expand Down
Expand Up @@ -137,6 +137,14 @@ public function provideHashedSignature()
yield [1, 13, 'protected function prot($a = [123]) {}'];
yield [0, 14, '/** priv docblock */'];
yield [0, 15, ''];

if (\PHP_VERSION_ID >= 70400) {
// PHP7.4 typed properties without default value are
// undefined, make sure this doesn't throw an error
yield [1, 5, 'public array $pub;'];
yield [0, 7, 'protected int $prot;'];
yield [0, 9, 'private string $priv;'];
}
}

public function testEventSubscriber()
Expand Down

0 comments on commit bad2a2c

Please sign in to comment.