App\Listeners\JobPublished

Complexity

Total Complexity

7

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	handle()	0	10	6

<?php

Missing file doc comment

namespace App\Listeners;

use App\Events\JobSaved;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Support\Facades\Log;

class JobPublished

Missing doc comment for class JobPublished

{
    /**
     * Create the event listener.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Handle the event.
     *
     * @param  JobSaved  $event

Missing parameter comment

Expected 1 spaces after parameter type; 2 found

     * @return void

Tag cannot be grouped with parameter tags in a doc comment

     */
    public function handle(JobSaved $event)
    {
        $job = $event->job;

        //If job has just been created, log if its being published now
        //If job is being modified, only want to log when it goes from unpublished to published
        if ( ($job->wasRecentlyCreated && $job->published) ||
                (!$job->wasRecentlyCreated && $job->published && !$job->getOriginal('published'))) {

Multi-line IF statement not indented correctly; expected 12 spaces but found 16

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

            Log::notice('Job published: job {id='.$job->id."} published by manager {id=".$job->manager->id.", email=".$job->manager->user->email."}");
        }
    }
}