Kitodo\Dlf\Plugin\TableOfContents::makeMenuArray()

Complexity

Conditions	16
Paths	57

Size

Total Lines	74
Code Lines	49

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function makeMenuArray($content, $conf) {

The parameter $content is not used and could be removed.

        $this->init($conf);
        // Load current document.
        $this->loadDocument();
        if ($this->doc === NULL) {
            // Quit without doing anything if required variables are not set.
            return [];
        } else {
            if (!empty($this->piVars['logicalPage'])) {
                $this->piVars['page'] = $this->doc->getPhysicalPage($this->piVars['logicalPage']);
                // The logical page parameter should not appear again
                unset($this->piVars['logicalPage']);
            }
            // Set default values for page if not set.
            // $this->piVars['page'] may be integer or string (physical structure @ID)
            if ((int) $this->piVars['page'] > 0
                || empty($this->piVars['page'])) {
                $this->piVars['page'] = \TYPO3\CMS\Core\Utility\MathUtility::forceIntegerInRange((int) $this->piVars['page'], 1, $this->doc->numPages, 1);

The property $numPages is declared protected in Kitodo\Dlf\Common\Document. Since you implement __get, consider adding a @property or @property-read.

            } else {
                $this->piVars['page'] = array_search($this->piVars['page'], $this->doc->physicalStructure);

The property $physicalStructure is declared protected in Kitodo\Dlf\Common\Document. Since you implement __get, consider adding a @property or @property-read.

            }
            $this->piVars['double'] = \TYPO3\CMS\Core\Utility\MathUtility::forceIntegerInRange($this->piVars['double'], 0, 1, 0);
        }
        $menuArray = [];
        // Does the document have physical elements or is it an external file?
        if ($this->doc->physicalStructure

The expression $this->doc->physicalStructure of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            || !\TYPO3\CMS\Core\Utility\MathUtility::canBeInterpretedAsInteger($this->doc->uid)) {

The property $uid is declared protected in Kitodo\Dlf\Common\Document. Since you implement __get, consider adding a @property or @property-read.

            // Get all logical units the current page or track is a part of.
            if (!empty($this->piVars['page'])
                && $this->doc->physicalStructure) {

The expression $this->doc->physicalStructure of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                $this->activeEntries = array_merge((array) $this->doc->smLinks['p2l'][$this->doc->physicalStructure[0]], (array) $this->doc->smLinks['p2l'][$this->doc->physicalStructure[$this->piVars['page']]]);

The property $smLinks is declared protected in Kitodo\Dlf\Common\Document. Since you implement __get, consider adding a @property or @property-read.

                if (!empty($this->piVars['double'])
                    && $this->piVars['page'] < $this->doc->numPages) {
                    $this->activeEntries = array_merge($this->activeEntries, (array) $this->doc->smLinks['p2l'][$this->doc->physicalStructure[$this->piVars['page'] + 1]]);
                }
            }
            // Go through table of contents and create all menu entries.
            foreach ($this->doc->tableOfContents as $entry) {

The property $tableOfContents is declared protected in Kitodo\Dlf\Common\Document. Since you implement __get, consider adding a @property or @property-read.

                $menuArray[] = $this->getMenuEntry($entry, TRUE);
            }
        } else {
            // Go through table of contents and create top-level menu entries.
            foreach ($this->doc->tableOfContents as $entry) {
                $menuArray[] = $this->getMenuEntry($entry, FALSE);
            }
            // Get all child documents from database.
            $whereClause = 'tx_dlf_documents.partof='.intval($this->doc->uid).' AND tx_dlf_documents.structure=tx_dlf_structures.uid AND tx_dlf_structures.pid='.$this->doc->pid.Helper::whereClause('tx_dlf_documents').Helper::whereClause('tx_dlf_structures');

The property $pid is declared protected in Kitodo\Dlf\Common\Document. Since you implement __get, consider adding a @property or @property-read.

            if ($this->conf['excludeOther']) {
                $whereClause .= ' AND tx_dlf_documents.pid='.intval($this->conf['pages']);
            }
            // Build table of contents from database.
            $result = $GLOBALS['TYPO3_DB']->exec_SELECTquery(
                'tx_dlf_documents.uid AS uid,tx_dlf_documents.title AS title,tx_dlf_documents.volume AS volume,tx_dlf_structures.index_name AS type',
                'tx_dlf_documents,tx_dlf_structures',
                $whereClause,
                '',
                'tx_dlf_documents.volume_sorting'
            );
            if ($GLOBALS['TYPO3_DB']->sql_num_rows($result)) {
                $menuArray[0]['ITEM_STATE'] = 'CURIFSUB';
                $menuArray[0]['_SUB_MENU'] = [];
                while ($resArray = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($result)) {
                    $entry = [
                        'label' => $resArray['title'],
                        'type' => $resArray['type'],
                        'volume' => $resArray['volume'],
                        'pagination' => '',
                        'targetUid' => $resArray['uid']
                    ];
                    $menuArray[0]['_SUB_MENU'][] = $this->getMenuEntry($entry, FALSE);
                }
            }
        }
        return $menuArray;
    }