phpmetrics / PhpMetrics

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

Spacing +7 added lines, -7 removed lines

@@ -61,7 +61,7 @@ 
     /**
      * @inheritdoc
      */
-    public function terminate(ResultCollection $collection, ResultCollection $groupedResults){
+    public function terminate(ResultCollection $collection, ResultCollection $groupedResults) {
         \Twig_Autoloader::register();
         $loader = new \Twig_Loader_Filesystem(__DIR__.'/../../../../../templates/html');
         $twig = new \Twig_Environment($loader, array('cache' => false));
@@ -92,14 +92,14 @@ 
         $array = array();
 
         // map of classes an relations
-        foreach($collection as $item) {
+        foreach ($collection as $item) {
 
             // case of oop is disabled
-            if(!$item->getOOP()) {
+            if (!$item->getOOP()) {
                 continue;
             }
 
-            foreach($item->getOOP()->getClasses() as $class) {
+            foreach ($item->getOOP()->getClasses() as $class) {
                 $array[$class->getFullname()] = (object) array(
                     'name' => $class->getFullname()
                     , 'size' => 3000
@@ -112,9 +112,9 @@ 
         }
 
         // dependency can not be in the parsed sources (for example, native PHP classes)
-        foreach($array as $class => $infos) {
-            foreach($infos->relations as $relation) {
-                if(!isset($array[$relation])) {
+        foreach ($array as $class => $infos) {
+            foreach ($infos->relations as $relation) {
+                if (!isset($array[$relation])) {
                     $array[$relation] = (object) array('name' => $relation, 'relations' => array(), 'size' => 3000);
                 }
 //                array_push($array[$relation]->imports, $class);

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