Lioo19\Models\IpGeo::fetchGeo()

Complexity

Conditions	1
Paths	1

Size

Total Lines	22
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function fetchGeo($url = "http://api.ipstack.com/")
    {
        $this->curl = curl_init();
        $apikey = require ANAX_INSTALL_PATH . "/config/apikeys.php";
        $accesskey = $apikey["ipstack"];
        //sets the url for curl to the correct one
        curl_setopt($this->curl, CURLOPT_URL, "$url" . $this->ipinput . "?access_key=" . $accesskey);

It seems like $this->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?

        //returns a string
        curl_setopt($this->curl, CURLOPT_RETURNTRANSFER, 1);
        //execute the started curl-session
        $output = curl_exec($this->curl);

It seems like $this->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?

        $exploded = json_decode($output, true);
        $data = [
            "country" => $exploded["country_name"],
            "city" => $exploded["city"],
            "latitude" => $exploded["latitude"],
            "longitude" => $exploded["longitude"],
        ];
        //close curl-session to free up space
        curl_close($this->curl);

It seems like $this->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 $data;
    }