@@ -44,10 +44,10 @@ discard block |
||
44 | 44 | { |
45 | 45 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) { |
46 | 46 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) { |
47 | - $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node)); |
|
47 | + $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node)); |
|
48 | 48 | $classOrFunction = $this->metrics->get($name); |
49 | 49 | } else { |
50 | - $classOrFunction = new FunctionMetric((string)$node->name); |
|
50 | + $classOrFunction = new FunctionMetric((string) $node->name); |
|
51 | 51 | $this->metrics->attach($classOrFunction); |
52 | 52 | } |
53 | 53 | |
@@ -55,7 +55,7 @@ discard block |
||
55 | 55 | $operands = []; |
56 | 56 | $operators = []; |
57 | 57 | |
58 | - iterate_over_node($node, function ($node) use (&$operators, &$operands) { |
|
58 | + iterate_over_node($node, function($node) use (&$operators, &$operands) { |
|
59 | 59 | if ($node instanceof Node\Expr\BinaryOp |
60 | 60 | || $node instanceof Node\Expr\AssignOp |
61 | 61 | || $node instanceof Stmt\If_ |
@@ -34,10 +34,10 @@ discard block |
||
34 | 34 | { |
35 | 35 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Function_ || $node instanceof Stmt\Trait_) { |
36 | 36 | if ($node instanceof Stmt\Class_ || $node instanceof Stmt\Trait_) { |
37 | - $name = (string)(isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node)); |
|
37 | + $name = (string) (isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node)); |
|
38 | 38 | $classOrFunction = $this->metrics->get($name); |
39 | 39 | } else { |
40 | - $classOrFunction = new FunctionMetric((string)$node->name); |
|
40 | + $classOrFunction = new FunctionMetric((string) $node->name); |
|
41 | 41 | $this->metrics->attach($classOrFunction); |
42 | 42 | } |
43 | 43 | |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | $code = preg_replace('!/\*.*?\*/!s', '', $code); |
58 | 58 | |
59 | 59 | // count and remove single line comments |
60 | - $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function (array $matches) use (&$cloc) { |
|
60 | + $code = preg_replace_callback('!(\'[^\']*\'|"[^"]*")|((?:#|\/\/).*$)!m', function(array $matches) use (&$cloc) { |
|
61 | 61 | if (isset($matches[2])) { |
62 | 62 | $cloc += 1; |
63 | 63 | } |
@@ -42,12 +42,12 @@ discard block |
||
42 | 42 | |
43 | 43 | foreach ($node->stmts as $stmt) { |
44 | 44 | if ($stmt instanceof Stmt\ClassMethod) { |
45 | - if (!$graph->has($stmt->name . '()')) { |
|
46 | - $graph->insert(new TreeNode($stmt->name . '()')); |
|
45 | + if (!$graph->has($stmt->name.'()')) { |
|
46 | + $graph->insert(new TreeNode($stmt->name.'()')); |
|
47 | 47 | } |
48 | - $from = $graph->get($stmt->name . '()'); |
|
48 | + $from = $graph->get($stmt->name.'()'); |
|
49 | 49 | |
50 | - \iterate_over_node($stmt, function ($node) use ($from, &$graph) { |
|
50 | + \iterate_over_node($stmt, function($node) use ($from, &$graph) { |
|
51 | 51 | if ($node instanceof Node\Expr\PropertyFetch && isset($node->var->name) && $node->var->name == 'this') { |
52 | 52 | $name = getNameOfNode($node); |
53 | 53 | // use of attribute $this->xxx; |
@@ -63,7 +63,7 @@ discard block |
||
63 | 63 | if (!$node->var instanceof Node\Expr\New_ && isset($node->var->name) && getNameOfNode($node->var) === 'this') { |
64 | 64 | // use of method call $this->xxx(); |
65 | 65 | // use of attribute $this->xxx; |
66 | - $name = getNameOfNode($node->name) . '()'; |
|
66 | + $name = getNameOfNode($node->name).'()'; |
|
67 | 67 | if (!$graph->has($name)) { |
68 | 68 | $graph->insert(new TreeNode($name)); |
69 | 69 | } |
@@ -27,7 +27,7 @@ discard block |
||
27 | 27 | public function enterNode(Node $node) |
28 | 28 | { |
29 | 29 | if ($node instanceof Namespace_) { |
30 | - $this->namespace = (string)$node->name; |
|
30 | + $this->namespace = (string) $node->name; |
|
31 | 31 | } |
32 | 32 | } |
33 | 33 | |
@@ -42,17 +42,17 @@ discard block |
||
42 | 42 | $package = $matches[1]; |
43 | 43 | } |
44 | 44 | if (preg_match('/^\s*\* @subpackage (.*)/m', $docBlockText, $matches)) { |
45 | - $package = $package . '\\' . $matches[1]; |
|
45 | + $package = $package.'\\'.$matches[1]; |
|
46 | 46 | } |
47 | 47 | |
48 | - $packageName = $package . '\\'; |
|
49 | - if (! $packageMetric = $this->metrics->get($packageName)) { |
|
48 | + $packageName = $package.'\\'; |
|
49 | + if (!$packageMetric = $this->metrics->get($packageName)) { |
|
50 | 50 | $packageMetric = new PackageMetric($packageName); |
51 | 51 | $this->metrics->attach($packageMetric); |
52 | 52 | } |
53 | 53 | /* @var PackageMetric $packageMetric */ |
54 | - $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@' . spl_object_hash($node); |
|
55 | - $elementName = (string)$elementName; |
|
54 | + $elementName = isset($node->namespacedName) ? $node->namespacedName : 'anonymous@'.spl_object_hash($node); |
|
55 | + $elementName = (string) $elementName; |
|
56 | 56 | $packageMetric->addClass($elementName); |
57 | 57 | |
58 | 58 | $this->metrics->get($elementName)->set('package', $packageName); |
@@ -18,7 +18,7 @@ discard block |
||
18 | 18 | public function addClass($name) |
19 | 19 | { |
20 | 20 | $elements = $this->get('classes'); |
21 | - $elements[] = (string)$name; |
|
21 | + $elements[] = (string) $name; |
|
22 | 22 | $this->set('classes', $elements); |
23 | 23 | } |
24 | 24 | |
@@ -26,7 +26,7 @@ discard block |
||
26 | 26 | public function setAbstraction($abstraction) |
27 | 27 | { |
28 | 28 | if ($abstraction !== null) { |
29 | - $abstraction = (float)$abstraction; |
|
29 | + $abstraction = (float) $abstraction; |
|
30 | 30 | } |
31 | 31 | $this->set('abstraction', $abstraction); |
32 | 32 | } |
@@ -41,7 +41,7 @@ discard block |
||
41 | 41 | public function setInstability($instability) |
42 | 42 | { |
43 | 43 | if ($instability !== null) { |
44 | - $instability = (float)$instability; |
|
44 | + $instability = (float) $instability; |
|
45 | 45 | } |
46 | 46 | $this->set('instability', $instability); |
47 | 47 | } |
@@ -63,11 +63,11 @@ discard block |
||
63 | 63 | } |
64 | 64 | $classDependencies = $this->getOutgoingClassDependencies(); |
65 | 65 | $packageDependencies = $this->getOutgoingPackageDependencies(); |
66 | - if (! in_array($className, $classDependencies)) { |
|
66 | + if (!in_array($className, $classDependencies)) { |
|
67 | 67 | $classDependencies[] = $className; |
68 | 68 | $this->set('outgoing_class_dependencies', $classDependencies); |
69 | 69 | } |
70 | - if (! in_array($packageName, $packageDependencies)) { |
|
70 | + if (!in_array($packageName, $packageDependencies)) { |
|
71 | 71 | $packageDependencies[] = $packageName; |
72 | 72 | $this->set('outgoing_package_dependencies', $packageDependencies); |
73 | 73 | } |
@@ -96,11 +96,11 @@ discard block |
||
96 | 96 | } |
97 | 97 | $classDependencies = $this->getIncomingClassDependencies(); |
98 | 98 | $packageDependencies = $this->getIncomingPackageDependencies(); |
99 | - if (! in_array($className, $classDependencies)) { |
|
99 | + if (!in_array($className, $classDependencies)) { |
|
100 | 100 | $classDependencies[] = $className; |
101 | 101 | $this->set('incoming_class_dependencies', $classDependencies); |
102 | 102 | } |
103 | - if (! in_array($packageName, $packageDependencies)) { |
|
103 | + if (!in_array($packageName, $packageDependencies)) { |
|
104 | 104 | $packageDependencies[] = $packageName; |
105 | 105 | $this->set('incoming_package_dependencies', $packageDependencies); |
106 | 106 | } |
@@ -9,9 +9,9 @@ |
||
9 | 9 | namespace Hal\Component\Ast; |
10 | 10 | |
11 | 11 | if (PHP_VERSION_ID >= 70000) { |
12 | - class_alias(Php7NodeTraverser::class, __NAMESPACE__ . '\\ActualNodeTraverser'); |
|
12 | + class_alias(Php7NodeTraverser::class, __NAMESPACE__.'\\ActualNodeTraverser'); |
|
13 | 13 | } else { |
14 | - class_alias(Php5NodeTraverser::class, __NAMESPACE__ . '\\ActualNodeTraverser'); |
|
14 | + class_alias(Php5NodeTraverser::class, __NAMESPACE__.'\\ActualNodeTraverser'); |
|
15 | 15 | } |
16 | 16 | |
17 | 17 | /** |
@@ -48,7 +48,7 @@ discard block |
||
48 | 48 | |
49 | 49 | $logDir = dirname($logFile); |
50 | 50 | if (!file_exists($logDir) || !is_writable($logDir)) { |
51 | - throw new RuntimeException('You do not have permissions to write CSV report in ' . $logFile); |
|
51 | + throw new RuntimeException('You do not have permissions to write CSV report in '.$logFile); |
|
52 | 52 | } |
53 | 53 | |
54 | 54 | $availableMetrics = (new Registry())->allForStructures(); |
@@ -59,9 +59,9 @@ discard block |
||
59 | 59 | if (!$metric instanceof ClassMetric) { |
60 | 60 | continue; |
61 | 61 | } |
62 | - $row = array_map(static function ($key) use ($metric) { |
|
62 | + $row = array_map(static function($key) use ($metric) { |
|
63 | 63 | $data = $metric->get($key); |
64 | - return (!is_scalar($data)) ? 'N/A': $data; |
|
64 | + return (!is_scalar($data)) ? 'N/A' : $data; |
|
65 | 65 | }, $availableMetrics); |
66 | 66 | |
67 | 67 | fputcsv($logPointer, $row); |
@@ -44,7 +44,7 @@ |
||
44 | 44 | |
45 | 45 | $logDir = dirname($logFile); |
46 | 46 | if (!file_exists($logDir) || !is_writable($logDir)) { |
47 | - throw new RuntimeException('You do not have permissions to write JSON report in ' . $logFile); |
|
47 | + throw new RuntimeException('You do not have permissions to write JSON report in '.$logFile); |
|
48 | 48 | } |
49 | 49 | |
50 | 50 | file_put_contents($logFile, json_encode($metrics, JSON_PRETTY_PRINT)); |
@@ -53,7 +53,7 @@ |
||
53 | 53 | $out .= $this->reportUnitTesting($metrics->get('unitTesting')); |
54 | 54 | } |
55 | 55 | |
56 | - $this->output->write($out . "\n"); |
|
56 | + $this->output->write($out."\n"); |
|
57 | 57 | } |
58 | 58 | |
59 | 59 | /** |