Lioo19\Models\IpGeo

Complexity

Total Complexity

2

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

17.65%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setInput()	0	3	1
A	fetchGeo()	0	22	1

<?php
namespace Lioo19\Models;

/**
 * Class for findning coordinates and place with ip
 *
 */
class IpGeo
{
    /**
    * @var string $ipinput   userinputted ip
    * @var string $curl      curl-object
    */
    public $ipinput;
    private $curl;

    /**
     * Constructor to assign user input and address to use
     *
     * @param null|string    $ipinp  User input
     */
    public function setInput(string $ipinp = "")
    {
        $this->ipinput = $ipinp;
    }

    /**
    * Method for retriving the geo-coordinates for given ip-address
    * @return array With parts of valid JSON-repsonse
    */
    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;
    }
}