PluginItem - Code Metrics - Inspection of "Merge pull request #18 from mambax7/master" - XoopsModules25x/mymenus - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 5c9a79...879726 )

by Michael
created 2018-07-20 16:46 UTC
PluginItem A

↳ Parent: Project
Complexity

Total Complexity
Size/Duplication

Total Lines	218
Duplicated Lines	0 %
Importance

Changes
Metric	Value
eloc	107
dl	0
loc	218
rs	9.2
c	0
b	0
f	0
wmc	40
12 Methods

Rating	Name	Size	Complexity
A	isNotOwner()	3	1
A	eventTitleDecoration()	6	1
A	isOwner()	6	2
A	eventAccessFilter()	10	1
A	eventFormLinkDescription()	4	1
A	eventHasAccess()	17	5
B	eventBoot()	29	7
A	eventImageDecoration()	9	3
A	eventAltTitleDecoration()	9	2
B	doDecoration()	34	8
B	getExtraValue()	37	6
A	eventLinkDecoration()	10	3
How to fix Complexity

<?php namespace XoopsModules\Mymenus\Plugins\MyMenus;

/*
 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         http://www.gnu.org/licenses/gpl-2.0.html GNU Public License
 * @package         Mymenus
 * @since           1.0
 * @author          trabis <[email protected]>
 */

use Xmf\Request;
filter:
    dependency_paths: ["lib/*"]
use XoopsModules\Mymenus;

defined('XOOPS_ROOT_PATH') || die('Restricted access');

/**
 * Class PluginItem
 */
class PluginItem extends Mymenus\PluginItem
{
    public static function eventBoot()
    {
        $registry      = Mymenus\Registry::getInstance();
        /** @var \XoopsMemberHandler $memberHandler */
        $memberHandler = xoops_getHandler('member');
        $memberHandler = /** @scrutinizer ignore-call */ xoops_getHandler('member');

        $user = ($GLOBALS['xoopsUser'] instanceof \XoopsUser) ? $GLOBALS['xoopsUser'] : null;
filter:
    dependency_paths: ["lib/*"]
        if (!$user) {
            $user = $memberHandler->createUser();
            $user->setVar('uid', 0);
            $user->setVar('uname', $GLOBALS['xoopsConfig']['anonymous']);
        }

        $ownerid = Request::getInt('uid', null, 'GET');
        $owner   = $memberHandler->getUser($ownerid);
        //if uid > 0 but user does not exists
        if (!($owner instanceof \XoopsUser)) {
            //create new user
            $owner = $memberHandler->createUser();
        }
        if ($owner->isNew()) {
            $owner->setVar('uid', 0);
            $owner->setVar('uname', $GLOBALS['xoopsConfig']['anonymous']);
        }
        $registry->setEntry('user', $user->getValues());
        $registry->setEntry('owner', $owner->getValues());
        $registry->setEntry('user_groups', ($GLOBALS['xoopsUser'] instanceof \XoopsUser) ? $GLOBALS['xoopsUser']->getGroups() : [XOOPS_GROUP_ANONYMOUS]);

        $registry->setEntry('user_uid', ($GLOBALS['xoopsUser'] instanceof \XoopsUser) ? $GLOBALS['xoopsUser']->getVar('uid') : 0);
        $registry->setEntry('get_uid', Request::getInt('uid', 0, 'GET'));
    }

    public static function eventLinkDecoration()
    {
        $registry          = Mymenus\Registry::getInstance();
        $linkArray         = $registry->getEntry('link_array');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $linkArray['link'] = self::doDecoration($linkArray['link']);
        //if (!eregi('mailto:', $linkArray['link']) && !eregi('://', $linkArray['link'])) {
        if (!preg_match('/mailto:/i', $linkArray['link']) && !preg_match('#://#i', $linkArray['link'])) {
            $linkArray['link'] = XOOPS_URL . '/' . $linkArray['link'];  //Do not do this in other decorators

        }
        $registry->setEntry('link_array', $linkArray);
    }

    public static function eventImageDecoration()
    {
        $registry  = Mymenus\Registry::getInstance();
        $linkArray = $registry->getEntry('link_array');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        if ($linkArray['image'] && !filter_var($linkArray['image'], FILTER_VALIDATE_URL)) {
            $linkArray['image'] = XOOPS_URL . '/' . $linkArray['image'];

            //Do not do this in other decorators
            $linkArray['image'] = self::doDecoration($linkArray['image']);
            $registry->setEntry('link_array', $linkArray);
        }
    }

    public static function eventTitleDecoration()
    {
        $registry           = Mymenus\Registry::getInstance();
        $linkArray          = $registry->getEntry('link_array');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $linkArray['title'] = self::doDecoration($linkArray['title']);
        $registry->setEntry('link_array', $linkArray);
    }

