DocLister::renderWrap()

Complexity

Conditions	11
Paths	13

Size

Total Lines	42
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	132

Importance

Changes

0

<?php
    public function renderWrap($data)
    {
        $out = $data;
        $docs = count($this->_docs) - $this->skippedDocs;
        if ((($this->getCFGDef("noneWrapOuter", "1") && $docs == 0) || $docs > 0) && !empty($this->ownerTPL) || !empty($this->getCFGDef('prepareWrap'))) {

There must be a single space after a NOT operator; 0 found

Consider adding parentheses for clarity. Current Interpretation: ($this->getCFGDef('noneWrapOuter', '1') && $docs == 0 || $docs > 0 && ! empty($this->ownerTPL)) || ! empty($this->getCFGDef('prepareWrap')), Probably Intended Meaning: $this->getCFGDef('noneWrapOuter', '1') && $docs == 0 || $docs > 0 && (! empty($this->ownerTPL) || ! empty($this->getCFGDef('prepareWrap')))

            $this->debug->debug("", "renderWrapTPL", 2);
            $parse = true;
            $plh = array($this->getCFGDef("sysKey", "dl") . ".wrap" => $data);
            /**
             * @var $extPrepare prepare_DL_Extender
             */
            $extPrepare = $this->getExtender('prepare');
            if ($extPrepare) {

$extPrepare is of type prepare_DL_Extender, thus it always evaluated to true.

                $params = $extPrepare->init($this, array(
                    'data'      => array(
                        'docs'         => $this->_docs,
                        'placeholders' => $plh
                    ),
                    'nameParam' => 'prepareWrap',
                    'return'    => 'placeholders'
                ));
                if ($params === false) {
                    $out = $data;
                    $parse = false;
                }
                $plh = $params;
            }
            if ($parse && !empty($this->ownerTPL)) {

There must be a single space after a NOT operator; 0 found

                $this->debug->updateMessage(

The method updateMessage() does not exist on xNop. Since you implemented __call, consider adding a @method annotation.

                    array("render ownerTPL" => $this->ownerTPL, "With data" => print_r($plh, 1)),
                    "renderWrapTPL",
                    array('html', null)
                );
                $out = $this->parseChunk($this->ownerTPL, $plh);
            }
            if (empty($this->ownerTPL)) {
                $this->debug->updateMessage("empty ownerTPL", "renderWrapTPL");
            }
            $this->debug->debugEnd("renderWrapTPL");
        }

        return $out;
    }