minor symfony#32561 [Config][ReflectionClassResource] Use ternary instead of null coalescing operator (fancyweb)

fabpot · fabpot · commit e5bd6ff59e1b · 2019-07-16T15:04:39.000+02:00
This PR was merged into the 3.4 branch. Discussion ---------- [Config][ReflectionClassResource] Use ternary instead of null coalescing operator | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - `??` cannot be on 3.4, tests are failing because of this. Commits ------- 4db953f [Config][ReflectionClassResource] Use ternary instead of null coaelscing operator
diff --git a/src/Symfony/Component/Config/Resource/ReflectionClassResource.php b/src/Symfony/Component/Config/Resource/ReflectionClassResource.php
@@ -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] ?? null, true);
+                yield print_r(isset($defaults[$p->name]) ? $defaults[$p->name] : null, true);
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ private function generateSignature(\ReflectionClass $class)`
`140`	`140`
`141`	`141`	`foreach ($class->getProperties(\ReflectionProperty::IS_PUBLIC \| \ReflectionProperty::IS_PROTECTED) as $p) {`
`142`	`142`	`yield $p->getDocComment().$p;`
`143`		`- yield print_r($defaults[$p->name] ?? null, true);`
	`143`	`+ yield print_r(isset($defaults[$p->name]) ? $defaults[$p->name] : null, true);`
`144`	`144`	`}`
`145`	`145`	`}`
`146`	`146`