adslightMaps() - Code Metrics - Inspection of "2.3-RC3" - mambax7/adslight - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d50017...68c32a )

by Michael

created 2019-04-22 17:29 UTC

adslightMaps() A

↳ Parent: Project

Complexity

Conditions	3
Paths	3

Size

Total Lines	52
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	36
c	0
b	0
f	0
dl	0
loc	52
rs	9.344
cc	3
nc	3
nop	0

How to fix Long Method

<?php
/*
-------------------------------------------------------------------------
                     ADSLIGHT 2 : Module for Xoops

        Redesigned and ameliorate By Luc Bizet user at www.frxoops.org
        Started with the Classifieds module and made MANY changes
        Website : http://www.luc-bizet.fr
        Contact : [email protected]
-------------------------------------------------------------------------
             Original credits below Version History
##########################################################################
#                    Classified Module for Xoops                         #
#  By John Mordo user jlm69 at www.xoops.org and www.jlmzone.com         #
#      Started with the MyAds module and made MANY changes               #
##########################################################################
 Original Author: Pascal Le Boustouller
 Author Website : [email protected]
 Licence Type   : GPL
-------------------------------------------------------------------------
*/

use Xmf\Request;

require_once __DIR__ . '/header.php';
//require_once XOOPS_ROOT_PATH . '/modules/adslight/include/gtickets.php';

$myts      = \MyTextSanitizer::getInstance();
$module_id = $xoopsModule->getVar('mid');

if (is_object($GLOBALS['xoopsUser'])) {
    $groups = $GLOBALS['xoopsUser']->getGroups();
} else {
    $groups = XOOPS_GROUP_ANONYMOUS;
}
/** @var \XoopsGroupPermHandler $grouppermHandler */
$grouppermHandler = xoops_getHandler('groupperm');
$perm_itemid      = Request::getInt('item_id', 0, 'POST');
//If no access
if (!$grouppermHandler->checkRight('adslight_view', $perm_itemid, $groups, $module_id)) {
    redirect_header(XOOPS_URL . '/index.php', 3, _NOPERM);
}
if (!$grouppermHandler->checkRight('adslight_premium', $perm_itemid, $groups, $module_id)) {
    $prem_perm = '0';
} else {
    $prem_perm = '1';
}

#  function adslightMaps
#####################################################
function adslightMaps()
{
    global $xoopsDB, $xoopsConfig, $xoopsModule, $myts, $prem_perm;

    $GLOBALS['xoopsOption']['template_main'] = 'adslight_maps.tpl';

    require_once XOOPS_ROOT_PATH . '/header.php';

    $GLOBALS['xoopsTpl']->assign('xmid', $xoopsModule->getVar('mid'));
    $GLOBALS['xoopsTpl']->assign('add_from', _ADSLIGHT_ADDFROM . ' ' . $xoopsConfig['sitename']);
    $GLOBALS['xoopsTpl']->assign('add_from_title', _ADSLIGHT_ADDFROM);
    $GLOBALS['xoopsTpl']->assign('add_from_sitename', $xoopsConfig['sitename']);
    $GLOBALS['xoopsTpl']->assign('search_listings', _ADSLIGHT_SEARCH_LISTINGS);
    $GLOBALS['xoopsTpl']->assign('all_words', _ADSLIGHT_ALL_WORDS);
    $GLOBALS['xoopsTpl']->assign('any_words', _ADSLIGHT_ANY_WORDS);
    $GLOBALS['xoopsTpl']->assign('exact_match', _ADSLIGHT_EXACT_MATCH);
    $GLOBALS['xoopsTpl']->assign('only_pix', _ADSLIGHT_ONLYPIX);
    $GLOBALS['xoopsTpl']->assign('search', _ADSLIGHT_SEARCH);
    $GLOBALS['xoopsTpl']->assign('permit', $prem_perm);
    $GLOBALS['xoopsTpl']->assign('imgscss', XOOPS_URL . '/modules/adslight/assets/css/adslight.css');
    $GLOBALS['xoopsTpl']->assign('adslight_logolink', _ADSLIGHT_LOGOLINK);

    $GLOBALS['xoTheme']->addMeta('meta', 'robots', 'noindex, nofollow');

    $header_cssadslight = '<link rel="stylesheet" href="' . XOOPS_URL . '/modules/adslight/assets/css/adslight.css" type="text/css" media="all" >';

    $GLOBALS['xoopsTpl']->assign('xoops_module_header', $header_cssadslight);

    $maps_name   = $GLOBALS['xoopsModuleConfig']['adslight_maps_set'];
    $maps_width  = $GLOBALS['xoopsModuleConfig']['adslight_maps_width'];
    $maps_height = $GLOBALS['xoopsModuleConfig']['adslight_maps_height'];

    $GLOBALS['xoopsTpl']->assign('maps_name', $maps_name);
    $GLOBALS['xoopsTpl']->assign('maps_width', $maps_width);
    $GLOBALS['xoopsTpl']->assign('maps_height', $maps_height);

    $GLOBALS['xoopsTpl']->assign('adlight_maps_title', _ADSLIGHT_MAPS_TITLE);
    $GLOBALS['xoopsTpl']->assign('bullinfotext', _ADSLIGHT_MAPS_TEXT);

    // adslight 2
    $GLOBALS['xoopsTpl']->assign('adslight_active_menu', $GLOBALS['xoopsModuleConfig']['adslight_active_menu']);
    $GLOBALS['xoopsTpl']->assign('adslight_active_rss', $GLOBALS['xoopsModuleConfig']['adslight_active_rss']);

    if ($GLOBALS['xoopsUser']) {
        $member_usid = $GLOBALS['xoopsUser']->getVar('uid');
        if ($usid = $member_usid) {

            $GLOBALS['xoopsTpl']->assign('istheirs', true);

            list($show_user) = $xoopsDB->fetchRow($xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('adslight_listing') . " WHERE usid=$member_usid"));

            $GLOBALS['xoopsTpl']->assign('show_user', $show_user);
            $GLOBALS['xoopsTpl']->assign('show_user_link', "members.php?usid=$member_usid");
        }
    }
}

