JoinTrait::crossJoin() - Code Metrics - Inspection of "release" - phuria/under-query - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 94029f...e94e78 )

by Beniamin

created 2016-11-20 14:45 UTC

JoinTrait::crossJoin() A

↳ Parent: JoinTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
cc	1
eloc	2
nc	1
nop	3
crap	1

<?php

/**
 * This file is part of UnderQuery package.
 *
 * Copyright (c) 2016 Beniamin Jonatan Šimko
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Phuria\UnderQuery\QueryBuilder\Clause;

use Phuria\UnderQuery\JoinType;
use Phuria\UnderQuery\Language\Expression\RelativeClause;
use Phuria\UnderQuery\Table\AbstractTable;
use Phuria\UnderQuery\Table\JoinMetadata;
use Phuria\UnderQuery\Table\TableInterface;

/**
 * @author Beniamin Jonatan Šimko <[email protected]>
 */
trait JoinTrait
{
    /**
     * @var array $joinTables
     */
    private $joinTables = [];

    /**
     * @param mixed       $table
     * @param string|null $alias
     *
     * @return TableInterface
     */
    abstract public function createTable($table, $alias = null);

    /**
     * @param string      $joinType
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return TableInterface
     */
    public function doJoin($joinType, $table, $alias = null, $joinOn = null)
    {
        $this->joinTables[] = $tableObject = $this->createTable($table, $alias);

        $joinMetadata = new JoinMetadata();
        $joinMetadata->setJoinType($joinType);

        if ($joinOn) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $relativeOn = new RelativeClause($tableObject, $joinOn, RelativeClause::SELF_DIRECTIVE);
            $joinMetadata->setJoinOn($relativeOn);
        }

        $tableObject->setJoinMetadata($joinMetadata);
        is_callable($table) && $table($tableObject, $joinMetadata);

        return $tableObject;
    }

    /**
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return TableInterface
     */
    public function join($table, $alias = null, $joinOn = null)
    {
        return $this->doJoin(JoinType::JOIN, $table, $alias, $joinOn);
    }

    /**
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return AbstractTable
     */
    public function straightJoin($table, $alias = null, $joinOn = null)
    {
        return $this->doJoin(JoinType::STRAIGHT_JOIN, $table, $alias, $joinOn);
    }

    /**
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return AbstractTable
     */
    public function crossJoin($table, $alias = null, $joinOn = null)
    {
        return $this->doJoin(JoinType::CROSS_JOIN, $table, $alias, $joinOn);
    }

    /**
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return AbstractTable
     */
    public function leftJoin($table, $alias = null, $joinOn = null)
    {
        return $this->doJoin(JoinType::LEFT_JOIN, $table, $alias, $joinOn);
    }

    /**
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return AbstractTable
     */
    public function rightJoin($table, $alias = null, $joinOn = null)
    {
        return $this->doJoin(JoinType::RIGHT_JOIN, $table, $alias, $joinOn);
    }

    /**
     * @param mixed       $table
     * @param string|null $alias
     * @param string|null $joinOn
     *
     * @return AbstractTable
     */
    public function innerJoin($table, $alias = null, $joinOn = null)
    {
        return $this->doJoin(JoinType::INNER_JOIN, $table, $alias, $joinOn);
    }

    /**
     * @return array
     */
    public function getJoinTables()
    {
        return $this->joinTables;
    }
}

