Api::call() - Code Metrics - Inspection of "Remove unused assignment" - wlbrough/clearbit-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( aae8d7...38ea04 )

by Bill

created 2018-08-14 02:23 UTC

Api::call() A

↳ Parent: Api

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3.054

Importance

Changes

Metric	Value
dl	0
loc	17
ccs	9
cts	11
cp	0.8182
rs	9.7
c	0
b	0
f	0
cc	3
nc	4
nop	2
crap	3.054

<?php

namespace wlbrough\clearbit\Abstracts;

use GuzzleHttp\Client;
use wlbrough\clearbit\Exceptions\ApiException;

abstract class Api
{
    protected $endpointUrl = null;
    protected $useStreaming = false;

    public function setWebhookEndpoint($endpointUrl = null)
    {
        self::validateUrl($endpointUrl);
        $this->endpointUrl = $endpointUrl;
    }

    public function enableStreaming()
    {
        $this->useStreaming = true;
    }

    private static function validateUrl($url)
    {
        if (!is_string($url)) {
            throw new \InvalidArgumentException('Webhook endpoint is not a string');
        }

        $isValid = preg_match('#((https?|ftp)://(\S*?\.\S*?))([\s)\[\]{},;"\':<]|\.\s|$)#i', $url);

        if (!$isValid) {
            throw new \InvalidArgumentException('Webhook endpoint is not a URL');
        }

        return true;
    }

    protected static function call($url, $client)
    {
        if (!$client) {
            $client = new Client(['http_errors' => false]);
        }

        $response = $client->get($url);
        $status = $response->getStatusCode();

        if ($status === 200) {
            $returnData = json_decode($response->getBody());
        } else {
            throw new ApiException($status);
        }

        return $returnData;
    }
}


1		<?php
2
3		namespace wlbrough\clearbit\Abstracts;
4
5		use GuzzleHttp\Client;
6		use wlbrough\clearbit\Exceptions\ApiException;
7
8		abstract class Api
9		{
10		protected $endpointUrl = null;
11		protected $useStreaming = false;
12
13	27	public function setWebhookEndpoint($endpointUrl = null)
14		{
15	27	self::validateUrl($endpointUrl);
16	12	$this->endpointUrl = $endpointUrl;
17	12	}
18
19	3	public function enableStreaming()
20		{
21	3	$this->useStreaming = true;
22	3	}
23
24	27	private static function validateUrl($url)
25		{
26	27	if (!is_string($url)) {
27	9	throw new \InvalidArgumentException('Webhook endpoint is not a string');
28		}
29
30	18	$isValid = preg_match('#((https?\|ftp)://(\S?\.\S?))([\s)\[\]{},;"\':<]\|\.\s\|$)#i', $url);
31
32	18	if (!$isValid) {
33	6	throw new \InvalidArgumentException('Webhook endpoint is not a URL');
34		}
35
36	12	return true;
37		}
38
39	18	protected static function call($url, $client)
40		{
41	18	if (!$client) {
42		$client = new Client(['http_errors' => false]);
43		}
44
45	18	$response = $client->get($url);
46	18	$status = $response->getStatusCode();
47
48	18	if ($status === 200) {
49	9	$returnData = json_decode($response->getBody());
50	6	} else {
51	9	throw new ApiException($status);
52		}
53
54	9	return $returnData;
55		}
56		}
57

wlbrough / clearbit-api

GitHub Access Token became invalid

Push — master ( aae8d7...38ea04 )

Api::call() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like