Overtrue\PHPLint\Process\Lint

Complexity

Total Complexity

18

Size/Duplication

Total Lines	105
Duplicated Lines	67.62 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	hasSyntaxError()	10	10	3
A	getSyntaxError()	10	10	2
A	parseError()	15	15	4
A	hasSyntaxIssue()	11	11	3
A	getSyntaxIssue()	10	10	2
A	parseIssue()	15	15	4

<?php

/*
 * This file is part of the overtrue/phplint
 *
 * (c) overtrue <i@overtrue.me>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Overtrue\PHPLint\Process;

use Symfony\Component\Process\Process;

/**
 * Class Lint.
 */
class Lint extends Process
{
    /**
     * @return bool
     */
    public function hasSyntaxError()

This method seems to be duplicated in your project.

    {
        $output = trim($this->getOutput());

        if (defined('HHVM_VERSION') && empty($output)) {
            return false;
        }

        return false === strpos($output, 'No syntax errors detected');
    }

    /**
     * @return bool|array
     */
    public function getSyntaxError()

This method seems to be duplicated in your project.

    {
        if ($this->hasSyntaxError()) {
            $out = explode("\n", trim($this->getOutput()));

            return $this->parseError(array_shift($out));
        }

        return false;
    }

    /**
     * Parse error message.
     *
     * @param string $message
     *
     * @return array
     */
    public function parseError($message)

This method seems to be duplicated in your project.

    {
        $pattern = '/^(PHP\s+)?(Parse|Fatal) error:\s*(?:\w+ error,\s*)?(?<error>.+?)\s+in\s+.+?\s*line\s+(?<line>\d+)/';

        $matched = preg_match($pattern, $message, $match);

        if (empty($message)) {
            $message = 'Unknown';
        }

        return [
            'error' => $matched ? "{$match['error']} in line {$match['line']}" : $message,
            'line' => $matched ? abs($match['line']) : 0,
        ];
    }

    /**
     * @return bool
     */
    public function hasSyntaxIssue()

This method seems to be duplicated in your project.

    {
        $output = trim($this->getOutput());

        if (defined('HHVM_VERSION') && empty($output)) {
            return false;
        }


        return (bool)preg_match('/(Warning:|Deprecated:|Notice:)/', $output);
    }

    /**
     * @return bool|array
     */
    public function getSyntaxIssue()

This method seems to be duplicated in your project.

    {
        if ($this->hasSyntaxIssue()) {
            $out = explode("\n", trim($this->getOutput()));

            return $this->parseIssue(array_shift($out));
        }

        return false;
    }

    /**
     * Parse error message.
     *
     * @param string $message
     *
     * @return array
     */
    private function parseIssue($message)

This method seems to be duplicated in your project.

    {
        $pattern = '/^(PHP\s+)?(Warning|Deprecated|Notice):\s*?(?<error>.+?)\s+in\s+.+?\s*line\s+(?<line>\d+)/';

        $matched = preg_match($pattern, $message, $match);

        if (empty($message)) {
            $message = 'Unknown';
        }

        return [
            'error' => $matched ? "{$match['error']} in line {$match['line']}" : $message,
            'line' => $matched ? abs($match['line']) : 0,
        ];
    }
}