Deel18 / weather-module

src/Ip/IPChecker.php

<?php
/**
 * Ip Checker, a class supporting IP validation.
 */

namespace Anax\Ip;

use Anax\Commons\ContainerInjectableInterface;
use Anax\Commons\ContainerInjectableTrait;

class IPChecker
{

    use ContainerInjectableTrait;

    public function checkIpv($ipv)
    {
        if (filter_var($ipv, FILTER_VALIDATE_IP, FILTER_FLAG_IPV6)) {
            $hostname = gethostbyaddr($ipv);
            return "$ipv is a valid IPv6 address and the host is: $hostname";
        } elseif (filter_var($ipv, FILTER_VALIDATE_IP)) {
            $hostname = gethostbyaddr($ipv);
            return "$ipv is a valid IPv4 address and the host is: $hostname";
        } else {
            return "$ipv is not a valid IP address";
        }
    }

    public function geoTag($ipv)
    {
        $apiKey = require ANAX_INSTALL_PATH . "/config/apikey.php";
        $apiKey = $apiKey["geotag"];
        $curl = curl_init();

        curl_setopt($curl, CURLOPT_URL, "http://api.ipstack.com/$ipv?access_key=$apiKey");

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

        curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);

        $result = curl_exec($curl);

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

        $decode = json_decode($result);

        return $decode;
    }

    public function verifyGeo($latitude, $longitude)
    {
        $validLat = preg_match('/^(\+|-)?(?:90(?:(?:\.0{1,6})?)|(?:[0-9]|[1-8][0-9])(?:(?:\.[0-9]{1,6})?))$/', $latitude);
        $validLon = preg_match('/^(\+|-)?(?:90(?:(?:\.0{1,6})?)|(?:[0-9]|[1-8][0-9])(?:(?:\.[0-9]{1,6})?))$/', $longitude);

        if (!$validLat && !$validLon) {
            return false;
        } else {
            return true;
        }
    }
}