Quantum\Libraries\Database\Sleekdb\Statements\Result

Complexity

Total Complexity

6

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	findOneBy()	0	8	1
A	get()	0	3	1
A	count()	0	3	1
A	first()	0	8	1
A	asArray()	0	3	1
A	findOne()	0	8	1

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

namespace Quantum\Libraries\Database\Sleekdb\Statements;

use Quantum\Libraries\Database\DbalInterface;

/**
 * Trait Result
 * @package Quantum\Libraries\Database\Sleekdb\Statements
 */
trait Result
{

    /**
     * @inheritDoc
     * @throws \Quantum\Exceptions\DatabaseException
     * @throws \SleekDB\Exceptions\IOException
     * @throws \SleekDB\Exceptions\InvalidArgumentException
     * @throws \SleekDB\Exceptions\InvalidConfigurationException
     */
    public function get()
    {
        return $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
     * @throws \Quantum\Exceptions\DatabaseException
     * @throws \SleekDB\Exceptions\IOException
     * @throws \SleekDB\Exceptions\InvalidArgumentException
     * @throws \SleekDB\Exceptions\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->data = $result;

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

        $this->modifiedFields = $result;

The property modifiedFields 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\Sleekdb\Statements\Result which is incompatible with the type-hinted return Quantum\Libraries\Database\DbalInterface.

    }

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

        $this->data = $result;

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

        $this->modifiedFields = $result;

The property modifiedFields 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\Sleekdb\Statements\Result which is incompatible with the type-hinted return Quantum\Libraries\Database\DbalInterface.

    }

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

        $this->data = $result;

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

        $this->modifiedFields = $result;

The property modifiedFields 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\Sleekdb\Statements\Result which is incompatible with the type-hinted return Quantum\Libraries\Database\DbalInterface.

    }

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

    /**
     * @inheritDoc
     */
    public function asArray(): array
    {
        return $this->data;
    }

}