Passed
Push — latest ( ba39d8...ca9086 )
by Colin
08:23
created

CommonMark/Parser/Block/HeadingParser.php (1 issue)

Labels
Severity
1
<?php
2
3
declare(strict_types=1);
4
5
/*
6
 * This file is part of the league/commonmark package.
7
 *
8
 * (c) Colin O'Dell <[email protected]>
9
 *
10
 * For the full copyright and license information, please view the LICENSE
11
 * file that was distributed with this source code.
12
 */
13
14
namespace League\CommonMark\Extension\CommonMark\Parser\Block;
15
16
use League\CommonMark\Extension\CommonMark\Node\Block\Heading;
17
use League\CommonMark\Node\Block\AbstractBlock;
18
use League\CommonMark\Parser\Block\AbstractBlockContinueParser;
19
use League\CommonMark\Parser\Block\BlockContinue;
20
use League\CommonMark\Parser\Block\BlockContinueParserInterface;
21
use League\CommonMark\Parser\Cursor;
22
use League\CommonMark\Parser\InlineParserEngineInterface;
23
24
final class HeadingParser extends AbstractBlockContinueParser
25
{
26
    /**
27
     * @var Heading
28
     *
29
     * @psalm-readonly
30
     */
31
    private $block;
32
33
    /** @var string */
34
    private $content;
35
36 207
    public function __construct(int $level, string $content)
37
    {
38 207
        $this->block   = new Heading($level);
39 207
        $this->content = $content;
40 207
    }
41
42 207
    public function getBlock(): AbstractBlock
43
    {
44 207
        return $this->block;
45
    }
46
47 126
    public function tryContinue(Cursor $cursor, BlockContinueParserInterface $activeBlockParser): ?BlockContinue
48
    {
49 126
        return BlockContinue::none();
1 ignored issue
show
Are you sure the usage of League\CommonMark\Parser...k\BlockContinue::none() targeting League\CommonMark\Parser...k\BlockContinue::none() seems to always return null.

This check looks for function or method calls that always return null and whose return value is used.

class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

The method getObject() can return nothing but null, so it makes no sense to use the return value.

The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.

Loading history...
50
    }
51
52 207
    public function parseInlines(InlineParserEngineInterface $inlineParser): void
53
    {
54 207
        $inlineParser->parse($this->content, $this->block);
55 207
    }
56
}
57