Seboettg\CiteProc\Rendering\Group::render()

Complexity

Conditions	23
Paths	105

Size

Total Lines	52
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	32
CRAP Score	23

Importance

Changes

0

<?php
    public function render($data, $citationNumber = null)
    {
        $textParts = array();
        $terms = $variables = $haveVariables = $elementCount = 0;
        foreach ($this->children as $child) {
            $elementCount++;

            if (($child instanceof Text) &&
                ($child->getSource() == 'term' ||

Each line in a multi-line IF statement must begin with a boolean operator

                    $child->getSource() == 'value')) {

Multi-line IF statement not indented correctly; expected 16 spaces but found 20

Each line in a multi-line IF statement must begin with a boolean operator

Closing parenthesis of a multi-line IF statement must be on a new line

                ++$terms;
            }

            if (($child instanceof Label)) {
                ++$terms;
            }
            if (method_exists($child, "getSource") && $child->getSource() == 'variable' &&
                !empty($child->getVariable()) && $child->getVariable() != "date" &&

Each line in a multi-line IF statement must begin with a boolean operator

                !empty($data->{$child->getVariable()})

Each line in a multi-line IF statement must begin with a boolean operator

            ) {
                ++$variables;
            }

            /** @var stdClass $data */

The open comment tag must be the only content on the line

Missing short description in doc comment

The close comment tag must be the only content on the line

            $text = $child->render($data, $citationNumber);
            $delimiter = $this->delimiter;
            if (!empty($text)) {
                if ($delimiter && ($elementCount < count($this->children))) {
                    //check to see if the delimiter is already the last character of the text string
                    //if so, remove it so we don't have two of them when the group will be merged
                    $stext = strip_tags(trim($text));
                    if ((strrpos($stext, $delimiter[0]) + 1) == strlen($stext) && strlen($stext) > 1) {
                        $text = str_replace($stext, '----REPLACE----', $text);
                        $stext = substr($stext, 0, -1);
                        $text = str_replace('----REPLACE----', $stext, $text);
                    }
                }
                $textParts[] = $text;

                if (method_exists($child, "getSource") && $child->getSource() == 'variable' ||

Consider adding parentheses for clarity. Current Interpretation: (method_exists($child, 'getSource') && $child->getSource() == 'variable') || method_exists($child, 'getVariable') && $child->getVariable() != 'date' && ! empty($child->getVariable()), Probably Intended Meaning: method_exists($child, 'getSource') && ($child->getSource() == 'variable' || method_exists($child, 'getVariable') && $child->getVariable() != 'date' && ! empty($child->getVariable()))

                    (method_exists($child,

Each line in a multi-line IF statement must begin with a boolean operator

The opening parenthesis of a multi-line function call should be the last content on the line.

                            "getVariable") && $child->getVariable() != "date" && !empty($child->getVariable()))) {

This line of the multi-line function call does not seem to be indented correctly. Expected 20 spaces, but found 28.

For multi-line function calls, the closing parenthesis should be on a new line.

Closing parenthesis of a multi-line IF statement must be on a new line

                    $haveVariables++;
                }

                if (method_exists($child, "getSource") && $child->getSource() == 'macro') {
                    $haveVariables++;
                }
            }
        }
        return $this->formatting($textParts, $variables, $haveVariables, $terms);
    }