mygroupperm.php ➔ myDeleteByModule()   A
last analyzed

Complexity

Conditions 4
Paths 6

Size

Total Lines 15

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 4
nc 6
nop 4
dl 0
loc 15
rs 9.7666
c 0
b 0
f 0
1
<?php
2
3
if (!defined('XOOPS_ROOT_PATH')) {
4
    exit;
5
}
6
7
/**
8
 * @param      $DB
9
 * @param      $gperm_modid
10
 * @param null $gperm_name
11
 * @param null $gperm_itemid
12
 * @return bool
13
 */
14
function myDeleteByModule($DB, $gperm_modid, $gperm_name = null, $gperm_itemid = null)
15
{
16
    $criteria = new CriteriaCompo(new Criteria('gperm_modid', (int)$gperm_modid));
17
    if (isset($gperm_name)) {
18
        $criteria->add(new Criteria('gperm_name', $gperm_name));
19
        if (isset($gperm_itemid)) {
20
            $criteria->add(new Criteria('gperm_itemid', (int)$gperm_itemid));
21
        }
22
    }
23
    $sql    = 'DELETE FROM ' . $DB->prefix('group_permission') . ' ' . $criteria->renderWhere();
24
    $result = $DB->query($sql);
25
    $result = $result ? true : false;
26
27
    return $result;
28
}
29
30
$modid = isset($_POST['modid']) ? (int)$_POST['modid'] : 1;
31
32
// we dont want system module permissions to be changed here ( 1 -> 0 GIJ)
33
if ($modid <= 0 || !is_object($xoopsUser) || !$xoopsUser->isAdmin($modid)) {
34
    redirect_header(XOOPS_URL . '/user.php', 1, _NOPERM);
35
    exit();
36
}
37
$moduleHandler = xoops_getHandler('module');
38
$module         = $moduleHandler->get($modid);
39
if (!is_object($module) || !$module->getVar('isactive')) {
40
    redirect_header(XOOPS_URL . '/admin.php', 1, _MODULENOEXIST);
41
    exit();
42
}
43
$memberHandler = xoops_getHandler('member');
44
$group_list     = $memberHandler->getGroupList();
45
if (is_array($_POST['perms']) && !empty($_POST['perms'])) {
46
    $gpermHandler = xoops_getHandler('groupperm');
47
    foreach ($_POST['perms'] as $perm_name => $perm_data) {
48
        foreach ($perm_data['itemname'] as $item_id => $item_name) {
49
            if (false != myDeleteByModule($gpermHandler->db, $modid, $perm_name, $item_id)) {
0 ignored issues
show
Coding Style Best Practice introduced by
It seems like you are loosely comparing two booleans. Considering using the strict comparison !== instead.

When comparing two booleans, it is generally considered safer to use the strict comparison operator.

Loading history...
50
                if (!empty($perm_data['groups'])) {
51
                    foreach ($perm_data['groups'] as $group_id => $item_ids) {
52
                        $selected = isset($item_ids[$item_id]) ? $item_ids[$item_id] : 0;
53
                        if ($selected == 1) {
54
                            // make sure that all parent ids are selected as well
55
                            if ($perm_data['parents'][$item_id] != '') {
56
                                $parent_ids = explode(':', $perm_data['parents'][$item_id]);
57
                                foreach ($parent_ids as $pid) {
58
                                    if ($pid != 0 && !in_array($pid, array_keys($item_ids))) {
59
                                        // one of the parent items were not selected, so skip this item
60
                                        $msg[] = sprintf(_MD_AM_PERMADDNG, '<strong>' . $perm_name . '</strong>', '<strong>' . $perm_data['itemname'][$item_id] . '</strong>', '<strong>' . $group_list[$group_id] . '</strong>') . ' (' . _MD_AM_PERMADDNGP . ')';
61
                                        continue 2;
62
                                    }
63
                                }
64
                            }
65
                            $gperm = $gpermHandler->create();
66
                            $gperm->setVar('gperm_groupid', $group_id);
67
                            $gperm->setVar('gperm_name', $perm_name);
68
                            $gperm->setVar('gperm_modid', $modid);
69
                            $gperm->setVar('gperm_itemid', $item_id);
70
                            if (!$gpermHandler->insert($gperm)) {
71
                                $msg[] = sprintf(_MD_AM_PERMADDNG, '<strong>' . $perm_name . '</strong>', '<strong>' . $perm_data['itemname'][$item_id] . '</strong>', '<strong>' . $group_list[$group_id] . '</strong>');
72
                            } else {
73
                                $msg[] = sprintf(_MD_AM_PERMADDOK, '<strong>' . $perm_name . '</strong>', '<strong>' . $perm_data['itemname'][$item_id] . '</strong>', '<strong>' . $group_list[$group_id] . '</strong>');
74
                            }
75
                            unset($gperm);
76
                        }
77
                    }
78
                }
79
            } else {
80
                $msg[] = sprintf(_MD_AM_PERMRESETNG, $module->getVar('name'));
81
            }
82
        }
83
    }
84
}
85