Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join

Complexity

Total Complexity

10

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	joinThrough()	0	21	3
A	join()	0	4	1
A	leftJoin()	0	4	1
A	innerJoin()	0	4	1
A	rightJoin()	0	4	1
A	joinTo()	0	21	3

<?php

/**
 * Quantum PHP Framework
 *
 * An open source software development framework for PHP
 *
 * @package Quantum
 * @author Arman Ag. <arman.ag@softberg.org>
 * @copyright Copyright (c) 2018 Softberg LLC (https://softberg.org)
 * @link http://quantum.softberg.org/
 * @since 2.9.5
 */

namespace Quantum\Libraries\Database\Adapters\Idiorm\Statements;

use Quantum\Libraries\Database\Exceptions\DatabaseException;
use Quantum\Libraries\Database\Adapters\Idiorm\IdiormPatch;
use Quantum\Libraries\Database\Exceptions\ModelException;
use Quantum\Libraries\Database\Contracts\DbalInterface;
use Quantum\Mvc\QtModel;

/**
 * Trait Join
 * @package Quantum\Libraries\Database
 */
trait Join
{

    /**
     * @inheritDoc
     * @throws DatabaseException
     */
    public function join(string $table, array $constraint, string $tableAlias = null): DbalInterface
    {
        $this->getOrmModel()->join($table, $constraint, $tableAlias);

It seems like getOrmModel() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this;

The expression return $this returns the type Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join which is incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     */
    public function innerJoin(string $table, array $constraint, ?string $tableAlias = null): DbalInterface
    {
        $this->getOrmModel()->inner_join($table, $constraint, $tableAlias);
        return $this;

The expression return $this returns the type Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join which is incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     */
    public function leftJoin(string $table, array $constraint, ?string $tableAlias = null): DbalInterface
    {
        IdiormPatch::getInstance()->use($this->getOrmModel())->leftJoin($table, $constraint, $tableAlias);
        return $this;

The expression return $this returns the type Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join which is incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     */
    public function rightJoin(string $table, array $constraint, ?string $tableAlias = null): DbalInterface
    {
        IdiormPatch::getInstance()->use($this->getOrmModel())->rightJoin($table, $constraint, $tableAlias);
        return $this;

The expression return $this returns the type Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join which is incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     * @throws ModelException
     */
    public function joinTo(QtModel $model, bool $switch = true): DbalInterface
    {
        if (!isset($model->foreignKeys[$this->table])) {
            throw ModelException::wrongRelation(get_class($model), $this->table);
        }

        $this->getOrmModel()->join($model->table,
            [
                $model->table . '.' . $model->foreignKeys[$this->table],
                '=',
                $this->table . '.' . $this->idColumn
            ]
        );

        if ($switch) {
            $this->table = $model->table;

The property table does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->idColumn = $model->idColumn;

The property idColumn does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->foreignKeys = $model->foreignKeys;

The property foreignKeys does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }

        return $this;

The expression return $this returns the type Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join which is incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     * @throws ModelException
     */
    public function joinThrough(QtModel $model, bool $switch = true): DbalInterface
    {
        if (!isset($this->foreignKeys[$model->table])) {
            throw ModelException::wrongRelation(get_class($model), $this->table);
        }

        $this->getOrmModel()->join($model->table,
            [
                $model->table . '.' . $model->idColumn,
                '=',
                $this->table . '.' . $this->foreignKeys[$model->table]
            ]
        );

        if ($switch) {
            $this->table = $model->table;

The property table does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->idColumn = $model->idColumn;

The property idColumn does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->foreignKeys = $model->foreignKeys;

The property foreignKeys does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }

        return $this;

The expression return $this returns the type Quantum\Libraries\Database\Adapters\Idiorm\Statements\Join&object which includes types incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }
}