mkusher / padawan.php

src/Padawan/Framework/Complete/Resolver/ContextResolver.php

Doc Comments +4 added lines

@@ -28,6 +28,10 @@
         $this->logger = $logger;
         $this->useParser = $useParser;
     }
+
+    /**
+     * @param string $badLine
+     */
     public function getContext($badLine, Index $index, Scope $scope = null)
     {
         if (empty($scope)) {

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
         if (strpos($badLine, '<?php') === false
             || strpos($badLine, '<?') === false
         ) {
-            $badLine = '<?php ' . $badLine;
+            $badLine = '<?php '.$badLine;
         }
         $badLine = str_replace(['elseif', 'else', 'catch'], '', $badLine);
         if ($wrapFunctionCall && $badLine[strlen($badLine) - 1] === '(') {

src/Padawan/Framework/Complete/Resolver/NodeTypeResolver.php

Doc Comments +1 added lines

@@ -145,6 +145,7 @@
     }
 
     /**
+     * @param \PhpParser\Node $node
      * @return Chain
      */
     protected function createChain($node)

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
         $chain = $this->createChain($node);
         $block = $chain;
         while ($block instanceof Chain) {
-            $this->logger->debug('looking for type of ' . $block->getName());
+            $this->logger->debug('looking for type of '.$block->getName());
             $event = new TypeResolveEvent($block, $type);
             $this->dispatcher->dispatch(self::BLOCK_START, $event);
             if ($block->getType() === 'var') {

src/Padawan/Framework/Generator/IndexGenerator.php

Doc Comments +4 added lines

@@ -93,6 +93,10 @@
         );
         $index->addParsedFile($file);
     }
+
+    /**
+     * @return null|FileScope
+     */
     public function createScopeForFile($file, Index $index, $createCache = true)
     {
         $startParser = microtime(1);

Spacing +4 added lines, -4 removed lines

@@ -66,7 +66,7 @@ 
             $end = microtime(1) - $start;
 
             $this->getLogger()->debug("Indexing: [$end]s");
-            $this->getLogger()->debug("Memory: " . memory_get_usage());
+            $this->getLogger()->debug("Memory: ".memory_get_usage());
             $globalTime += $end;
             ++$done;
             $process = floor($done / $all * 100);
@@ -119,17 +119,17 @@ 
         $this->getLogger()->debug(sprintf("Processing %s interfaces", count($scope->getInterfaces())));
         $this->getLogger()->debug(sprintf("Processing %s functions", count($scope->getFunctions())));
         foreach ($scope->getClasses() as $classData) {
-            $this->getLogger()->debug('Processing node ' . $classData->fqcn->toString());
+            $this->getLogger()->debug('Processing node '.$classData->fqcn->toString());
             $index->addFQCN($classData->fqcn);
             $index->addClass($classData);
         }
         foreach ($scope->getInterfaces() as $interfaceData) {
-            $this->getLogger()->debug('Processing node ' . $interfaceData->fqcn->toString());
+            $this->getLogger()->debug('Processing node '.$interfaceData->fqcn->toString());
             $index->addFQCN($interfaceData->fqcn);
             $index->addInterface($interfaceData);
         }
         foreach ($scope->getFunctions() as $functionData) {
-            $this->getLogger()->debug('Processing node ' . $functionData->name);
+            $this->getLogger()->debug('Processing node '.$functionData->name);
             $index->addFunction($functionData);
         }
     }

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace Padawan\Framework\Generator;
 
-use Padawan\Domain\Project\Node\ClassData;
-use Padawan\Domain\Project\Node\InterfaceData;
 use Padawan\Domain\Event\IndexGenerationEvent;
 use Padawan\Domain\Project\Index;
 use Padawan\Domain\Project;

src/Padawan/Parser/Walker/ScopeWalker.php

Doc Comments +4 added lines

@@ -88,6 +88,10 @@
         }
         return $line >= $startLine;
     }
+
+    /**
+     * @param Node $node
+     */
     public function getNodeLines($node)
     {
         $startLine = $endLine = -1;

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Padawan\Domain\Project\Node\Variable;
 use Padawan\Domain\Scope;
 use Padawan\Domain\Scope\FileScope;
-use Padawan\Domain\Scope\FunctionScope;
 use Padawan\Domain\Scope\MethodScope;
 use Padawan\Domain\Scope\ClassScope;
 use Padawan\Domain\Scope\ClosureScope;

src/Padawan/Parser/ClassParser.php

Braces +2 added lines, -4 removed lines

@@ -52,15 +52,13 @@
         foreach ($node->stmts AS $child) {
             if ($child instanceof ClassMethod) {
                 $classData->addMethod($this->parseMethod($child));
-            }
-            elseif ($child instanceof Property) {
+            } elseif ($child instanceof Property) {
                 foreach ($child->props AS $prop) {
                     $classData->addProp(
                         $this->parseProperty($prop, $child->type, $child->getAttribute('comments'))
                     );
                 }
-            }
-            elseif ($child instanceof ClassConst) {
+            } elseif ($child instanceof ClassConst) {
                 foreach ($child->consts AS $const) {
                     $classData->addConst($const->name);
                 }

src/Padawan/Parser/ErrorFreePhpParser.php

Braces +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
     public function parse($content) {
         try {
             return parent::parse($content);
-        }
-        catch (\Exception $e) {
+        } catch (\Exception $e) {
             return $this->semValue;
         }
     }

src/Padawan/Parser/ParamParser.php

Braces +3 added lines, -3 removed lines

@@ -12,9 +12,9 @@
     }
     public function parse(Param $node) {
         $param = new MethodParam($node->name);
-        if ($node->type instanceof Name)
-            $param->setFQCN($this->useParser->getFQCN($node->type));
-        else {
+        if ($node->type instanceof Name) {
+                    $param->setFQCN($this->useParser->getFQCN($node->type));
+        } else {
             $param->setType($node->type);
         }
         return $param;

src/Padawan/Parser/Parser.php

Braces +1 added lines, -2 removed lines

@@ -55,8 +55,7 @@
         if (!$createCache || $oldHash !== $hash || empty($ast)) {
             try {
                 $ast = $this->parser->parse($content);
-            }
-            catch (\Exception $e) {
+            } catch (\Exception $e) {
                 $this->logger->addError(sprintf("Parsing failed in file %s\n", $file));
                 $this->logger->error($e);
                 return null;

Doc Comments +8 added lines

@@ -35,6 +35,10 @@ 
         $content = $this->path->read($file);
         return $this->parseContent($file, $content, $uses, $createCache);
     }
+
+    /**
+     * @return \Padawan\Domain\Scope\FileScope
+     */
     public function parseContent($file, $content, Uses $uses = null, $createCache = true)
     {
         if ($createCache) {
@@ -110,6 +114,10 @@ 
         }
         return $nodes;
     }
+
+    /**
+     * @param \Padawan\Domain\Project\Index $index
+     */
     public function setIndex($index)
     {
         foreach ($this->walkers as $walker) {

src/Padawan/Parser/CommentParser.php

Braces +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
     public function parse($content) {
         try {
             return parent::parse($content);
-        }
-        catch (\Exception $e) {
+        } catch (\Exception $e) {
             return $this->semValue;
         }
     }