MergeConflictSolutionProvider::getSolutions() - Code Metrics - Inspection of "Merge pull request #58 from parallax/feature/merge..." - facade/ignition - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f1f5ae...d6cfaa )

by Freek

created 2019-09-02 21:05 UTC

MergeConflictSolutionProvider::getSolutions() A

↳ Parent: MergeConflictSolutionProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.8333
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

namespace Facade\Ignition\SolutionProviders;

use Throwable;
use ParseError;
use Illuminate\Support\Str;
use Facade\IgnitionContracts\BaseSolution;
use Facade\IgnitionContracts\HasSolutionsForThrowable;
use Symfony\Component\Debug\Exception\FatalThrowableError;

class MergeConflictSolutionProvider implements HasSolutionsForThrowable
{
    public function canSolve(Throwable $throwable): bool
    {
        if (! ($throwable instanceof FatalThrowableError || $throwable instanceof ParseError)) {
            return false;
        }

        if (! Str::startsWith($throwable->getMessage(), 'syntax error, unexpected \'<<\'')) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            return false;
        }

        $file = file_get_contents($throwable->getFile());
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        if (strpos($file, '=======') === false) {
            return false;
        }

        if (strpos($file, '>>>>>>>') === false) {
            return false;
        }

        return true;
    }

    public function getSolutions(Throwable $throwable): array
    {
        $file = file_get_contents($throwable->getFile());
        preg_match('/\>\>\>\>\>\>\> (.*?)\n/', $file, $matches);
        $source = $matches[1];

        $target = $this->getCurrentBranch(basename($throwable->getFile()));

        return [
            BaseSolution::create("Merge conflict from branch '$source' into $target")
                ->setSolutionDescription('You have a Git merge conflict. To undo your merge do `git reset --hard HEAD`'),
        ];
    }

    private function getCurrentBranch($folder)
    {
        $branch = "'".trim(`cd $folder; git branch | grep \* | cut -d ' ' -f2`)."'";

        if (! isset($branch) || $branch === "''") {
            $branch = 'current branch';
        }

        return $branch;
    }
}


1			<?php
2
3			namespace Facade\Ignition\SolutionProviders;
4
5			use Throwable;
6			use ParseError;
7			use Illuminate\Support\Str;
8			use Facade\IgnitionContracts\BaseSolution;
9			use Facade\IgnitionContracts\HasSolutionsForThrowable;
10			use Symfony\Component\Debug\Exception\FatalThrowableError;
11
12			class MergeConflictSolutionProvider implements HasSolutionsForThrowable
13			{
14			public function canSolve(Throwable $throwable): bool
15			{
16			if (! ($throwable instanceof FatalThrowableError \|\| $throwable instanceof ParseError)) {
17			return false;
18			}
19
20			if (! Str::startsWith($throwable->getMessage(), 'syntax error, unexpected \'<<\'')) {
			0 ignored issues – show Bug introduced 2019-09-02 17:01 UTC by Report Bug Copy Issue Report The method `getMessage` does only exist in `ParseError`, but not in `Symfony\Component\Debug\...ion\FatalThrowableError`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
21			return false;
22			}
23
24			$file = file_get_contents($throwable->getFile());
			0 ignored issues – show Bug introduced 2019-09-02 17:01 UTC by Report Bug Copy Issue Report The method `getFile` does only exist in `ParseError`, but not in `Symfony\Component\Debug\...ion\FatalThrowableError`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
25
26			if (strpos($file, '=======') === false) {
27			return false;
28			}
29
30			if (strpos($file, '>>>>>>>') === false) {
31			return false;
32			}
33
34			return true;
35			}
36
37			public function getSolutions(Throwable $throwable): array
38			{
39			$file = file_get_contents($throwable->getFile());
40			preg_match('/\>\>\>\>\>\>\> (.*?)\n/', $file, $matches);
41			$source = $matches[1];
42
43			$target = $this->getCurrentBranch(basename($throwable->getFile()));
44
45			return [
46			BaseSolution::create("Merge conflict from branch '$source' into $target")
47			->setSolutionDescription('You have a Git merge conflict. To undo your merge do `git reset --hard HEAD`'),
48			];
49			}
50
51			private function getCurrentBranch($folder)
52			{
53			$branch = "'".trim(`cd $folder; git branch \| grep \* \| cut -d ' ' -f2`)."'";
54
55			if (! isset($branch) \|\| $branch === "''") {
56			$branch = 'current branch';
57			}
58
59			return $branch;
60			}
61			}
62

facade / ignition

Push — master ( f1f5ae...d6cfaa )

MergeConflictSolutionProvider::getSolutions() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like