Skip to content

Commit 11d6832

Browse files
committed
Convert JS objects to JSON for easier handling
1 parent bb22b4b commit 11d6832

File tree

2 files changed

+6
-9
lines changed

2 files changed

+6
-9
lines changed

web/includes/sb-callback.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2106,6 +2106,9 @@ function EditGroup($gid, $web_flags, $srv_flags, $type, $name, $overrides, $newO
21062106
$GLOBALS['db']->Execute("UPDATE `".DB_PREFIX."_admins` SET `srv_group` = ? WHERE `aid` = '" . (int)$o['aid'] . "'", array($name));
21072107
}
21082108

2109+
$overrides = json_decode(html_entity_decode($overrides, ENT_QUOTES), true);
2110+
$newOverride = json_decode(html_entity_decode($newOverride, ENT_QUOTES), true);
2111+
21092112
// Update group overrides
21102113
if(!empty($overrides))
21112114
{
@@ -2136,12 +2139,6 @@ function EditGroup($gid, $web_flags, $srv_flags, $type, $name, $overrides, $newO
21362139
}
21372140
}
21382141

2139-
$newOverride = explode(';', $newOverride);
2140-
foreach ($newOverride as $input) {
2141-
$input = explode('::', $input);
2142-
$newOverride[$input[0]] = $input[1];
2143-
}
2144-
21452142
// Add a new override
21462143
if(!empty($newOverride))
21472144
{

web/scripts/sourcebans.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -755,18 +755,18 @@ function ProcessEditGroup(type, name)
755755
if($type(override_access) == "element")
756756
override_access = [override_access];
757757

758-
overrides = new Array(override_id.length);
758+
overrides = {};
759759

760760
for(var i=0;i<override_id.length;i++)
761761
{
762762
overrides[i] = {'id': override_id[i].value, 'type': override_type[i][override_type[i].selectedIndex].value, 'name': override_name[i].value, 'access': override_access[i][override_access[i].selectedIndex].value};
763763
}
764764
}
765765

766-
new_override = {'type::': $('new_override_type')[$('new_override_type').selectedIndex].value, ';name::': $('new_override_name').value, ';access::': $('new_override_access')[$('new_override_access').selectedIndex].value};
766+
new_override = {'type': $('new_override_type')[$('new_override_type').selectedIndex].value, 'name': $('new_override_name').value, 'access': $('new_override_access')[$('new_override_access').selectedIndex].value};
767767
}
768768

769-
xajax_EditGroup(group, Mask, srvMask, type, name, overrides, new_override);
769+
xajax_EditGroup(group, Mask, srvMask, type, name, JSON.stringify(overrides), JSON.stringify(new_override));
770770
}
771771

772772
function update_server()

0 commit comments

Comments
 (0)