NullCoalesceReplacer - Code Metrics - Inspection of "Merge pull request #26 from sergiusd/null_collapse..." - spatie/7to5 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e4a352...0b5c23 )

by Freek

created 2017-09-25 10:14 UTC

NullCoalesceReplacer A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	6
c	1
b	0
f	0
lcom	0
cbo	7
dl	0
loc	27
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
B	leaveNode()	0	21	6

<?php

namespace Spatie\Php7to5\NodeVisitors;

use PhpParser\Node;
use PhpParser\Node\Expr\BinaryOp\Coalesce;
use PhpParser\NodeVisitorAbstract;

class NullCoalesceReplacer extends NodeVisitorAbstract
{
    /**
     * {@inheritdoc}
     */
    public function leaveNode(Node $node)
    {
        if (!$node instanceof Coalesce) {
            return;
        }
        switch(true)
        {
            case $node->left instanceof Node\Expr\FuncCall:
            case $node->left instanceof Node\Expr\MethodCall:
            case $node->left instanceof Node\Expr\StaticCall:
                $notEmptyCall = new Node\Expr\BooleanNot(new Node\Expr\FuncCall(new Node\Name('empty'), [$node->left]));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                return new Node\Expr\Ternary($notEmptyCall, $node->left, $node->right);
            case $node->left instanceof Node\Expr\BinaryOp:
                $issetCall = new Node\Expr\FuncCall(new Node\Name('isset'), [$node->left->right]);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                $node->left->right = new Node\Expr\Ternary($issetCall, $node->left->right, $node->right);
                return $node->left;
            default:
                $issetCall = new Node\Expr\FuncCall(new Node\Name('isset'), [$node->left]);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                return new Node\Expr\Ternary($issetCall, $node->left, $node->right);
        }
    }
}

1			<?php
2
3			namespace Spatie\Php7to5\NodeVisitors;
4
5			use PhpParser\Node;
6			use PhpParser\Node\Expr\BinaryOp\Coalesce;
7			use PhpParser\NodeVisitorAbstract;
8
9			class NullCoalesceReplacer extends NodeVisitorAbstract
10			{
11			/**
12			* {@inheritdoc}
13			*/
14			public function leaveNode(Node $node)
15			{
16			if (!$node instanceof Coalesce) {
17			return;
18			}
19			switch(true)
20			{
21			case $node->left instanceof Node\Expr\FuncCall:
22			case $node->left instanceof Node\Expr\MethodCall:
23			case $node->left instanceof Node\Expr\StaticCall:
24			$notEmptyCall = new Node\Expr\BooleanNot(new Node\Expr\FuncCall(new Node\Name('empty'), [$node->left]));
			0 ignored issues – show Documentation introduced 2017-09-25 09:11 UTC by Report Bug Copy Issue Report `array($node->left)` is of type `array<integer,object<Php...e\\Expr\\StaticCall>"}>`, but the function expects a `array<integer,object<PhpParser\Node\Arg>>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
25			return new Node\Expr\Ternary($notEmptyCall, $node->left, $node->right);
26			case $node->left instanceof Node\Expr\BinaryOp:
27			$issetCall = new Node\Expr\FuncCall(new Node\Name('isset'), [$node->left->right]);
			0 ignored issues – show Documentation introduced 2017-09-25 09:11 UTC by Report Bug Copy Issue Report `array($node->left->right)` is of type `array<integer,object<Php...pParser\\Node\\Expr>"}>`, but the function expects a `array<integer,object<PhpParser\Node\Arg>>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
28			$node->left->right = new Node\Expr\Ternary($issetCall, $node->left->right, $node->right);
29			return $node->left;
30			default:
31			$issetCall = new Node\Expr\FuncCall(new Node\Name('isset'), [$node->left]);
			0 ignored issues – show Documentation introduced 2017-09-24 10:04 UTC by Report Bug Copy Issue Report `array($node->left)` is of type `array<integer,object<Php...pParser\\Node\\Expr>"}>`, but the function expects a `array<integer,object<PhpParser\Node\Arg>>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
32			return new Node\Expr\Ternary($issetCall, $node->left, $node->right);
33			}
34			}
35			}

spatie / 7to5

Push — master ( e4a352...0b5c23 )

NullCoalesceReplacer A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Duplication Side-by-Side

Filter issues like