HipChat::getMessageUrl() - Code Metrics - Inspection of "Add file sharing capability." - laravel-notification-channels/hipchat - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 820cfa...bfe298 )

unknown

created 2016-08-16 21:15 UTC

HipChat::getMessageUrl() A

↳ Parent: HipChat::shareFile()

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace NotificationChannels\HipChat;

use GuzzleHttp\Client as HttpClient;
use GuzzleHttp\Psr7\MultipartStream;
use GuzzleHttp\Psr7\Request;
use function GuzzleHttp\Psr7\modify_request;
use function GuzzleHttp\Psr7\stream_for;

class HipChat
{
    /** @var string */
    protected $token;

    /** @var HttpClient */
    protected $http;

    /** @var string */
    protected $url;

    /** @var string */
    protected $room;

    /**
     * @param HttpClient $http
     * @param string $token
     * @param string|null $url
     */
    public function __construct(HttpClient $http, $url, $token, $room)
    {
        $this->http = $http;
        $this->url = rtrim($url ?: 'https://api.hipchat.com', '/');
        $this->token = $token;
        $this->room = $room;
    }

    /**
     * Returns default room id or name.
     *
     * @return string
     */
    public function room()
    {
        return $this->room;
    }

    /**
     * Returns HipChat base url.
     *
     * @return string
     */
    public function url()
    {
        return $this->url;
    }

    /**
     * Send a message.
     *
     * @param string|int $to
     * @param array $message
     * @return \Psr\Http\Message\ResponseInterface
     */
    public function sendMessage($to, $message)
    {
        $url = $this->url.'/v2/room/'.urlencode($to).'/notification';

        return $this->post($url, [
            'headers' => $this->getHeaders(),
            'json' => $message,
        ]);
    }

    /**
     * Share a file.
     *
     * @param string|int $to
     * @param array $file
     * @return \Psr\Http\Message\ResponseInterface
     */
    public function shareFile($to, $file)
    {
        $parts[] = [
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            'headers' => [
                'Content-Type' => $file['file_type'] ?: 'application/octet-stream',
            ],
            'name' => 'file',
            'contents' => stream_for($file['content']),
            'filename' => $file['filename'] ?: 'untitled',
        ];

        if (!empty($file['message'])) {
            $parts[] = [
                'headers' => [
                    'Content-Type' => 'application/json',
                ],
                'name' => 'metadata',
                'contents' => json_encode(['message' => $file['message']]),
            ];
        }

        $url = $this->url.'/v2/room/'.urlencode($to).'/share/file';

        return $this->postMultipartRelated($url, [
            'headers' => $this->getHeaders(),
            'multipart' => $parts,
        ]);
    }

    /**
     * Make a simple post request.
     *
     * @param string $url
     * @param array $options
     * @return \Psr\Http\Message\ResponseInterface
     */
    protected function post($url, $options)
    {
        return $this->http->post($url, $options);
    }

    /**
     * Make a multipart/related request.
     * Unfortunately Guzzle doesn't support multipart/related requests out of the box.
     *
     * @param $url
     * @param $options
     * @return \Psr\Http\Message\ResponseInterface
     */
    protected function postMultipartRelated($url, $options)
    {
        $headers = isset($options['headers']) ? $options['headers'] : [];

        $body = new MultipartStream($options['multipart']);

        $version = isset($options['version']) ? $options['version'] : '1.1';

        $request = new Request('POST', $url, $headers, $body, $version);

        $changeContentType['set_headers']['Content-Type'] = 'multipart/related; boundary=' . $request->getBody()->getBoundary();
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

        $request = modify_request($request, $changeContentType);

        return $this->http->send($request);
    }

    /**
     * Get common request headers.
     *
     * @return array
     */
    protected function getHeaders()
    {
        return [
            'Authorization' => 'Bearer '.$this->token,
        ];
    }
}


