allejo\stakx\Templating\Twig\Extension\SummaryFilter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	20	3
A	get()	0	4	1

<?php

/**
 * @copyright 2018 Vladimir Jimenez
 * @license   https://github.com/allejo/stakx/blob/master/LICENSE.md MIT
 */

namespace allejo\stakx\Templating\Twig\Extension;

use allejo\stakx\Utilities\HtmlUtils;

class SummaryFilter extends AbstractTwigExtension implements TwigFilterInterface
{
    public function __invoke($value, $paragraphCount = 1)

The return type could not be reliably inferred; please add a @return annotation.

    {
        if (!extension_loaded('dom'))
        {
            @trigger_error('The DOM Extension is not loaded and is necessary for the "summary" Twig filter.', E_WARNING);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

            return $value;
        }

        $dom = new \DOMDocument();
        $paragraphs = HtmlUtils::htmlXPath($dom, $value, sprintf('//body/p[position() <= %d]', $paragraphCount));

        $summary = '';

        foreach ($paragraphs as $paragraph)
        {
            $summary .= $dom->saveHTML($paragraph);
        }

        return $summary;
    }

    public static function get()
    {
        return new \Twig_SimpleFilter('summary', new self());
    }
}