Cortex\Foundation\DataTables\MediaDataTable

Complexity

Total Complexity

3

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	query()	0	6	1
A	getColumns()	0	12	1
A	getBuilderParameters()	0	16	1

<?php

declare(strict_types=1);

namespace Cortex\Foundation\DataTables;

use Spatie\MediaLibrary\Models\Media;
use Cortex\Foundation\Transformers\MediaTransformer;

class MediaDataTable extends AbstractDataTable
{
    /**
     * {@inheritdoc}
     */
    protected $model = Media::class;

    /**
     * {@inheritdoc}
     */
    protected $transformer = MediaTransformer::class;

    /**
     * Get the query object to be processed by dataTables.
     *
     * @return \Illuminate\Database\Eloquent\Builder|\Illuminate\Database\Query\Builder|\Illuminate\Support\Collection

Consider making the return type a bit more specific; maybe use \Illuminate\Database\Query\Builder|\Illuminate\Database\Eloquent\Builder.

     */
    public function query()
    {
        $query = $this->resource->media()->orderBy('order_column', 'ASC');

The property resource does not exist on object<Cortex\Foundation\DataTables\MediaDataTable>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this->applyScopes($query);
    }

    /**
     * Get default builder parameters.
     *
     * @return array

Consider making the return type a bit more specific; maybe use array<string,boolean|string|array<string|array<string,string>>>.

     */
    protected function getBuilderParameters()
    {
        return [
            'keys' => true,
            'autoWidth' => false,
            'dom' => "<'row'<'col-sm-6'B><'col-sm-6'f>> <'row'r><'row'<'col-sm-12't>> <'row'<'col-sm-5'i><'col-sm-7'p>>",

This line exceeds maximum limit of 120 characters; contains 121 characters

            'buttons' => [
                'print', 'reset', 'reload', 'export',
                ['extend' => 'colvis', 'text' => '<i class="fa fa-columns"></i> '.trans('cortex/foundation::common.columns').' <span class="caret"/>'],

This line exceeds maximum limit of 120 characters; contains 151 characters

                ['extend' => 'pageLength', 'text' => '<i class="fa fa-list-ol"></i> '.trans('cortex/foundation::common.limit').' <span class="caret"/>'],

This line exceeds maximum limit of 120 characters; contains 153 characters

            ],
            'initComplete' => 'function (settings) {
                implicitForms.initialize();
            }',
        ];
    }

    /**
     * Get columns.
     *
     * @return array

Consider making the return type a bit more specific; maybe use array<string,array>.

     */
    protected function getColumns()
    {
        return [
            'name' => ['title' => trans('cortex/foundation::common.name'), 'responsivePriority' => 0],
            'file_name' => ['title' => trans('cortex/foundation::common.file_name')],
            'mime_type' => ['title' => trans('cortex/foundation::common.mime_type')],
            'size' => ['title' => trans('cortex/foundation::common.size')],
            'created_at' => ['title' => trans('cortex/foundation::common.created_at'), 'render' => "moment(data).format('MMM Do, YYYY')"],

This line exceeds maximum limit of 120 characters; contains 138 characters

            'updated_at' => ['title' => trans('cortex/foundation::common.updated_at'), 'render' => "moment(data).format('MMM Do, YYYY')"],

This line exceeds maximum limit of 120 characters; contains 138 characters

            'delete' => ['title' => trans('cortex/foundation::common.delete'), 'searchable' => false, 'render' => '"<a href=\""+data+"\" data-method=\"delete\" data-token=\"'.csrf_token().'\" class=\"close\"><span class=\"text-danger fa fa-close\"></span></a>"'],

This line exceeds maximum limit of 120 characters; contains 263 characters

        ];
    }
}