######################################################

$pa = Request::getInt('pa', null, 'GET');

switch ($pa) {
    default:
        $GLOBALS['xoopsOption']['template_main'] = 'adslight_maps.tpl';
        adslightMaps();
        break;
}
require_once XOOPS_ROOT_PATH . '/footer.php';


1			<?php
2			/*
3			-------------------------------------------------------------------------
4			ADSLIGHT 2 : Module for Xoops
5
6			Redesigned and ameliorate By Luc Bizet user at www.frxoops.org
7			Started with the Classifieds module and made MANY changes
8			Website : http://www.luc-bizet.fr
9			Contact : [email protected]
10			-------------------------------------------------------------------------
11			Original credits below Version History
12			##########################################################################
13			# Classified Module for Xoops #
14			# By John Mordo user jlm69 at www.xoops.org and www.jlmzone.com #
15			# Started with the MyAds module and made MANY changes #
16			##########################################################################
17			Original Author: Pascal Le Boustouller
18			Author Website : [email protected]
19			Licence Type : GPL
20			-------------------------------------------------------------------------
21			*/
22
23			use Xmf\Request;
24
25			require_once __DIR__ . '/header.php';
26			//require_once XOOPS_ROOT_PATH . '/modules/adslight/include/gtickets.php';
27
28			$myts = \MyTextSanitizer::getInstance();
29			$module_id = $xoopsModule->getVar('mid');
30
31			if (is_object($GLOBALS['xoopsUser'])) {
32			$groups = $GLOBALS['xoopsUser']->getGroups();
33			} else {
34			$groups = XOOPS_GROUP_ANONYMOUS;
35			}
36			/** @var \XoopsGroupPermHandler $grouppermHandler */
37			$grouppermHandler = xoops_getHandler('groupperm');
38			$perm_itemid = Request::getInt('item_id', 0, 'POST');
39			//If no access
40			if (!$grouppermHandler->checkRight('adslight_view', $perm_itemid, $groups, $module_id)) {
41			redirect_header(XOOPS_URL . '/index.php', 3, _NOPERM);
42			}
43			if (!$grouppermHandler->checkRight('adslight_premium', $perm_itemid, $groups, $module_id)) {
44			$prem_perm = '0';
45			} else {
46			$prem_perm = '1';
47			}
48
49			# function adslightMaps
50			#####################################################
51			function adslightMaps()
52			{
53			global $xoopsDB, $xoopsConfig, $xoopsModule, $myts, $prem_perm;
54
55			$GLOBALS['xoopsOption']['template_main'] = 'adslight_maps.tpl';
56
57			require_once XOOPS_ROOT_PATH . '/header.php';
58
59			$GLOBALS['xoopsTpl']->assign('xmid', $xoopsModule->getVar('mid'));
60			$GLOBALS['xoopsTpl']->assign('add_from', _ADSLIGHT_ADDFROM . ' ' . $xoopsConfig['sitename']);
61			$GLOBALS['xoopsTpl']->assign('add_from_title', _ADSLIGHT_ADDFROM);
62			$GLOBALS['xoopsTpl']->assign('add_from_sitename', $xoopsConfig['sitename']);
63			$GLOBALS['xoopsTpl']->assign('search_listings', _ADSLIGHT_SEARCH_LISTINGS);
64			$GLOBALS['xoopsTpl']->assign('all_words', _ADSLIGHT_ALL_WORDS);
65			$GLOBALS['xoopsTpl']->assign('any_words', _ADSLIGHT_ANY_WORDS);
66			$GLOBALS['xoopsTpl']->assign('exact_match', _ADSLIGHT_EXACT_MATCH);
67			$GLOBALS['xoopsTpl']->assign('only_pix', _ADSLIGHT_ONLYPIX);
68			$GLOBALS['xoopsTpl']->assign('search', _ADSLIGHT_SEARCH);
69			$GLOBALS['xoopsTpl']->assign('permit', $prem_perm);
70			$GLOBALS['xoopsTpl']->assign('imgscss', XOOPS_URL . '/modules/adslight/assets/css/adslight.css');
71			$GLOBALS['xoopsTpl']->assign('adslight_logolink', _ADSLIGHT_LOGOLINK);
72
73			$GLOBALS['xoTheme']->addMeta('meta', 'robots', 'noindex, nofollow');
74
75			$header_cssadslight = '<link rel="stylesheet" href="' . XOOPS_URL . '/modules/adslight/assets/css/adslight.css" type="text/css" media="all" >';
76
77			$GLOBALS['xoopsTpl']->assign('xoops_module_header', $header_cssadslight);
78
79			$maps_name = $GLOBALS['xoopsModuleConfig']['adslight_maps_set'];
80			$maps_width = $GLOBALS['xoopsModuleConfig']['adslight_maps_width'];
81			$maps_height = $GLOBALS['xoopsModuleConfig']['adslight_maps_height'];
82
83			$GLOBALS['xoopsTpl']->assign('maps_name', $maps_name);
84			$GLOBALS['xoopsTpl']->assign('maps_width', $maps_width);
85			$GLOBALS['xoopsTpl']->assign('maps_height', $maps_height);
86
87			$GLOBALS['xoopsTpl']->assign('adlight_maps_title', _ADSLIGHT_MAPS_TITLE);
88			$GLOBALS['xoopsTpl']->assign('bullinfotext', _ADSLIGHT_MAPS_TEXT);
89
90			// adslight 2
91			$GLOBALS['xoopsTpl']->assign('adslight_active_menu', $GLOBALS['xoopsModuleConfig']['adslight_active_menu']);
92			$GLOBALS['xoopsTpl']->assign('adslight_active_rss', $GLOBALS['xoopsModuleConfig']['adslight_active_rss']);
93
94			if ($GLOBALS['xoopsUser']) {
95			$member_usid = $GLOBALS['xoopsUser']->getVar('uid');
96			if ($usid = $member_usid) {
			0 ignored issues – show Unused Code introduced 2018-03-16 07:40 UTC by Report Bug Copy Issue Report The assignment to `$usid` is dead and can be removed. Loading history...
97			$GLOBALS['xoopsTpl']->assign('istheirs', true);
98
99			list($show_user) = $xoopsDB->fetchRow($xoopsDB->query('SELECT COUNT(*) FROM ' . $xoopsDB->prefix('adslight_listing') . " WHERE usid=$member_usid"));
100
101			$GLOBALS['xoopsTpl']->assign('show_user', $show_user);
102			$GLOBALS['xoopsTpl']->assign('show_user_link', "members.php?usid=$member_usid");
103			}
104			}
105			}
106
107			######################################################
108
109			$pa = Request::getInt('pa', null, 'GET');
110
111			switch ($pa) {
112			default:
113			$GLOBALS['xoopsOption']['template_main'] = 'adslight_maps.tpl';
114			adslightMaps();
115			break;
116			}
117			require_once XOOPS_ROOT_PATH . '/footer.php';
118

mambax7 / adslight

Push — master ( d50017...68c32a )

adslightMaps() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like