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

Completed

Pull Request — master (#1)

by Romain

created 2016-11-22 10:09 UTC

File::__construct() A

↳ Parent: File

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	8
nc	2
nop	3

<?php
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
     */
    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 $value
     * @return bool
     */
    private function isAttachmentId($value): bool
    {
        return preg_match('/^[\d]+$/', $value);
    }

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

        $json = parent::jsonSerialize();
        $json += [
            'payload' => array_filter($payload),
        ];

        return $json;
    }
}


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

ker0x / messenger

Pull Request — master (#1)

File::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like