ElfSundae\Laravel\Datatables\Engines\EloquentEngine

Complexity

Total Complexity

7

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
C	addColumn()	0	23	7

<?php

namespace ElfSundae\Laravel\Datatables\Engines;

use Yajra\Datatables\Engines\EloquentEngine as BaseEngine;

class EloquentEngine extends BaseEngine
{
    /**
     * Add column in collection.
     *
     * @param  string|string[]  $name
     * @param  string|callable|bool|int|null  $content
     * @param  bool|int  $order
     * @return $this
     */
    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?

    }
}