wggithub_notify_iteminfo() - Code Metrics - ggoffy/wggithub - Measure and Improve Code Quality continuously with Scrutinizer

wggithub_notify_iteminfo() A
last analyzed 2025-03-21 14:15 UTC

↳ Parent: Project

Complexity

Conditions	4
Paths	6

Size

Total Lines	24
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	4
eloc	18
c	1
b	0
f	1
nc	6
nop	2
dl	0
loc	24
rs	9.6666

<?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.
*/

/**
 * wgGitHub module for xoops
 *
 * @copyright      2020 XOOPS Project (https://xooops.org)
 * @license        GPL 2.0 or later
 * @package        wggithub
 * @since          1.0
 * @min_xoops      2.5.10
 * @author         Goffy - XOOPS Development Team - Email:<[email protected]> - Website:<https://wedega.com>
 */

/**
 * comment callback functions
 *
 * @param  $category
 * @param  $item_id
 * @return array item|null
 */
function wggithub_notify_iteminfo($category, $item_id)
{
    global $xoopsDB;

    if (!\defined('WGGITHUB_URL')) {
        \define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');

    }

    switch ($category) {
        case 'global':
            $item['name'] = '';

            $item['url']  = '';
            return $item;
            break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        case 'repositories':
            $sql          = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
            $result       = $xoopsDB->query($sql);
            $result_array = $xoopsDB->fetchArray($result);
            $item['name'] = $result_array['repo_name'];
            $item['url']  = \WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
            return $item;
            break;
    }
    return null;
}


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			* wgGitHub module for xoops
14			*
15			* @copyright 2020 XOOPS Project (https://xooops.org)
16			* @license GPL 2.0 or later
17			* @package wggithub
18			* @since 1.0
19			* @min_xoops 2.5.10
20			* @author Goffy - XOOPS Development Team - Email:<[email protected]> - Website:<https://wedega.com>
21			*/
22
23			/**
24			* comment callback functions
25			*
26			* @param $category
27			* @param $item_id
28			* @return array item\|null
29			*/
30			function wggithub_notify_iteminfo($category, $item_id)
31			{
32			global $xoopsDB;
33
34			if (!\defined('WGGITHUB_URL')) {
35			\define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
			0 ignored issues – show Bug introduced 2020-11-26 17:45 UTC by Report Bug Copy Issue Report The constant `XOOPS_URL` was not found. Maybe you did not declare it correctly or list all dependencies? Loading history...
36			}
37
38			switch ($category) {
39			case 'global':
40			$item['name'] = '';
			0 ignored issues – show Comprehensibility Best Practice introduced 2020-11-26 17:45 UTC by Report Bug Copy Issue Report `$item` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$item = array();` before regardless. Loading history...
41			$item['url'] = '';
42			return $item;
43			break;
			0 ignored issues – show Unused Code introduced 2020-11-26 17:45 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
44			case 'repositories':
45			$sql = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
46			$result = $xoopsDB->query($sql);
47			$result_array = $xoopsDB->fetchArray($result);
48			$item['name'] = $result_array['repo_name'];
49			$item['url'] = \WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
50			return $item;
51			break;
52			}
53			return null;
54			}
55

ggoffy / wggithub

wggithub_notify_iteminfo() A last analyzed 2025-03-21 14:15 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

wggithub_notify_iteminfo() A
last analyzed 2025-03-21 14:15 UTC