App\Queue\UserMailJob

Complexity

Total Complexity

3

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	8	2
A	__construct()	0	4	1

<?php

namespace App\Queue;

use App\Facades\Mail;
use App\Model\User;
use Ronanchilvers\Foundation\Queue\Exception\FailedJobException;
use Ronanchilvers\Foundation\Queue\Job\Job;

/**
 * Base job for sending user emails
 *
 * @author Ronan Chilvers <ronan@d3r.com>
 */
abstract class UserMailJob extends Job
{
    /**
     * @var string
     */
    protected $queue = 'emails';

    /**
     * @var string
     */
    protected $emailClass = null;

    /**
     * @var App\Model\User

The type App\Queue\App\Model\User was not found. Did you mean App\Model\User? If so, make sure to prefix the type with \.

     */
    protected $user;

    /**
     * @var array
     */
    protected $internalProperties = [
        'delay',
        'queue',
        'emailClass'
    ];

    /**
     * Class constructor
     *
     * @author Ronan Chilvers <ronan@d3r.com>
     */
    public function __construct(
        User $user,
    ) {
        $this->user = $user;

It seems like $user of type App\Model\User is incompatible with the declared type App\Queue\App\Model\User of property $user.

    }

    /**
     * {@inheritdoc}
     *
     * @author Ronan Chilvers <ronan@d3r.com>
     */
    public function execute()
    {
        $class = $this->emailClass;
        $mail = new $class(
            $this->user
        );
        if (!Mail::send($mail)) {

The method send() does not exist on App\Facades\Mail. Since you implemented __callStatic, consider adding a @method annotation.

            throw new FailedJobException('Unable to send User email : ' . $class);
        }
    }
}