App\DataTables\AlertsDataTable

Complexity

Total Complexity

6

Size/Duplication

Total Lines	112
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

55%

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
B	ajax()	0	25	1
A	query()	0	5	1
A	html()	0	8	1
A	getColumns()	0	19	1
A	filename()	0	4	1
A	getBuilderParameters()	0	10	1

<?php
/*
 * Copyright (C) 2016 Neil Lathwood <neil@lathwood.co.uk>
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
namespace App\DataTables;

use App\Models\Alerting\Alerts;
use Yajra\Datatables\Services\DataTable;

class AlertsDataTable extends DataTable
{
    // protected $printPreview  = 'path.to.print.preview.view';

45% of this comment could be valid code. Did you maybe forget this after debugging?

    /**
     * Display ajax response.
     *
     * @return \Illuminate\Http\JsonResponse
     */
    public function ajax()
    {
        return $this->datatables
            ->eloquent($this->query())
            ->editColumn('state', '@if ($state == 0)
                                        <div class="label label-success">SUCCESS</div>
                                    @elseif ($state == 1)
                                        <div class="label label-danger">FAILED</div>
                                    @elseif ($state == 2)
                                        <div class="label label-warning">MUTED</div>
                                    @else
                                        <div class="label label-primary">UNKNOWN</div>
                                    @endif')
            ->editColumn('rule.name', function ($this) {

function ($this) { return '<a href="' . url('alerting/rules/' . $this['rule']['id']) . '">' . $this['rule']['name'] . '</a>'; } is of type object<Closure>, but the function expects a string.

                return '<a href="' . url("alerting/rules/".$this['rule']['id']) . '">' . $this['rule']['name'] . '</a>';
            })
            ->editColumn('device.hostname', function ($this) {

function ($this) { return '<a href="' . url('devices/' . $this['device']['device_id']) . '">' . $this['device']['hostname'] . '</a>'; } is of type object<Closure>, but the function expects a string.

                return '<a href="' . url("devices/".$this['device']['device_id']) . '">' . $this['device']['hostname'] . '</a>';
            })
            ->addColumn('actions', function ($this) {

function ($this) { return ''; } is of type object<Closure>, but the function expects a string.

The parameter $this is not used and could be removed.

                return '';
            })
            ->make(true);
            //            ->addColumn('action', 'path.to.action.view')

67% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    /**
     * Get the query object to be processed by datatables.
     *
     * @return \Illuminate\Database\Query\Builder|\Illuminate\Database\Eloquent\Builder
     */
    public function query()
    {
        $alerts = Alerts::query()->where('state', '!=', '0')->with('device')->with('user')->with('rule')->select('alerts.*');
        return $this->applyScopes($alerts);
    }

    /**
     * Optional method if you want to use html builder.
     *
     * @return \Yajra\Datatables\Html\Builder
     */
    public function html()
    {
        return $this->builder()
                    ->columns($this->getColumns())
                    ->parameters($this->getBuilderParameters());
                    //                    ->ajax('')

67% of this comment could be valid code. Did you maybe forget this after debugging?

                    //                    ->addAction(['width' => '80px'])

67% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    /**
     * Get columns.
     *
     * @return array
     */
    private function getColumns()
    {
        return [
            'state'     => [
                'title' => trans('alerting.alerts.text.state'),
            ],
            'rule.name' => [
                'title' => trans('alerting.alerts.text.rule'),
            ],
            'device.hostname' => [
                'title'       => trans('devices.label.hostname'),
            ],
            'timestamp',
            'rule.severity' => [
                'title'     => trans('alerting.alerts.text.severity'),
            ],
            'actions',
        ];
    }

    /**
     * Get filename for export.
     *
     * @return string
     */
    protected function filename()
    {
        return 'alerts';
    }

    /**
     * Get Builder Params
     *
     * @return array
     */
    protected function getBuilderParameters()
    {
        return [
            'dom' => 'Blfrtip',
            'lengthMenu' => [[25, 50, 100, -1], [25, 50, 100, "All"]],
            'buttons' => [
                'csv', 'excel', 'pdf', 'print', 'reset', 'reload',
            ],
        ];
    }

}