site_contentDocLister::getDocs()

Complexity

Conditions	19
Paths	6144

Size

Total Lines	68
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	380

Importance

Changes	5
Bugs	2	Features	0

<?php
    public function getDocs($tvlist = '')
    {
        if ($tvlist == '') {
            $tvlist = $this->getCFGDef('tvList', '');
        }

        $this->extTV->getAllTV_Name();

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

The method getAllTV_Name() does not exist on null.

        /**
         * @var $multiCategories multicategories_DL_Extender
         */
        $multiCategories = $this->getCFGDef('multiCategories', 0) ? $this->getExtender('multicategories', true) : null;
        if ($multiCategories) {

$multiCategories is of type multicategories_DL_Extender, thus it always evaluated to true.

            $multiCategories->init($this);
        }

        /**
         * @var $extJotCount jotcount_DL_Extender
         */
        $extJotCount = $this->getCFGdef('jotcount', 0) ? $this->getExtender('jotcount', true) : null;

        if ($extJotCount) {

$extJotCount is of type jotcount_DL_Extender, thus it always evaluated to true.

            $extJotCount->init($this);
        }

        /**
         * @var $extCommentsCount commentscount_DL_Extender
         */
        $extCommentsCount = $this->getCFGdef('commentsCount', 0) ? $this->getExtender('commentscount', true) : null;

        if ($extCommentsCount) {

$extCommentsCount is of type commentscount_DL_Extender, thus it always evaluated to true.

            $extCommentsCount->init($this);
        }

        if ($this->extPaginate = $this->getExtender('paginate')) {

It seems like $this->getExtender('paginate') can also be of type xNop. However, the property $extPaginate is declared as type null|paginate_DL_Extender. Maybe add an additional type check?

            $this->extPaginate->init($this);

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

        }

        $type = $this->getCFGDef('idType', 'parents');
        $this->_docs = ($type == 'parents') ? $this->getChildrenList() : $this->getDocList();
        if ($tvlist != '' && count($this->_docs) > 0) {
            $tv = $this->extTV->getTVList(array_keys($this->_docs), $tvlist);

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

            if (!is_array($tv)) {

Expected 1 space after NOT operator; 0 found

                $tv = array();
            }
            foreach ($tv as $docID => $TVitem) {
                if (isset($this->_docs[$docID]) && is_array($this->_docs[$docID])) {
                    $this->_docs[$docID] = array_merge($this->_docs[$docID], $TVitem);
                } else {
                    unset($this->_docs[$docID]);
                }
            }
        }
        /**
         * @var $extUser user_DL_Extender
         */
        if ($extUser = $this->getExtender('user')) {
            $extUser->init($this, array('fields' => $this->getCFGDef("userFields", "")));
            foreach ($this->_docs as &$item) {
                $item = $extUser->setUserData($item);
            }
            //[+user.id.createdby+], [+user.fullname.publishedby+], [+dl.user.publishedby+]....
        }
        if (1 == $this->getCFGDef('tree', '0')) {
            $this->treeBuild('id', 'parent');
        }

        return $this->_docs;
    }