Jiad\Modules\GeoTag

Complexity

Total Complexity

1

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getGeoInfo()	0	14	1

<?php

namespace Jiad\Modules;

use Anax\Session\SessionInterface;

/**
 * REM Server using session to store information.
 */
class GeoTag
{
    /**
     * Get geo information about an IP address.
     *
     * @param string $ip string with IP to fetch geoinformation about.
     *
     * @return self
     */
    public function getGeoInfo(string $ip) : object
    {
        $key = require(ANAX_INSTALL_PATH . "/config/api_key.php");
        $api_key = $key["ipstack"];
        $url = "http://api.ipstack.com/$ip?access_key=$api_key";
        $ch = curl_init();

        curl_setopt($ch, CURLOPT_URL, $url);

It seems like $ch 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($ch, CURLOPT_HEADER, 0);
        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);

        $info = curl_exec($ch);

It seems like $ch 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($ch);

It seems like $ch 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 json_decode($info);
    }
}