Curl::send() - Code Metrics - Inspection of "Added support for web hooks" - waltertamboer/communicator - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bd9b49...e48eea )

by Walter

created 2017-06-09 10:55 UTC

Curl::send() A

↳ Parent: Curl

Complexity

Conditions	1
Paths	1

Size

Total Lines	21
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	21
rs	9.3142
c	0
b	0
f	0
cc	1
eloc	10
nc	1
nop	1

<?php
/**
 * Communicator (https://github.com/waltertamboer/communicator)
 *
 * @link https://github.com/waltertamboer/communicator for the canonical source repository
 * @copyright Copyright (c) 2017 Communicator (https://github.com/waltertamboer/communicator)
 * @license https://github.com/waltertamboer/communicator/blob/master/LICENSE.md MIT
 */

namespace Communicator\Transport\Webhook\Adapter;

use Communicator\Transport\Webhook\SendData;

/**
 * A curl adapter.
 */
final class Curl implements AdapterInterface
{
    /**
     * Sends a message over the transport.
     *
     * @param SendData $data The data that should be send in the request.
     * @return bool Returns true when the call succeeded; false otherwise.
     */
    public function send(SendData $data): bool
    {
        // @codeCoverageIgnoreStart

        $ch = curl_init();

        curl_setopt($ch, CURLOPT_URL, "example.com");
        curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
        curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);

        curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
        curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);

        $output = curl_exec($ch);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        curl_close($ch);

        // @codeCoverageIgnoreEnd

        return true;
    }
}


1			<?php
2			/**
3			* Communicator (https://github.com/waltertamboer/communicator)
4			*
5			* @link https://github.com/waltertamboer/communicator for the canonical source repository
6			* @copyright Copyright (c) 2017 Communicator (https://github.com/waltertamboer/communicator)
7			* @license https://github.com/waltertamboer/communicator/blob/master/LICENSE.md MIT
8			*/
9
10			namespace Communicator\Transport\Webhook\Adapter;
11
12			use Communicator\Transport\Webhook\SendData;
13
14			/**
15			* A curl adapter.
16			*/
17			final class Curl implements AdapterInterface
18			{
19			/**
20			* Sends a message over the transport.
21			*
22			* @param SendData $data The data that should be send in the request.
23			* @return bool Returns true when the call succeeded; false otherwise.
24			*/
25			public function send(SendData $data): bool
26			{
27			// @codeCoverageIgnoreStart
28
29			$ch = curl_init();
30
31			curl_setopt($ch, CURLOPT_URL, "example.com");
32			curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
33			curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
34
35			curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
36			curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
37
38			$output = curl_exec($ch);
			0 ignored issues – show Unused Code introduced 2017-06-09 11:08 UTC by Report Bug Copy Issue Report `$output` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
39
40			curl_close($ch);
41
42			// @codeCoverageIgnoreEnd
43
44			return true;
45			}
46			}
47

waltertamboer / communicator

Push — master ( bd9b49...e48eea )

Curl::send() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like