phpDocumentor / Reflection

tests/example.file.php

Doc Comments -1 removed lines

@@ -99,7 +99,6 @@
         /**
          * Calculates the price for this specific component.
          *
-         * @param float[] $...additionalPrices Additional costs may be passed
          *
          * @return float
          */

example.php

Indentation +12 added lines, -12 removed lines

@@ -1,17 +1,17 @@
 <?php
 /**
-     * Create a project map (similar to a sitemap) of project.
-     *
-     * The Reflection component is capable of analyzing one or more files into a hierarchy of objects representing the
-     * structure of your project. It does this by analyzing the source code of each individual file using the
-     * `analyze()` method in the Analyzer class.
-     *
-     * Because the Analyzer class requires a whole series of objects that interact together a factory method `create()`
-     * is available. This method instantiates all objects and provides a reasonable default to start using the Analyzer.
-     *
-     * There is also a Service Provider (`\phpDocumentor\Descriptor\ServiceProvider`) that can be used with either Silex
-     * or Cilex instead of using the factory method; this will make it easier to plug in your own features if you want to.
-     */
+ * Create a project map (similar to a sitemap) of project.
+ *
+ * The Reflection component is capable of analyzing one or more files into a hierarchy of objects representing the
+ * structure of your project. It does this by analyzing the source code of each individual file using the
+ * `analyze()` method in the Analyzer class.
+ *
+ * Because the Analyzer class requires a whole series of objects that interact together a factory method `create()`
+ * is available. This method instantiates all objects and provides a reasonable default to start using the Analyzer.
+ *
+ * There is also a Service Provider (`\phpDocumentor\Descriptor\ServiceProvider`) that can be used with either Silex
+ * or Cilex instead of using the factory method; this will make it easier to plug in your own features if you want to.
+ */
 
 // use Composer's autoloader to allow the application to automatically load all classes on request.
 use phpDocumentor\Reflection\Php\Project;

Spacing +3 added lines, -3 removed lines

@@ -22,16 +22,16 @@
 $projectFactory = \phpDocumentor\Reflection\Php\ProjectFactory::createInstance();
 
 // Create an array of files to analize.
-$files = [ new \phpDocumentor\Reflection\File\LocalFile('tests/example.file.php') ];
+$files = [new \phpDocumentor\Reflection\File\LocalFile('tests/example.file.php')];
 
 //create a new project 'MyProject' containing all elements in the files.
 /** @var Project $project */
 $project = $projectFactory->create('MyProject', $files);
 
 // As an example of what you can do, let's list all class names in the file 'tests/example.file.php'.
-echo 'List all classes in the example source file: ' . PHP_EOL;
+echo 'List all classes in the example source file: '.PHP_EOL;
 
 /** @var \phpDocumentor\Reflection\Php\Class_ $class */
 foreach ($project->getFiles()['tests/example.file.php']->getClasses() as $class) {
-    echo '- ' . $class->getFqsen() . PHP_EOL;
+    echo '- '.$class->getFqsen().PHP_EOL;
 }

