App\Http\Controllers\Api\CriteriaController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	151
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
B	batchUpdate()	0	34	6
A	deleteCriteria()	0	11	1
A	updateCriteria()	0	19	3
A	createCriteria()	0	10	1
A	indexByJob()	0	4	1
A	makeAssessmentPlanNotification()	0	19	1

<?php

namespace App\Http\Controllers\Api;

use App\Http\Controllers\Controller;
use App\Http\Resources\CriteriaResource;
use App\Models\Assessment;
use App\Models\AssessmentPlanNotification;
use App\Models\Criteria;
use App\Models\JobPoster;
use Illuminate\Http\Request;

class CriteriaController extends Controller
{
    /**
     * Returns all criteria by JobPoster ID.
     *
     * @param  \App\Models\JobPoster $jobPoster Incoming Job Poster object.
     * @return \Illuminate\Http\Response
     */
    public function indexByJob(JobPoster $jobPoster)
    {
        $criteriaByJob = Criteria::where('job_poster_id', $jobPoster->id)->get();
        return CriteriaResource::collection($criteriaByJob);
    }

    /**
     * Update the set of criteria associated with a Job.
     *
     * @param  Request $request

Missing parameter comment

Expected 3 spaces after parameter type; 1 found

     * @param  JobPoster $jobPoster

Missing parameter comment

     * @return \Illuminate\Http\Response
     */
    public function batchUpdate(Request $request, JobPoster $jobPoster)
    {
        $newCriteria = collect($request->input()); // TODO: switch to validated
        $oldCriteria = $jobPoster->criteria;

        $updatedIds = [];

        // First, delete old criteria that weren't resubmitted, and update those that were.
        foreach ($oldCriteria as $criteria) {
            $newData = $newCriteria->firstWhere('id', $criteria['id']);
            if ($newData) {
                $updatedIds[] = $criteria->id;
                $this->updateCriteria($criteria, $newData);
            } else {
                $this->deleteCriteria($criteria);
            }
        }

        $isUnsaved = function ($criteria, $savedIds): bool {
            return !array_key_exists('id', $criteria) || !in_array($criteria['id'], $savedIds);
        };

        // Now, save any new criteria that remain.
        foreach ($newCriteria as $criteriaData) {
            if ($isUnsaved($criteriaData, $updatedIds)) {
                $criteria = new Criteria();
                $criteria->job_poster_id = $jobPoster->id;
                $fillableData = collect($criteriaData)->except(['id', 'job_poster_id'])->toArray();
                $criteria->fill($fillableData);
                $this->createCriteria($criteria);
            }
        }

        return CriteriaResource::collection($jobPoster->fresh()->criteria);
    }

    /**
     * Save a new criteria and create a notification.
     *
     * @param Criteria $criteria

Missing parameter comment

     * @return void

Function return type is void, but function contains return statement

     */
    protected function createCriteria(Criteria $criteria)
    {
        $criteria->save();
        $notification = $this->makeAssessmentPlanNotification(
            'CREATE',
            $criteria
        );
        $notification->save();

        return $criteria;
    }

    /**
     * Update an existing Job Criteria and create a notification if necessary.
     *
     * @param  \App\Models\Criteria $oldCriteria Existing Criteria.
     * @param  mixed[] $newData Updated version of the Criteria.

Expected 14 spaces after parameter type; 1 found

Expected 5 spaces after parameter name; 1 found

     * @return void
     */
    protected function updateCriteria(Criteria $oldCriteria, $newData): void

Type hint "array" missing for $newData

    {
        // We only need to create a notification when the non-descriptive fields change.
        if ($oldCriteria->skill_level_id != $newData['skill_level_id']
            || $oldCriteria->skill_id != $newData['skill_id']
        ) {
            $notification = $this->makeAssessmentPlanNotification(
                'UPDATE',
                $oldCriteria,
                $newData['skill_id'],
                $newData['skill_level_id'],
                $newData['criteria_type_id']
            );
            $notification->save();
        }
        // Get just the data that can be changed.
        $fillableData = collect($newData)->except(['id', 'job_poster_id'])->toArray();
        $oldCriteria->fill($fillableData);
        $oldCriteria->save();
    }

    /**
     * Delete existing Job Criteria and create a notification.
     *
     * @param  \App\Models\Criteria $criteria Incoming Criteria.
     * @return void
     */
    protected function deleteCriteria(Criteria $criteria): void
    {
        $notification = $notification = $this->makeAssessmentPlanNotification(

The assignment to $notification is dead and can be removed.

            'DELETE',
            $criteria
        );
        $notification->save();

        // Delete assessments related to this criteria.
        Assessment::where('criterion_id', $criteria->id)->delete();
        $criteria->delete();
    }

    /**
     * Create a new AssessmentPlanNotification for a modification to a Criteria
     *
     * @param  string               $type            Can be CREATE, UPDATE or DELETE.

Expected 14 spaces after parameter name; 12 found

     * @param  \App\Models\Criteria $criteria        The Criteria (the OLD criteria if updating or deleting)

Expected 10 spaces after parameter name; 8 found

Parameter comment must end with a full stop

     * @param  integer|null         $newSkillId      Only used for UPDATE type notifications.

Expected 8 spaces after parameter name; 6 found

     * @param  integer|null         $newSkillLevelId Only used for UPDATE type notifications.

Expected 3 spaces after parameter name; 1 found

     * @param  integer|null         $newCriteriaTypeId Only used for UPDATE type notifications.
     * @return \App\Models\AssessmentPlanNotification
     */
    protected function makeAssessmentPlanNotification(
        string $type,
        Criteria $criteria,
        $newSkillId = null,
        $newSkillLevelId = null,
        $newCriteriaTypeId = null
    ) {
        $notification = new AssessmentPlanNotification();
        $notification->job_poster_id = $criteria->job_poster_id;
        $notification->type = $type;
        $notification->criteria_id = $criteria->id;
        $notification->skill_id = $criteria->skill_id;
        $notification->criteria_type_id = $criteria->criteria_type_id;
        $notification->skill_level_id = $criteria->skill_level_id;
        $notification->skill_id_new = $newSkillId;
        $notification->skill_level_id_new = $newSkillLevelId;
        $notification->criteria_type_id_new = $newCriteriaTypeId;
        $notification->acknowledged = false;
        return $notification;
    }
}