juliushaertl / phpdoc-to-rst

src/Extension/Extension.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     /** @var Project */
     protected $project;
 
-    public function __construct(Project &$project) {
+    public function __construct(Project&$project) {
         $this->project = $project;
     }
 

src/Builder/RstBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
 
     private $indentLevel = 0;
     /** @var string */
-    protected $content = '.. rst-class:: phpdoctorst' . PHP_EOL . PHP_EOL ;
+    protected $content = '.. rst-class:: phpdoctorst' . PHP_EOL . PHP_EOL;
 
     public function getContent() {
         return $this->content;
@@ -58,7 +58,7 @@ 
     }
 
     public function addFieldList($key, $value) {
-        $this->addLine(':'.self::escape($key).':');
+        $this->addLine(':' . self::escape($key) . ':');
         $this->indent()->addMultiline($value, true)->unindent();
         return $this;
     }

src/ApiDocBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -209,7 +209,7 @@
         $this->log('Build indexes.');
         $namespaces = $this->project->getNamespaces();
         $namespaces['\\'] = $this->project->getRootNamespace();
-        usort($namespaces, function (Namespace_ $a, Namespace_ $b) {
+        usort($namespaces, function(Namespace_ $a, Namespace_ $b) {
             return strcmp($a->getFqsen(), $b->getFqsen());
         });
         /** @var Namespace_ $namespace */

src/GenerateDocumentationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
             $apiDocBuilder->setVerboseOutput(true);
             $apiDocBuilder->setDebugOutput(true);
         }
-        if($input->getOption('public-only')) {
+        if ($input->getOption('public-only')) {
             $apiDocBuilder->addExtension(PublicOnlyExtension::class);
         }
         $apiDocBuilder->addExtension(AddFullElementNameExtension::class);

src/Builder/PhpDomainBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
      * @return mixed
      */
     public static function getNamespace(Element $element) {
-        return substr($element->getFqsen(), 0, strlen($element->getFqsen())-strlen('\\'. $element->getName()));
+        return substr($element->getFqsen(), 0, strlen($element->getFqsen()) - strlen('\\' . $element->getName()));
         //return str_replace('\\' . $element->getName(), '', $element->getFqsen());
     }
 

Braces +27 added lines, -10 removed lines

@@ -249,7 +249,9 @@ 
             foreach ($method->getArguments() as $argument) {
                 /** @var Param $param */
                 $param = $params[$argument->getName()];
-                if ($param !== null) $this->addMultiline(':param ' . self::escape($param->getType()) . ' $' . $argument->getName() . ': ' . $param->getDescription(), true);
+                if ($param !== null) {
+                    $this->addMultiline(':param ' . self::escape($param->getType()) . ' $' . $argument->getName() . ': ' . $param->getDescription(), true);
+                }
             }
         }
         $this->endPhpDomain('method');
@@ -312,44 +314,58 @@ 
         $tags = $docBlock->getTagsByName($tagName);
         switch ($tagName) {
             case 'return':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var Return_ $return */
                 $return = $tags[0];
                 $this->addMultiline(':Returns: ' . $return->getType() . ' ' . RstBuilder::escape($return->getDescription()), true);
                 break;
             case 'var':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var DocBlock\Tags\Var_ $return */
                 $return = $tags[0];
                 $this->addMultiline(':Type: ' . self::typesToRst($return->getType()) . ' ' . RstBuilder::escape($return->getDescription()), true);
                 break;
             case 'throws':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var Throws $tag */
                 foreach ($tags as $tag) {
                     $this->addMultiline(':Throws: ' . $tag->getType() . ' ' . RstBuilder::escape($tag->getDescription()), true);
                 }
                 break;
             case 'since':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var Since $return */
                 $return = $tags[0];
                 $this->addMultiline(':Since: ' . $return->getVersion() . ' ' . RstBuilder::escape($return->getDescription()), true);
                 break;
             case 'deprecated':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var Deprecated $return */
                 $return = $tags[0];
                 $this->addMultiline(':Deprecated: ' . $return->getVersion() . ' ' . RstBuilder::escape($return->getDescription()), true);
                 break;
             case 'see':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var See $return */
                 $return = $tags[0];
                 $this->addMultiline(':See: ' . $return->getReference() . ' ' . RstBuilder::escape($return->getDescription()), true);
                 break;
             case 'license':
-                if (count($tags) === 0) continue;
+                if (count($tags) === 0) {
+                    continue;
+                }
                 /** @var DocBlock\Tags\BaseTag $return */
                 $return = $tags[0];
                 $this->addMultiline(':License: ' . RstBuilder::escape($return->getDescription()), true);
@@ -379,8 +395,9 @@ 
                 $result .= $type . ' | ';
                 continue;
             }
-            if (0 === strpos($type, '\\'))
-                $type = substr($type, 1);
+            if (0 === strpos($type, '\\')) {
+                            $type = substr($type, 1);
+            }
             // we could use :any: here but resolve_any_xref is not implemented by sphinxcontrib.phpdomain
             // FIXME: once https://github.com/markstory/sphinxcontrib-phpdomain/pull/14 is merged
             // $result .= ':any:`' . RstBuilder::escape($type) . '` | ';

src/Extension/TocExtension.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
      */
     public function render($type, &$builder, $element) {
         if ($type === PhpDomainBuilder::SECTION_AFTER_DESCRIPTION) {
-            if($element instanceof Class_ || $element instanceof Interface_ || $element instanceof Trait_) {
+            if ($element instanceof Class_ || $element instanceof Interface_ || $element instanceof Trait_) {
                 $builder->addLine();
                 /** @var Interface_ $interface */
                 $interface = $builder->getElement();

src/Extension/AddFullElementNameExtension.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
             return;
         }
         if ($type === PhpDomainBuilder::SECTION_BEFORE_DESCRIPTION) {
-            if($element instanceof Class_) {
+            if ($element instanceof Class_) {
                 $modifiers = $element->isAbstract() ? 'abstract' : '';
                 $modifiers = $element->isFinal() ? ' final' : $modifiers;
                 $builder->addLine(':php:`' . $modifiers . ' class ' . RstBuilder::escape($builder->getElement()->getName()) . ' {}`');