BristolSU\Module\AssignRoles\Http\Controllers\ParticipantPageController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

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

<?php

Missing file doc comment

namespace BristolSU\Module\AssignRoles\Http\Controllers;

use BristolSU\Module\AssignRoles\Support\PositionSettingRetrieval;
use BristolSU\Module\AssignRoles\Support\SettingRetrievalException;
use BristolSU\Support\Authentication\Contracts\Authentication;

class ParticipantPageController extends Controller

Missing doc comment for class ParticipantPageController

{

    public function index(Authentication $authentication)

Missing doc comment for function index()

    {
        $this->authorize('view-page');
        
        try {
            $setting = app(PositionSettingRetrieval::class)->getSettings(
                $authentication->getGroup()

It seems like $authentication->getGroup() can also be of type null; however, parameter $group of BristolSU\Module\AssignRoles\Support\PositionSettingRetrieval::getSettings() does only seem to accept BristolSU\ControlDB\Contracts\Models\Group, maybe add an additional type check?

            );
            $setting = app(PositionSettingRetrieval::class)->parse($setting);
        } catch (SettingRetrievalException $e) {
            return view('assign-roles::nosettings');
        }
        return view('assign-roles::participant')->with('positionSetting', $setting);
    }
    
}