10000 minor #22663 [DI] Fix PhpDumper blank lines around namespace (ogizanagi) · symfony/symfony@62cb443 · GitHub
[go: up one dir, main page]
Skip to content

Commit 62cb443

Browse files
fabpotfabpot
committed
minor #22663 [DI] Fix PhpDumper blank lines around namespace (ogizanagi)
This PR was merged into the 2.7 branch. Discussion ---------- [DI] Fix PhpDumper blank lines around namespace | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | N/A | License | MIT | Doc PR | N/A 219bce9 tries to fix CS in an expected PhpDumper output, which actually shows the PhpDumper does not create proper blank lines around the namespace. That's why tests are failing on #22660. Commits ------- c9f6c16 [DI] Fix PhpDumper blank lines around namespace
2 parents 219bce9 + c9f6c16 commit 62cb443

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

‎src/Symfony/Component/DependencyInjection/Dumper/PhpDumper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -827,7 +827,7 @@ private function addNewInstance($id, Definition $definition, $return, $instantia
827827
private function startClass($class, $baseClass, $namespace)
828828
{
829829
$bagClass = $this->container->isFrozen() ? 'use Symfony\Component\DependencyInjection\ParameterBag\FrozenParameterBag;' : 'use Symfony\Component\DependencyInjection\ParameterBag\\ParameterBag;';
830-
$namespaceLine = $namespace ? "namespace $namespace;\n" : '';
830+
$namespaceLine = $namespace ? "\nnamespace $namespace;\n" : '';
831831

832832
return <<<EOF
833833
<?php

0 commit comments

Comments
 (0)
0