juliushaertl / phpdoc-to-rst

src/Builder/PhpDomainBuilder.php

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) . '` | ';