App\Http\Controllers\UserController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

95.45%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	20	2
A	delete()	0	13	3

<?php

namespace App\Http\Controllers;

use App\Models\User;
use App\Services\UserTableService;

class UserController extends Controller
{
    protected $userTableService;

    public function __construct(UserTableService $userTableService)
    {
        $this->userTableService = $userTableService;
    }

    public function index()
    {
        $title = trans('general.users');

        $dataset = User::select('id', 'name', 'email', 'created_at')
            ->paginate($this->getItemsPerPage());

        if ($dataset->isEmpty()) {
            $this->addFlashMessage(trans('general.no_users_in_database'), 'alert-danger');
        }

        $viewData = [
            'columns'       => $this->userTableService->getColumns(),
            'dataset'       => $dataset,
            'routeName'     => $this->userTableService->getRouteName(),
            'title'         => $title,
            'deleteMessage' => mb_strtolower(trans('general.user')).' '.mb_strtolower(trans('general.number')),

It seems like trans('general.user') can also be of type array; however, parameter $str of mb_strtolower() does only seem to accept string, maybe add an additional type check?

        ];

        return view('list', $viewData);
    }

    public function delete($objectId)
    {
        $object = User::find($objectId);
        $data = ['class' => 'alert-success', 'message' => trans('general.deleted')];

        // TODO: isAdmin()
        if (!$object || $object->id === 1 /*|| $object->isAdmin()*/) {

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

            $data = ['class' => 'alert-danger', 'message' => trans('general.cannot_delete_admins')];
        } else {
            User::destroy($objectId);
        }

        return response()->json($data);
    }
}