Skip to content

Commit 62cb443

Browse files
committed
minor symfony#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 symfony@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 symfony#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)