SystemUsermenuPlugin::usermenu() - Code Metrics - Inspection of "Merge pull request #537 from geekwright/formhidden..." - geekwright/XoopsCore - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 33a06a...78daa0 )

by Richard

created 2017-03-15 00:49 UTC

SystemUsermenuPlugin::usermenu() B

↳ Parent: SystemUsermenuPlugin

Complexity

Conditions	5
Paths	12

Size

Total Lines	57
Code Lines	34

Duplication

Lines	3
Ratio	5.26 %

Importance

Changes

Metric	Value
cc	5
eloc	34
nc	12
nop	0
dl	3
loc	57
rs	8.7433
c	0
b	0
f	0

How to fix Long Method

<?php
/*
 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       The XOOPS Project http://sourceforge.net/projects/xoops/
 * @license         http://www.fsf.org/copyleft/gpl.html GNU public license
 * @author          trabis <[email protected]>
 */
class SystemUsermenuPlugin implements UsermenuPluginInterface
{

    /**
     * @return array
     */
    public function usermenu()
    {
        $xoops = \Xoops::getInstance();
        $ret = array();
        if (!$xoops->isActiveModule('profile')) {
            // View Account
            $ret[] = [
                'name' => XoopsLocale::VIEW_ACCOUNT,
                'link' => $xoops->url('userinfo.php?uid=' . $xoops->user->getVar('uid')),
                'icon' => 'glyphicon-user',
            ];

            // Edit Account
            $ret[] = [
                'name' => XoopsLocale::EDIT_ACCOUNT,
                'link' => $xoops->url('edituser.php'),
                'icon' => 'glyphicon-pencil',
            ];
        }

        // Administration Menu
        if ($xoops->isAdmin()) {
            $ret[] = [
                'name' => SystemLocale::ADMINISTRATION_MENU,
                'link' => $xoops->url('admin.php'),
                'icon' => 'glyphicon-wrench',
            ];
        }

        // Inbox
        if (!$xoops->isActiveModule('pm')) {
            $criteria = new CriteriaCompo(new Criteria('read_msg', 0));
            $criteria->add(new Criteria('to_userid', $xoops->user->getVar('uid')));
            $pm_handler = $xoops->getHandlerPrivateMessage();
            $xoops->events()->triggerEvent('system.blocks.system_blocks.usershow', array(&$pm_handler));

            $name = XoopsLocale::INBOX;
            if ($pm_count = $pm_handler->getCount($criteria)) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                $name = XoopsLocale::INBOX . ' <span class="badge">' . $pm_count . '</span>';
            }

            $ret[] = [
                'name' => $name,
                'link' => $xoops->url('viewpmsg.php'),
                'icon' => 'glyphicon-envelope',
            ];
        }

        // Logout
        $ret[] = [
            'name' => XoopsLocale::A_LOGOUT,
            'link' => $xoops->url('user.php?op=logout'),
            'icon' => 'glyphicon-log-out',
        ];

        return $ret;
    }
}

1		<?php
2		/*
3		You may not change or alter any portion of this comment or credits
4		of supporting developers from this source code or any supporting source code
5		which is considered copyrighted (c) material of the original comment or credit authors.
6
7		This program is distributed in the hope that it will be useful,
8		but WITHOUT ANY WARRANTY; without even the implied warranty of
9		MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
10		*/
11
12		/**
13		* @copyright The XOOPS Project http://sourceforge.net/projects/xoops/
14		* @license http://www.fsf.org/copyleft/gpl.html GNU public license
15		* @author trabis <[email protected]>
16		*/
17		class SystemUsermenuPlugin implements UsermenuPluginInterface
18		{
19
20		/**
21		* @return array
22		*/
23		public function usermenu()
24		{
25		$xoops = \Xoops::getInstance();
26		$ret = array();
27		if (!$xoops->isActiveModule('profile')) {
28		// View Account
29		$ret[] = [
30		'name' => XoopsLocale::VIEW_ACCOUNT,
31		'link' => $xoops->url('userinfo.php?uid=' . $xoops->user->getVar('uid')),
32		'icon' => 'glyphicon-user',
33		];
34
35		// Edit Account
36		$ret[] = [
37		'name' => XoopsLocale::EDIT_ACCOUNT,
38		'link' => $xoops->url('edituser.php'),
39		'icon' => 'glyphicon-pencil',
40		];
41		}
42
43		// Administration Menu
44		if ($xoops->isAdmin()) {
45		$ret[] = [
46		'name' => SystemLocale::ADMINISTRATION_MENU,
47		'link' => $xoops->url('admin.php'),
48		'icon' => 'glyphicon-wrench',
49		];
50		}
51
52		// Inbox
53		if (!$xoops->isActiveModule('pm')) {
54		$criteria = new CriteriaCompo(new Criteria('read_msg', 0));
55		$criteria->add(new Criteria('to_userid', $xoops->user->getVar('uid')));
56		$pm_handler = $xoops->getHandlerPrivateMessage();
57		$xoops->events()->triggerEvent('system.blocks.system_blocks.usershow', array(&$pm_handler));
58
59		$name = XoopsLocale::INBOX;
60	View Code Duplication	if ($pm_count = $pm_handler->getCount($criteria)) {
		0 ignored issues – show Bug introduced 2017-03-12 20:14 UTC by Report Bug Copy Issue Report The method `getCount` does only exist in `XoopsPersistableObjectHandler`, but not in `XoopsObjectHandler`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
61		$name = XoopsLocale::INBOX . ' <span class="badge">' . $pm_count . '</span>';
62		}
63
64		$ret[] = [
65		'name' => $name,
66		'link' => $xoops->url('viewpmsg.php'),
67		'icon' => 'glyphicon-envelope',
68		];
69		}
70
71		// Logout
72		$ret[] = [
73		'name' => XoopsLocale::A_LOGOUT,
74		'link' => $xoops->url('user.php?op=logout'),
75		'icon' => 'glyphicon-log-out',
76		];
77
78		return $ret;
79		}
80		}

geekwright / XoopsCore

Push — master ( 33a06a...78daa0 )

SystemUsermenuPlugin::usermenu() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Available Fixes

Duplication Side-by-Side

Filter issues like