Anax\Models\GeoApi::findGeoLocation()

Complexity

Conditions	1
Paths	1

Size

Total Lines	28
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function findGeoLocation($ipAdress)
    {
        global $di;

        // get the secret api key
        $config = $di->get("configuration")->load("api_keys.php") ?? null;
        $apiKey = $config["config"]["ipStack"]["apiKey"] ?? null;

        // make curl api call with ip address and api key
        $ch1 = curl_init('http://api.ipstack.com/'.$ipAdress.'?access_key='.$apiKey.'');
        curl_setopt($ch1, CURLOPT_RETURNTRANSFER, true);

It seems like $ch1 can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        $json = curl_exec($ch1);

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

It seems like $ch1 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 the json result
         * for usage in both view and rest-api controller
         */
        $result = json_decode($json, true);

        $data = [
            "city" => $result['city'] ?? "-",
            "country_name" => $result['country_name'] ?? "-",
            "longitude" => $result['longitude'] ?? "-",
            "latitude" => $result['latitude'] ?? "-"
        ];

        return $data;
    }