fixed a couple of stupid things

Signed-off-by: emanuele <[email protected]>

Sources/ManageBans.php

@@ -890,7 +890,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,
@@ -972,18 +972,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']))
@@ -992,7 +992,7 @@ function addTriggers($group_id = 0, $triggers = array(), $logs = array())
 	$smcFunc['db_insert']('',
 		'{db_prefix}ban_items',
 		$insertKeys,
-		$triggers,
+		$insertTriggers,
 		array('id_ban')
 	);
 

Themes/default/ManageBans.template.php

@@ -126,7 +126,7 @@ function template_ban_edit()
 								<input type="text" name="email" value="', $context['ban_suggestions']['email'], '" size="44" onfocus="document.getElementById(\'email_check\').checked = true;" class="input_text" />
 							</dd>
 							<dt>
-								<input type="checkbox" name="ban_suggestions[]" id="user_check" value="user" class="input_check" ', !empty($context['ban_suggestions']['user']) ? 'checked="checked" ' : '', '/>
+								<input type="checkbox" name="ban_suggestions[]" id="user_check" value="user" class="input_check" ', !empty($context['ban_suggestions']['user']) ||  isset($context['ban']['from_user']) ? 'checked="checked" ' : '', '/>
 								<label for="user_check">', $txt['ban_on_username'], '</label>:
 							</dt>
 							<dd>
@@ -140,13 +140,14 @@ function template_ban_edit()
 					<div>', $txt['ips_in_messages'], ':</div>
 					<dl class="settings">';
 
+			$count = 0;
 			foreach ($context['ban_suggestions']['message_ips'] as $ip)
 				echo '
 							<dt>
-								<input type="checkbox" name="ban_suggestions[ips_m][]" ', !empty($context['ban_suggestions']['ips_m']) && in_array($ip, $context['ban_suggestions']['ips_m']) ? 'checked="checked" ' : '', 'value="', $ip, '" class="input_check" />
+								<input type="checkbox" id="ban_suggestions_ips_m_', $count, '" name="ban_suggestions[ips_m][]" ', !empty($context['ban_suggestions']['ips_m']) && in_array($ip, $context['ban_suggestions']['ips_m']) ? 'checked="checked" ' : '', 'value="', $ip, '" class="input_check" />
 							</dt>
 							<dd>
-								', $ip, '
+								<label for="ban_suggestions_ips_m_', $count++, '">', $ip, '</label>
 							</dd>';
 		}
 
@@ -157,13 +158,14 @@ function template_ban_edit()
 					<div>', $txt['ips_in_errors'], '</div>
 					<dl class="settings">';
 
+			$count = 0;
 			foreach ($context['ban_suggestions']['error_ips'] as $ip)
 				echo '
 							<dt>
-								<input type="checkbox" name="ban_suggestions[ips_e][]" ', !empty($context['ban_suggestions']['ips_e']) && in_array($ip, $context['ban_suggestions']['ips_e']) ? 'checked="checked" ' : '', 'value="', $ip, '" class="input_check" />
+								<input type="checkbox" id="ban_suggestions_ips_e_', $count, '" name="ban_suggestions[ips_e][]" ', !empty($context['ban_suggestions']['ips_e']) && in_array($ip, $context['ban_suggestions']['ips_e']) ? 'checked="checked" ' : '', 'value="', $ip, '" class="input_check" />
 							</dt>
 							<dd>
-								', $ip, '
+								<label for="ban_suggestions_ips_m_', $count++, '">', $ip, '</label>
 							</dd>';
 		}