minor #502 Running php-cs-fixer (weaverryan)

This PR was merged into the 1.0-dev branch.

Discussion
----------

Running php-cs-fixer

Commits
-------

0f84fcd Running php-cs-fixer

Author: weaverryan

src/ApplicationAwareMakerInterface.php

@@ -20,8 +20,5 @@
  */
 interface ApplicationAwareMakerInterface
 {
-    /**
-     * @param Application $application
-     */
     public function setApplication(Application $application);
 }

src/Doctrine/DoctrineHelper.php

@@ -70,8 +70,6 @@ public function getEntityNamespace(): string
     }
 
     /**
-     * @param string $className
-     *
      * @return MappingDriver|null
      *
      * @throws \Exception
@@ -118,9 +116,6 @@ public function getEntitiesForAutocomplete(): array
     }
 
     /**
-     * @param string|null $classOrNamespace
-     * @param bool        $disconnected
-     *
      * @return array|ClassMetadata
      */
     public function getMetadata(string $classOrNamespace = null, bool $disconnected = false)
@@ -180,8 +175,6 @@ public function getMetadata(string $classOrNamespace = null, bool $disconnected
     }
 
     /**
-     * @param string $entityClassName
-     *
      * @return EntityDetails|null
      */
     public function createDoctrineDetails(string $entityClassName)

src/Doctrine/ORMDependencyBuilder.php

