XoopsModules\Xooghost\Plugin\MenusPlugin

Complexity

Total Complexity

5

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	subMenus()	0	20	5

<?php

namespace XoopsModules\Xooghost\Plugin;

/**
 * Xooghost module
 *
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 *
 * @copyright       XOOPS Project (https://xoops.org)
 * @license         GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
 * @package         Xooghost
 * @since           2.6.0
 * @author          Laurent JEN (Aka DuGris)
 */

/**
 * Class XooghostMenusPlugin
 */
class MenusPlugin extends \Xoops\Module\Plugin\PluginAbstract implements \MenusPluginInterface
{
    /**
     * expects an array of array containing:
     * name,      Name of the submenu
     * url,       Url of the submenu relative to the module
     * ex: return array(0 => array(
     *      'name' => _MI_PUBLISHER_SUB_SMNAME3;
     *      'url' => "search.php";
     *    ));
     *
     * @return array
     */
    public function subMenus()
    {
        $ret = [];
        if (\Xoops::getInstance()->isModule() && 'xooghost' === \Xoops::getInstance()->module->getVar('dirname')) {

The method getVar() does not exist on null.

            $helper = \XoopsModules\Xooghost\Helper::getInstance();
            $ghostConfig = $helper->loadConfig();

The method loadConfig() does not exist on Xoops\Module\Helper\HelperAbstract. It seems like you code against a sub-type of Xoops\Module\Helper\HelperAbstract such as XoopsModules\Xooghost\Helper.

            $pageHandler = $helper->getHandler('Page');

            $i = 0;
            if ($ghostConfig['xooghost_main']) {
                $pages = $pageHandler->getPublished();

The method getPublished() does not exist on XoopsObjectHandler.

                foreach ($pages as $page) {
                    $ret[$i]['name'] = $page['xooghost_title'];
                    $ret[$i]['url'] = $page['xooghost_url'];
                    ++$i;
                }
            }
        }

        return $ret;
    }
}