BaseVisitor::getAbsoluteFilePath() - Code Metrics - Inspection of "Deal with help in symfony forms (#113)" - php-translation/extractor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ba40ee...261a9c )

by Tobias

created 2018-06-24 19:18 UTC

BaseVisitor::getAbsoluteFilePath() A

↳ Parent: BaseVisitor

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	4
ccs	2
cts	2
cp	1
rs	10
c	1
b	0
f	0
cc	1
nc	1
nop	0
crap	1

<?php

/*
 * This file is part of the PHP Translation package.
 *
 * (c) PHP Translation team <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Translation\Extractor\Visitor;

use Doctrine\Common\Annotations\DocParser;
use PhpParser\Node;
use Symfony\Component\Finder\SplFileInfo;
use Translation\Extractor\Annotation\Desc;
use Translation\Extractor\Annotation\Ignore;
use Translation\Extractor\Model\Error;
use Translation\Extractor\Model\SourceCollection;
use Translation\Extractor\Model\SourceLocation;

/**
 * Base class for any visitor.
 *
 * @author Tobias Nyholm <[email protected]>
 */
abstract class BaseVisitor implements Visitor
{
    /**
     * @var SourceCollection
     */
    protected $collection;

    /**
     * @var SplFileInfo
     */
    protected $file;

    /**
     * @var DocParser
     */
    private $docParser;

    public function init(SourceCollection $collection, SplFileInfo $file)
    {
        $this->collection = $collection;
        $this->file = $file;
    }

    protected function getAbsoluteFilePath()
    {
        return $this->file->getRealPath();
    }

    /**
     * @param Node   $node
     * @param string $errorMessage
     */
    protected function addError(Node $node, $errorMessage)
    {
        $docComment = $node->getDocComment();
        $file = $this->getAbsoluteFilePath();

        if (property_exists($node, 'value')) {
            $line = $node->value->getAttribute('startLine');
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        } else {
            $line = $node->getAttribute('startLine');
        }
        if (null !== $docComment) {
            $context = 'file '.$file.' near line '.$line;
            foreach ($this->getDocParser()->parse($docComment->getText(), $context) as $annotation) {
                if ($annotation instanceof Ignore) {
                    return;
                }
            }
        }

        $this->collection->addError(new Error($errorMessage, $file, $line));
    }

    /**
     * @param string    $text
     * @param int       $line
     * @param Node|null $node
     * @param array     $context
     */
    protected function addLocation($text, $line, Node $node = null, array $context = [])
    {
        if (null === $location = $this->getLocation($text, $line, $node, $context)) {
            return;
        }

        $this->collection->addLocation($location);
    }

    /**
     * @param string    $text
     * @param int       $line
     * @param Node|null $node
     * @param array     $context
     *
     * @return SourceLocation|null
     */
    protected function getLocation($text, $line, Node $node = null, array $context = [])
    {
        $file = $this->getAbsoluteFilePath();
        if (null !== $node && null !== $docComment = $node->getDocComment()) {
            $parserContext = 'file '.$file.' near line '.$line;
            foreach ($this->getDocParser()->parse($docComment->getText(), $parserContext) as $annotation) {
                if ($annotation instanceof Ignore) {
                    return null;
                } elseif ($annotation instanceof Desc) {
                    $context['desc'] = $annotation->text;
                }
            }
        }

        return new SourceLocation($text, $file, $line, $context);
    }

    /**
     * @return DocParser
     */
    private function getDocParser()
    {
        if (null === $this->docParser) {
            $this->docParser = new DocParser();

            $this->docParser->setImports([
                'ignore' => Ignore::class,
                'desc' => Desc::class,
            ]);
            $this->docParser->setIgnoreNotImportedAnnotations(true);
        }

        return $this->docParser;
    }

    /**
     * @param DocParser $docParser
     */
    public function setDocParser(DocParser $docParser)
    {
        $this->docParser = $docParser;
    }
}