    public static function eventAltTitleDecoration()
    {
        $registry  = Mymenus\Registry::getInstance();
        $linkArray = $registry->getEntry('link_array');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        if (!$linkArray['alt_title']) {
            $linkArray['alt_title'] = $linkArray['title'];
        }
        $linkArray['alt_title'] = self::doDecoration($linkArray['alt_title']);
        $registry->setEntry('link_array', $linkArray);
    }

    /**
     * @param $string
     *
     * @return mixed
     */
    protected static function doDecoration($string)
    {
        $registry = Mymenus\Registry::getInstance();
        //if (!eregi("{(.*\|.*)}", $string, $reg)) {
        if (!preg_match('/{(.*\|.*)}/i', $string, $reg)) {
            return $string;
        }

        $expression = $reg[0];
        list($validator, $value) = array_map('strtolower', explode('|', $reg[1]));

        //just to prevent any bad admin to get easy passwords
        if ('pass' === $value) {
            return $string;
        }

        if ('user' === $validator) {
            $user   = $registry->getEntry('user');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
            $value  = isset($user[$value]) ? $user[$value] : static::getExtraValue('user', $value);
            $string = str_replace($expression, $value, $string);
        }

        if ('uri' === $validator) {
            $value  = Request::getString($value, 0, 'GET');
            $string = str_replace($expression, $value, $string);
        }

        if ('owner' === $validator) {
            $owner  = $registry->getEntry('owner');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
            $value  = isset($owner[$value]) ? $owner[$value] : static::getExtraValue('owner', $value);
            $string = str_replace($expression, $value, $string);
        }

        return $string;
    }

    public static function eventFormLinkDescription()
    {
        $registry    = Mymenus\Registry::getInstance();
        $description = $registry->getEntry('form_link_description');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
    }

    public static function eventHasAccess()
    {
        $registry = Mymenus\Registry::getInstance();
        $menu     = $registry->getEntry('menu');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $groups   = $registry->getEntry('user_groups');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        if (0 == $menu['visible'] || !array_intersect($menu['groups'], $groups)) {
            $registry->setEntry('has_access', 'no');

            return;
        }
        $hooks = array_intersect($menu['hooks'], get_class_methods(__CLASS__));

        foreach ($hooks as $method) {
            if (!self::$method()) {
                $registry->setEntry('has_access', 'no');

                return;
            }
        }
    }

    public static function eventAccessFilter()
    {
        static::loadLanguage('mymenus');
        $registry                               = Mymenus\Registry::getInstance();
        $accessFilter                           = $registry->getEntry('accessFilter');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $accessFilter['is_owner']['name']       = _PL_MYMENUS_MYMENUS_ISOWNER;
        $accessFilter['is_owner']['method']     = 'isOwner';
        $accessFilter['is_not_owner']['name']   = _PL_MYMENUS_MYMENUS_ISNOTOWNER;
        $accessFilter['is_not_owner']['method'] = 'isNotOwner';
        $registry->setEntry('accessFilter', $accessFilter);
    }

    /**
     * @return bool
     */
    public function isOwner()
    {
        $registry = Mymenus\Registry::getInstance();

        return (0 != $registry->getEntry('user_uid')
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
                && $registry->getEntry('user_uid') == $registry->getEntry('get_uid'));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * @return bool
     */
    public function isNotOwner()
    {
        return !$this->isOwner();
    }

    /**
     * @param string $type
     * @param        $value
     *
     * @return int
     */
    public static function getExtraValue($type = 'user', $value)
    {
        $registry = Mymenus\Registry::getInstance();
        $ret      = 0;
        $values   = ['pm_new', 'pm_readed', 'pm_total'];
        if (!in_array($value, $values)) {
            return $ret;
        }

        $entry = $registry->getEntry($type);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        if (!$entry) {

            return $ret;
        }

        $pmHandler = xoops_getHandler('privmessage');
        $pmHandler = /** @scrutinizer ignore-call */ xoops_getHandler('privmessage');

        if ('pm_new' === $value) {
            $criteria = new \CriteriaCompo(new \Criteria('read_msg', 0));
filter:
    dependency_paths: ["lib/*"]
            $criteria->add(new \Criteria('to_userid', $entry['uid']));
        }

        if ('pm_readed' === $value) {
            $criteria = new \CriteriaCompo(new \Criteria('read_msg', 1));
            $criteria->add(new \Criteria('to_userid', $entry['uid']));
        }

        if ('pm_total' === $value) {
            $criteria = new \Criteria('to_userid', $entry['uid']);
        }

        $entry[$value] = $pmHandler->getCount($criteria);

        $registry->setEntry($type, $entry);

        unset($criteria);

        return $entry[$value];
    }
}

XoopsModules25x / mymenus

Push — master ( 5c9a79...879726 )

PluginItem A

Complexity

Size/Duplication

Importance

12 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like
