EmojiParser::parse() - Code Metrics - Inspection of "Discuss Forum" - XetaIO/Xetaravel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#34)

by Fèvre

created 2017-07-11 09:00 UTC

EmojiParser::parse() C

↳ Parent: EmojiParser

Complexity

Conditions	7
Paths	5

Size

Total Lines	51
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	51
rs	6.9743
c	0
b	0
f	0
cc	7
eloc	29
nc	5
nop	1

How to fix Long Method

<?php
namespace Xetaravel\Markdown\Emoji;

use League\CommonMark\Inline\Element\Image;
use League\CommonMark\Inline\Parser\AbstractInlineParser;
use League\CommonMark\InlineParserContext;

class EmojiParser extends AbstractInlineParser
{
    /**
     * The emoji mappings.
     *
     * @var array
     */
    protected $map;

    /**
     * The emoji extension.
     *
     * @var string
     */
    protected $ext = '.png';

    /**
     * The emoji path directory.
     *
     * @var string
     */
    protected $path;

    /**
     * Create a new emoji parser instance.
     *
     * @return void

     */
    public function __construct()
    {
        $this->map = include __DIR__ . "/EmojiSheet.php";
        $this->path = config('app.url') . '/images/emojis/';
    }

    /**
     * Get the characters that must be matched.
     *
     * @return array
     */
    public function getCharacters()
    {
        return [':'];
    }

    /**
     * Parse a line and determine if it contains an emoji. If it does,
     * then we do the necessary.
     *
     * @param \League\CommonMark\InlineParserContext $inlineContext
     *
     * @return bool
     */
    public function parse(InlineParserContext $inlineContext)
    {
        $cursor = $inlineContext->getCursor();

        $previous = $cursor->peek(-1);
        if ($previous !== null && $previous !== ' ') {
            return false;
        }

        $saved = $cursor->saveState();

        $cursor->advance();

        $handle = $cursor->match('/^[a-z0-9\+\-_]+:/');

        if (!$handle) {
            $cursor->restoreState($saved);

            return false;
        }

        $next = $cursor->peek(0);

        if ($next !== null && $next !== ' ') {
            $cursor->restoreState($saved);

            return false;
        }

        $key = substr($handle, 0, -1);

        if (!in_array($key, $this->map)) {
            $cursor->restoreState($saved);

            return false;
        }

        $fileName = $this->path . $key . $this->ext;

        $inline = new Image($fileName, $key);
        $inline->data['attributes'] = [
            'class' => 'emoji',
            'data-emoji' => $key,
            'width' => '24',
            'height' => '24',
            'title' => ':' . $key . ':'
        ];
        $inlineContext->getContainer()->appendChild($inline);

        return true;
    }
}


1			<?php
2			namespace Xetaravel\Markdown\Emoji;
3
4			use League\CommonMark\Inline\Element\Image;
5			use League\CommonMark\Inline\Parser\AbstractInlineParser;
6			use League\CommonMark\InlineParserContext;
7
8			class EmojiParser extends AbstractInlineParser
9			{
10			/**
11			* The emoji mappings.
12			*
13			* @var array
14			*/
15			protected $map;
16
17			/**
18			* The emoji extension.
19			*
20			* @var string
21			*/
22			protected $ext = '.png';
23
24			/**
25			* The emoji path directory.
26			*
27			* @var string
28			*/
29			protected $path;
30
31			/**
32			* Create a new emoji parser instance.
33			*
34			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2017-05-03 18:29 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
35			*/
36			public function __construct()
37			{
38			$this->map = include __DIR__ . "/EmojiSheet.php";
39			$this->path = config('app.url') . '/images/emojis/';
40			}
41
42			/**
43			* Get the characters that must be matched.
44			*
45			* @return array
46			*/
47			public function getCharacters()
48			{
49			return [':'];
50			}
51
52			/**
53			* Parse a line and determine if it contains an emoji. If it does,
54			* then we do the necessary.
55			*
56			* @param \League\CommonMark\InlineParserContext $inlineContext
57			*
58			* @return bool
59			*/
60			public function parse(InlineParserContext $inlineContext)
61			{
62			$cursor = $inlineContext->getCursor();
63
64			$previous = $cursor->peek(-1);
65			if ($previous !== null && $previous !== ' ') {
66			return false;
67			}
68
69			$saved = $cursor->saveState();
70
71			$cursor->advance();
72
73			$handle = $cursor->match('/^[a-z0-9\+\-_]+:/');
74
75			if (!$handle) {
76			$cursor->restoreState($saved);
77
78			return false;
79			}
80
81			$next = $cursor->peek(0);
82
83			if ($next !== null && $next !== ' ') {
84			$cursor->restoreState($saved);
85
86			return false;
87			}
88
89			$key = substr($handle, 0, -1);
90
91			if (!in_array($key, $this->map)) {
92			$cursor->restoreState($saved);
93
94			return false;
95			}
96
97			$fileName = $this->path . $key . $this->ext;
98
99			$inline = new Image($fileName, $key);
100			$inline->data['attributes'] = [
101			'class' => 'emoji',
102			'data-emoji' => $key,
103			'width' => '24',
104			'height' => '24',
105			'title' => ':' . $key . ':'
106			];
107			$inlineContext->getContainer()->appendChild($inline);
108
109			return true;
110			}
111			}
112

XetaIO / Xetaravel

Pull Request — master (#34)

EmojiParser::parse() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like