phpmetrics / PhpMetrics

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

Spacing +5 added lines, -5 removed lines

@@ -92,7 +92,7 @@ 
 
         $files = $this->finder->find($this->path);
 
-        if(0 == sizeof($files, COUNT_NORMAL)) {
+        if (0 == sizeof($files, COUNT_NORMAL)) {
             throw new \LogicException('No file found');
         }
 
@@ -116,12 +116,12 @@ 
             , $classMap
         );
 
-        foreach($files as $k => $filename) {
+        foreach ($files as $k => $filename) {
 
             $progress->advance();
 
             // Integrity
-            if(!$this->ignoreErrors && !$syntaxChecker->isCorrect($filename)) {
+            if (!$this->ignoreErrors && !$syntaxChecker->isCorrect($filename)) {
                 $this->output->writeln(sprintf('<error>file %s is not valid and has been skipped</error>', $filename));
                 unset($files[$k]);
                 continue;
@@ -130,7 +130,7 @@ 
             // Analyze
             try {
                 $resultSet = $fileAnalyzer->execute($filename);
-            } catch(NoTokenizableException $e) {
+            } catch (NoTokenizableException $e) {
                 $this->output->writeln(sprintf("<error>file %s has been skipped: \n%s</error>", $filename, $e->getMessage()));
                 unset($files[$k]);
                 continue;
@@ -151,7 +151,7 @@ 
         $progress->finish();
 
 
-        if($this->withOOP) {
+        if ($this->withOOP) {
             // COUPLING (should be done after parsing files)
             $this->output->write(str_pad("\x0DAnalyzing coupling. This will take few minutes...", 80, "\x20"));
             $couplingAnalyzer = new CouplingAnalyzer($classMap, $collection);

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

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        foreach($this->jobs as $job) {
+        foreach ($this->jobs as $job) {
             $job->execute($collection, $aggregatedResults);
         }
     }

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

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        if(!$this->enabled) {
+        if (!$this->enabled) {
             return;
         }
 

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

Spacing +2 added lines, -2 removed lines

@@ -61,12 +61,12 @@
      * @inheritdoc
      */
     public function execute(ResultCollection $collection, ResultCollection $aggregatedResults) {
-        if(!$this->destination) {
+        if (!$this->destination) {
             return;
         }
 
         $dir = dirname($this->destination);
-        if(!file_exists($dir)) {
+        if (!file_exists($dir)) {
             mkdir($dir, 0777, true);
         }
         $this->output->writeln(sprintf('Generating %s Report...', $this->formater->getName()));

src/Hal/Application/Command/RunMetricsCommand.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
                     'path', InputArgument::OPTIONAL, 'Path to explore'
                 )
                 ->addOption(
-                    'report-html',null, InputOption::VALUE_REQUIRED, 'Path to save report in HTML format. Example: /tmp/report.html'
+                    'report-html', null, InputOption::VALUE_REQUIRED, 'Path to save report in HTML format. Example: /tmp/report.html'
                 )
                 ->addOption(
                     'report-xml', null, InputOption::VALUE_REQUIRED, 'Path to save summary report in XML format. Example: /tmp/report.xml'
@@ -102,7 +102,7 @@ 
         $config = $configFactory->factory($input);
 
         // files
-        if(null === $config->getPath()->getBasePath()) {
+        if (null === $config->getPath()->getBasePath()) {
             throw new \LogicException('Please provide a path to analyze');
         }
 
@@ -129,7 +129,7 @@ 
 
         // evaluation of success
         $rule = $config->getFailureCondition();
-        if(null !== $rule) {
+        if (null !== $rule) {
             $evaluator = new Evaluator($collection, $aggregatedResults, $bounds);
             $result = $evaluator->evaluate($rule);
             // fail if failure-condition is realized

src/Hal/Application/Config/ConfigFactory.php

Spacing +12 added lines, -12 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,18 +46,18 @@ 
 
         
         // 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('ignore-errors')) > 0 && $config->setIgnoreErrors(true);
 

src/Hal/Application/Config/ConfigLocator.php

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
      * @param array $defaults
      */
     public function __construct(array $defaults = null) {
-        if(is_null($defaults)) {
+        if (is_null($defaults)) {
             $defaults = array('.phpmetrics.yml', '.phpmetrics.yml.dist', '.phpmetrics-dist.yml');
         }
         $this->defaults = $defaults;
@@ -42,9 +42,9 @@ 
      */
     public function locate($filename) {
 
-        if(null === $filename) {
+        if (null === $filename) {
             // try to use default configfile : .phpmetrics.yml or .phpmetrics.yml.dist
-            foreach($this->defaults as $filenameToCheck) {
+            foreach ($this->defaults as $filenameToCheck) {
                 $filenameToCheck = getcwd().DIRECTORY_SEPARATOR.$filenameToCheck;
                 if (true === $this->isFileAccessible($filenameToCheck)) {
                     return $filenameToCheck;
@@ -53,7 +53,7 @@ 
             return null;
         }
 
-        if(false === $this->isFileAccessible($filename)) {
+        if (false === $this->isFileAccessible($filename)) {
             throw new \RuntimeException('configuration file is not accessible');
         }
 

src/Hal/Application/Console/PhpMetricsApplication.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
     {
         $available = ['metrics', 'self-update'];
         $arg = $input->getFirstArgument();
-        if(!in_array($arg, $available) ||'metrics' === $arg) {
+        if (!in_array($arg, $available) || 'metrics' === $arg) {
             // default argument : we don't want to provide the name of the command by default
             $inputDefinition = $this->getDefinition();
             $inputDefinition->setArguments();

src/Hal/Application/Formater/Chart/Bubbles.php

Spacing +6 added lines, -6 removed lines

@@ -51,19 +51,19 @@ 
     /**
      * @inheritdoc
      */
-    public function terminate(ResultCollection $collection, ResultCollection $groupedResults){
+    public function terminate(ResultCollection $collection, ResultCollection $groupedResults) {
 
         $text = 'digraph G {'
-            . PHP_EOL .'bgcolor=white;'
-            . PHP_EOL .'node [shape=circle, color=lightblue2, style=filled];';
+            . PHP_EOL.'bgcolor=white;'
+            . PHP_EOL.'node [shape=circle, color=lightblue2, style=filled];';
 
         $width = 300;
 
-        foreach($collection as $item) {
+        foreach ($collection as $item) {
 
             // color
             $valid = $this->validator->validate('maintainabilityIndex', $item->getMaintainabilityIndex()->getMaintainabilityIndex());
-            switch($valid) {
+            switch ($valid) {
                 case Validator::CRITICAL:   $color = 'red'; break;
                 case Validator::GOOD:       $color = 'chartreuse4'; break;
                 case Validator::WARNING:    $color = 'gold1'; break;
@@ -73,7 +73,7 @@ 
             // size
             $size = round($item->getMcCabe()->getCyclomaticComplexityNumber() * $width / 100);
 
-            $text .= PHP_EOL. sprintf('"%1$s" [color=%2$s, tooltip="%3$s", width=%4$s, height=%4$s, label=""];'
+            $text .= PHP_EOL.sprintf('"%1$s" [color=%2$s, tooltip="%3$s", width=%4$s, height=%4$s, label=""];'
                 , $item->getName()
                 , $color
                 , $item->getName()