ElfSundae\Laravel\Datatables\Engines\EloquentEngine::addColumn()

Complexity

Conditions	7
Paths	12

Size

Total Lines	23
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes

0

<?php
    public function addColumn($name, $content = null, $order = false)
    {
        if (is_bool($content) || is_int($content)) {
            $order = $content;
            $content = null;
        }

        if (is_null($content) && is_string($name)) {
            $content = function ($model) use ($name) {
                return $model->{$name};
            };
        }

        if (is_array($name)) {
            foreach ($name as $n) {
                $this->addColumn($n, $content, $order);
            }

            return $this;
        }

        return parent::addColumn($name, $content, $order);

It seems like $content can also be of type null; however, Yajra\Datatables\Engines\QueryBuilderEngine::addColumn() does only seem to accept callable, maybe add an additional type check?

    }
}