O2System\Framework\Models\Sql\DataObjects\Result

Complexity

Total Complexity

15

Size/Duplication

Total Lines	99
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setInfo()	0	5	1
B	__construct()	0	35	10
A	pagination()	0	9	3
A	getInfo()	0	3	1

<?php
/**
 * This file is part of the O2System Framework package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @author         Steeve Andrian Salim
 * @copyright      Copyright (c) Steeve Andrian Salim
 */

// ------------------------------------------------------------------------

namespace O2System\Framework\Models\Sql\DataObjects;

// ------------------------------------------------------------------------

use O2System\Database\DataObjects\Result\Info;
use O2System\Framework\Libraries\Ui\Components\Pagination;
use O2System\Framework\Models\Sql\Model;
use O2System\Spl\Iterators\ArrayIterator;

/**
 * Class Result
 *
 * @package O2System\Database\DataStructures
 */
class Result extends ArrayIterator
{
    /**
     * Result::$info
     *
     * @var Info
     */
    public $info;

    // ------------------------------------------------------------------------

    /**
     * Result::__construct
     *
     * @param \O2System\Database\DataObjects\Result $result
     * @param \O2System\Framework\Models\Sql\Model  $model
     */
    public function __construct(\O2System\Database\DataObjects\Result $result, Model &$model)
    {
        if ($result->count() > 0) {
            $ormResult = new \SplFixedArray($result->count());

            foreach ($result as $key => $row) {
                if (method_exists($model, 'rebuildRow')) {
                    $row = $model->rebuildRow($row);

The method rebuildRow() does not exist on O2System\Framework\Models\Sql\Model. Since you implemented __call, consider adding a @method annotation.

                }

                // Visible Columns
                if (count($model->visibleColumns)) {
                    $columns = $row->getColumns();
                    foreach ($columns as $column) {
                        if ( ! in_array($column, $model->visibleColumns)) {
                            array_push($model->hideColumns, $column);
                        }
                    }
                }

                // Hide Columns
                if (count($model->hideColumns)) {
                    foreach ($model->hideColumns as $column) {
                        if ($row->offsetExists($column)) {
                            $row->offsetUnset($column);
                        }
                    }
                }

                $ormResult[ $key ] = new Result\Row($row, $model);

It seems like $row can also be of type boolean; however, parameter $row of O2System\Framework\Models\Sql\DataObjects\Result\Row::__construct() does only seem to accept O2System\Database\DataObjects\Result\Row|array, maybe add an additional type check?

            }

            parent::__construct($ormResult->toArray());

            unset($ormResult);
        }
    }

    // ------------------------------------------------------------------------

    /**
     * Result::setInfo
     *
     * @param \O2System\Database\DataObjects\Result\Info $info
     *
     * @return static
     */
    public function setInfo(Info $info)
    {
        $this->info = $info;

        return $this;
    }

    // ------------------------------------------------------------------------

    /**
     * Result::getInfo
     *
     * @return \O2System\Database\DataObjects\Result\Info
     */
    public function getInfo()
    {
        return $this->info;
    }

    // ------------------------------------------------------------------------

    /**
     * Result::pagination
     *
     * @return \O2System\Framework\Libraries\Ui\Components\Pagination
     */
    public function pagination()
    {
        $rows = $this->info->getTotal()->rows;
        $rows = empty($rows) ? 0 : $rows;

        $limit = input()->get('limit');
        $limit = empty($limit) ? $this->info->limit : $limit;

        return new Pagination($rows, $limit);

It seems like $limit can also be of type O2System\Spl\DataStructures\SplArrayObject; however, parameter $limit of O2System\Framework\Libraries\Ui\Components\Pagination::__construct() does only seem to accept integer, maybe add an additional type check?

    }
}