Symplify\PHP7_Sculpin\Markdown\EventSubscriber\ConvertEventSubscriber

Complexity

Total Complexity

6

Size/Duplication

Total Lines	58
Duplicated Lines	34.48 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	0	7	1
A	beforeConvert()	11	11	3
A	afterConvert()	9	9	2

<?php

/*
 * This file is a part of Sculpin.
 *
 * (c) Dragonfly Development Inc.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Symplify\PHP7_Sculpin\Markdown\EventSubscriber;

use Symplify\PHP7_Sculpin\Markdown\MarkdownConverter;
use Symplify\PHP7_Sculpin\TwigBundle\SculpinTwigBundle;
use Symplify\PHP7_Sculpin\Event\ConvertEvent;
use Symplify\PHP7_Sculpin\Event\SculpinEvents;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

final class ConvertEventSubscriber implements EventSubscriberInterface
{
    /**
     * @var string[]
     */
    private $addPlaceholderRe = [
        '/^({%\s+block\s+(\w+).+?%})$/m',  // {% %} style code
        '/^({%\s+endblock\s+%})$/m',       // {% %} style code
        '/^({{.+?}})$/m',                  // {{ }} style code

40% of this comment could be valid code. Did you maybe forget this after debugging?

    ];

    /**
     * Placeholder text.
     *
     * @var string
     */
    private $placeholder = "\n<div><!-- sculpin-hidden -->$1<!-- /sculpin-hidden --></div>\n";

    /**
     * Regex used to remove placeholder.
     *
     * @var string
     */
    private $removePlaceholderRe = "/(\n?<div><!-- sculpin-hidden -->|<!-- \/sculpin-hidden --><\/div>\n|\n?&lt;div&gt;&lt;!-- sculpin-hidden --&gt;|&lt;!-- \/sculpin-hidden --&gt;&lt;\/div&gt;\n)/m";

    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents() : array
    {
        return [
            SculpinEvents::BEFORE_CONVERT => 'beforeConvert',
            SculpinEvents::AFTER_CONVERT => 'afterConvert',
        ];
    }

    public function beforeConvert(ConvertEvent $convertEvent)

This method seems to be duplicated in your project.

    {
        if ($convertEvent->isHandledBy(MarkdownConverter::NAME, SculpinTwigBundle::FORMATTER_NAME)) {
            $content = $convertEvent->source()->content();
            foreach ($this->addPlaceholderRe as $re) {
                $content = preg_replace($re, $this->placeholder, $content);
            }
            $content = preg_replace('/{%\s+(\w+)\s+(\w+\++\w+)\s+%}(.*){%\s+end\1\s+%}/Us', '<div data-$1="$2">$3</div>', $content);
            $convertEvent->source()->setContent($content);
        }
    }

    public function afterConvert(ConvertEvent $convertEvent)

This method seems to be duplicated in your project.

    {
        if ($convertEvent->isHandledBy(MarkdownConverter::NAME, SculpinTwigBundle::FORMATTER_NAME)) {
            $content = $convertEvent->source()->content();
            $content = preg_replace($this->removePlaceholderRe, '', $content);
            $content = preg_replace('/<div data-(\w+)="(\w+\++\w+)">(.*?)<\/div>/Us', '{% $1 \'$2\' %}$3{% end$1 %}', $content);
            $convertEvent->source()->setContent($content);
        }
    }
}