MailerPipe::process() - Code Metrics - Inspection of "Merge branch 'pipeline'" - alexmpunkt/mail - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 38a62f...ed2944 )

by Alex

created 2017-07-03 17:32 UTC

MailerPipe::process() A

↳ Parent: MailerPipe

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	15
c	0
b	0
f	0
ccs	8
cts	8
cp	1
rs	9.4285
cc	3
eloc	9
nc	3
nop	2
crap	3

<?php

namespace Conversio\Mail\Pipeline\Pipe;

use Conversio\Mail\Mail;
use Conversio\Mail\Mailer\MailerInterface;
use Conversio\Mail\Pipeline\MailPipeInterface;
use Conversio\Mail\Pipeline\ProcessResult;

/**
 * Class MailerPipe
 * @package Conversio\Mail\Pipeline\Pipe
 */
final class MailerPipe implements MailPipeInterface
{
    /**
     * @var MailerInterface
     */
    private $mailer;

    /**
     * MailerPipe constructor.
     *
     * @param MailerInterface $mailer
     */
    public function __construct(MailerInterface $mailer)
    {
        $this->mailer = $mailer;
    }

    /**
     * @param Mail          $mail
     * @param ProcessResult $result
     *
     * @return ProcessResult
     */
    public function process(Mail $mail, ProcessResult $result): ProcessResult
    {
        if ($result->errored()) {
            return $result;
        }

        if ($this->mailer->send($mail)) {
            $result->setStatus(ProcessResult::SUCCEEDED);
        } else {
            $result->setStatus(ProcessResult::FAILED);
            $result->addInfo(self::class, $this->mailer->getErrorInfo());
        }

        return $result;
    }
}

1		<?php
2
3		namespace Conversio\Mail\Pipeline\Pipe;
4
5		use Conversio\Mail\Mail;
6		use Conversio\Mail\Mailer\MailerInterface;
7		use Conversio\Mail\Pipeline\MailPipeInterface;
8		use Conversio\Mail\Pipeline\ProcessResult;
9
10		/**
11		* Class MailerPipe
12		* @package Conversio\Mail\Pipeline\Pipe
13		*/
14		final class MailerPipe implements MailPipeInterface
15		{
16		/**
17		* @var MailerInterface
18		*/
19		private $mailer;
20
21		/**
22		* MailerPipe constructor.
23		*
24		* @param MailerInterface $mailer
25		*/
26	1	public function __construct(MailerInterface $mailer)
27		{
28	1	$this->mailer = $mailer;
29	1	}
30
31		/**
32		* @param Mail $mail
33		* @param ProcessResult $result
34		*
35		* @return ProcessResult
36		*/
37	1	public function process(Mail $mail, ProcessResult $result): ProcessResult
38		{
39	1	if ($result->errored()) {
40	1	return $result;
41		}
42
43	1	if ($this->mailer->send($mail)) {
44	1	$result->setStatus(ProcessResult::SUCCEEDED);
45		} else {
46	1	$result->setStatus(ProcessResult::FAILED);
47	1	$result->addInfo(self::class, $this->mailer->getErrorInfo());
48		}
49
50	1	return $result;
51		}
52		}

alexmpunkt / mail

GitHub Access Token became invalid

Push — master ( 38a62f...ed2944 )

MailerPipe::process() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like