mkusher / padawan.php

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/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;
         }
     }

src/Padawan/Framework/Application/Socket.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
             return;
         }
         $arrayForInput = ['command' => $request->command];
-        foreach($request->params as $key=>$value) {
-            $arrayForInput[$key] =  $value;
+        foreach ($request->params as $key=>$value) {
+            $arrayForInput[$key] = $value;
         }
         $input = new ArrayInput($arrayForInput);
         $command = $this->find($request->command);

src/Padawan/Parser/UseParser.php

Spacing +14 added lines, -14 removed lines

@@ -10,8 +10,8 @@ 
 class UseParser {
     /** @var Uses */
     private $uses;
-    public function parse(Use_ $node){
-        foreach($node->uses AS $use){
+    public function parse(Use_ $node) {
+        foreach ($node->uses AS $use) {
             $fqcn = $this->parseFQCN($use->name->toString());
             $this->uses->add($fqcn, $use->alias);
         }
@@ -21,44 +21,44 @@ 
     /**
      * @param string $type
      */
-    public function parseType($type){
+    public function parseType($type) {
         $pureFQCN = $this->parseFQCN($type);
-        if($pureFQCN->isScalar()){
+        if ($pureFQCN->isScalar()) {
             return $pureFQCN;
         }
-        if(strpos($type, '\\') === 0){
+        if (strpos($type, '\\') === 0) {
             return $pureFQCN;
         }
         $fqcn = $this->uses->find($type);
-        if(!empty($fqcn)){
+        if (!empty($fqcn)) {
             return $fqcn;
         }
         return $this->createFQCN($pureFQCN);
     }
-    public function getFQCN(Name $node = null){
-        if($node === null) {
+    public function getFQCN(Name $node = null) {
+        if ($node === null) {
             return $node;
         }
-        if($node->isFullyQualified()){
+        if ($node->isFullyQualified()) {
             return $this->parseFQCN($node->toString());
         }
         $fqcn = $this->uses->find($node->getFirst());
-        if($fqcn){
+        if ($fqcn) {
             return $fqcn;
         }
         return $this->createFQCN($node->toString());
     }
-    public function parseFQCN($fqcn){
+    public function parseFQCN($fqcn) {
         $fqcn = trim($fqcn, '\\');
-        if(empty($fqcn)){
+        if (empty($fqcn)) {
             return new FQCN('');
         }
         $parts = explode('\\', $fqcn);
         $name = array_pop($parts);
         $regex = '/(\w+)(\\[\\])?/';
         preg_match($regex, $name, $matches);
-        if(count($matches) === 0){
-            throw new \Exception("Could not parse FQCN for empty class name: " . $fqcn);
+        if (count($matches) === 0) {
+            throw new \Exception("Could not parse FQCN for empty class name: ".$fqcn);
         }
         $name = $matches[1];
         $isArray = count($matches) === 3 && $matches[2] = '[]';

src/Padawan/Plugin/Loader.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
                 explode("-", $part)
             ));
         }, $parts));
-        $className .= "\\" . $this->pluginClassName;
+        $className .= "\\".$this->pluginClassName;
         try {
             /** @var \Padawan\Plugin\PluginInterface */
             $plugin = $this->container->get($className);

src/Padawan/Framework/Generator/IndexGenerator.php

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);
         }
     }

Doc Comments +8 added lines

@@ -102,6 +102,10 @@ 
             );
         }
     }
+
+    /**
+     * @param string $content
+     */
     public function createScopeForFile(File $file, $content, Index $index, $rewrite = true)
     {
         $startParser = microtime(1);
@@ -114,6 +118,10 @@ 
         $this->getLogger()->info("Parsing: [$end]s");
         return $scope;
     }
+
+    /**
+     * @param string $hash
+     */
     public function processFileScope(File $file, Index $index, FileScope $scope = null, $hash = null)
     {
         if (empty($scope)) {

Unused Use Statements -3 removed lines

@@ -2,10 +2,7 @@
 
 namespace Padawan\Framework\Generator;
 
-use Padawan\Domain\Project\Node\ClassData;
-use Padawan\Domain\Project\Node\InterfaceData;
 use Padawan\Domain\Event\IndexGenerationEvent;
-use Padawan\Framework\Domain\Project\InMemoryIndex;
 use Padawan\Domain\Project;
 use Padawan\Domain\Project\File;
 use Padawan\Domain\Project\Index;