1		<?php
2
3		/**
4		* This file is part of UnderQuery package.
5		*
6		* Copyright (c) 2016 Beniamin Jonatan Šimko
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Phuria\UnderQuery\QueryBuilder\Clause;
13
14		use Phuria\UnderQuery\JoinType;
15		use Phuria\UnderQuery\Language\Expression\RelativeClause;
16		use Phuria\UnderQuery\Table\AbstractTable;
17		use Phuria\UnderQuery\Table\JoinMetadata;
18		use Phuria\UnderQuery\Table\TableInterface;
19
20		/**
21		* @author Beniamin Jonatan Šimko <[email protected]>
22		*/
23		trait JoinTrait
24		{
25		/**
26		* @var array $joinTables
27		*/
28		private $joinTables = [];
29
30		/**
31		* @param mixed $table
32		* @param string\|null $alias
33		*
34		* @return TableInterface
35		*/
36		abstract public function createTable($table, $alias = null);
37
38		/**
39		* @param string $joinType
40		* @param mixed $table
41		* @param string\|null $alias
42		* @param string\|null $joinOn
43		*
44		* @return TableInterface
45		*/
46	3	public function doJoin($joinType, $table, $alias = null, $joinOn = null)
47		{
48	3	$this->joinTables[] = $tableObject = $this->createTable($table, $alias);
49
50	3	$joinMetadata = new JoinMetadata();
51	3	$joinMetadata->setJoinType($joinType);
52
53	3	if ($joinOn) {
		0 ignored issues – show Bug Best Practice introduced 2016-11-20 14:47 UTC by Report Bug Copy Issue Report The expression `$joinOn` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
54	1	$relativeOn = new RelativeClause($tableObject, $joinOn, RelativeClause::SELF_DIRECTIVE);
55	1	$joinMetadata->setJoinOn($relativeOn);
56	1	}
57
58	3	$tableObject->setJoinMetadata($joinMetadata);
59	3	is_callable($table) && $table($tableObject, $joinMetadata);
60
61	3	return $tableObject;
62		}
63
64		/**
65		* @param mixed $table
66		* @param string\|null $alias
67		* @param string\|null $joinOn
68		*
69		* @return TableInterface
70		*/
71	3	public function join($table, $alias = null, $joinOn = null)
72		{
73	3	return $this->doJoin(JoinType::JOIN, $table, $alias, $joinOn);
74		}
75
76		/**
77		* @param mixed $table
78		* @param string\|null $alias
79		* @param string\|null $joinOn
80		*
81		* @return AbstractTable
82		*/
83	1	public function straightJoin($table, $alias = null, $joinOn = null)
84		{
85	1	return $this->doJoin(JoinType::STRAIGHT_JOIN, $table, $alias, $joinOn);
86		}
87
88		/**
89		* @param mixed $table
90		* @param string\|null $alias
91		* @param string\|null $joinOn
92		*
93		* @return AbstractTable
94		*/
95	1	public function crossJoin($table, $alias = null, $joinOn = null)
96		{
97	1	return $this->doJoin(JoinType::CROSS_JOIN, $table, $alias, $joinOn);
98		}
99
100		/**
101		* @param mixed $table
102		* @param string\|null $alias
103		* @param string\|null $joinOn
104		*
105		* @return AbstractTable
106		*/
107	1	public function leftJoin($table, $alias = null, $joinOn = null)
108		{
109	1	return $this->doJoin(JoinType::LEFT_JOIN, $table, $alias, $joinOn);
110		}
111
112		/**
113		* @param mixed $table
114		* @param string\|null $alias
115		* @param string\|null $joinOn
116		*
117		* @return AbstractTable
118		*/
119	1	public function rightJoin($table, $alias = null, $joinOn = null)
120		{
121	1	return $this->doJoin(JoinType::RIGHT_JOIN, $table, $alias, $joinOn);
122		}
123
124		/**
125		* @param mixed $table
126		* @param string\|null $alias
127		* @param string\|null $joinOn
128		*
129		* @return AbstractTable
130		*/
131	1	public function innerJoin($table, $alias = null, $joinOn = null)
132		{
133	1	return $this->doJoin(JoinType::INNER_JOIN, $table, $alias, $joinOn);
134		}
135
136		/**
137		* @return array
138		*/
139	1	public function getJoinTables()
140		{
141	1	return $this->joinTables;
142		}
143		}

phuria / under-query

Push — master ( 94029f...e94e78 )

JoinTrait::crossJoin() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like