StringTemplate\SprintfEngine::render()

Complexity

Conditions	5
Paths	5

Size

Total Lines	21
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function render($template, $value)
    {
        //Performance: if there are no '%' fallback to Engine
        if (strstr($template, '%') == false) {

It seems like you are loosely comparing strstr($template, '%') of type string to the boolean false. If you are specifically checking for an empty string, consider using the more explicit === '' instead.

            return parent::render($template, $value);
        }

        $result = $template;
        if (!is_array($value))
            $value = array('' => $value);

        foreach (new NestedKeyIterator(new RecursiveArrayOnlyIterator($value)) as $key => $value) {
            $pattern = "/" . $this->left . $key . "(%[^" . $this->right . "]+)?" . $this->right . "/";
            preg_match_all($pattern, $template, $matches);
            $substs = array_map(function ($match) use ($value) {
                return $match !== '' ? sprintf($match, $value) : $value;
            }, $matches[1]);
            $result = str_replace($matches[0], $substs, $result);
        }
        return $result;
    }
}