Merge pull request #1081 from jrushlow/fix/php-cs-fixer

[ci] adhere to php-cs-fixer standards

Author: jrushlow

src/EventRegistry.php

@@ -79,7 +79,7 @@ public function __construct(EventDispatcherInterface $eventDispatcher)
 
         // Loop through the new event classes
         foreach (self::$newEventsMap as $eventName => $newEventClass) {
-            //Check if the new event classes exist, if so replace the old one with the new.
+            // Check if the new event classes exist, if so replace the old one with the new.
             if (isset(self::$eventsMap[$eventName]) && class_exists($newEventClass)) {
                 self::$eventsMap[$eventName] = $newEventClass;
             }

src/Resources/skeleton/authenticator/Security52LoginFormAuthenticator.tpl.php

@@ -51,7 +51,7 @@ public function onAuthenticationSuccess(Request $request, TokenInterface $token,
         }
 
         // For example:
-        //return new RedirectResponse($this->urlGenerator->generate('some_route'));
+        // return new RedirectResponse($this->urlGenerator->generate('some_route'));
         throw new \Exception('TODO: provide a valid redirect inside '.__FILE__);
     }
 

src/Resources/skeleton/test/KernelTestCase.tpl.php

@@ -11,7 +11,7 @@ public function testSomething(): void
         $kernel = self::bootKernel();
 
         $this->assertSame('test', $kernel->getEnvironment());
-        //$routerService = <?= $use_legacy_container_property ? 'self::$container' : 'static::getContainer()'; ?>->get('router');
-        //$myCustomService = <?= $use_legacy_container_property ? 'self::$container' : 'static::getContainer()'; ?>->get(CustomService::class);
+        // $routerService = <?= $use_legacy_container_property ? 'self::$container' : 'static::getContainer()'; ?>->get('router');
+        // $myCustomService = <?= $use_legacy_container_property ? 'self::$container' : 'static::getContainer()'; ?>->get(CustomService::class);
     }
 }

src/Util/ClassSourceManipulator.php

@@ -694,7 +694,7 @@ private function addCollectionRelation(BaseCollectionRelation $relation): void
         $paramBuilder->setTypeHint($typeHint);
         $adderNodeBuilder->addParam($paramBuilder->getNode());
 
-        //if (!$this->avatars->contains($avatar))
+        // if (!$this->avatars->contains($avatar))
         $containsMethodCallNode = new Node\Expr\MethodCall(
             new Node\Expr\PropertyFetch(new Node\Expr\Variable('this'), $relation->getPropertyName()),
             'contains',
@@ -749,7 +749,7 @@ private function addCollectionRelation(BaseCollectionRelation $relation): void
             // $this->avatars->removeElement($avatar);
             $removerNodeBuilder->addStmt(BuilderHelpers::normalizeStmt($removeElementCall));
         } else {
-            //if ($this->avatars->removeElement($avatar))
+            // if ($this->avatars->removeElement($avatar))
             $ifRemoveElementStmt = new Node\Stmt\If_($removeElementCall);
             $removerNodeBuilder->addStmt($ifRemoveElementStmt);
             if ($relation instanceof RelationOneToMany) {

src/Util/YamlSourceManipulator.php

@@ -835,7 +835,7 @@ private function findEndPositionOfValue($value, $offset = null)
             return $endKeyPosition;
         }
 
-        if (is_scalar($value) || null === $value) {
+        if (\is_scalar($value) || null === $value) {
             $offset = null === $offset ? $this->currentPosition : $offset;
 
             if (\is_bool($value)) {

tests/Util/ClassSourceManipulatorTest.php

@@ -648,7 +648,6 @@ public function getAddOneToOneRelationTests(): \Generator
             (new RelationOneToOne())
                 ->setPropertyName('userProfile')
                 ->setTargetClassName('App\Entity\UserProfile')
-                //->setTargetPropertyName('user')
                 ->setIsNullable(true)
                 ->setIsOwning(true)
                 ->setMapInverseRelation(false),
@@ -660,7 +659,6 @@ public function getAddOneToOneRelationTests(): \Generator
             (new RelationOneToOne())
                 ->setPropertyName('userProfile')
                 ->setTargetClassName('App\Entity\UserProfile')
-                //->setTargetPropertyName('user')
                 ->setIsNullable(false)
                 ->setIsOwning(true)
                 ->setMapInverseRelation(false),

tests/Util/YamlSourceManipulatorTest.php

@@ -30,7 +30,7 @@ public function testSetData(string $startingSource, array $newData, string $expe
 
         $logger = $this->createLogger();
         // uncomment to enhance debugging
-        //$manipulator->setLogger($logger);
+        // $manipulator->setLogger($logger);
 
         $manipulator->setData($newData);