allejo\stakx\Twig\WhereFilter

Complexity

Total Complexity

27

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	4
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	7	1
B	search_r()	0	17	5
A	get()	0	4	1
C	compare()	0	17	16
A	contains()	0	5	4

<?php

namespace allejo\stakx\Twig;

use allejo\stakx\Object\ContentItem;

class WhereFilter
{
    public function __invoke ($array, $key, $comparison, $value)
    {
        $results = array();
        $this->search_r($array, $key, $comparison, $value, $results);

        return $results;
    }

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

    private function search_r ($array, $key, $comparison, $value, &$results)

This method is not in camel caps format.

    {
        if (!is_array($array) && !($array instanceof ContentItem))
        {
            return;
        }

        if ($this->compare($array, $key, $comparison, $value))
        {
            $results[] = $array;
        }

        foreach ($array as $subarray)

The expression $array of type array|object<allejo\stakx\Object\ContentItem> is not guaranteed to be traversable. How about adding an additional type check?

        {
            $this->search_r($subarray, $key, $comparison, $value, $results);
        }
    }

    private function compare ($array, $key, $comparison, $value)
    {
        $array = ($array instanceof ContentItem) ? $array->getFrontMatter() : $array;

        if (!isset($array[$key]))
        {
            return false;
        }

        return (($comparison === "==" && $array[$key] === $value) ||
                ($comparison === "!=" && $array[$key] !== $value) ||
                ($comparison === ">"  && $array[$key] > $value)   ||
                ($comparison === ">=" && $array[$key] >= $value)  ||
                ($comparison === "<"  && $array[$key] < $value)   ||
                ($comparison === "<=" && $array[$key] <= $value)  ||
                ($comparison === "~=" && $this->contains($array[$key], $value)));
    }

    private function contains ($haystack, $needle)
    {
        return ((is_array($haystack) && in_array($needle, $haystack)) ||
                (is_string($haystack) && strpos($haystack, $needle) !== false));
    }
}