Afrittella\BackProject\Listeners\SendRegistrationEmail

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	4	4	1
A	handle()	10	10	1
A	failed()	4	4	1

<?php

namespace Afrittella\BackProject\Listeners;

use Afrittella\BackProject\Events\UserRegistered;
use Illuminate\Contracts\Queue\ShouldQueue;
use Afrittella\BackProject\Repositories\Users;
use Illuminate\Support\Facades\Log;
use Afrittella\BackProject\Notifications\RegistrationEmail;

class SendRegistrationEmail implements ShouldQueue

This class seems to be duplicated in your project.

{
    protected $users;
    /**
     * Create the event listener.
     *
     */
    public function __construct(Users $users)
    {
        $this->users = $users;
    }

    /**
     * Handle the event.
     *
     * @param UserRegistered $event
     * @return void
     * @internal param $Event =UserRegistered  $event
     */
    public function handle(UserRegistered $event)
    {
        //try {
        $user = $this->users->findBy('id', $event->user_id);
        $user->notify(new RegistrationEmail($user));
        //} catch(\Exception $e) {

60% of this comment could be valid code. Did you maybe forget this after debugging?

        //Log::error('lavoro fallito');

72% of this comment could be valid code. Did you maybe forget this after debugging?

        //}

    }

    public function failed(UserRegistered $event, $exception)

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

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

    {
        Log::error('fallito ');
    }
}