phpmetrics / PhpMetrics

src/Hal/Component/Token/Dumper.php

Spacing +3 added lines, -3 removed lines

@@ -26,13 +26,13 @@
     public function dump(TokenCollection $tokens, $start = 0, $len = 0)
     {
         $str = '';
-        if($len == 0) {
+        if ($len == 0) {
             $len = sizeof($tokens);
         }
-        if($start < 0) {
+        if ($start < 0) {
             $start = 0;
         }
-        for($i = $start; $i < $len; $i++) {
+        for ($i = $start; $i < $len; $i++) {
             $str .= sprintf("\n%s : %s", token_name($tokens[$i]->getType()), $tokens[$i]->getValue());
         }
         return $str;

src/Hal/Component/OOP/Extractor/Extractor.php

Spacing +9 added lines, -9 removed lines

@@ -48,7 +48,7 @@ 
 
         $this->tokenizer = $tokenizer;
         $this->searcher = new Searcher();
-        $this->result= new Result;
+        $this->result = new Result;
 
         $this->extractors = (object) array(
             'class' => new ClassExtractor($this->searcher)
@@ -80,9 +80,9 @@ 
         $endAnonymous = 0;
         $mainContextClass = null; // class containing a anonymous class
 
-        for($n = 0; $n < $len; $n++) {
+        for ($n = 0; $n < $len; $n++) {
 
-            if($mainContextClass && $n > $endAnonymous) {
+            if ($mainContextClass && $n > $endAnonymous) {
                 // anonymous class is finished. We back to parent class
                 // methods will be added to the main class now
                 $class = $mainContextClass;
@@ -91,7 +91,7 @@ 
 
             $token = $tokens[$n];
 
-            switch($token->getType()) {
+            switch ($token->getType()) {
 
                 case T_USE:
                     $alias = $this->extractors->alias->extract($n, $tokens);
@@ -131,7 +131,7 @@ 
                 case T_CLASS:
                     $i = $n;
                     // avoid MyClass::class syntax
-                    if($this->searcher->isPrecededBy(T_DOUBLE_COLON, $i, $tokens, 1)) {
+                    if ($this->searcher->isPrecededBy(T_DOUBLE_COLON, $i, $tokens, 1)) {
                         continue;
                     }
 
@@ -142,14 +142,14 @@ 
                     $result->pushClass($c);
 
                     // PHP 7 and inner classes
-                    if($c instanceof ReflectedAnonymousClass) {
+                    if ($c instanceof ReflectedAnonymousClass) {
                         // avoid to consider anonymous class as main class
                         $p = $n;
                         $endAnonymous = $this->searcher->getPositionOfClosingBrace($p, $tokens);
                         $mainContextClass = $class;
 
                         // add anonymous class in method
-                        if($method) {
+                        if ($method) {
                             $method->pushAnonymousClass($c);
                         }
                     }
@@ -157,10 +157,10 @@ 
                     break;
 
                 case T_FUNCTION:
-                    if($class) {
+                    if ($class) {
                         // avoid closure
                         $next = $tokens[$n + 1];
-                        if(T_WHITESPACE != $next->getType()) {
+                        if (T_WHITESPACE != $next->getType()) {
                             continue;
                         }
                         $method = $this->extractors->method->extract($n, $tokens, $class);

src/Hal/Application/Command/Job/QueueAnalyzeFactory.php

Unused Use Statements -4 removed lines

@@ -9,10 +9,6 @@
 
 namespace Hal\Application\Command\Job;
 use Hal\Application\Extension\ExtensionService;
-use Hal\Application\Formater\Chart;
-use Hal\Application\Formater\Details;
-use Hal\Application\Formater\Summary;
-use Hal\Application\Formater\Violations;
 use Hal\Application\Score\Scoring;
 use Hal\Component\Aggregator\DirectoryAggregatorFlat;
 use Hal\Component\Bounds\BoundsInterface;

src/Hal/Application/Command/Job/QueueReportFactory.php

Unused Use Statements -3 removed lines

@@ -10,13 +10,10 @@
 namespace Hal\Application\Command\Job;
 use Hal\Application\Extension\ExtensionService;
 use Hal\Application\Extension\extensionsService;
-use Hal\Application\Extension\Repository;
 use Hal\Application\Formater\Chart;
 use Hal\Application\Formater\Details;
 use Hal\Application\Formater\Summary;
 use Hal\Application\Formater\Violations;
-use Hal\Application\Score\Scoring;
-use Hal\Component\Aggregator\DirectoryAggregatorFlat;
 use Hal\Component\Bounds\BoundsInterface;
 use Hal\Component\Config\ConfigurationInterface;
 use Hal\Component\File\Finder;

src/Hal/Application/Config/ConfigFactory.php

Spacing +13 added lines, -13 removed lines

@@ -37,7 +37,7 @@ 
         $locator = new ConfigLocator();
         $filename = $locator->locate($input->getOption('config'));
 
-        if(null !== $filename) {
+        if (null !== $filename) {
             $loader = new Loader($hydrator);
             $config = $loader->load($filename);
         } else {
@@ -46,21 +46,21 @@ 
 
         
         // then, overwrite configuration by arguments provided in run
-        strlen($input->getArgument('path')) > 0         && $config->getPath()->setBasePath($input->getArgument('path'));
-        strlen($input->getOption('extensions')) > 0     && $config->getPath()->setExtensions($input->getOption('extensions'));
-        strlen($input->getOption('excluded-dirs')) > 0  && $config->getPath()->setExcludedDirs($input->getOption('excluded-dirs'));
-        strlen($input->getOption('symlinks')) > 0       && $config->getPath()->setFollowSymlinks(true);
-        strlen($input->getOption('report-xml')) > 0     && $config->getLogging()->setReport('xml', $input->getOption('report-xml'));
-        strlen($input->getOption('report-cli')) > 0     && $config->getLogging()->setReport('cli', $input->getOption('report-cli'));
-        strlen($input->getOption('report-json')) > 0    && $config->getLogging()->setReport('json', $input->getOption('report-json'));
-        strlen($input->getOption('report-html')) > 0    && $config->getLogging()->setReport('html', $input->getOption('report-html'));
-        strlen($input->getOption('report-csv')) > 0     && $config->getLogging()->setReport('csv', $input->getOption('report-csv'));
+        strlen($input->getArgument('path')) > 0 && $config->getPath()->setBasePath($input->getArgument('path'));
+        strlen($input->getOption('extensions')) > 0 && $config->getPath()->setExtensions($input->getOption('extensions'));
+        strlen($input->getOption('excluded-dirs')) > 0 && $config->getPath()->setExcludedDirs($input->getOption('excluded-dirs'));
+        strlen($input->getOption('symlinks')) > 0 && $config->getPath()->setFollowSymlinks(true);
+        strlen($input->getOption('report-xml')) > 0 && $config->getLogging()->setReport('xml', $input->getOption('report-xml'));
+        strlen($input->getOption('report-cli')) > 0 && $config->getLogging()->setReport('cli', $input->getOption('report-cli'));
+        strlen($input->getOption('report-json')) > 0 && $config->getLogging()->setReport('json', $input->getOption('report-json'));
+        strlen($input->getOption('report-html')) > 0 && $config->getLogging()->setReport('html', $input->getOption('report-html'));
+        strlen($input->getOption('report-csv')) > 0 && $config->getLogging()->setReport('csv', $input->getOption('report-csv'));
         strlen($input->getOption('violations-xml')) > 0 && $config->getLogging()->setViolation('xml', $input->getOption('violations-xml'));
-        strlen($input->getOption('chart-bubbles')) > 0  && $config->getLogging()->setChart('bubbles', $input->getOption('chart-bubbles'));
-        strlen($input->getOption('failure-condition')) > 0  && $config->setFailureCondition($input->getOption('failure-condition'));
+        strlen($input->getOption('chart-bubbles')) > 0 && $config->getLogging()->setChart('bubbles', $input->getOption('chart-bubbles'));
+        strlen($input->getOption('failure-condition')) > 0 && $config->setFailureCondition($input->getOption('failure-condition'));
         strlen($input->getOption('template-title')) > 0 && $config->getTemplate()->setTitle($input->getOption('template-title'));
         strlen($input->getOption('offline')) > 0 && $config->getTemplate()->setOffline($input->getOption('offline'));
-        strlen($input->getOption('plugins')) > 0 && $config->getExtensions()->setExtensions(explode(',',  $input->getOption('plugins')));
+        strlen($input->getOption('plugins')) > 0 && $config->getExtensions()->setExtensions(explode(',', $input->getOption('plugins')));
         strlen($input->getOption('ignore-errors')) > 0 && $config->setIgnoreErrors(true);
 
         return $config;

src/Hal/Application/Extension/ExtensionService.php

Indentation +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     /**
      * @var Repository
      */
-   private $repository;
+    private $repository;
 
     /**
      * ExtensionService constructor.

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function receive(Configuration $config, ResultCollection $collection, ResultCollection $aggregatedResults, Bounds $bounds)
     {
         // search controller
-        foreach($this->repository->all() as $item) {
+        foreach ($this->repository->all() as $item) {
             $item->receive($config, $collection, $aggregatedResults, $bounds);
         }
     }

src/Hal/Application/Extension/Repository.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
 namespace Hal\Application\Extension;
 
-class Repository{
+class Repository {
 
     /**
      * @var array

src/Hal/Application/Formater/Twig/FormatingExtension.php

Spacing +8 added lines, -8 removed lines

@@ -52,7 +52,7 @@ 
      */
     public function textify($v)
     {
-        return ucfirst(preg_replace( '/([a-z0-9])([A-Z])/', "$1 $2", $v ));
+        return ucfirst(preg_replace('/([a-z0-9])([A-Z])/', "$1 $2", $v));
     }
 
     /**
@@ -74,12 +74,12 @@ 
     public function extensionsMenu(ExtensionService $extensions)
     {
         $html = '';
-        foreach($extensions->getRepository()->all() as $extension) {
+        foreach ($extensions->getRepository()->all() as $extension) {
             $helper = $extension->getReporterHtmlSummary();
-            if(!$helper) {
+            if (!$helper) {
                 continue;
             }
-            foreach($helper->getMenus() as $name => $label) {
+            foreach ($helper->getMenus() as $name => $label) {
                 $html .= sprintf('<li id="link-%s"><a>%s</a></li>', $name, $label);
             }
         }
@@ -93,9 +93,9 @@ 
     public function extensionsJs(ExtensionService $extensions)
     {
         $html = '';
-        foreach($extensions->getRepository()->all() as $extension) {
+        foreach ($extensions->getRepository()->all() as $extension) {
             $helper = $extension->getReporterHtmlSummary();
-            if(!$helper) {
+            if (!$helper) {
                 continue;
             }
             $html .= $helper->renderJs();
@@ -110,9 +110,9 @@ 
     public function extensionsContent(ExtensionService $extensions)
     {
         $html = '';
-        foreach($extensions->getRepository()->all() as $extension) {
+        foreach ($extensions->getRepository()->all() as $extension) {
             $helper = $extension->getReporterHtmlSummary();
-            if(!$helper) {
+            if (!$helper) {
                 continue;
             }
             $html .= $helper->renderHtml();

src/Hal/Application/Formater/Summary/Cli.php

Spacing +4 added lines, -4 removed lines

@@ -68,7 +68,7 @@ 
     /**
      * @inheritdoc
      */
-    public function terminate(ResultCollection $collection, ResultCollection $groupedResults){
+    public function terminate(ResultCollection $collection, ResultCollection $groupedResults) {
 
         $this->output->write(str_pad("\x0D", 80, "\x20"));
         $this->output->writeln('');
@@ -76,14 +76,14 @@ 
         // score
         $score = $collection->getScore();
         foreach ($score->all() as $name => $value) {
-            $this->output->writeln(sprintf('%s %s', str_pad($name, 35, '.'),  str_pad($value, 5, ' ', STR_PAD_LEFT). ' / ' . Scoring::MAX));
+            $this->output->writeln(sprintf('%s %s', str_pad($name, 35, '.'), str_pad($value, 5, ' ', STR_PAD_LEFT).' / '.Scoring::MAX));
         }
         $this->output->writeln('');
 
         // extensions
-        foreach($this->extensionsService->getRepository()->all() as $plugin) {
+        foreach ($this->extensionsService->getRepository()->all() as $plugin) {
             $helper = $plugin->getReporterCliSummary();
-            if(!$helper) {
+            if (!$helper) {
                 continue;
             }
             $this->output->write($helper->render());