crocodicstudio\crudbooster\Modules\PrivilegeModule\PrivilegeRepo

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	grantAllPermissions()	0	10	1
A	deleteByRoleId()	0	3	1

<?php

namespace crocodicstudio\crudbooster\Modules\PrivilegeModule;

class PrivilegeRepo
{
    public static function deleteByRoleId($id)
    {
        return \DB::table('cms_privileges_roles')->where('cms_privileges_id', $id)->delete();
    }

    /**
     * @param $id
     */
    public function grantAllPermissions($id)
    {
        \DB::table('cms_privileges_roles')->insert([
            'cms_modules_id' => $id,
            'cms_privileges_id' => cbUser()->cms_privileges_id,

Accessing cms_privileges_id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            'can_see_module' => 1,
            'can_create' => 1,
            'can_read' => 1,
            'can_edit' => 1,
            'can_delete' => 1,
        ]);
    }
}