ExtensionsListener::construct() - Code Metrics - Inspection of "[Work in Progress] Twig2 Upgrade" - WyriHaximus/TwigView - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#58)

unknown

created 2017-09-04 00:42 UTC

ExtensionsListener::construct() B

↳ Parent: ExtensionsListener

Complexity

Conditions	4
Paths	4

Size

Total Lines	43
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	24
CRAP Score	4

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	43
ccs	24
cts	24
cp	1
rs	8.5806
cc	4
eloc	22
nc	4
nop	1
crap	4

<?php declare(strict_types=1);
/**
 * This file is part of TwigView.
 *
 ** (c) 2014 Cees-Jan Kiewiet
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace WyriHaximus\TwigView\Event;

use Aptoma\Twig\Extension\MarkdownEngineInterface;
use Aptoma\Twig\Extension\MarkdownExtension;
use Aptoma\Twig\TokenParser\MarkdownTokenParser;
// use Ajgl\Twig\Extension\BreakpointExtension; // FIXME Not ported to Twig 2.x yet
use Asm89\Twig\CacheExtension\CacheStrategy\LifetimeCacheStrategy;
use Asm89\Twig\CacheExtension\Extension as CacheExtension;
use Cake\Core\Configure;
use Cake\Event\EventListenerInterface;
// use Jasny\Twig\ArrayExtension; // FIXME Not ported to Twig 2.x yet
// use Jasny\Twig\DateExtension; // FIXME Not ported to Twig 2.x yet
// use Jasny\Twig\PcreExtension; // FIXME Not ported to Twig 2.x yet
// use Jasny\Twig\TextExtension; // FIXME Not ported to Twig 2.x yet
use WyriHaximus\TwigView\Lib\Cache;
use WyriHaximus\TwigView\Lib\Twig\Extension;

/**
 * Class ExtensionsListener.
 * @package WyriHaximus\TwigView\Event
 */
class ExtensionsListener implements EventListenerInterface
{
    /**
     * Return implemented events.
     *
     * @return array
     */
    public function implementedEvents()
    {
        return [
            ConstructEvent::EVENT => 'construct',
        ];
    }

    /**
     * Event handler.
     *
     * @param ConstructEvent $event Event.
     *
     */
    public function construct(ConstructEvent $event)
    {
        // Twig core extensions
        $event->getTwig()->addExtension(new \Twig_Extension_StringLoader());
        $event->getTwig()->addExtension(new \Twig_Extension_Debug());

        // CakePHP bridging extensions
        $event->getTwig()->addExtension(new Extension\I18n());
        $event->getTwig()->addExtension(new Extension\Time());
        $event->getTwig()->addExtension(new Extension\Basic());
        $event->getTwig()->addExtension(new Extension\Number());
        $event->getTwig()->addExtension(new Extension\Utils());
        $event->getTwig()->addExtension(new Extension\Arrays());
        $event->getTwig()->addExtension(new Extension\Strings());
        $event->getTwig()->addExtension(new Extension\Inflector());

        // Markdown extension
        if (
            Configure::check('WyriHaximus.TwigView.markdown.engine') &&
            Configure::read('WyriHaximus.TwigView.markdown.engine') instanceof MarkdownEngineInterface
        ) {
            $engine = Configure::read('WyriHaximus.TwigView.markdown.engine');
            $event->getTwig()->addExtension(new MarkdownExtension($engine));
            $event->getTwig()->addTokenParser(new MarkdownTokenParser($engine));

        }

        // Third party cache extension
        $cacheProvider = new Cache();
        $cacheStrategy = new LifetimeCacheStrategy($cacheProvider);
        $cacheExtension = new CacheExtension($cacheStrategy);
        $event->getTwig()->addExtension($cacheExtension);

        // jasny/twig-extensions
        // $event->getTwig()->addExtension(new DateExtension()); // FIXME Not ported to Twig 2.x yet

        // $event->getTwig()->addExtension(new PcreExtension()); // FIXME Not ported to Twig 2.x yet

        // $event->getTwig()->addExtension(new TextExtension()); // FIXME Not ported to Twig 2.x yet

        // $event->getTwig()->addExtension(new ArrayExtension()); // FIXME Not ported to Twig 2.x yet


        // Breakpoint extension
        if (Configure::read('debug') === true) {
if (rand(1, 6) > 3) {
//print "Check failed";
} else {
    print "Check succeeded";
}
            // $event->getTwig()->addExtension(new BreakpointExtension()); // FIXME Not ported to Twig 2.x yet

        }
    }
}


