FixMissingSemicolonSolution::generateProposedFileFromTokens() - Code Metrics - Inspection of "Automagically fix missing semicolons, uses tokeniz..." - facade/ignition - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#94)

by James

created 2019-11-09 11:50 UTC

generateProposedFileFromTokens() B

↳ Parent: FixMissingSemicolonSolution

Complexity

Conditions	10
Paths	17

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
nc	17
nop	3
dl	0
loc	24
rs	7.6666
c	0
b	0
f	0

How to fix Complexity

<?php

namespace Facade\Ignition\Solutions;

use Facade\IgnitionContracts\RunnableSolution;
use Illuminate\Support\Str;

class FixMissingSemicolonSolution implements RunnableSolution
{
    private $filePath;
    private $lineNumber;
    private $unexpected;

    public function __construct($filePath = null, $lineNumber = null, $unexpected = null)
    {
        $this->filePath = $filePath;
        $this->lineNumber = $lineNumber;
        $this->unexpected = $unexpected;
    }

    public function getSolutionTitle(): string
    {
        return 'You are missing a semicolon.';
    }

    public function getDocumentationLinks(): array
    {
        return [];
    }

    public function getSolutionActionDescription(): string
    {
        return 'Each instruction requires termination with a semicolon.';
    }

    public function getRunButtonText(): string
    {
        return 'Insert missing semicolon';
    }

    public function getSolutionDescription(): string
    {
        return '';
    }

    public function getRunParameters(): array
    {
        return [
            'filePath' => $this->filePath,
            'lineNumber' => $this->lineNumber,
            'unexpected' => $this->unexpected,
        ];
    }

    public function isRunnable()
    {
        return $this->insertSemicolon($this->getRunParameters()) !== false;
    }

    public function run(array $parameters = [])
    {
        $output = $this->insertSemicolon($parameters);
        if ($output !== false) {
            file_put_contents(app_path().$parameters['filePath'], $output);
        }
    }

    public function insertSemicolon(array $parameters = [])
    {
        if (strpos($parameters['filePath'], 'ignition/tests/stubs') !== false) {
            $file = $parameters['filePath'];
        } else {
            $file = app_path().$parameters['filePath'];
        }
        if (! is_file($file)) {
            return false;
        }
        $contents = file_get_contents($file);
        $tokens = token_get_all($contents);

        $proposedFix = $this->generateProposedFileFromTokens($tokens, $parameters['lineNumber'], $parameters['unexpected']);

        $result = exec(sprintf('echo %s | php -l', escapeshellarg($proposedFix)), $output, $exit);

        if (! Str::contains($result, 'No syntax errors')) {
            return false;
        }

        return $proposedFix;
    }

    protected function generateProposedFileFromTokens(array $tokens, int $lineNumber, string $unexpectedChar)
    {
        $reverseTokens = array_reverse($tokens);
        $output = [];
        $insertSemicolon = false;
        $line = 0;
        foreach ($reverseTokens as $token) {
            $char = isset($token[1]) ? $token[1] : $token;

            $output[] = $char;

            if (isset($token[2])) {
                $line = $token[2];
            }
            if (is_string($token) && $line == $lineNumber && $char == $unexpectedChar) {
                $insertSemicolon = true;
            }
            if ($insertSemicolon && isset($token[0]) && $token[0] == T_WHITESPACE) {
                $insertSemicolon = false;
                $output[] = ';';
            }
        }
        return implode('', array_reverse($output));
    }
}


1			<?php
2
3			namespace Facade\Ignition\Solutions;
4
5			use Facade\IgnitionContracts\RunnableSolution;
6			use Illuminate\Support\Str;
7
8			class FixMissingSemicolonSolution implements RunnableSolution
9			{
10			private $filePath;
11			private $lineNumber;
12			private $unexpected;
13
14			public function __construct($filePath = null, $lineNumber = null, $unexpected = null)
15			{
16			$this->filePath = $filePath;
17			$this->lineNumber = $lineNumber;
18			$this->unexpected = $unexpected;
19			}
20
21			public function getSolutionTitle(): string
22			{
23			return 'You are missing a semicolon.';
24			}
25
26			public function getDocumentationLinks(): array
27			{
28			return [];
29			}
30
31			public function getSolutionActionDescription(): string
32			{
33			return 'Each instruction requires termination with a semicolon.';
34			}
35
36			public function getRunButtonText(): string
37			{
38			return 'Insert missing semicolon';
39			}
40
41			public function getSolutionDescription(): string
42			{
43			return '';
44			}
45
46			public function getRunParameters(): array
47			{
48			return [
49			'filePath' => $this->filePath,
50			'lineNumber' => $this->lineNumber,
51			'unexpected' => $this->unexpected,
52			];
53			}
54
55			public function isRunnable()
56			{
57			return $this->insertSemicolon($this->getRunParameters()) !== false;
58			}
59
60			public function run(array $parameters = [])
61			{
62			$output = $this->insertSemicolon($parameters);
63			if ($output !== false) {
64			file_put_contents(app_path().$parameters['filePath'], $output);
65			}
66			}
67
68			public function insertSemicolon(array $parameters = [])
69			{
70			if (strpos($parameters['filePath'], 'ignition/tests/stubs') !== false) {
71			$file = $parameters['filePath'];
72			} else {
73			$file = app_path().$parameters['filePath'];
74			}
75			if (! is_file($file)) {
76			return false;
77			}
78			$contents = file_get_contents($file);
79			$tokens = token_get_all($contents);
80
81			$proposedFix = $this->generateProposedFileFromTokens($tokens, $parameters['lineNumber'], $parameters['unexpected']);
82
83			$result = exec(sprintf('echo %s \| php -l', escapeshellarg($proposedFix)), $output, $exit);
84
85			if (! Str::contains($result, 'No syntax errors')) {
86			return false;
87			}
88
89			return $proposedFix;
90			}
91
92			protected function generateProposedFileFromTokens(array $tokens, int $lineNumber, string $unexpectedChar)
93			{
94			$reverseTokens = array_reverse($tokens);
95			$output = [];
96			$insertSemicolon = false;
97			$line = 0;
98			foreach ($reverseTokens as $token) {
99			$char = isset($token[1]) ? $token[1] : $token;
100
101			$output[] = $char;
102
103			if (isset($token[2])) {
104			$line = $token[2];
105			}
106			if (is_string($token) && $line == $lineNumber && $char == $unexpectedChar) {
107			$insertSemicolon = true;
108			}
109			if ($insertSemicolon && isset($token[0]) && $token[0] == T_WHITESPACE) {
110			$insertSemicolon = false;
111			$output[] = ';';
112			}
113			}
114			return implode('', array_reverse($output));
115			}
116			}
117

facade / ignition

Pull Request — master (#94)

generateProposedFileFromTokens() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like