exussum12\CoverageChecker\CodeClimateLoader

Complexity

Total Complexity

10

Size/Duplication

Total Lines	82
Duplicated Lines	10.98 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	handleNotFoundFile()	0	4	1
A	getDescription()	0	4	1
A	addError()	0	12	2
A	__construct()	0	5	1
A	parseLines()	0	8	2
A	getErrorsOnLine()	9	9	2
A	convertToJson()	0	6	1

<?php
namespace exussum12\CoverageChecker;

/**
 * Class CodeClimateLoader
 * Used for parsing reports in CodeClimate format
 * @package exussum12\CoverageChecker
 */
class CodeClimateLoader implements FileChecker
{
    /**
     * @var string
     */
    protected $file;

    /**
     * @var array
     */
    protected $errors = [];

    /**
     * @param string $file the path to the codeclimate file
     */
    public function __construct($file)
    {
        $json = $this->convertToJson(file_get_contents($file));
        $this->file = json_decode($json);
    }

    /**
     * {@inheritdoc}
     */
    public function parseLines()
    {
        foreach ($this->file as $line) {

The expression $this->file of type string is not traversable.

            $this->addError($line);
        }

        return array_keys($this->errors);
    }

    /**
     * {@inheritdoc}
     */
    public function getErrorsOnLine($file, $lineNumber)

This method seems to be duplicated in your project.

    {
        $errors = [];
        if (isset($this->errors[$file][$lineNumber])) {
            $errors = $this->errors[$file][$lineNumber];
        }

        return $errors;
    }

    /**
     * {@inheritdoc}
     */
    public function handleNotFoundFile()
    {
        return true;
    }

    /**
     * {@inheritdoc}
     */
    public static function getDescription()
    {
        return 'Parse codeclimate output';
    }

    private function addError($line)
    {
        $trim = './';
        $fileName = substr($line->location->path, strlen($trim));
        $start = $line->location->lines->begin;
        $end = $line->location->lines->end;
        $message = $line->description;

        for ($lineNumber = $start; $lineNumber <= $end; $lineNumber++) {
            $this->errors[$fileName][$lineNumber][] = $message;
        }
    }

    private function convertToJson($codeClimateFormat)
    {
        $codeClimateFormat = str_replace("\0", ',', $codeClimateFormat);

        return '[' . $codeClimateFormat . ']';
    }
}