Coyote\Http\Controllers\Adm\Forum\PermissionsController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	17

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	index()	0	6	1
B	edit()	0	42	5
A	save()	0	24	3

<?php

namespace Coyote\Http\Controllers\Adm\Forum;

use Coyote\Events\ForumWasSaved;
use Coyote\Http\Controllers\Adm\BaseController;
use Coyote\Http\Grids\Adm\Forum\PermissionsGrid;
use Coyote\Permission;
use Coyote\Repositories\Contracts\ForumRepositoryInterface as ForumRepository;
use Coyote\Repositories\Contracts\GroupRepositoryInterface as GroupRepository;
use Boduch\Grid\Source\CollectionSource;
use Coyote\Services\Forum\TreeBuilder;
use Illuminate\Http\Request;

class PermissionsController extends BaseController
{
    /**
     * @var ForumRepository
     */
    private $forum;

    /**
     * @var GroupRepository
     */
    private $group;

    /**
     * @param ForumRepository $forum
     * @param GroupRepository $group
     */
    public function __construct(ForumRepository $forum, GroupRepository $group)
    {
        parent::__construct();

        $this->forum = $forum;
        $this->group = $group;
        $this->breadcrumb->push('Prawa dostępu', route('adm.forum.permissions'));
    }

    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function index()
    {
        $categoriesList = (new TreeBuilder())->listById($this->forum->list());

        return $this->view('adm.forum.permissions.home')->with('categoriesList', $categoriesList);
    }

    /**
     * @param Request $request
     * @return \Illuminate\View\View
     */
    public function edit(Request $request)
    {
        /** @var \Coyote\Forum $forum */
        $forum = $this->forum->findOrFail((int) $request->input('id'));
        $permissions = $forum->permissions()->get();
        $groups = $this->group->all();

        $this->breadcrumb->push($forum->name);

        $data = collect();

        // ugly way to get only forum permissions
        foreach (Permission::where('name', 'ilike', 'forum%')->get() as $permission) {
            $row = collect([
                'name' => $permission->name,
                'description' => $permission->description,
                'permission_id' => $permission->id
            ]);

            foreach ($groups as $group) {
                $filtered = $permissions->filter(function ($value) use ($group, $permission) {
                    return $value->group_id == $group->id && $value->permission_id == $permission->id;
                })->first();

                $row['group_' . $group->id] = isset($filtered->value) ? $filtered->value : $permission->default;
            }

            $data->push($row);
        }

        $grid = $this
            ->gridBuilder()
            ->createGrid(PermissionsGrid::class)
            ->setEnablePagination(false)
            ->setSource(new CollectionSource($data))
            ->setViewData(['http_method' => 'post', 'form_url' => route('adm.forum.permissions.save', ['id' => $forum->id])]);

        return $this->view('adm.forum.permissions.home', [
            'grid' => $grid,
            'categoriesList' => (new TreeBuilder())->listById($this->forum->list())
        ]);
    }

    /**
     * @param Request $request
     * @return \Illuminate\Http\RedirectResponse
     */
    public function save(Request $request)
    {
        /** @var \Coyote\Forum $forum */
        $forum = $this->forum->findOrFail($request->input('id'));

        $this->transaction(function () use ($forum, $request) {
            $forum->permissions()->delete();

            foreach ($request->input('group') as $groupId => $permissions) {

The expression $request->input('group') of type string|array|null is not guaranteed to be traversable. How about adding an additional type check?

                foreach ($permissions as $permissionId => $value) {
                    $forum->permissions()->create([
                        'group_id' => $groupId,
                        'permission_id' => $permissionId,
                        'value' => $value
                    ]);
                }
            }

            $this->flushPermission();
            event(new ForumWasSaved($forum));
        });

        return redirect()->route('adm.forum.permissions')->with('success', 'Zmiany uprawnień zostały zapisane.');
    }
}