Coco\SourceWatcher\Core\Api\ApiReader::read()

Complexity

Conditions	3
Paths	2

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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;
    }