1			<?php
2
3			namespace NotificationChannels\HipChat;
4
5			use GuzzleHttp\Client as HttpClient;
6			use GuzzleHttp\Psr7\MultipartStream;
7			use GuzzleHttp\Psr7\Request;
8			use function GuzzleHttp\Psr7\modify_request;
9			use function GuzzleHttp\Psr7\stream_for;
10
11			class HipChat
12			{
13			/** @var string */
14			protected $token;
15
16			/** @var HttpClient */
17			protected $http;
18
19			/** @var string */
20			protected $url;
21
22			/** @var string */
23			protected $room;
24
25			/**
26			* @param HttpClient $http
27			* @param string $token
28			* @param string\|null $url
29			*/
30			public function __construct(HttpClient $http, $url, $token, $room)
31			{
32			$this->http = $http;
33			$this->url = rtrim($url ?: 'https://api.hipchat.com', '/');
34			$this->token = $token;
35			$this->room = $room;
36			}
37
38			/**
39			* Returns default room id or name.
40			*
41			* @return string
42			*/
43			public function room()
44			{
45			return $this->room;
46			}
47
48			/**
49			* Returns HipChat base url.
50			*
51			* @return string
52			*/
53			public function url()
54			{
55			return $this->url;
56			}
57
58			/**
59			* Send a message.
60			*
61			* @param string\|int $to
62			* @param array $message
63			* @return \Psr\Http\Message\ResponseInterface
64			*/
65			public function sendMessage($to, $message)
66			{
67			$url = $this->url.'/v2/room/'.urlencode($to).'/notification';
68
69			return $this->post($url, [
70			'headers' => $this->getHeaders(),
71			'json' => $message,
72			]);
73			}
74
75			/**
76			* Share a file.
77			*
78			* @param string\|int $to
79			* @param array $file
80			* @return \Psr\Http\Message\ResponseInterface
81			*/
82			public function shareFile($to, $file)
83			{
84			$parts[] = [
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-08-16 21:19 UTC by Report Bug Copy Issue Report `$parts` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$parts = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
85			'headers' => [
86			'Content-Type' => $file['file_type'] ?: 'application/octet-stream',
87			],
88			'name' => 'file',
89			'contents' => stream_for($file['content']),
90			'filename' => $file['filename'] ?: 'untitled',
91			];
92
93			if (!empty($file['message'])) {
94			$parts[] = [
95			'headers' => [
96			'Content-Type' => 'application/json',
97			],
98			'name' => 'metadata',
99			'contents' => json_encode(['message' => $file['message']]),
100			];
101			}
102
103			$url = $this->url.'/v2/room/'.urlencode($to).'/share/file';
104
105			return $this->postMultipartRelated($url, [
106			'headers' => $this->getHeaders(),
107			'multipart' => $parts,
108			]);
109			}
110
111			/**
112			* Make a simple post request.
113			*
114			* @param string $url
115			* @param array $options
116			* @return \Psr\Http\Message\ResponseInterface
117			*/
118			protected function post($url, $options)
119			{
120			return $this->http->post($url, $options);
121			}
122
123			/**
124			* Make a multipart/related request.
125			* Unfortunately Guzzle doesn't support multipart/related requests out of the box.
126			*
127			* @param $url
128			* @param $options
129			* @return \Psr\Http\Message\ResponseInterface
130			*/
131			protected function postMultipartRelated($url, $options)
132			{
133			$headers = isset($options['headers']) ? $options['headers'] : [];
134
135			$body = new MultipartStream($options['multipart']);
136
137			$version = isset($options['version']) ? $options['version'] : '1.1';
138
139			$request = new Request('POST', $url, $headers, $body, $version);
140
141			$changeContentType['set_headers']['Content-Type'] = 'multipart/related; boundary=' . $request->getBody()->getBoundary();
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-08-16 21:19 UTC by Report Bug Copy Issue Report `$changeContentType` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$changeContentType = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
142
143			$request = modify_request($request, $changeContentType);
144
145			return $this->http->send($request);
146			}
147
148			/**
149			* Get common request headers.
150			*
151			* @return array
152			*/
153			protected function getHeaders()
154			{
155			return [
156			'Authorization' => 'Bearer '.$this->token,
157			];
158			}
159			}
160

laravel-notification-channels / hipchat

Push — master ( 820cfa...bfe298 )

HipChat::getMessageUrl() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like