Facade\Ignition\Solutions\SuggestCorrectVariableNameSolution

Complexity

Total Complexity

16

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getSolutionTitle()	0	4	1
A	getDocumentationLinks()	0	4	1
A	getSolutionActionDescription()	0	9	1
A	getRunButtonText()	0	4	1
A	getSolutionDescription()	0	4	1
A	getRunParameters()	0	8	1
A	isRunnable()	0	4	1
A	run()	0	7	2
A	fixTypo()	0	17	2
A	generateExpectedTokens()	0	10	4

<?php

namespace Facade\Ignition\Solutions;

use Illuminate\Support\Facades\Blade;
use Facade\IgnitionContracts\RunnableSolution;

class SuggestCorrectVariableNameSolution implements RunnableSolution
{
    private $variableName;
    private $viewFile;

    public function __construct($variableName = null, $viewFile = null, $suggested = null)
    {
        $this->variableName = $variableName;
        $this->viewFile = $viewFile;
        $this->suggested = $suggested;

The property suggested does not exist. Did you maybe forget to declare it?

    }

    public function getSolutionTitle(): string
    {
        return 'Possible typo $'.$this->variableName;
    }

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

    public function getSolutionActionDescription(): string
    {
        $path = str_replace(base_path().'/', '', $this->viewFile);

$path is not used, you could remove the assignment.

        $output = [
            'Did you mean `$'.$this->suggested.'`?',
        ];

        return implode(PHP_EOL, $output);
    }

    public function getRunButtonText(): string
    {
        return 'Fix typo';
    }

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

    public function getRunParameters(): array
    {
        return [
            'variableName' => $this->variableName,
            'viewFile' => $this->viewFile,
            'suggested' => $this->suggested,
        ];
    }

    public function isRunnable(array $parameters = [])

The parameter $parameters is not used and could be removed.

    {
        return $this->fixTypo($this->getRunParameters()) !== false;
    }

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

    protected function fixTypo(array $parameters = [])
    {
        // Make sure suggested variable is valid alpha-numeric with underscore, or return false
        if (! preg_match('/^[a-zA-Z]+[a-zA-Z0-9_]+$/', $parameters['suggested'])) {
            return false;
        }

        $originalContents = file_get_contents($parameters['viewFile']);
        $newContents = str_replace('$'.$parameters['variableName'], '$'.$parameters['suggested'], $originalContents);

        $originalTokens = token_get_all(Blade::compileString($originalContents));
        $newTokens = token_get_all(Blade::compileString($newContents));

$newTokens is not used, you could remove the assignment.

        $expectedTokens = $this->generateExpectedTokens($originalTokens, $parameters['variableName'], $parameters['suggested']);

$expectedTokens is not used, you could remove the assignment.

        return $newContents;
    }

    protected function generateExpectedTokens(array $originalTokens, string $variableName, string $suggested): array
    {
        $expectedTokens = $originalTokens;
        foreach ($expectedTokens as $key => $token) {
            if ($token[0] === T_VARIABLE && $token[1] === '$'.$variableName) {
                $expectedTokens[$key][1] = '$'.$suggested;
            }
        }
        return $expectedTokens;
    }
}