seregazhuk\PinterestBot\Helpers\CsrfHelper

Complexity

Total Complexity

7

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	6
Bugs	3	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getTokenFromFile()	0	15	4
A	_parseLineForToken()	0	13	3

<?php

namespace seregazhuk\PinterestBot\Helpers;

class CsrfHelper
{
    const TOKEN_NAME = 'csrftoken';

    /**
     * Get a CSRF token from the given cookie file
     *
     * @param string $file
     * @return string
     */
    public static function getTokenFromFile($file)
    {
        if ( ! file_exists($file)) {
            return null;
        }

        foreach (file($file) as $line) {

            if ($token = self::_parseLineForToken($line)) {
                return $token;
            }
        }

        return null;
    }

    /**
     * @param string $line
     * @return bool
     */
    protected static function _parseLineForToken($line)
    {
        if (empty(strstr($line, self::TOKEN_NAME))) {
            return false;
        }

        preg_match('/' . self::TOKEN_NAME . '\s(\w*)/', $line, $matches);
        if ($matches) {

The expression $matches of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $matches[1];
        }

        return false;
    }
}