juliushaertl / phpdoc-to-rst

src/GenerateDocumentationCommand.php

Spacing +1 added lines, -1 removed lines

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

src/Builder/TraitFileBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
         $this->addH1(self::escape($trait->getFqsen()));
 
         $namespace = str_replace('\\' . $trait->getName(), '', $trait->getFqsen());
-        if($namespace !== '') {
+        if ($namespace !== '') {
             $this->beginPhpDomain('namespace', substr($namespace, 1), false);
         }
 
@@ -69,7 +69,7 @@ 
         foreach ($trait->getMethods() as $method) {
             $docBlock = $method->getDocBlock();
             $params = [];
-            if($docBlock !== null) {
+            if ($docBlock !== null) {
                 /** @var Param $param */
                 foreach ($docBlock->getTagsByName('param') as $param) {
                     $params[$param->getVariableName()] = $param;
@@ -79,7 +79,7 @@ 
             /** @var Argument $argument */
             foreach ($method->getArguments() as $argument) {
                 // TODO: defaults, types
-                $args .=  ' $' . $argument->getName() . ', ';
+                $args .= ' $' . $argument->getName() . ', ';
             }
             $args = substr($args, 0, -2);
 
@@ -89,7 +89,7 @@ 
             $modifiers .= $method->isStatic() ? ' static' : '';
             $this->addLine('.. rst-class:: ' . $modifiers)->addLine();
             $this->indent();
-            $this->beginPhpDomain('method', $method->getName().'('.$args.')');
+            $this->beginPhpDomain('method', $method->getName() . '(' . $args . ')');
             $this->addDocBlockDescription($docBlock);
             if (!empty($params)) {
                 foreach ($method->getArguments() as $argument) {

Braces +3 added lines, -2 removed lines

@@ -95,8 +95,9 @@
                 foreach ($method->getArguments() as $argument) {
                     /** @var Param $param */
                     $param = $params[$argument->getName()];
-                    if ($param !== null)
-                        $this->addMultiline(':param ' . $param->getType() . ' $' . $argument->getName() . ': ' . $param->getDescription(), true);
+                    if ($param !== null) {
+                                            $this->addMultiline(':param ' . $param->getType() . ' $' . $argument->getName() . ': ' . $param->getDescription(), true);
+                    }
                 }
             }
             $this->endPhpDomain('method');

src/Builder/PhpDomainBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -55,7 +55,7 @@ 
         $docBlock = $constant->getDocBlock();
         if ($docBlock) {
             foreach ($docBlock->getTags() as $tag) {
-                $this->addDocblockTag( $tag->getName(), $docBlock);
+                $this->addDocblockTag($tag->getName(), $docBlock);
             }
         }
         $this->endPhpDomain();
@@ -97,9 +97,9 @@ 
      * @param $name string
      * @param $indent bool Should indent after the section started
      */
-    public function beginPhpDomain($type, $name, $indent=true) {
+    public function beginPhpDomain($type, $name, $indent = true) {
         // FIXME: Add checks if it is properly ended
-        $this->addLine('.. php:' . $type . ':: '. $name)->addLine();
+        $this->addLine('.. php:' . $type . ':: ' . $name)->addLine();
         if ($indent === true) {
             $this->indent();
         }
@@ -108,7 +108,7 @@ 
     /**
      * @param string $type
      */
-    public function endPhpDomain($type='') {
+    public function endPhpDomain($type = '') {
         $this->unindent();
         $this->addLine();
     }

Braces +18 added lines, -6 removed lines

@@ -134,38 +134,50 @@
         $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 '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);

src/Builder/ClassFileBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -42,8 +42,8 @@ 
 
         $this->addH1(self::escape($class->getName()));
 
-        $namespace = substr($class->getFqsen(), 0, strlen($class->getFqsen())-strlen('\\' . $class->getName()));
-        if($namespace !== '') {
+        $namespace = substr($class->getFqsen(), 0, strlen($class->getFqsen()) - strlen('\\' . $class->getName()));
+        if ($namespace !== '') {
             $this->beginPhpDomain('namespace', substr($namespace, 1), false);
         }
 
@@ -57,7 +57,7 @@ 
         $this->indent();
         $this->addDocBlockDescription($docBlock);
 
-        if($class instanceof Class_) {
+        if ($class instanceof Class_) {
             // Add class details
             $parent = $class->getParent();
             if ($parent !== null) {

Braces +3 added lines, -1 removed lines

@@ -132,7 +132,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');