1		<?php
2
3		/*
4		* This file is part of the PHP Translation package.
5		*
6		* (c) PHP Translation team <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Translation\Extractor\Visitor;
13
14		use Doctrine\Common\Annotations\DocParser;
15		use PhpParser\Node;
16		use Symfony\Component\Finder\SplFileInfo;
17		use Translation\Extractor\Annotation\Desc;
18		use Translation\Extractor\Annotation\Ignore;
19		use Translation\Extractor\Model\Error;
20		use Translation\Extractor\Model\SourceCollection;
21		use Translation\Extractor\Model\SourceLocation;
22
23		/**
24		* Base class for any visitor.
25		*
26		* @author Tobias Nyholm <[email protected]>
27		*/
28		abstract class BaseVisitor implements Visitor
29		{
30		/**
31		* @var SourceCollection
32		*/
33		protected $collection;
34
35		/**
36		* @var SplFileInfo
37		*/
38		protected $file;
39
40		/**
41		* @var DocParser
42		*/
43		private $docParser;
44
45	32	public function init(SourceCollection $collection, SplFileInfo $file)
46		{
47	32	$this->collection = $collection;
48	32	$this->file = $file;
49	32	}
50
51	29	protected function getAbsoluteFilePath()
52		{
53	29	return $this->file->getRealPath();
54		}
55
56		/**
57		* @param Node $node
58		* @param string $errorMessage
59		*/
60	8	protected function addError(Node $node, $errorMessage)
61		{
62	8	$docComment = $node->getDocComment();
63	8	$file = $this->getAbsoluteFilePath();
64
65	8	if (property_exists($node, 'value')) {
66	6	$line = $node->value->getAttribute('startLine');
		0 ignored issues – show Bug introduced 2017-03-11 13:56 UTC by Report Bug Copy Issue Report Accessing `value` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
67		} else {
68	2	$line = $node->getAttribute('startLine');
69		}
70	8	if (null !== $docComment) {
71	3	$context = 'file '.$file.' near line '.$line;
72	3	foreach ($this->getDocParser()->parse($docComment->getText(), $context) as $annotation) {
73	3	if ($annotation instanceof Ignore) {
74	3	return;
75		}
76		}
77		}
78
79	8	$this->collection->addError(new Error($errorMessage, $file, $line));
80	8	}
81
82		/**
83		* @param string $text
84		* @param int $line
85		* @param Node\|null $node
86		* @param array $context
87		*/
88	9	protected function addLocation($text, $line, Node $node = null, array $context = [])
89		{
90	9	if (null === $location = $this->getLocation($text, $line, $node, $context)) {
91		return;
92		}
93
94	9	$this->collection->addLocation($location);
95	9	}
96
97		/**
98		* @param string $text
99		* @param int $line
100		* @param Node\|null $node
101		* @param array $context
102		*
103		* @return SourceLocation\|null
104		*/
105	15	protected function getLocation($text, $line, Node $node = null, array $context = [])
106		{
107	15	$file = $this->getAbsoluteFilePath();
108	15	if (null !== $node && null !== $docComment = $node->getDocComment()) {
109	3	$parserContext = 'file '.$file.' near line '.$line;
110	3	foreach ($this->getDocParser()->parse($docComment->getText(), $parserContext) as $annotation) {
111	3	if ($annotation instanceof Ignore) {
112	2	return null;
113	2	} elseif ($annotation instanceof Desc) {
114	2	$context['desc'] = $annotation->text;
115		}
116		}
117		}
118
119	15	return new SourceLocation($text, $file, $line, $context);
120		}
121
122		/**
123		* @return DocParser
124		*/
125	5	private function getDocParser()
126		{
127	5	if (null === $this->docParser) {
128	5	$this->docParser = new DocParser();
129
130	5	$this->docParser->setImports([
131	5	'ignore' => Ignore::class,
132		'desc' => Desc::class,
133		]);
134	5	$this->docParser->setIgnoreNotImportedAnnotations(true);
135		}
136
137	5	return $this->docParser;
138		}
139
140		/**
141		* @param DocParser $docParser
142		*/
143		public function setDocParser(DocParser $docParser)
144		{
145		$this->docParser = $docParser;
146		}
147		}
148

php-translation / extractor

Push — master ( ba40ee...261a9c )

BaseVisitor::getAbsoluteFilePath() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like