Coco\SourceWatcher\Core\Api\ApiReader

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setEndpoint()	0	3	1
A	read()	0	19	3
A	__construct()	0	2	1
A	getEndpoint()	0	3	1

<?php

namespace Coco\SourceWatcher\Core\Api;

use Coco\SourceWatcher\Core\SourceWatcherException;

class ApiReader implements Reader
{
    /**
     * @var string
     */
    protected string $endpoint;

    /**
     * ApiReader constructor.
     */
    public function __construct ()
    {

    }

    /**
     * @return string
     */
    public function getEndpoint () : string
    {
        return $this->endpoint;
    }

    /**
     * @param string $endpoint
     */
    public function setEndpoint ( string $endpoint ) : void
    {
        $this->endpoint = $endpoint;
    }

    /**
     * @return bool|string
     * @throws SourceWatcherException
     */
    public function read ()
    {
        if ( $this->endpoint == null || $this->endpoint == "" ) {
            throw new SourceWatcherException( "No endpoint found." );
        }

        $curl = curl_init();

        $timeout = 5;

        curl_setopt( $curl, CURLOPT_URL, $this->endpoint );

It seems like $curl can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        curl_setopt( $curl, CURLOPT_RETURNTRANSFER, 1 );
        curl_setopt( $curl, CURLOPT_CONNECTTIMEOUT, $timeout );

        $response = curl_exec( $curl );

It seems like $curl can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        curl_close( $curl );

It seems like $curl can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        return $response;
    }
}