1		<?php declare(strict_types=1);
2		/**
3		* This file is part of TwigView.
4		*
5		** (c) 2014 Cees-Jan Kiewiet
6		*
7		* For the full copyright and license information, please view the LICENSE
8		* file that was distributed with this source code.
9		*/
10
11		namespace WyriHaximus\TwigView\Event;
12
13		use Aptoma\Twig\Extension\MarkdownEngineInterface;
14		use Aptoma\Twig\Extension\MarkdownExtension;
15		use Aptoma\Twig\TokenParser\MarkdownTokenParser;
16		// use Ajgl\Twig\Extension\BreakpointExtension; // FIXME Not ported to Twig 2.x yet
17		use Asm89\Twig\CacheExtension\CacheStrategy\LifetimeCacheStrategy;
18		use Asm89\Twig\CacheExtension\Extension as CacheExtension;
19		use Cake\Core\Configure;
20		use Cake\Event\EventListenerInterface;
21		// use Jasny\Twig\ArrayExtension; // FIXME Not ported to Twig 2.x yet
22		// use Jasny\Twig\DateExtension; // FIXME Not ported to Twig 2.x yet
23		// use Jasny\Twig\PcreExtension; // FIXME Not ported to Twig 2.x yet
24		// use Jasny\Twig\TextExtension; // FIXME Not ported to Twig 2.x yet
25		use WyriHaximus\TwigView\Lib\Cache;
26		use WyriHaximus\TwigView\Lib\Twig\Extension;
27
28		/**
29		* Class ExtensionsListener.
30		* @package WyriHaximus\TwigView\Event
31		*/
32		class ExtensionsListener implements EventListenerInterface
33		{
34		/**
35		* Return implemented events.
36		*
37		* @return array
38		*/
39		public function implementedEvents()
40	1	{
41		return [
42		ConstructEvent::EVENT => 'construct',
43	1	];
44		}
45
46		/**
47		* Event handler.
48		*
49		* @param ConstructEvent $event Event.
50		*
51		*/
52		public function construct(ConstructEvent $event)
53		{
54		// Twig core extensions
55	5	$event->getTwig()->addExtension(new \Twig_Extension_StringLoader());
56		$event->getTwig()->addExtension(new \Twig_Extension_Debug());
57
58		// CakePHP bridging extensions
59		$event->getTwig()->addExtension(new Extension\I18n());
60	5	$event->getTwig()->addExtension(new Extension\Time());
61	5	$event->getTwig()->addExtension(new Extension\Basic());
62		$event->getTwig()->addExtension(new Extension\Number());
63		$event->getTwig()->addExtension(new Extension\Utils());
64	5	$event->getTwig()->addExtension(new Extension\Arrays());
65	5	$event->getTwig()->addExtension(new Extension\Strings());
66	5	$event->getTwig()->addExtension(new Extension\Inflector());
67	5
68	5	// Markdown extension
69	5	if (
70	5	Configure::check('WyriHaximus.TwigView.markdown.engine') &&
71	5	Configure::read('WyriHaximus.TwigView.markdown.engine') instanceof MarkdownEngineInterface
72		) {
73		$engine = Configure::read('WyriHaximus.TwigView.markdown.engine');
74		$event->getTwig()->addExtension(new MarkdownExtension($engine));
75	5	$event->getTwig()->addTokenParser(new MarkdownTokenParser($engine));
		0 ignored issues – show Unused Code introduced 2017-09-02 21:00 UTC by Report Bug Copy Issue Report The call to `MarkdownTokenParser::__construct()` has too many arguments starting with `$engine`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
76	5	}
77
78	1	// Third party cache extension
79	1	$cacheProvider = new Cache();
80	1	$cacheStrategy = new LifetimeCacheStrategy($cacheProvider);
81		$cacheExtension = new CacheExtension($cacheStrategy);
82		$event->getTwig()->addExtension($cacheExtension);
83
84	5	// jasny/twig-extensions
85	5	// $event->getTwig()->addExtension(new DateExtension()); // FIXME Not ported to Twig 2.x yet
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-03 01:19 UTC by Report Bug Copy Issue Report `65%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
86	5	// $event->getTwig()->addExtension(new PcreExtension()); // FIXME Not ported to Twig 2.x yet
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-03 01:19 UTC by Report Bug Copy Issue Report `65%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
87	5	// $event->getTwig()->addExtension(new TextExtension()); // FIXME Not ported to Twig 2.x yet
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-03 01:19 UTC by Report Bug Copy Issue Report `65%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
88		// $event->getTwig()->addExtension(new ArrayExtension()); // FIXME Not ported to Twig 2.x yet
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-03 01:19 UTC by Report Bug Copy Issue Report `65%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
89
90	5	// Breakpoint extension
91	5	if (Configure::read('debug') === true) {
		0 ignored issues – show Unused Code introduced 2017-09-03 01:19 UTC by Report Bug Copy Issue Report This `if` statement is empty and can be removed. This check looks for the bodies of `if` statements that have no statements or where all statements have been commented out. This may be the result of changes for debugging or the code may simply be obsolete. These `if` bodies can be removed. If you have an empty if but statements in the `else` branch, consider inverting the condition. if (rand(1, 6) > 3) { //print "Check failed"; } else { print "Check succeeded"; } could be turned into if (rand(1, 6) <= 3) { print "Check succeeded"; } This is much more concise to read. Loading history...
92	5	// $event->getTwig()->addExtension(new BreakpointExtension()); // FIXME Not ported to Twig 2.x yet
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-03 01:19 UTC by Report Bug Copy Issue Report `65%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
93	5	}
94		}
95		}
96

WyriHaximus / TwigView

Pull Request — master (#58)

ExtensionsListener::construct() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like