itstructure / yii2-template-simple

helpers/BaseHelper.php

<?php

namespace app\helpers;

/**
 * Class BaseHelper
 *
 * @package app\helpers
 */
class BaseHelper
{
    /**
     * @param string $date
     * @param string $format
     *
     * @return string
     */
    public static function getDateAt(string $date, string $format = 'd.m.Y H:i:s'): string
    {
        $inDateTime = new \DateTime($date);

        return date($format, $inDateTime->getTimestamp());
    }

    /**
     * @return string
     */
    public static function ip_address()
    {
        $fields = array('HTTP_CLIENT_IP', 'HTTP_X_REAL_IP', 'HTTP_X_FORWARDED_FOR', 'REMOTE_ADDR');

        $ip = "0.0.0.0";
        foreach($fields as $k)
        {
            if(!empty($_SERVER[$k]) && ip2long($_SERVER[$k]) != false)

It seems like you are loosely comparing ip2long($_SERVER[$k]) of type integer to the boolean false. If you are specifically checking for non-zero, consider using something more explicit like > 0 or !== 0 instead.

            {
                $ip = $_SERVER[$k];
            }
        }

        return $ip;
    }
}