App\Http\Controllers\SkillsController

Complexity

Total Complexity

18

Size/Duplication

Total Lines	133
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	edit()	0	8	1
C	update()	0	74	14
A	destroy()	0	10	2

<?php

Missing file doc comment

namespace App\Http\Controllers;

use Illuminate\Support\Facades\Lang;
use Illuminate\Http\Request;
use Barryvdh\Debugbar\Facade as Debugbar;
use App\Models\Skill;
use App\Models\Lookup\SkillStatus;
use App\Models\SkillDeclaration;
use App\Models\Applicant;
use App\Http\Controllers\Controller;

class SkillsController extends Controller

Missing doc comment for class SkillsController

{

    /**
     * Display the Skills page associated with the applicant.
     *
     * @param  \App\Models\Applicant  $applicant

Missing parameter comment

Expected 1 spaces after parameter type; 2 found

     * @return \Illuminate\Http\Response

Tag cannot be grouped with parameter tags in a doc comment

     */
    public function show(Applicant $applicant)

The parameter $applicant is not used and could be removed.

    {
        //

    }

    /**
     * Show the form for editing the applicant's skills
     *
     * @param  Request  $request

Missing parameter comment

Expected 15 spaces after parameter type; 2 found

     * @param  \App\Models\Applicant  $applicant

Missing parameter comment

Expected 1 spaces after parameter type; 2 found

     * @return \Illuminate\Http\Response

Tag cannot be grouped with parameter tags in a doc comment

     */
    public function edit(Request $request, Applicant $applicant)
    {
        $skills = Skill::all();
        return view('applicant/profile_03_skills', [

The opening parenthesis of a multi-line function call should be the last content on the line.

The expression return view('applicant/profile_03_skills', array('applicant' => $applicant, 'profile' => Illuminate\Support\Facades\Lang::get('applicant/profile_skills'), 'form_submit_action' => route('profile.skills.update', $applicant), 'skills' => $skills)) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

            'applicant' => $applicant,
            'profile' => Lang::get('applicant/profile_skills'),
            'form_submit_action' => route('profile.skills.update', $applicant),

$applicant of type App\Models\Applicant is incompatible with the type array expected by parameter $parameters of route().

            'skills' => $skills
        ]);

For multi-line function calls, the closing parenthesis should be on a new line.

    }

    /**
     * Update the applicant's profile in storage.
     *
     * @param  \Illuminate\Http\Request  $request

Missing parameter comment

Expected 1 spaces after parameter type; 2 found

     * @param  \App\Models\Applicant  $applicant

Missing parameter comment

Expected 4 spaces after parameter type; 2 found

     * @return \Illuminate\Http\Response

Tag cannot be grouped with parameter tags in a doc comment

     */
    public function update(Request $request, Applicant $applicant)
    {

        $input = $request->input();

        Debugbar::info($input);

        $skillDeclarations = $input['skill_declarations'];
        $claimedStatusId = SkillStatus::where('name', 'claimed')->firstOrFail()->id;

        //Delete old skill declarations that weren't resubmitted
        //Note: this must be done before adding new ones, so we don't delete
        // them right after adding them
        foreach($applicant->skill_declarations as $oldDeclaration) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

            //Check if none were resubmitted, or if this specific one wasn't
            $type = $oldDeclaration->skill->skill_type->name;
            if (!isset($skillDeclarations['old']) ||
                !isset($skillDeclarations['old'][$type]) ||

Each line in a multi-line IF statement must begin with a boolean operator

                !isset($skillDeclarations['old'][$type][$oldDeclaration->id])) {

Each line in a multi-line IF statement must begin with a boolean operator

Closing parenthesis of a multi-line IF statement must be on a new line

                $oldDeclaration->delete();
            }
        }

        //Save new skill declarartions
        if (isset($skillDeclarations['new'])) {
            foreach($skillDeclarations['new'] as $skillType => $typeInput) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

                foreach($typeInput as $skillDeclarationInput) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

                    $skillDeclaration = new SkillDeclaration();
                    $skillDeclaration->applicant_id = $applicant->id;
                    $skillDeclaration->skill_id = $skillDeclarationInput['skill_id'];
                    $skillDeclaration->skill_status_id = $claimedStatusId;
                    $skillDeclaration->fill([

The opening parenthesis of a multi-line function call should be the last content on the line.

                        'description' => $skillDeclarationInput['description'],
                        'skill_level_id' => isset($skillDeclarationInput['skill_level_id']) ? $skillDeclarationInput['skill_level_id'] : null,
                    ]);

For multi-line function calls, the closing parenthesis should be on a new line.

                    $skillDeclaration->save();

                    $referenceIds = $this->getRelativeIds($skillDeclarationInput, 'references');
                    $skillDeclaration->references()->sync($referenceIds);

                    $sampleIds = $this->getRelativeIds($skillDeclarationInput, 'samples');
                    $skillDeclaration->work_samples()->sync($sampleIds);
                }
            }
        }

        //Update old declarations
        if (isset($skillDeclarations['old'])) {
            foreach($skillDeclarations['old'] as $skillType => $typeInput) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

                foreach($typeInput as $id=>$skillDeclarationInput) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

                    //Ensure this declaration belongs to this applicant
                    $skillDeclaration = $applicant->skill_declarations->firstWhere('id', $id);
                    if ($skillDeclaration != null) {
                        //skill_id and skill_status cannot be changed
                        $skillDeclaration->fill([

The opening parenthesis of a multi-line function call should be the last content on the line.

                            'description' => $skillDeclarationInput['description'],
                            'skill_level_id' => isset($skillDeclarationInput['skill_level_id']) ? $skillDeclarationInput['skill_level_id'] : null,
                        ]);

For multi-line function calls, the closing parenthesis should be on a new line.

                        $skillDeclaration->save();

                        $referenceIds = $this->getRelativeIds($skillDeclarationInput, 'references');
                        $skillDeclaration->references()->sync($referenceIds);

                        $sampleIds = $this->getRelativeIds($skillDeclarationInput, 'samples');
                        $skillDeclaration->work_samples()->sync($sampleIds);
                    } else {
                        Debugbar::warning('Applicant '.$applicant->id.' attempted to update skill declaration with invalid id '.$id);
                    }
                }
            }
        }


        return redirect( route('profile.skills.edit', $applicant) );

The expression return redirect(route('profile.skills.edit', $applicant)) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

$applicant of type App\Models\Applicant is incompatible with the type array expected by parameter $parameters of route().

    }

    /**
     * Delete the particular skill declaration in storage.
     *
     * @param  \Illuminate\Http\Request  $request

Missing parameter comment

Expected 5 spaces after parameter type; 2 found

     * @param  \App\Models\SkillDeclaration  $skillDeclaration

Missing parameter comment

Expected 1 spaces after parameter type; 2 found

     * @return \Illuminate\Http\Response

Tag cannot be grouped with parameter tags in a doc comment

     */
    public function destroy(Request $request, SkillDeclaration $skillDeclaration)
    {
        $skillDeclaration->delete();
        debugbar()->info('deleted skill declaration '.$skillDeclaration->id);

        if($request->ajax()) {

Expected "if (...) {\n"; found "if(...) {\n"

            return 'Skill deleted';

The expression return 'Skill deleted' returns the type string which is incompatible with the documented return type Illuminate\Http\Response.

        }

        return redirect()->back();

The expression return redirect()->back() returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

    }

}