App\Http\Controllers\Api\JobController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	109
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	12	1
A	submitForReview()	0	19	3
A	__construct()	0	4	1
A	store()	0	11	1
A	destroy()	0	2	1
A	index()	0	2	1
A	show()	0	4	1

<?php

namespace App\Http\Controllers\Api;

use App\Http\Controllers\Controller;
use App\Http\Requests\StoreJobPoster;
use App\Http\Requests\UpdateJobPoster;
use App\Mail\JobPosterReviewRequested;
use App\Models\JobPoster;
use App\Models\Lookup\JobTerm;
use Illuminate\Http\Resources\Json\JsonResource;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Log;
use Illuminate\Support\Facades\Mail;
use Jenssegers\Date\Date;

class JobController extends Controller
{
    /**
     * Class constructor
     */
    public function __construct()
    {
        // This applies the appropriate policy to each resource route.
        $this->authorizeResource(JobPoster::class, 'job');
    }

    /**
     * Display a listing of the resource.
     *
     * @return void
     */
    public function index()
    {
        // TODO: complete.
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \App\Http\Requests\StoreJobPoster $request Incoming request.
     * @return \Illuminate\Http\Response
     */
    public function store(StoreJobPoster $request)
    {
        $data = $request->validated();
        $job = new JobPoster();
        $job->manager_id = $request->user()->manager->id;
        // Defaulting JPB created Jobs to monthly terms for now.
        $job->job_term_id = JobTerm::where('name', 'month')->value('id');
        $job->fill($data);
        $job->save();
        $job->load('criteria');
        return new JsonResource($job);
    }

    /**
     * Display the specified resource.
     *
     * @param  \App\Models\JobPoster $job Incoming Job Poster.
     * @return \Illuminate\Http\Response
     */
    public function show(JobPoster $job)
    {
        $job->load('criteria');
        return new JsonResource($job);
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \App\Http\Requests\UpdateJobPoster $request Validates input.
     * @param  \App\Models\JobPoster              $job     Incoming Job Poster.
     * @return \Illuminate\Http\Response
     */
    public function update(UpdateJobPoster $request, JobPoster $job)
    {
        $data = $request->validated();
        // Only values both in the JobPoster->fillable array,
        // and returned by UpdateJobPoster->validatedData(), will be set.
        $job->fill($data);
        // Defaulting JPB updated jobs to monthly for now.
        $job->job_term_id = JobTerm::where('name', 'month')->value('id');
        $job->save();
        $job->fresh();
        $job->load('criteria');
        return new JsonResource($job);
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  integer $id Job Poster ID.
     * @return void
     */
    public function destroy(int $id)

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

    {
        // TODO: complete.
    }

    /**
     * Submit the Job Poster for review.
     *
     * @param  \App\Models\JobPoster $job Job Poster object.
     * @return \Illuminate\Http\Response
     */
    public function submitForReview(JobPoster $job)
    {
        // Check to avoid submitting for review multiple times.
        if ($job->review_requested_at === null) {
            // Update review request timestamp.
            $job->review_requested_at = new Date();
            $job->save();

            // Send email.
            $reviewer_email = config('mail.reviewer_email');
            if (isset($reviewer_email)) {
                Mail::to($reviewer_email)->send(new JobPosterReviewRequested($job, Auth::user()));

It seems like Illuminate\Support\Facades\Auth::user() can also be of type null; however, parameter $manager of App\Mail\JobPosterReviewRequested::__construct() does only seem to accept App\Models\User, maybe add an additional type check?

            } else {
                Log::error('The reviewer email environment variable is not set.');
            }
        }
        $job->load('criteria');

        return new JsonResource($job);
    }
}