A17\Twill\Repositories\GroupRepository

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getFormFields()	0	7	1
A	afterSave()	0	5	1
A	bulkDelete()	0	11	2
A	delete()	0	7	2
A	filter()	0	5	1

<?php

namespace A17\Twill\Repositories;

use A17\Twill\Models\Group;
use A17\Twill\Repositories\Behaviors\HandleGroupPermissions;

class GroupRepository extends ModuleRepository
{
    use HandleGroupPermissions;

The trait A17\Twill\Repositories\Behaviors\HandleGroupPermissions requires some properties which are not provided by A17\Twill\Repositories\GroupRepository: $name, $id

    public function __construct(Group $model)
    {
        $this->model = $model;

It seems like $model of type A17\Twill\Models\Group is incompatible with the declared type A17\Twill\Models\Model of property $model.

    }

    public function getFormFields($group)
    {
        $fields = parent::getFormFields($group);

        $fields['browsers']['users'] = $this->getFormFieldsForBrowser($group, 'users');

        return $fields;
    }

    public function afterSave($group, $fields)
    {
        $this->updateBrowser($group, $fields, 'users');

        parent::afterSave($group, $fields);
    }

    public function delete($id)
    {
        if ($this->model->find($id)->is_everyone_group) {

The property is_everyone_group does not seem to exist on A17\Twill\Models\Model. Are you sure there is no database migration missing?

            return false;
        }

        return parent::delete($id);
    }

    public function bulkDelete($ids)
    {
        $includes_everyone_group = $this->model->whereIn('id', $ids)
            ->where('is_everyone_group', true)
            ->first();

        if ($includes_everyone_group) {
            return false;
        }

        return parent::bulkDelete($ids);
    }

    public function filter($query, array $scopes = [])
    {
        $this->searchIn($query, $scopes, 'search', ['name']);

        return parent::filter($query, $scopes);
    }
}