src/phpDocumentor/Reflection/Php/Factory/File.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
     {
         $this->nodesFactory = $nodesFactory;
 
-        $lastCallable = function ($command) {
+        $lastCallable = function($command) {
             return $this->createFile($command);
         };
 

Doc Comments +1 added lines

@@ -155,6 +155,7 @@
 
     /**
      * @param Node[] $nodes
+     * @param StrategyContainer $strategies
      */
     protected function createFileDocBlock(
         ?Doc $docBlock = null,

Unused Use Statements -2 removed lines

@@ -16,7 +16,6 @@ 
 
 use phpDocumentor\Reflection\DocBlock as DocBlockInstance;
 use phpDocumentor\Reflection\File as FileSystemFile;
-use phpDocumentor\Reflection\Fqsen;
 use phpDocumentor\Reflection\Middleware\ChainFactory;
 use phpDocumentor\Reflection\Middleware\Middleware;
 use phpDocumentor\Reflection\Php\Factory\File\CreateCommand;
@@ -25,7 +24,6 @@ 
 use phpDocumentor\Reflection\Php\ProjectFactoryStrategy;
 use phpDocumentor\Reflection\Php\StrategyContainer;
 use phpDocumentor\Reflection\Types\Context;
-use phpDocumentor\Reflection\Types\ContextFactory;
 use phpDocumentor\Reflection\Types\NamespaceNodeToContext;
 use PhpParser\Comment\Doc;
 use PhpParser\Node;

src/phpDocumentor/Reflection/Php/ProjectFactory.php

Doc Comments +1 added lines

@@ -98,6 +98,7 @@
 
     /**
      * Gets Namespace from the project if it exists, otherwise returns a new namepace
+     * @param string $name
      */
     private function getNamespaceByName(Project $project, $name): Namespace_
     {

Spacing +6 added lines, -6 removed lines

@@ -90,7 +90,7 @@ 
     {
         foreach ($project->getFiles() as $file) {
             foreach ($file->getNamespaces() as $namespaceFqsen) {
-                $namespace = $this->getNamespaceByName($project, (string) $namespaceFqsen);
+                $namespace = $this->getNamespaceByName($project, (string)$namespaceFqsen);
                 $this->buildNamespace($file, $namespace);
             }
         }
@@ -118,31 +118,31 @@ 
     private function buildNamespace(File $file, Namespace_ $namespace): void
     {
         foreach ($file->getClasses() as $class) {
-            if ($namespace->getFqsen() . '\\' . $class->getName() === (string) $class->getFqsen()) {
+            if ($namespace->getFqsen().'\\'.$class->getName() === (string)$class->getFqsen()) {
                 $namespace->addClass($class->getFqsen());
             }
         }
 
         foreach ($file->getInterfaces() as $interface) {
-            if ($namespace->getFqsen() . '\\' . $interface->getName() === (string) $interface->getFqsen()) {
+            if ($namespace->getFqsen().'\\'.$interface->getName() === (string)$interface->getFqsen()) {
                 $namespace->addInterface($interface->getFqsen());
             }
         }
 
         foreach ($file->getFunctions() as $function) {
-            if ($namespace->getFqsen() . '\\' . $function->getName() . '()' === (string) $function->getFqsen()) {
+            if ($namespace->getFqsen().'\\'.$function->getName().'()' === (string)$function->getFqsen()) {
                 $namespace->addFunction($function->getFqsen());
             }
         }
 
         foreach ($file->getConstants() as $constant) {
-            if ($namespace->getFqsen() . '::' . $constant->getName() === (string) $constant->getFqsen()) {
+            if ($namespace->getFqsen().'::'.$constant->getName() === (string)$constant->getFqsen()) {
                 $namespace->addConstant($constant->getFqsen());
             }
         }
 
         foreach ($file->getTraits() as $trait) {
-            if ($namespace->getFqsen() . '\\' . $trait->getName() === (string) $trait->getFqsen()) {
+            if ($namespace->getFqsen().'\\'.$trait->getName() === (string)$trait->getFqsen()) {
                 $namespace->addTrait($trait->getFqsen());
             }
         }

tests/component/ProjectNamespaceTest.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 use Mockery as m;
 use phpDocumentor\Reflection\File\LocalFile;
-use phpDocumentor\Reflection\Php\Factory\File;
 use phpDocumentor\Reflection\Php\ProjectFactory;
 use PHPUnit\Framework\TestCase;
 

src/phpDocumentor/Reflection/NodeVisitor/ElementNameResolver.php

Doc Comments +1 added lines

@@ -111,6 +111,7 @@
 
     /**
      * Resets the state of the object to an empty state.
+     * @param string $namespace
      */
     private function resetState(?string $namespace = null): void
     {

Spacing +5 added lines, -5 removed lines

@@ -77,7 +77,7 @@ 
     {
         switch (get_class($node)) {
             case Namespace_::class:
-                $this->resetState('\\' . $node->name . '\\');
+                $this->resetState('\\'.$node->name.'\\');
                 $node->fqsen = new Fqsen($this->buildName());
                 break;
             case Class_::class:
@@ -91,11 +91,11 @@ 
                 $node->fqsen = new Fqsen($this->buildName());
                 break;
             case Function_::class:
-                $this->parts->push($node->name . '()');
+                $this->parts->push($node->name.'()');
                 $node->fqsen = new Fqsen($this->buildName());
                 return NodeTraverser::DONT_TRAVERSE_CHILDREN;
             case ClassMethod::class:
-                $this->parts->push('::' . $node->name . '()');
+                $this->parts->push('::'.$node->name.'()');
                 $node->fqsen = new Fqsen($this->buildName());
                 return NodeTraverser::DONT_TRAVERSE_CHILDREN;
             case ClassConst::class:
@@ -106,7 +106,7 @@ 
                 $node->fqsen = new Fqsen($this->buildName());
                 break;
             case PropertyProperty::class:
-                $this->parts->push('::$' . $node->name);
+                $this->parts->push('::$'.$node->name);
                 $node->fqsen = new Fqsen($this->buildName());
                 break;
         }
@@ -133,6 +133,6 @@ 
             $name .= $part;
         }
 
-        return rtrim((string) $name, '\\');
+        return rtrim((string)$name, '\\');
     }
 }

src/phpDocumentor/Reflection/Middleware/ChainFactory.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
                 );
             }
 
-            $lastCallable = function ($command) use ($middleware, $lastCallable) {
+            $lastCallable = function($command) use ($middleware, $lastCallable) {
                 return $middleware->execute($command, $lastCallable);
             };
         }

src/phpDocumentor/Reflection/Php/Factory/Class_.php

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
         $classElement = new ClassElement(
             $object->fqsen,
             $docBlock,
-            $object->extends ? new Fqsen('\\' . $object->extends) : null,
+            $object->extends ? new Fqsen('\\'.$object->extends) : null,
             $object->isAbstract(),
             $object->isFinal(),
             new Location($object->getLine())
@@ -71,7 +71,7 @@ 
         if (isset($object->implements)) {
             foreach ($object->implements as $interfaceClassName) {
                 $classElement->addInterface(
-                    new Fqsen('\\' . $interfaceClassName->toString())
+                    new Fqsen('\\'.$interfaceClassName->toString())
                 );
             }
         }
@@ -81,7 +81,7 @@ 
                 switch (get_class($stmt)) {
                     case TraitUse::class:
                         foreach ($stmt->traits as $use) {
-                            $classElement->addUsedTrait(new Fqsen('\\' . $use->toString()));
+                            $classElement->addUsedTrait(new Fqsen('\\'.$use->toString()));
                         }
                         break;
                     case PropertyNode::class:

src/phpDocumentor/Reflection/Php/Factory/Interface_.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
         $docBlock = $this->createDocBlock($strategies, $object->getDocComment(), $context);
         $parents = [];
         foreach ($object->extends as $extend) {
-            $parents['\\' . (string) $extend] = new Fqsen('\\' . (string) $extend);
+            $parents['\\'.(string)$extend] = new Fqsen('\\'.(string)$extend);
         }
 
         $interface = new InterfaceElement($object->fqsen, $parents, $docBlock, new Location($object->getLine()));