File::__construct() - Code Metrics - Inspection of "Feature/messenger 2.2" - ker0x/messenger - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#83)

by Romain

created 2017-12-21 11:22 UTC

File::__construct() A

↳ Parent: File

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	6
cts	6
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	7
nc	2
nop	3
crap	2

<?php

declare(strict_types=1);

namespace Kerox\Messenger\Model\Message\Attachment;

use Kerox\Messenger\Model\Message\Attachment;

class File extends Attachment
{
    /**
     * @var null|string
     */
    protected $url;

    /**
     * @var null|bool
     */
    protected $reusable;

    /**
     * @var null|string
     */
    protected $attachmentId;

    /**
     * File constructor.
     *
     * @param string    $url
     * @param bool|null $reusable
     * @param string    $type
     *
     * @throws \InvalidArgumentException
     */
    public function __construct($url, ?bool $reusable = null, $type = Attachment::TYPE_FILE)
    {
        parent::__construct($type);

        if ($this->isAttachmentId($url)) {
            $this->attachmentId = $url;
        } else {
            $this->isValidUrl($url);
            $this->url = $url;
        }

        $this->reusable = $reusable;
    }

    /**
     * @param string    $url
     * @param bool|null $reusable
     *
     * @throws \InvalidArgumentException
     *
     * @return \Kerox\Messenger\Model\Message\Attachment\File
     */
    public static function create($url, ?bool $reusable = null): self
    {
        return new self($url, $reusable);
    }

    /**
     * @param $value
     *
     * @return bool
     */
    private function isAttachmentId($value): bool
    {
        return (bool) preg_match('/^[\d]+$/', $value);
    }

    /**
     * @return array
     */
    public function toArray(): array
    {
        $array = parent::toArray();
        $array += [
            'payload' => [
                'url'           => $this->url,
                'is_reusable'   => $this->reusable,
                'attachment_id' => $this->attachmentId,
            ],
        ];

        return $this->arrayFilter($array);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Kerox\Messenger\Model\Message\Attachment;
6
7		use Kerox\Messenger\Model\Message\Attachment;
8
9		class File extends Attachment
10		{
11		/**
12		* @var null\|string
13		*/
14		protected $url;
15
16		/**
17		* @var null\|bool
18		*/
19		protected $reusable;
20
21		/**
22		* @var null\|string
23		*/
24		protected $attachmentId;
25
26		/**
27		* File constructor.
28		*
29		* @param string $url
30		* @param bool\|null $reusable
31	6	* @param string $type
32		*
33	6	* @throws \InvalidArgumentException
34		*/
35	6	public function __construct($url, ?bool $reusable = null, $type = Attachment::TYPE_FILE)
36	1	{
37		parent::__construct($type);
38	5
39	5	if ($this->isAttachmentId($url)) {
40		$this->attachmentId = $url;
41		} else {
42	6	$this->isValidUrl($url);
43	6	$this->url = $url;
44		}
45
46		$this->reusable = $reusable;
47		}
48
49		/**
50	6	* @param string $url
51		* @param bool\|null $reusable
52	6	*
53		* @throws \InvalidArgumentException
54		*
55		* @return \Kerox\Messenger\Model\Message\Attachment\File
56		*/
57		public static function create($url, ?bool $reusable = null): self
58	6	{
59		return new self($url, $reusable);
60	6	}
61
62		/**
63	6	* @param $value
64	6	*
65	6	* @return bool
66		*/
67		private function isAttachmentId($value): bool
68		{
69	6	return (bool) preg_match('/^[\d]+$/', $value);
70		}
71
72		/**
73		* @return array
74		*/
75		public function toArray(): array
76		{
77		$array = parent::toArray();
78		$array += [
79		'payload' => [
80		'url' => $this->url,
81		'is_reusable' => $this->reusable,
82		'attachment_id' => $this->attachmentId,
83		],
84		];
85
86		return $this->arrayFilter($array);
87		}
88		}
89

ker0x / messenger

Pull Request — master (#83)

File::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like