Quantum\Libraries\Database\Adapters\Sleekdb\Statements\Result

Complexity

Total Complexity

10

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	setHidden()	0	3	1
A	findOneBy()	0	7	1
A	count()	0	3	1
A	first()	0	7	1
A	get()	0	9	1
A	findOne()	0	7	1
A	asArray()	0	9	4

<?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.6
 */

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

use Quantum\Libraries\Database\Exceptions\DatabaseException;
use Quantum\Libraries\Database\Contracts\DbalInterface;
use SleekDB\Exceptions\InvalidConfigurationException;
use SleekDB\Exceptions\InvalidArgumentException;
use Quantum\Model\Exceptions\ModelException;
use SleekDB\Exceptions\IOException;

/**
 * Trait Result
 * @package Quantum\Libraries\Database
 */
trait Result
{
    /**
     * @inheritDoc
     * @return array
     * @throws DatabaseException
     * @throws ModelException
     * @throws IOException
     * @throws InvalidArgumentException
     * @throws InvalidConfigurationException
     */
    public function get(): array
    {
        return array_map(function ($element) {
            $item = clone $this;
            $item->data = $element;

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

            $item->modifiedFields = $element;

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

            $item->isNew = false;

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

            return $item;
        }, $this->getBuilder()->getQuery()->fetch());

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

    }

    /**
     * @inheritDoc
     * @return DbalInterface
     * @throws DatabaseException
     * @throws IOException
     * @throws InvalidArgumentException
     * @throws InvalidConfigurationException
     */
    public function findOne(int $id): DbalInterface
    {
        $result = $this->getOrmModel()->findById($id);

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

        $this->updateOrmModel($result);

It seems like updateOrmModel() 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\Sleekdb\Statements\Result which is incompatible with the type-hinted return Quantum\Libraries\Database\Contracts\DbalInterface.

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     * @throws IOException
     * @throws InvalidArgumentException
     * @throws InvalidConfigurationException
     */
    public function findOneBy(string $column, $value): DbalInterface
    {
        $result = $this->getOrmModel()->findOneBy([$column, '=', $value]);

        $this->updateOrmModel($result);

        return $this;

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

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     * @throws ModelException
     * @throws IOException
     * @throws InvalidArgumentException
     * @throws InvalidConfigurationException
     */
    public function first(): DbalInterface
    {
        $result = $this->getBuilder()->getQuery()->first();

        $this->updateOrmModel($result);

        return $this;

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

    }

    /**
     * @inheritDoc
     * @throws DatabaseException
     * @throws ModelException
     * @throws IOException
     * @throws InvalidArgumentException
     * @throws InvalidConfigurationException
     */
    public function count(): int
    {
        return count($this->getBuilder()->getQuery()->fetch());
    }

    /**
     * @inheritDoc
     */
    public function asArray(): array
    {
        $result = $this->data ?: [];

        if (count($this->hidden) > 0 && count($result)) {
            $result = $this->setHidden($result);
        }

        return $result;
    }

    /**
     * @param $result
     * @return array
     */
    public function setHidden($result): array
    {
        return array_diff_key($result, array_flip($this->hidden));
    }
}