O2System\Framework\Models\Sql\DataObjects\Result::offsetSet()

Complexity

Conditions	5
Paths	16

Size

Total Lines	26
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function offsetSet($offset, $row)
    {
        $model = models($this->model->getClass());

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

        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.

        }

        $hideColumns = [];

        // Visible Columns
        if (count($model->visibleColumns)) {

It seems like $model->visibleColumns can also be of type boolean; however, parameter $var of count() does only seem to accept Countable|array, maybe add an additional type check?

The property visibleColumns does not exist on O2System\Framework\Containers\Models. Since you implemented __get, consider adding a @property annotation.

The property visibleColumns does not exist on O2System\Framework\Models\NoSql\Model. Since you implemented __get, consider adding a @property annotation.

            $hideColumns = array_diff($row->getColumns(), $model->visibleColumns);

It seems like $model->visibleColumns can also be of type boolean; however, parameter $array2 of array_diff() does only seem to accept array, maybe add an additional type check?

        }

        // Hide Columns
        if (count($model->hideColumns)) {

The property hideColumns does not exist on O2System\Framework\Containers\Models. Since you implemented __get, consider adding a @property annotation.

The property hideColumns does not exist on O2System\Framework\Models\NoSql\Model. Since you implemented __get, consider adding a @property annotation.

            $hideColumns = array_merge($model->hideColumns);

It seems like $model->hideColumns can also be of type boolean; however, parameter $array1 of array_merge() does only seem to accept array, maybe add an additional type check?

        }

        // Unset Columns
        foreach ($hideColumns as $column) {
            $row->offsetUnset($column);
        }

        parent::offsetSet($offset, new Result\Row($row, $model));

It seems like $model can also be of type O2System\Framework\Containers\Models and O2System\Framework\Models\NoSql\Model; however, parameter $model of O2System\Framework\Models\Sql\DataObjects\Result\Row::__construct() does only seem to accept O2System\Framework\Models\Sql\Model, maybe add an additional type check?

    }
}