ApnConnection::open() - Code Metrics - Inspection of "Break APNS feedback out of the channel" - laravel-notification-channels/apn - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#35)

by Dwight

created 2018-02-09 04:53 UTC

ApnConnection::open() A

↳ Parent: ApnConnection

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace NotificationChannels\Apn;

use ZendService\Apple\Apns\Message;
use ZendService\Apple\Apns\Client\Message as Client;
use NotificationChannels\Apn\Exceptions\ConnectionFailed;

class ApnConnection
{
    /**
     * The sandbox environment identifier.
     *
     * @var int
     */
    const SANDBOX = 0;

    /**
     * The production environment identifier.
     *
     * @var int
     */
    const PRODUCTION = 1;

    /**
     * The APNS client.
     *
     * @var \ZendService\Apple\Apns\Client\Message
     */
    protected $client;

    /**
     * Create a new connection instance.
     *
     * @param  \ZendService\Apple\Apns\Client\Message  $client
     * @return void

     */
    public function __construct(Client $client)
    {
        $this->client = $client;
    }

    /**
     * Open a new connection.
     *
     * @param  string  $environment
     * @param  string  $certificate
     * @param  string  $passPhrase
     * @return void
     * @throws \NotificationChannels\Apn\Exceptions\ConnectionFailed
     */
    public function open($environment, $certificate, $passPhrase = null)
    {
        try {
            $this->client->open($environment, $certificate, $passPhrase);
        } catch (Exception $exception) {

            throw ConnectionFailed::create($exception);
        }
    }

    /**
     * Send the provided packet over the existing connection.
     *
     * @param  \ZendService\Apple\Apns\Message  $message
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     * @return \ZendService\Apple\Apns\Response\Message
     */
    public function send(Message $packet)
    {
        return $this->client->send($packet);
    }

    /**
     * Close the existing connection.
     *
     * @return void
     */
    public function close()
    {
        $this->client->close();
    }
}


1			<?php
2
3			namespace NotificationChannels\Apn;
4
5			use ZendService\Apple\Apns\Message;
6			use ZendService\Apple\Apns\Client\Message as Client;
7			use NotificationChannels\Apn\Exceptions\ConnectionFailed;
8
9			class ApnConnection
10			{
11			/**
12			* The sandbox environment identifier.
13			*
14			* @var int
15			*/
16			const SANDBOX = 0;
17
18			/**
19			* The production environment identifier.
20			*
21			* @var int
22			*/
23			const PRODUCTION = 1;
24
25			/**
26			* The APNS client.
27			*
28			* @var \ZendService\Apple\Apns\Client\Message
29			*/
30			protected $client;
31
32			/**
33			* Create a new connection instance.
34			*
35			* @param \ZendService\Apple\Apns\Client\Message $client
36			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2018-02-09 04:54 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
37			*/
38			public function __construct(Client $client)
39			{
40			$this->client = $client;
41			}
42
43			/**
44			* Open a new connection.
45			*
46			* @param string $environment
47			* @param string $certificate
48			* @param string $passPhrase
49			* @return void
50			* @throws \NotificationChannels\Apn\Exceptions\ConnectionFailed
51			*/
52			public function open($environment, $certificate, $passPhrase = null)
53			{
54			try {
55			$this->client->open($environment, $certificate, $passPhrase);
56			} catch (Exception $exception) {
			0 ignored issues – show Bug introduced 2018-02-09 04:54 UTC by Report Bug Copy Issue Report The class `NotificationChannels\Apn\Exception` does not exist. Did you forget a USE statement, or did you not list all dependencies? Scrutinizer analyzes your `composer.json`/`composer.lock` file if available to determine the classes, and functions that are defined by your dependencies. It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis. Loading history...
57			throw ConnectionFailed::create($exception);
58			}
59			}
60
61			/**
62			* Send the provided packet over the existing connection.
63			*
64			* @param \ZendService\Apple\Apns\Message $message
			0 ignored issues – show Bug introduced 2018-02-09 04:54 UTC by Report Bug Copy Issue Report There is no parameter named `$message`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
65			* @return \ZendService\Apple\Apns\Response\Message
66			*/
67			public function send(Message $packet)
68			{
69			return $this->client->send($packet);
70			}
71
72			/**
73			* Close the existing connection.
74			*
75			* @return void
76			*/
77			public function close()
78			{
79			$this->client->close();
80			}
81			}
82

laravel-notification-channels / apn

GitHub Access Token became invalid

Pull Request — master (#35)

ApnConnection::open() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like