WithKeyword::build() - Code Metrics - Inspection of "Add WITH support" - phpmyadmin/sql-parser - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#334)

by Antoine

created 2021-03-11 11:16 UTC

WithKeyword::build() A

↳ Parent: WithKeyword

Complexity

Conditions	4
Paths	5

Size

Total Lines	21
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	10
c	1
b	0
f	0
nc	5
nop	2
dl	0
loc	21
rs	9.9332

<?php
/**
 * `UNION` keyword builder.
 */

declare(strict_types=1);

namespace PhpMyAdmin\SqlParser\Components;

use PhpMyAdmin\SqlParser\Component;
use PhpMyAdmin\SqlParser\Parser;
use RuntimeException;

/**
 * `WITH` keyword builder.
 */
class WithKeyword extends Component
{
    /** @var string $statement */
    public $name;
    /** @var ArrayObj[] $statement */
    public $columns = [];
    /** @var Parser $statement */
    public $statement;

    public function __construct(string $name)
    {
        $this->name = $name;
    }

    /**
     * @param WithKeyword $component
     *
     * @return string
     */
    public static function build($component, array $options = [])
    {
        if (! $component instanceof WithKeyword) {

            throw new RuntimeException('Can not build a component that is not a WithKeyword');
        }

        $str = $component->name;

        if ($component->columns) {

            $str .= ArrayObj::build($component->columns);
        }

        $str .= ' AS (';

        foreach ($component->statement->statements as $statement) {
            $str .= $statement->build();
        }

        $str .= ')';

        return $str;
    }
}


1			<?php
2			/**
3			* `UNION` keyword builder.
4			*/
5
6			declare(strict_types=1);
7
8			namespace PhpMyAdmin\SqlParser\Components;
9
10			use PhpMyAdmin\SqlParser\Component;
11			use PhpMyAdmin\SqlParser\Parser;
12			use RuntimeException;
13
14			/**
15			* `WITH` keyword builder.
16			*/
17			class WithKeyword extends Component
18			{
19			/** @var string $statement */
20			public $name;
21			/** @var ArrayObj[] $statement */
22			public $columns = [];
23			/** @var Parser $statement */
24			public $statement;
25
26			public function __construct(string $name)
27			{
28			$this->name = $name;
29			}
30
31			/**
32			* @param WithKeyword $component
33			*
34			* @return string
35			*/
36			public static function build($component, array $options = [])
37			{
38			if (! $component instanceof WithKeyword) {
			0 ignored issues – show introduced 2021-03-11 11:16 UTC by Report Bug Copy Issue Report `$component` is always a sub-type of `PhpMyAdmin\SqlParser\Components\WithKeyword`. Loading history...
39			throw new RuntimeException('Can not build a component that is not a WithKeyword');
40			}
41
42			$str = $component->name;
43
44			if ($component->columns) {
			0 ignored issues – show Bug Best Practice introduced 2021-03-11 11:26 UTC by Report Bug Copy Issue Report The expression `$component->columns` of type `PhpMyAdmin\SqlParser\Components\ArrayObj[]` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
45			$str .= ArrayObj::build($component->columns);
46			}
47
48			$str .= ' AS (';
49
50			foreach ($component->statement->statements as $statement) {
51			$str .= $statement->build();
52			}
53
54			$str .= ')';
55
56			return $str;
57			}
58			}
59

phpmyadmin / sql-parser

Pull Request — master (#334)

WithKeyword::build() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like