ddlzz\AmoAPI\Auth

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	isAuthenticated()	0	16	4
A	__construct()	0	4	1

<?php


namespace ddlzz\AmoAPI;


/**
 * Class Auth
 * @package ddlzz\AmoAPI
 * @author ddlzz
 */
class Auth
{
    /** @var string */
    private $login;

    /** @var string */
    private $cookiePath;

    /**
     * Auth constructor.
     * @param string $login
     * @param string $cookiePath
     */
    public function __construct($login, $cookiePath)
    {
        $this->login = $login;
        $this->cookiePath = $cookiePath;
    }

    /**
     * Checks if the cookie file exists and if it's valid.
     * @return bool
     */
    public function isAuthenticated()
    {
        $cookie = $this->cookiePath;
        $cookieLifetime = time() - 60 * 14; // 14 minutes

        if ((!file_exists($cookie)) || (filemtime($cookie) <= $cookieLifetime)) {
            return false;
        }

        // If login has been changed, we need to delete the cookie file for the changes to take effect
        if (false === (strpos(file_get_contents($cookie), (str_replace('@', '%40', $this->login))))) {

It seems like file_get_contents($cookie) can also be of type false; however, parameter $haystack of strpos() does only seem to accept string, maybe add an additional type check?

            unlink($cookie);
            return false;
        }

        return true;
    }
}