diff --git a/sources/admin/ManageBans.php b/sources/admin/ManageBans.php index 9c296cc9e2..b83eae43e3 100644 --- a/sources/admin/ManageBans.php +++ b/sources/admin/ManageBans.php @@ -887,7 +887,7 @@ function validateTriggers(&$triggers) else $context['ban_erros'][] = 'no_bantype_selected'; - if (isset($value)) + if (isset($value) && !is_array($value)) $log_info[] = array( 'value' => $value, 'bantype' => $key, @@ -969,18 +969,18 @@ function addTriggers($group_id = 0, $triggers = array(), $logs = array()) 'ip_high8' => 'int', ); - $ins_triggers = array(); + $insertTriggers = array(); foreach ($triggers as $key => $trigger) { // Exceptions, exceptions, exceptions...always exceptions... :P - if (is_array($trigger)) + if ($key == 'ips_m' || $key == 'ips_e') foreach ($trigger as $real_trigger) - $ins_triggers[$key] = array_merge($values, $trigger); + $insertTriggers[] = array_merge($values, $real_trigger); else - $ins_triggers[$key] = array_merge($values, $trigger); + $insertTriggers[] = array_merge($values, $trigger); } - if (empty($triggers)) + if (empty($insertTriggers)) $context['ban_errors'][] = 'ban_no_triggers'; if (!empty($context['ban_errors'])) @@ -989,7 +989,7 @@ function addTriggers($group_id = 0, $triggers = array(), $logs = array()) $smcFunc['db_insert']('', '{db_prefix}ban_items', $insertKeys, - $triggers, + $insertTriggers, array('id_ban') ); diff --git a/themes/default/ManageBans.template.php b/themes/default/ManageBans.template.php index d99fefb8d6..1c5a0bfeef 100644 --- a/themes/default/ManageBans.template.php +++ b/themes/default/ManageBans.template.php @@ -130,7 +130,7 @@ function template_ban_edit()
- + :
@@ -144,13 +144,14 @@ function template_ban_edit()
', $txt['ips_in_messages'], ':
'; + $count = 0; foreach ($context['ban_suggestions']['message_ips'] as $ip) echo '
- +
- ', $ip, ' +
'; } @@ -161,13 +162,14 @@ function template_ban_edit()
', $txt['ips_in_errors'], '
'; + $count = 0; foreach ($context['ban_suggestions']['error_ips'] as $ip) echo '
- +
- ', $ip, ' +
'; }