Anax\Models\GetGeo::getGeo()

Complexity

Conditions	1
Paths	1

Size

Total Lines	28
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getGeo($ip)
    {
        global $di;

        $config = $di->get("configuration")->load("apikey.php");
        $access_key = $config["config"]["keyHolder"]["apiKey"];

        // Initialize CURL:
        $ch = curl_init('http://api.ipstack.com/'.$ip.'?access_key='.$access_key.'');
        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);

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?

        // Store the data:
        $json = 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?

        // Decode JSON response:
        $api_result = json_decode($json, true);

        // Output the "capital" object inside "location"
        $data = [
            "city" => $api_result['city'] ?? null,
            "country_name" => $api_result['country_name'] ?? null,
            "region_name" => $api_result['region_name'] ?? null,
            "longitude" => $api_result['longitude'] ?? null,
            "latitude" => $api_result['latitude'] ?? null
        ];

        return $data;
    }