App\Utils\Candidates\CandidateDeleter

Complexity

Total Complexity

3

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	hashData()	0	5	1
A	deleteCandidate()	0	17	2

<?php

namespace App\Utils\Candidates;

use App\Events\CandidateDeleted;
use App\Http\Requests\CandidateDeleteRequest;
use App\Models\Candidate;
use Illuminate\Support\Facades\Storage;

class CandidateDeleter
{
    public static function deleteCandidate(CandidateDeleteRequest $request, $candidateId)
    {
        $candidate = Candidate::findOrFail($candidateId);

        $notificationEmailAddress = null;
        if ($request->get('notify_candidate')) {

The method get() does not exist on App\Http\Requests\CandidateDeleteRequest.

            $notificationEmailAddress = $candidate->email;
        }

        Storage::delete(storage_path('app/'.$candidate->path_to_cv));
        $candidate->path_to_cv = '';

The property path_to_cv does not seem to exist on Illuminate\Database\Eloquent\Collection.

        self::hashData($candidate);
        $candidate->save();

        $candidate->delete(); //safe since we use softdeletes

        event(new CandidateDeleted($candidate, $notificationEmailAddress));
    }

    protected static function hashData(Candidate $candidate)
    {
        $candidate->name = sha1($candidate->first_name);

The property first_name does not seem to exist on App\Models\Candidate. Are you sure there is no database migration missing?

        $candidate->email = sha1($candidate->email);
        $candidate->phone_number = sha1($candidate->phone_number);
    }
}