FortySixElksMedia::__construct() - Code Metrics - Inspection of "Feat/rebase sms options" - laravel-notification-channels/46elks - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#12)

unknown

created 2022-09-25 13:48 UTC

FortySixElksMedia::__construct() A

↳ Parent: FortySixElksMedia

Complexity

Conditions	3
Paths	2

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	10
c	1
b	0
f	0
dl	0
loc	16
rs	9.9332
cc	3
nc	2
nop	0

<?php

namespace NotificationChannels\FortySixElks;

use GuzzleHttp\Client;
use NotificationChannels\FortySixElks\Exceptions\MissingConfigNotification;

class FortySixElksMedia
{
    /**
     * @var string
     */
    const ENDPOINT = null;
    /**
     * @var array
     */
    protected $payload = [
        'lines'   => [],
    ];

    /**
     * @var int
     */
    protected $phone_number;
    /**
     * @var string
     */
    protected $from;

    /**
     * @var string
     */
    protected $username;
    /**
     * @var string
     */
    protected $password;

    /**
     * FortySixElksMedia constructor.
     */
    public function __construct()
    {
        $this->username = config('services.46elks.username');
        $this->password = config('services.46elks.password');
        if (!$this->username || !$this->password) {
            throw MissingConfigNotification::missingConfig();
        }

        $this->client = new Client(

            [
                'headers' => [
                    'Content-Type' => 'application/x-www-urlencoded',
                ],
                'auth'    => [
                    $this->username,
                    $this->password,
                ],
            ]
        );
    }

    /**
     * @param $line
     *
     * @return $this
     */
    public function line($line)
    {
        $this->payload['lines'][] = $line;

        return $this;
    }

    /**
     * @param $phone_number
     *
     * @return $this
     */
    public function to($phoneNumber)
    {
        $this->phone_number = $phoneNumber;

        return $this;
    }

    /**
     * @param $string
     *
     * @return $this
     */
    public function from($string)
    {
        $this->from = $string;

        return $this;
    }

    /**
     * @return string
     */
    public function getContent()
    {
        return implode(PHP_EOL, $this->payload['lines']);
    }
}


1			<?php
2
3			namespace NotificationChannels\FortySixElks;
4
5			use GuzzleHttp\Client;
6			use NotificationChannels\FortySixElks\Exceptions\MissingConfigNotification;
7
8			class FortySixElksMedia
9			{
10			/**
11			* @var string
12			*/
13			const ENDPOINT = null;
14			/**
15			* @var array
16			*/
17			protected $payload = [
18			'lines' => [],
19			];
20
21			/**
22			* @var int
23			*/
24			protected $phone_number;
25			/**
26			* @var string
27			*/
28			protected $from;
29
30			/**
31			* @var string
32			*/
33			protected $username;
34			/**
35			* @var string
36			*/
37			protected $password;
38
39			/**
40			* FortySixElksMedia constructor.
41			*/
42			public function __construct()
43			{
44			$this->username = config('services.46elks.username');
45			$this->password = config('services.46elks.password');
46			if (!$this->username \|\| !$this->password) {
47			throw MissingConfigNotification::missingConfig();
48			}
49
50			$this->client = new Client(
			0 ignored issues – show Bug Best Practice introduced 2019-10-04 13:05 UTC by Report Bug Copy Issue Report The property `client` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
51			[
52			'headers' => [
53			'Content-Type' => 'application/x-www-urlencoded',
54			],
55			'auth' => [
56			$this->username,
57			$this->password,
58			],
59			]
60			);
61			}
62
63			/**
64			* @param $line
65			*
66			* @return $this
67			*/
68			public function line($line)
69			{
70			$this->payload['lines'][] = $line;
71
72			return $this;
73			}
74
75			/**
76			* @param $phone_number
77			*
78			* @return $this
79			*/
80			public function to($phoneNumber)
81			{
82			$this->phone_number = $phoneNumber;
83
84			return $this;
85			}
86
87			/**
88			* @param $string
89			*
90			* @return $this
91			*/
92			public function from($string)
93			{
94			$this->from = $string;
95
96			return $this;
97			}
98
99			/**
100			* @return string
101			*/
102			public function getContent()
103			{
104			return implode(PHP_EOL, $this->payload['lines']);
105			}
106			}
107

laravel-notification-channels / 46elks

Pull Request — master (#12)

FortySixElksMedia::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like