SmscRuChannel::sendMessage() - Code Metrics - Inspection of "add support for old version laravel" - laravel-notification-channels/smsc-ru - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#11)

by Evgeny

created 2016-10-25 06:52 UTC

SmscRuChannel::sendMessage() A

↳ Parent: SmscRuChannel

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2.0054

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	8
cts	9
cp	0.8889
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	8
nc	2
nop	2
crap	2.0054

<?php

namespace NotificationChannels\SmscRu;

use Illuminate\Notifications\Notification;
use NotificationChannels\SmscRu\Exceptions\CouldNotSendNotification;

class SmscRuChannel
{
    /** @var \NotificationChannels\SmscRu\SmscRuApi */
    protected $smsc;

    public function __construct(SmscRuApi $smsc)
    {
        $this->smsc = $smsc;
    }

    /**
     * Send the given notification.
     *
     * @param  mixed  $notifiable
     * @param  \Illuminate\Notifications\Notification  $notification
     *
     * @throws  \NotificationChannels\SmscRu\Exceptions\CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        $to = $notifiable->routeNotificationFor('smscru');

        if (empty($to)) {
            throw CouldNotSendNotification::missingRecipient();
        }

        $message = $notification->toSmscRu($notifiable);

        if (is_string($message)) {
            $message = new SmscRuMessage($message);
        }

        $this->sendMessage($to, $message);
    }

    protected function sendMessage($recipient, SmscRuMessage $message)
    {
        if (mb_strlen($message->content) > 800) {
            throw CouldNotSendNotification::contentLengthLimitExceeded();
        }

        $params = [
            'phones'  => $recipient,
            'mes'     => $message->content,
            'sender'  => $message->from,
        ];

        $this->smsc->send($params);
    }
}


1		<?php
2
3		namespace NotificationChannels\SmscRu;
4
5		use Illuminate\Notifications\Notification;
6		use NotificationChannels\SmscRu\Exceptions\CouldNotSendNotification;
7
8		class SmscRuChannel
9		{
10		/** @var \NotificationChannels\SmscRu\SmscRuApi */
11		protected $smsc;
12
13	2	public function __construct(SmscRuApi $smsc)
14		{
15	2	$this->smsc = $smsc;
16	2	}
17
18		/**
19		* Send the given notification.
20		*
21		* @param mixed $notifiable
22		* @param \Illuminate\Notifications\Notification $notification
23		*
24		* @throws \NotificationChannels\SmscRu\Exceptions\CouldNotSendNotification
25		*/
26	2	public function send($notifiable, Notification $notification)
27		{
28	2	$to = $notifiable->routeNotificationFor('smscru');
29
30	2	if (empty($to)) {
31	1	throw CouldNotSendNotification::missingRecipient();
32		}
33
34	1	$message = $notification->toSmscRu($notifiable);
35
36	1	if (is_string($message)) {
37		$message = new SmscRuMessage($message);
38		}
39
40	1	$this->sendMessage($to, $message);
41	1	}
42
43	1	protected function sendMessage($recipient, SmscRuMessage $message)
44		{
45	1	if (mb_strlen($message->content) > 800) {
46		throw CouldNotSendNotification::contentLengthLimitExceeded();
47		}
48
49		$params = [
50	1	'phones' => $recipient,
51	1	'mes' => $message->content,
52	1	'sender' => $message->from,
53	1	];
54
55	1	$this->smsc->send($params);
56	1	}
57		}
58

laravel-notification-channels / smsc-ru

GitHub Access Token became invalid

Pull Request — master (#11)

SmscRuChannel::sendMessage() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like