@@ -22,8 +22,6 @@ final class ORMDependencyBuilder
 {
     /**
      * Central method to add dependencies needed for Doctrine ORM.
-     *
-     * @param DependencyBuilder $dependencies
      */
     public static function buildDependencies(DependencyBuilder $dependencies)
     {

src/FileManager.php

@@ -87,8 +87,6 @@ public function fileExists($path): bool
      *
      * @param string $absolutePath
      *
-     * @return string
-     *
      * @throws \Exception
      */
     public function relativizePath($absolutePath): string
@@ -148,8 +146,6 @@ public function absolutizePath($path): string
     }
 
     /**
-     * @param string $className
-     *
      * @return string|null
      *
      * @throws \Exception
@@ -189,8 +185,6 @@ public function getPathForTemplate(string $filename): string
      * Resolve '../' in paths (like real_path), but for non-existent files.
      *
      * @param string $absolutePath
-     *
-     * @return string
      */
     private function realPath($absolutePath): string
     {

src/Generator.php

@@ -64,10 +64,6 @@ public function generateClass(string $className, string $templateName, array $va
 
     /**
      * Generate a normal file from a template.
-     *
-     * @param string $targetPath
-     * @param string $templateName
-     * @param array  $variables
      */
     public function generateFile(string $targetPath, string $templateName, array $variables)
     {
@@ -124,12 +120,9 @@ public function getFileContentsForPendingOperation(string $targetPath): string
      *      // Cool\Stuff\BalloonController
      *      $gen->createClassNameDetails('Cool\\Stuff\\Balloon', 'Controller', 'Controller');
      *
-     * @param string $name                   The short "name" that will be turned into the class name
-     * @param string $namespacePrefix        Recommended namespace where this class should live, but *without* the "App\\" part
-     * @param string $suffix                 Optional suffix to guarantee is on the end of the class
-     * @param string $validationErrorMessage
-     *
-     * @return ClassNameDetails
+     * @param string $name            The short "name" that will be turned into the class name
+     * @param string $namespacePrefix Recommended namespace where this class should live, but *without* the "App\\" part
+     * @param string $suffix          Optional suffix to guarantee is on the end of the class
      */
     public function createClassNameDetails(string $name, string $namespacePrefix, string $suffix = '', string $validationErrorMessage = ''): ClassNameDetails
     {
@@ -160,10 +153,7 @@ public function getRootDirectory(): string
     private function addOperation(string $targetPath, string $templateName, array $variables)
     {
         if ($this->fileManager->fileExists($targetPath)) {
-            throw new RuntimeCommandException(sprintf(
-                'The file "%s" can\'t be generated because it already exists.',
-                $this->fileManager->relativizePath($targetPath)
-            ));
+            throw new RuntimeCommandException(sprintf('The file "%s" can\'t be generated because it already exists.', $this->fileManager->relativizePath($targetPath)));
         }
 
         $variables['relative_path'] = $this->fileManager->relativizePath($targetPath);
@@ -228,10 +218,6 @@ public function generateController(string $controllerClassName, string $controll
 
     /**
      * Generate a template file.
-     *
-     * @param string $targetPath
-     * @param string $templateName
-     * @param array  $variables
      */
     public function generateTemplate(string $targetPath, string $templateName, array $variables)
     {

src/InputConfiguration.php

@@ -18,8 +18,6 @@ final class InputConfiguration
     /**
      * Call in MakerInterface::configureCommand() to disable the automatic interactive
      * prompt for an argument.
-     *
-     * @param string $argumentName
      */
     public function setArgumentAsNonInteractive(string $argumentName)
     {

src/MakerInterface.php

@@ -23,8 +23,6 @@ interface MakerInterface
 {
     /**
      * Return the command name for your maker (e.g. make:report).
-     *
-     * @return string
      */
     public static function getCommandName(): string;
 
@@ -33,34 +31,21 @@ public static function getCommandName(): string;
      *
      * By default, all arguments will be asked interactively. If you want
      * to avoid that, use the $inputConfig->setArgumentAsNonInteractive() method.
-     *
-     * @param Command            $command
-     * @param InputConfiguration $inputConfig
      */
     public function configureCommand(Command $command, InputConfiguration $inputConfig);
 
     /**
      * Configure any library dependencies that your maker requires.
-     *
-     * @param DependencyBuilder $dependencies
      */
     public function configureDependencies(DependencyBuilder $dependencies);
 
     /**
      * If necessary, you can use this method to interactively ask the user for input.
-     *
-     * @param InputInterface $input
-     * @param ConsoleStyle   $io
-     * @param Command        $command
      */
     public function interact(InputInterface $input, ConsoleStyle $io, Command $command);
 
     /**
      * Called after normal code generation: allows you to do anything.
-     *
-     * @param InputInterface $input
-     * @param ConsoleStyle   $io
-     * @param Generator      $generator
      */
     public function generate(InputInterface $input, ConsoleStyle $io, Generator $generator);
 }

src/Security/SecurityConfigUpdater.php

@@ -24,12 +24,6 @@ final class SecurityConfigUpdater
 
     /**
      * Updates security.yaml contents based on a new User class.
-     *
-     * @param string                 $yamlSource
-     * @param UserClassConfiguration $userConfig
-     * @param string                 $userClass
-     *
-     * @return string
      */
     public function updateForUserClass(string $yamlSource, UserClassConfiguration $userConfig, string $userClass): string
     {

src/Test/MakerTestProcess.php

@@ -48,12 +48,7 @@ public function run($allowToFail = false, array $envVars = []): self
         $this->process->run(null, $envVars);
 
         if (!$allowToFail && !$this->process->isSuccessful()) {
-            throw new \Exception(sprintf(
-                'Error running command: "%s". Output: "%s". Error: "%s"',
-                $this->process->getCommandLine(),
-                $this->process->getOutput(),
-                $this->process->getErrorOutput()
-            ));
+            throw new \Exception(sprintf('Error running command: "%s". Output: "%s". Error: "%s"', $this->process->getCommandLine(), $this->process->getOutput(), $this->process->getErrorOutput()));
         }
 
         return $this;

src/Util/AutoloaderUtil.php

@@ -33,8 +33,6 @@ public function __construct(ComposerAutoloaderFinder $autoloaderFinder)
     /**
      * Returns the relative path to where a new class should live.
      *
-     * @param string $className
-     *
      * @return string|null
      *
      * @throws \Exception

src/Util/ClassNameDetails.php

@@ -44,8 +44,6 @@ public function getShortName(): string
      *
      * For example, assuming the namespace is App\Entity:
      *      App\Entity\Admin\User => Admin\User
-     *
-     * @return string
      */
     public function getRelativeName(): string
     {

src/Util/ClassSourceManipulator.php

@@ -687,8 +687,6 @@ private function getConstructorNode()
     }
 
     /**
-     * @param string $class
-     *
      * @return string The alias to use when referencing this class
      */
     public function addUseStatementIfNecessary(string $class): string
@@ -837,8 +835,6 @@ private function getNamespaceNode(): Node\Stmt\Namespace_
     }
 
     /**
-     * @param callable $filterCallback
-     *
      * @return Node|null
      */
     private function findFirstNode(callable $filterCallback)
@@ -852,9 +848,6 @@ private function findFirstNode(callable $filterCallback)
     }
 
     /**
-     * @param callable $filterCallback
-     * @param array    $ast
-     *
      * @return Node|null
      */
     private function findLastNode(callable $filterCallback, array $ast)
@@ -1045,8 +1038,6 @@ private function getThisFullClassName(): string
      * Adds this new node where a new property should go.
      *
      * Useful for adding properties, or adding a constructor.
-     *
-     * @param Node $newNode
      */
     private function addNodeAfterProperties(Node $newNode)
     {

src/Util/PrettyPrinter.php

@@ -33,8 +33,6 @@ final class PrettyPrinter extends Standard
      * changes) adds 4 spaces onto that. This is why we don't
      * also need to handle indent levels of 5, 9, etc: these
      * do not occur (at least in the code we generate);
-     *
-     * @param int $level
      */
     protected function setIndentLevel(int $level)
     {

src/Util/YamlSourceManipulator.php

@@ -904,8 +904,6 @@ private function guessNextArrayTypeAndAdvance(): string
 
     /**
      * Advance until you find *one* of the characters in $chars.
-     *
-     * @param array $chars
      */
     private function findPositionOfNextCharacter(array $chars)
     {