Da\Mailer\Queue\Backend\Sqs\SqsMailJob

Complexity

Total Complexity

13

Size/Duplication

Total Lines	107
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

37.04%

Importance

Changes	5
Bugs	0	Features	1

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	isNewRecord()	0	4	2
A	getReceiptHandle()	0	4	1
A	setReceiptHandle()	0	4	1
A	getDelaySeconds()	0	4	1
A	setDelaySeconds()	0	7	3
A	getVisibilityTimeout()	0	4	1
A	setVisibilityTimeout()	0	4	1
A	getDeleted()	0	4	1
A	setDeleted()	0	4	1

<?php
namespace Da\Mailer\Queue\Backend\Sqs;

use BadMethodCallException;
use Da\Mailer\Event\EventHandlerTrait;
use Da\Mailer\Model\MailJob;
use Da\Mailer\Model\MailMessage;

class SqsMailJob extends MailJob
{
    use EventHandlerTrait;
    /**
     * @var string
     */
    private $receiptHandle;
    /**
     * @var MailMessage|string the message to store
     */
    private $message;

Consider using a different property name as you override a private property of the parent class.

The property $message is not used and could be removed.

    /**
     * @var int
     */
    private $delaySeconds;
    /**
     * @var int between 0 and 900 seconds
     */
    private $visibilityTimeout;
    /**
     * @var bool
     */
    private $deleted = false;

    /**
     * {@inheritdoc}
     */
    public function __construct(array $config = [])
    {
        parent::__construct($config);
    }

    /**
     * @return bool
     */
    public function isNewRecord()
    {
        return $this->getId() === null || $this->receiptHandle === null;
    }

    /**
     * @return string
     */
    public function getReceiptHandle()
    {
        return $this->receiptHandle;
    }

    /**
     * @param string $receiptHandle
     */
    public function setReceiptHandle($receiptHandle)
    {
        $this->receiptHandle = $receiptHandle;
    }

    /**
     * @return int
     */
    public function getDelaySeconds()
    {
        return $this->delaySeconds;
    }

    /**
     * @param int $delaySeconds
     */
    public function setDelaySeconds($delaySeconds)
    {
        if ($delaySeconds < 0 || $delaySeconds > 900) {
            throw new BadMethodCallException('Delay seconds must be between 0 and 900 seconds interval');
        }
        $this->delaySeconds = $delaySeconds;
    }

    /**
     * @return int
     */
    public function getVisibilityTimeout()
    {
        return $this->visibilityTimeout;
    }

    /**
     * @param int $visibilityTimeout
     */
    public function setVisibilityTimeout($visibilityTimeout)
    {
        $this->visibilityTimeout = $visibilityTimeout;
    }

    /**
     * @return bool
     */
    public function getDeleted()
    {
        return $this->deleted;
    }

    /**
     * @param bool $deleted
     */
    public function setDeleted($deleted)
    {
        $this->deleted = $deleted;
    }
}