phpmetrics / PhpMetrics

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

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
     /**
      * @inheritdoc
      */
-    public function terminate(ResultCollection $collection, ResultCollection $groupedResults){
+    public function terminate(ResultCollection $collection, ResultCollection $groupedResults) {
 
         $bounds = $this->bound->calculate($collection);
 
@@ -63,7 +63,7 @@ 
 
         // modules
         $modules = $xml->createElement('modules');
-        foreach($groupedResults as $result) {
+        foreach ($groupedResults as $result) {
             $module = $xml->createElement('module');
             $this->injectsBounds($module, $result->getBounds());
             $module->setAttribute('namespace', $result->getName());
@@ -98,7 +98,7 @@ 
         $node->setAttribute('length', $bound->getAverage('length'));
 
         $hasOOP = null !== $bound->getSum('instability');
-        if($hasOOP) {
+        if ($hasOOP) {
             $node->setAttribute('lcom', $bound->getAverage('lcom'));
             $node->setAttribute('instability', $bound->getAverage('instability'));
             $node->setAttribute('efferentCoupling', $bound->getAverage('efferentCoupling'));

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

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,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));
     }
 
     /**

src/Hal/Application/Formater/Violations/Xml.php

Spacing +6 added lines, -6 removed lines

@@ -50,7 +50,7 @@ 
     /**
      * @inheritdoc
      */
-    public function terminate(ResultCollection $collection, ResultCollection $groupedResults){
+    public function terminate(ResultCollection $collection, ResultCollection $groupedResults) {
 
 
         // root
@@ -61,18 +61,18 @@ 
         $root->setAttribute('timestamp', date('c'));
 
         // violations
-        foreach($collection as $item) {
+        foreach ($collection as $item) {
             $file = $xml->createElement('file');
             $file->setAttribute('name', realpath($item->getFilename()));
 
             $array = $item->asArray();
             $hasViolation = false;
-            foreach($array as $key => $value) {
+            foreach ($array as $key => $value) {
                 $result = $this->validator->validate($key, $value);
-                if(Validator::GOOD !== $result && Validator::UNKNOWN !== $result) {
+                if (Validator::GOOD !== $result && Validator::UNKNOWN !== $result) {
                     $hasViolation = true;
                     $violation = $xml->createElement('violation');
-                    $violation->setAttribute('beginline' , 1);
+                    $violation->setAttribute('beginline', 1);
                     $violation->setAttribute('endline', $array['loc']);
                     $violation->setAttribute('rule', $key);
                     $violation->setAttribute('ruleset', $key);
@@ -89,7 +89,7 @@ 
                 }
             }
 
-            if($hasViolation) {
+            if ($hasViolation) {
                 $root->appendChild($file);
             }
         }

src/Hal/Application/Rule/RuleSet.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      */
     public function __construct(array $rules = array())
     {
-        if(!$rules) {
+        if (!$rules) {
             $rules = array(
                 'cyclomaticComplexity' => array(10, 6, 2)
                 , 'maintainabilityIndex' => array(0, 69, 85)

src/Hal/Application/Rule/Validator.php

Spacing +4 added lines, -4 removed lines

@@ -62,14 +62,14 @@ 
     public function validate($key, $value) {
         $rule = $this->ruleSet->getRule($key);
 
-        if(!is_array($rule) || !is_numeric($value)) {
+        if (!is_array($rule) || !is_numeric($value)) {
             return self::UNKNOWN;
         }
 
         // according order
-        if($rule[0] < $rule[2]) {
+        if ($rule[0] < $rule[2]) {
             // critical < warn < good
-            switch(true) {
+            switch (true) {
                 case $value < $rule[1]:
                     return self::CRITICAL;
                 case $value >= $rule[1] && $value < $rule[2]:
@@ -79,7 +79,7 @@ 
             }
         }
         // critical > warn > good
-        switch(true) {
+        switch (true) {
             case $value > $rule[1]:
                 return self::CRITICAL;
             case $value < $rule[2]:

src/Hal/Application/Score/Calculator.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 
         $score = (($note - $bad) / ($good - $bad)) * $this->limit;
         $score = max(0, $score);
-        $score = min ($this->limit, $score);
+        $score = min($this->limit, $score);
         return round($score, 2);
     }
 
@@ -45,7 +45,7 @@ 
 
         $score = $this->limit - ($note - $good) / ($bad - $good) * $this->limit;
         $score = max(0, $score);
-        $score = min ($this->limit, $score);
+        $score = min($this->limit, $score);
         return round($score, 2);
     }
 }
\ No newline at end of file

src/Hal/Application/Score/Result.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  *
  * @author Jean-François Lépine <https://twitter.com/Halleck45>
  */
-class Result implements ScoreInterface{
+class Result implements ScoreInterface {
 
     /**
      * @var array

src/Hal/Application/Score/Scoring.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
  *
  * @author Jean-François Lépine <https://twitter.com/Halleck45>
  */
-class Scoring implements ScoringInterface{
+class Scoring implements ScoringInterface {
 
     /**
      * Maximal score
@@ -66,7 +66,7 @@ 
 
         // score
         $result = new Result;
-        foreach($factors as $qualityFactor) {
+        foreach ($factors as $qualityFactor) {
             $score = $qualityFactor->calculate($collection, $groupedResults, $bound);
             $result->push($qualityFactor->getName(), $score);
         }

src/Hal/Component/Aggregator/DirectoryAggregatorFlat.php

Spacing +4 added lines, -4 removed lines

@@ -40,7 +40,7 @@ 
      */
     public function aggregates(ResultCollection $results) {
         $array = array();
-        foreach($results as $result) {
+        foreach ($results as $result) {
             $basename = dirname($result->getFilename());
 
             // from 'folder1/folder2/file.php', we want an array with ('folder1', 'folder1/folder2')
@@ -49,14 +49,14 @@ 
                 return $v;
             }, array());
 
-            if($this->depth) {
+            if ($this->depth) {
                 array_splice($namespaces, $this->depth);
             }
 
             // merge infos for each namespace in the DirectoryResultCollection
-            foreach($namespaces as $namespace) {
+            foreach ($namespaces as $namespace) {
 
-                if(!isset($array[$namespace])) {
+                if (!isset($array[$namespace])) {
                     $array[$namespace] = new ResultCollection();
                 }