elkarte / Elkarte

themes/default/ManageCalendar.template.php

Braces +15 added lines, -6 removed lines

@@ -42,8 +42,10 @@ 
 
 	// Show a list of all the years we allow...
 	for ($year = $context['cal_minyear']; $year <= $context['cal_maxyear']; $year++)
-		echo '
+	{
+			echo '
 							<option value="', $year, '"', $year == $context['holiday']['year'] ? ' selected="selected"' : '', '>', $year, '</option>';
+	}
 
 	echo '
 						</select>
@@ -52,8 +54,10 @@ 
 
 	// There are 12 months per year - ensure that they all get listed.
 	for ($month = 1; $month <= 12; $month++)
-		echo '
+	{
+			echo '
 							<option value="', $month, '"', $month == $context['holiday']['month'] ? ' selected="selected"' : '', '>', $txt['months'][$month], '</option>';
+	}
 
 	echo '
 						</select>
@@ -62,8 +66,10 @@ 
 
 	// This prints out all the days in the current month - this changes dynamically as we switch months.
 	for ($day = 1; $day <= $context['holiday']['last_day']; $day++)
-		echo '
+	{
+			echo '
 							<option value="', $day, '"', $day == $context['holiday']['day'] ? ' selected="selected"' : '', '>', $day, '</option>';
+	}
 
 	echo '
 						</select>
@@ -73,13 +79,16 @@ 
 				<div class="submitbutton">';
 
 	if ($context['is_new'])
-		echo '
+	{
+			echo '
 						<input type="submit" value="', $txt['holidays_button_add'], '" />';
-	else
-		echo '
+	}
+	else {
+			echo '
 						<input type="submit" name="edit" value="', $txt['holidays_button_edit'], '" />
 						<input type="submit" name="delete" value="', $txt['holidays_button_remove'], '" />
 						<input type="hidden" name="holiday" value="', $context['holiday']['id'], '" />';
+	}
 
 	echo '
 						<input type="hidden" name="', $context['session_var'], '" value="', $context['session_id'], '" />

themes/default/ManageBans.template.php

Braces +22 added lines, -8 removed lines

@@ -28,8 +28,10 @@ 
 			</h2>';
 
 	if ($context['ban']['is_new'])
-		echo '
+	{
+			echo '
 			<div class="information">', $txt['ban_add_notes'], '</div>';
+	}
 
 	// If there were errors creating the ban, show them.
 	template_show_error('ban_errors');
@@ -45,7 +47,8 @@ 
 					</dd>';
 
 	if (isset($context['ban']['reason']))
-		echo '
+	{
+			echo '
 				<dt>
 					<label for="reason">', $txt['ban_reason'], ':</label><br />
 					<span class="smalltext">', $txt['ban_reason_desc'], '</span>
@@ -53,9 +56,11 @@ 
 				<dd>
 					<textarea name="reason" id="reason" cols="40" rows="3" class="ban_text">', $context['ban']['reason'], '</textarea>
 				</dd>';
+	}
 
 	if (isset($context['ban']['notes']))
-		echo '
+	{
+			echo '
 				<dt>
 					<label for="ban_notes">', $txt['ban_notes'], ':</label><br />
 					<span class="smalltext">', $txt['ban_notes_desc'], '</span>
@@ -63,6 +68,7 @@ 
 				<dd>
 					<textarea name="notes" id="ban_notes" cols="40" rows="3" class="ban_text">', $context['ban']['notes'], '</textarea>
 				</dd>';
+	}
 
 	echo '
 				</dl>
@@ -102,7 +108,8 @@ 
 						</dd>';
 
 		if (empty($modSettings['disableHostnameLookup']))
-			echo '
+		{
+					echo '
 						<dt>
 							<input type="checkbox" name="ban_suggestions[]" id="hostname_check" value="hostname" ', !empty($context['ban_suggestions']['hostname']) ? 'checked="checked" ' : '', '/>
 							<label for="hostname_check">', $txt['ban_on_hostname'], '</label>
@@ -110,6 +117,7 @@ 
 						<dd>
 							<input type="text" name="hostname" value="', $context['ban_suggestions']['hostname'], '" size="44" onfocus="document.getElementById(\'hostname_check\').checked = true;" class="input_text" />
 						</dd>';
+		}
 
 		echo '
 						<dt>
@@ -140,13 +148,15 @@ 
 
 					$count = 0;
 					foreach ($ban_ips as $ip)
-						echo '
+					{
+											echo '
 						<dt>
 							<input type="checkbox" id="suggestions_', $key, '_', $count, '" name="ban_suggestions[', $key, '][]" ', !empty($context['ban_suggestions']['saved_triggers'][$key]) && in_array($ip, $context['ban_suggestions']['saved_triggers'][$key]) ? 'checked="checked" ' : '', 'value="', $ip, '" />
 						</dt>
 						<dd>
 							<label for="suggestions_', $key, '_', ($count++), '">', $ip, '</label>
 						</dd>';
+					}
 
 					echo '
 					</dl>';
@@ -183,7 +193,8 @@ 
 
 	// Auto suggest only needed for adding new bans, not editing
 	if (!empty($context['use_autosuggest']))
-		echo '
+	{
+			echo '
 	<script>
 		var oAddMemberSuggest = new smc_AutoSuggest({
 			sSelf: \'oAddMemberSuggest\',
@@ -198,7 +209,8 @@ 
 
 		oAddMemberSuggest.registerCallback(\'onBeforeUpdate\', \'onUpdateName\');
 	</script>';
-}
+	}
+	}
 
 /**
  * Template to edit ban triggers
@@ -228,7 +240,8 @@ 
 						</dd>';
 
 	if (empty($modSettings['disableHostnameLookup']))
-		echo '
+	{
+			echo '
 						<dt>
 							<input type="checkbox" name="ban_suggestions[]" id="hostname_check" value="hostname" ', $context['ban_trigger']['hostname']['selected'] ? 'checked="checked" ' : '', '/>
 							<label for="hostname_check">', $txt['ban_on_hostname'], '</label>
@@ -236,6 +249,7 @@ 
 						<dd>
 							<input type="text" name="hostname" value="', $context['ban_trigger']['hostname']['value'], '" size="44" onfocus="document.getElementById(\'hostname_check\').checked = true;" class="input_text" />
 						</dd>';
+	}
 
 	echo '
 						<dt>

themes/default/Emailuser.template.php

Braces +12 added lines, -5 removed lines

@@ -116,7 +116,8 @@ 
 
 	// Can the user see the persons email?
 	if ($context['can_view_recipient_email'])
-		echo '
+	{
+			echo '
 					<dt>
 						<strong>', $txt['sendtopic_receiver_email'], ':</strong>
 					</dt>
@@ -126,10 +127,12 @@ 
 				</dl>
 				<hr />
 				<dl class="settings send_mail">';
+	}
 
 	// If it's a guest we need their details.
 	if ($context['user']['is_guest'])
-		echo '
+	{
+			echo '
 					<dt>
 						<label for="y_name">', $txt['sendtopic_sender_name'], ':</label>
 					</dt>
@@ -143,9 +146,10 @@ 
 					<dd>
 						<input type="text" id="y_mail" name="y_email" size="24" maxlength="50" value="', $context['user']['email'], '" class="input_text" />
 					</dt>';
+	}
 	// Otherwise show the user that we know their email.
-	else
-		echo '
+	else {
+			echo '
 					<dt>
 						<strong>', $txt['sendtopic_sender_email'], ':</strong><br />
 						<span class="smalltext">', $txt['send_email_disclosed'], '</span>
@@ -153,6 +157,7 @@ 
 					<dd>
 						<em>', $context['user']['email'], '</em>
 					</dd>';
+	}
 
 	echo '
 					<dt>
@@ -174,8 +179,10 @@ 
 
 
 	foreach ($context['form_hidden_vars'] as $key => $value)
-		echo '
+	{
+			echo '
 					<input type="hidden" name="', $key, '" value="', $value, '" />';
+	}
 
 	echo '
 					<input type="hidden" name="', $context['session_var'], '" value="', $context['session_id'], '" />

sources/SiteCombiner.class.php

Braces +36 added lines, -14 removed lines

@@ -168,7 +168,9 @@ 
 
 		// Nothing to do, then we are done
 		if (count($this->_combine_files) === 0)
-			return true;
+		{
+					return true;
+		}
 
 		// Create the archive name
 		$this->_buildName('.js');
@@ -226,7 +228,9 @@ 
 
 		// Nothing to do so return
 		if (count($this->_combine_files) === 0)
-			return true;
+		{
+					return true;
+		}
 
 		// Create the css archive name
 		$this->_buildName('.css');
@@ -373,7 +377,9 @@ 
 		foreach ($this->_combine_files as $file)
 		{
 			if ($file['filemtime'] > $filemtime)
-				return true;
+			{
+							return true;
+			}
 		}
 
 		return false;
@@ -390,7 +396,9 @@ 
 
 		// Create this groups archive name
 		foreach ($this->_combine_files as $file)
-			$this->_archive_filenames .= $file['basename'] . ' ';
+		{
+					$this->_archive_filenames .= $file['basename'] . ' ';
+		}
 
 		// Add in the actual theme url to make the sha1 unique to this hive
 		$this->_archive_filenames = $settings['actual_theme_url'] . '/' . trim($this->_archive_filenames);
@@ -400,7 +408,9 @@ 
 
 		// Create a unique cache stale for this hive ?x12345
 		if (!empty($this->_stales))
-			$this->_archive_stale = '?x' . hash('crc32b', implode(' ', $this->_stales));
+		{
+					$this->_archive_stale = '?x' . hash('crc32b', implode(' ', $this->_stales));
+		}
 	}
 
 	/**
@@ -445,9 +455,12 @@ 
 
 			// Add the file to the correct array for processing
 			if ($file['minimized'] === false)
-				$_cache[] = $tempfile;
-			else
-				$_min_cache[] = $tempfile;
+			{
+							$_cache[] = $tempfile;
+			}
+			else {
+							$_min_cache[] = $tempfile;
+			}
 		}
 
 		// Build out our combined file strings
@@ -463,7 +476,9 @@ 
 	{
 		// Add in the file header if available
 		if (!empty($this->_archive_header))
-			$this->_minified_cache = $this->_archive_header . $this->_minified_cache;
+		{
+					$this->_minified_cache = $this->_archive_header . $this->_minified_cache;
+		}
 
 		// First the plain text version
 		file_put_contents($this->_archive_dir . '/' . $this->_archive_name, $this->_minified_cache, LOCK_EX);
@@ -500,7 +515,9 @@ 
 			// To prevent a stack overflow segmentation fault, which silently kills Apache, we need to limit
 			// recursion on windows.  This may cause JSqueeze to fail, but at least its then catchable.
 			if (detectServer()->is('windows'))
-				@ini_set('pcre.recursion_limit', '524');
+			{
+							@ini_set('pcre.recursion_limit', '524');
+			}
 
 			require_once(EXTDIR . '/JSqueeze.php');
 			$jsqueeze = new Patchwork\JSqueeze;
@@ -509,7 +526,9 @@ 
 
 		// If we still have no data, then try the post js_code method to the closure compiler
 		if ($fetch_data === false || trim($fetch_data) == '')
-			$fetch_data = $this->_closure_js_code();
+		{
+					$fetch_data = $this->_closure_js_code();
+		}
 
 		// If we have nothing to return, use the original data
 		$fetch_data = ($fetch_data === false || trim($fetch_data) == '') ? $this->_cache : $fetch_data;
@@ -534,7 +553,9 @@ 
 		foreach ($this->_combine_files as $file)
 		{
 			if ($file['minimized'] === false)
-				$post_data .= '&code_url=' . urlencode($file['url'] . '/scripts/' . $file['basename'] . $this->_archive_stale);
+			{
+							$post_data .= '&code_url=' . urlencode($file['url'] . '/scripts/' . $file['basename'] . $this->_archive_stale);
+			}
 		}
 
 		return fetch_web_data($this->_url, $this->_post_header . $post_data);
@@ -558,8 +579,9 @@ 
 			$fetch_data = fetch_web_data($this->_url, $this->_post_header . $post_data);
 		}
 		// Simply to much data for a single post so break it down in to as few as possible
-		else
-			$fetch_data = $this->_closure_js_code_chunks();
+		else {
+					$fetch_data = $this->_closure_js_code_chunks();
+		}
 
 		return $fetch_data;
 	}

sources/Subs.php

Braces +375 added lines, -151 removed lines

@@ -97,7 +97,9 @@ 
 	$cache = Cache::instance();
 
 	if (empty($changeArray) || !is_array($changeArray))
-		return;
+	{
+			return;
+	}
 
 	// In some cases, this may be better and faster, but for large sets we don't want so many UPDATEs.
 	if ($update)
@@ -128,10 +130,14 @@ 
 	{
 		// Don't bother if it's already like that ;).
 		if (isset($modSettings[$variable]) && $modSettings[$variable] == $value)
-			continue;
+		{
+					continue;
+		}
 		// If the variable isn't set, but would only be set to nothing'ness, then don't bother setting it.
 		elseif (!isset($modSettings[$variable]) && empty($value))
-			continue;
+		{
+					continue;
+		}
 
 		$replaceArray[] = array($variable, $value);
 
@@ -139,7 +145,9 @@ 
 	}
 
 	if (empty($replaceArray))
-		return;
+	{
+			return;
+	}
 
 	$db->insert('replace',
 		'{db_prefix}settings',
@@ -164,10 +172,14 @@ 
 	$db = database();
 
 	if (empty($toRemove))
-		return;
+	{
+			return;
+	}
 
 	if (!is_array($toRemove))
-		$toRemove = array($toRemove);
+	{
+			$toRemove = array($toRemove);
+	}
 
 	// Remove the setting from the db
 	$db->query('', '
@@ -180,8 +192,10 @@ 
 
 	// Remove it from $modSettings now so it does not persist
 	foreach ($toRemove as $setting)
-		if (isset($modSettings[$setting]))
+	{
+			if (isset($modSettings[$setting]))
 			unset($modSettings[$setting]);
+	}
 
 	// Kill the cache - it needs redoing now, but we won't bother ourselves with that here.
 	Cache::instance()->remove('modSettings');
@@ -228,13 +242,18 @@ 
 
 	// Make sure $start is a proper variable - not less than 0.
 	if ($start_invalid)
-		$start = 0;
+	{
+			$start = 0;
+	}
 	// Not greater than the upper bound.
 	elseif ($start >= $max_value)
-		$start = max(0, (int) $max_value - (((int) $max_value % (int) $num_per_page) == 0 ? $num_per_page : ((int) $max_value % (int) $num_per_page)));
+	{
+			$start = max(0, (int) $max_value - (((int) $max_value % (int) $num_per_page) == 0 ? $num_per_page : ((int) $max_value % (int) $num_per_page)));
+	}
 	// And it has to be a multiple of $num_per_page!
-	else
-		$start = max(0, (int) $start - ((int) $start % (int) $num_per_page));
+	else {
+			$start = max(0, (int) $start - ((int) $start % (int) $num_per_page));
+	}
 
 	$context['current_page'] = $start / $num_per_page;
 
@@ -249,12 +268,16 @@ 
 		// Show all the pages.
 		$display_page = 1;
 		for ($counter = 0; $counter < $max_value; $counter += $num_per_page)
-			$pageindex .= $start == $counter && !$start_invalid && empty($show['all_selected']) ? sprintf($settings['page_index_template']['current_page'], $display_page++) : sprintf($base_link, $counter, $display_page++);
+		{
+					$pageindex .= $start == $counter && !$start_invalid && empty($show['all_selected']) ? sprintf($settings['page_index_template']['current_page'], $display_page++) : sprintf($base_link, $counter, $display_page++);
+		}
 
 		// Show the right arrow.
 		$display_page = ($start + $num_per_page) > $max_value ? $max_value : ($start + $num_per_page);
 		if ($start != $counter - $max_value && !$start_invalid && $show['prev_next'] && empty($show['all_selected']))
-			$pageindex .= $display_page > $counter - $num_per_page ? ' ' : sprintf($base_link, $display_page, str_replace('{next_txt}', $txt['next'], $settings['page_index_template']['next_page']));
+		{
+					$pageindex .= $display_page > $counter - $num_per_page ? ' ' : sprintf($base_link, $display_page, str_replace('{next_txt}', $txt['next'], $settings['page_index_template']['next_page']));
+		}
 	}
 	else
 	{
@@ -263,61 +286,84 @@ 
 
 		// Show the "prev page" link. (>prev page< 1 ... 6 7 [8] 9 10 ... 15 next page)
 		if (!empty($start) && $show['prev_next'])
-			$pageindex = sprintf($base_link, $start - $num_per_page, str_replace('{prev_txt}', $txt['prev'], $settings['page_index_template']['previous_page']));
-		else
-			$pageindex = '';
+		{
+					$pageindex = sprintf($base_link, $start - $num_per_page, str_replace('{prev_txt}', $txt['prev'], $settings['page_index_template']['previous_page']));
+		}
+		else {
+					$pageindex = '';
+		}
 
 		// Show the first page. (prev page >1< ... 6 7 [8] 9 10 ... 15)
 		if ($start > $num_per_page * $PageContiguous)
-			$pageindex .= sprintf($base_link, 0, '1');
+		{
+					$pageindex .= sprintf($base_link, 0, '1');
+		}
 
 		// Show the ... after the first page.  (prev page 1 >...< 6 7 [8] 9 10 ... 15 next page)
 		if ($start > $num_per_page * ($PageContiguous + 1))
-			$pageindex .= str_replace('{custom}', 'data-baseurl="' . htmlspecialchars(JavaScriptEscape(($flexible_start ? $base_url : strtr($base_url, array('%' => '%%')) . ';start=%1$d')), ENT_COMPAT, 'UTF-8') . '" data-perpage="' . $num_per_page . '" data-firstpage="' . $num_per_page . '" data-lastpage="' . ($start - $num_per_page * $PageContiguous) . '"', $settings['page_index_template']['expand_pages']);
+		{
+					$pageindex .= str_replace('{custom}', 'data-baseurl="' . htmlspecialchars(JavaScriptEscape(($flexible_start ? $base_url : strtr($base_url, array('%' => '%%')) . ';start=%1$d')), ENT_COMPAT, 'UTF-8') . '" data-perpage="' . $num_per_page . '" data-firstpage="' . $num_per_page . '" data-lastpage="' . ($start - $num_per_page * $PageContiguous) . '"', $settings['page_index_template']['expand_pages']);
+		}
 
 		// Show the pages before the current one. (prev page 1 ... >6 7< [8] 9 10 ... 15 next page)
 		for ($nCont = $PageContiguous; $nCont >= 1; $nCont--)
-			if ($start >= $num_per_page * $nCont)
+		{
+					if ($start >= $num_per_page * $nCont)
 			{
 				$tmpStart = $start - $num_per_page * $nCont;
+		}
 				$pageindex .= sprintf($base_link, $tmpStart, $tmpStart / $num_per_page + 1);
 			}
 
 		// Show the current page. (prev page 1 ... 6 7 >[8]< 9 10 ... 15 next page)
 		if (!$start_invalid && empty($show['all_selected']))
-			$pageindex .= sprintf($settings['page_index_template']['current_page'], ($start / $num_per_page + 1));
-		else
-			$pageindex .= sprintf($base_link, $start, $start / $num_per_page + 1);
+		{
+					$pageindex .= sprintf($settings['page_index_template']['current_page'], ($start / $num_per_page + 1));
+		}
+		else {
+					$pageindex .= sprintf($base_link, $start, $start / $num_per_page + 1);
+		}
 
 		// Show the pages after the current one... (prev page 1 ... 6 7 [8] >9 10< ... 15 next page)
 		$tmpMaxPages = (int) (($max_value - 1) / $num_per_page) * $num_per_page;
 		for ($nCont = 1; $nCont <= $PageContiguous; $nCont++)
-			if ($start + $num_per_page * $nCont <= $tmpMaxPages)
+		{
+					if ($start + $num_per_page * $nCont <= $tmpMaxPages)
 			{
 				$tmpStart = $start + $num_per_page * $nCont;
+		}
 				$pageindex .= sprintf($base_link, $tmpStart, $tmpStart / $num_per_page + 1);
 			}
 
 		// Show the '...' part near the end. (prev page 1 ... 6 7 [8] 9 10 >...< 15 next page)
 		if ($start + $num_per_page * ($PageContiguous + 1) < $tmpMaxPages)
-			$pageindex .= str_replace('{custom}', 'data-baseurl="' . htmlspecialchars(JavaScriptEscape(($flexible_start ? $base_url : strtr($base_url, array('%' => '%%')) . ';start=%1$d')), ENT_COMPAT, 'UTF-8') . '" data-perpage="' . $num_per_page . '" data-firstpage="' . ($start + $num_per_page * ($PageContiguous + 1)) . '" data-lastpage="' . $tmpMaxPages . '"', $settings['page_index_template']['expand_pages']);
+		{
+					$pageindex .= str_replace('{custom}', 'data-baseurl="' . htmlspecialchars(JavaScriptEscape(($flexible_start ? $base_url : strtr($base_url, array('%' => '%%')) . ';start=%1$d')), ENT_COMPAT, 'UTF-8') . '" data-perpage="' . $num_per_page . '" data-firstpage="' . ($start + $num_per_page * ($PageContiguous + 1)) . '" data-lastpage="' . $tmpMaxPages . '"', $settings['page_index_template']['expand_pages']);
+		}
 
 		// Show the last number in the list. (prev page 1 ... 6 7 [8] 9 10 ... >15<  next page)
 		if ($start + $num_per_page * $PageContiguous < $tmpMaxPages)
-			$pageindex .= sprintf($base_link, $tmpMaxPages, $tmpMaxPages / $num_per_page + 1);
+		{
+					$pageindex .= sprintf($base_link, $tmpMaxPages, $tmpMaxPages / $num_per_page + 1);
+		}
 
 		// Show the "next page" link. (prev page 1 ... 6 7 [8] 9 10 ... 15 >next page<)
 		if ($start != $tmpMaxPages && $show['prev_next'] && empty($show['all_selected']))
-			$pageindex .= sprintf($base_link, $start + $num_per_page, str_replace('{next_txt}', $txt['next'], $settings['page_index_template']['next_page']));
+		{
+					$pageindex .= sprintf($base_link, $start + $num_per_page, str_replace('{next_txt}', $txt['next'], $settings['page_index_template']['next_page']));
+		}
 	}
 
 	// The "all" button
 	if ($show['all'])
 	{
 		if (!empty($show['all_selected']))
-			$pageindex .= sprintf($settings['page_index_template']['current_page'], $txt['all']);
-		else
-			$pageindex .= sprintf(str_replace('%1$d', '%1$s', $base_link), '0;all', str_replace('{all_txt}', $txt['all'], $settings['page_index_template']['all']));
+		{
+					$pageindex .= sprintf($settings['page_index_template']['current_page'], $txt['all']);
+		}
+		else {
+					$pageindex .= sprintf(str_replace('%1$d', '%1$s', $base_link), '0;all', str_replace('{all_txt}', $txt['all'], $settings['page_index_template']['all']));
+		}
 	}
 
 	return $pageindex;
@@ -347,7 +393,9 @@ 
 	{
 		// Not set for whatever reason?
 		if (empty($txt['number_format']) || preg_match('~^1([^\d]*)?234([^\d]*)(0*?)$~', $txt['number_format'], $matches) != 1)
-			return $number;
+		{
+					return $number;
+		}
 
 		// Cache these each load...
 		$thousands_separator = $matches[1];
@@ -436,16 +484,23 @@ 
 
 	// Offset the time.
 	if (!$offset_type)
-		$time = $log_time + ($user_info['time_offset'] + $modSettings['time_offset']) * 3600;
+	{
+			$time = $log_time + ($user_info['time_offset'] + $modSettings['time_offset']) * 3600;
+	}
 	// Just the forum offset?
 	elseif ($offset_type === 'forum')
-		$time = $log_time + $modSettings['time_offset'] * 3600;
-	else
-		$time = $log_time;
+	{
+			$time = $log_time + $modSettings['time_offset'] * 3600;
+	}
+	else {
+			$time = $log_time;
+	}
 
 	// We can't have a negative date (on Windows, at least.)
 	if ($log_time < 0)
-		$log_time = 0;
+	{
+			$log_time = 0;
+	}
 
 	// Today and Yesterday?
 	if ($modSettings['todayMod'] >= 1 && $show_today === true)
@@ -463,16 +518,21 @@ 
 			$h = strpos($user_info['time_format'], '%l') === false ? '%I' : '%l';
 			$today_fmt = $h . ':%M' . $s . ' %p';
 		}
-		else
-			$today_fmt = '%H:%M' . $s;
+		else {
+					$today_fmt = '%H:%M' . $s;
+		}
 
 		// Same day of the year, same year.... Today!
 		if ($then['yday'] == $now['yday'] && $then['year'] == $now['year'])
-			return sprintf($txt['today'], standardTime($log_time, $today_fmt, $offset_type));
+		{
+					return sprintf($txt['today'], standardTime($log_time, $today_fmt, $offset_type));
+		}
 
 		// Day-of-year is one less and same year, or it's the first of the year and that's the last of the year...
 		if ($modSettings['todayMod'] == '2' && (($then['yday'] == $now['yday'] - 1 && $then['year'] == $now['year']) || ($now['yday'] == 0 && $then['year'] == $now['year'] - 1) && $then['mon'] == 12 && $then['mday'] == 31))
-			return sprintf($txt['yesterday'], standardTime($log_time, $today_fmt, $offset_type));
+		{
+					return sprintf($txt['yesterday'], standardTime($log_time, $today_fmt, $offset_type));
+		}
 	}
 
 	$str = !is_bool($show_today) ? $show_today : $user_info['time_format'];
@@ -480,28 +540,40 @@ 
 	if (setlocale(LC_TIME, $txt['lang_locale']))
 	{
 		if (!isset($non_twelve_hour))
-			$non_twelve_hour = trim(Util::strftime('%p')) === '';
+		{
+					$non_twelve_hour = trim(Util::strftime('%p')) === '';
+		}
 		if ($non_twelve_hour && strpos($str, '%p') !== false)
-			$str = str_replace('%p', (Util::strftime('%H', $time) < 12 ? $txt['time_am'] : $txt['time_pm']), $str);
+		{
+					$str = str_replace('%p', (Util::strftime('%H', $time) < 12 ? $txt['time_am'] : $txt['time_pm']), $str);
+		}
 
 		foreach (array('%a', '%A', '%b', '%B') as $token)
-			if (strpos($str, $token) !== false)
+		{
+					if (strpos($str, $token) !== false)
 				$str = str_replace($token, !empty($txt['lang_capitalize_dates']) ? Util::ucwords(Util::strftime($token, $time)) : Util::strftime($token, $time), $str);
+		}
 	}
 	else
 	{
 		// Do-it-yourself time localization.  Fun.
 		foreach (array('%a' => 'days_short', '%A' => 'days', '%b' => 'months_short', '%B' => 'months') as $token => $text_label)
-			if (strpos($str, $token) !== false)
+		{
+					if (strpos($str, $token) !== false)
 				$str = str_replace($token, $txt[$text_label][(int) Util::strftime($token === '%a' || $token === '%A' ? '%w' : '%m', $time)], $str);
+		}
 
 		if (strpos($str, '%p') !== false)
-			$str = str_replace('%p', (Util::strftime('%H', $time) < 12 ? $txt['time_am'] : $txt['time_pm']), $str);
+		{
+					$str = str_replace('%p', (Util::strftime('%H', $time) < 12 ? $txt['time_am'] : $txt['time_pm']), $str);
+		}
 	}
 
 	// Windows doesn't support %e; on some versions, Util::strftime fails altogether if used, so let's prevent that.
 	if ($support_e && strpos($str, '%e') !== false)
-		$str = str_replace('%e', ltrim(Util::strftime('%d', $time), '0'), $str);
+	{
+			$str = str_replace('%e', ltrim(Util::strftime('%d', $time), '0'), $str);
+	}
 
 	// Format any other characters..
 	return Util::strftime($str, $time);
@@ -519,7 +591,9 @@ 
 	global $txt, $context;
 
 	if (empty($timestamp))
-		return '';
+	{
+			return '';
+	}
 
 	$forumtime = forum_time(false, $timestamp);
 	$timestamp = forum_time(true, $timestamp);
@@ -547,9 +621,13 @@ 
 	global $user_info, $modSettings;
 
 	if ($timestamp === null)
-		$timestamp = time();
+	{
+			$timestamp = time();
+	}
 	elseif ($timestamp == 0)
-		return 0;
+	{
+			return 0;
+	}
 
 	return $timestamp + ($modSettings['time_offset'] + ($use_user_offset ? $user_info['time_offset'] : 0)) * 3600;
 }
@@ -568,7 +646,9 @@ 
 function un_htmlspecialchars($string)
 {
 	if (empty($string))
-		return $string;
+	{
+			return $string;
+	}
 
 	$string = htmlspecialchars_decode($string, ENT_QUOTES);
 	$string = str_replace('&nbsp;', ' ', $string);
@@ -609,7 +689,9 @@ 
 		$array[$j] = $temp;
 
 		for ($i = 1; $p[$i] == 0; $i++)
-			$p[$i] = $i;
+		{
+					$p[$i] = $i;
+		}
 
 		$orders[] = $array;
 	}
@@ -691,7 +773,9 @@ 
 {
 	// Don't waste cycles
 	if ($message === '')
-		return '';
+	{
+			return '';
+	}
 
 	$parser = \BBC\ParserWrapper::instance();
 
@@ -779,29 +863,40 @@ 
 
 	// In case we have mail to send, better do that - as obExit doesn't always quite make it...
 	if (!empty($context['flush_mail']))
-		// @todo this relies on 'flush_mail' being only set in AddMailQueue itself... :\
+	{
+			// @todo this relies on 'flush_mail' being only set in AddMailQueue itself... :\
 		AddMailQueue(true);
+	}
 
 	Notifications::instance()->send();
 
 	$add = preg_match('~^(ftp|http)[s]?://~', $setLocation) == 0 && substr($setLocation, 0, 6) != 'about:';
 
 	if ($add)
-		$setLocation = $scripturl . ($setLocation != '' ? '?' . $setLocation : '');
+	{
+			$setLocation = $scripturl . ($setLocation != '' ? '?' . $setLocation : '');
+	}
 
 	// Put the session ID in.
 	if (empty($_COOKIE) && defined('SID') && SID != '')
-		$setLocation = preg_replace('/^' . preg_quote($scripturl, '/') . '(?!\?' . preg_quote(SID, '/') . ')\\??/', $scripturl . '?' . SID . ';', $setLocation);
+	{
+			$setLocation = preg_replace('/^' . preg_quote($scripturl, '/') . '(?!\?' . preg_quote(SID, '/') . ')\\??/', $scripturl . '?' . SID . ';', $setLocation);
+	}
 	// Keep that debug in their for template debugging!
 	elseif (isset($_GET['debug']))
-		$setLocation = preg_replace('/^' . preg_quote($scripturl, '/') . '\\??/', $scripturl . '?debug;', $setLocation);
+	{
+			$setLocation = preg_replace('/^' . preg_quote($scripturl, '/') . '\\??/', $scripturl . '?debug;', $setLocation);
+	}
 
 	if (!empty($modSettings['queryless_urls']) && detectServer()->supportRewrite())
 	{
 		if (defined('SID') && SID != '')
-			$setLocation = preg_replace_callback('~^' . preg_quote($scripturl, '~') . '\?(?:' . SID . '(?:;|&|&amp;))((?:board|topic)=[^#]+?)(#[^"]*?)?$~', 'redirectexit_callback', $setLocation);
-		else
-			$setLocation = preg_replace_callback('~^' . preg_quote($scripturl, '~') . '\?((?:board|topic)=[^#"]+?)(#[^"]*?)?$~', 'redirectexit_callback', $setLocation);
+		{
+					$setLocation = preg_replace_callback('~^' . preg_quote($scripturl, '~') . '\?(?:' . SID . '(?:;|&|&amp;))((?:board|topic)=[^#]+?)(#[^"]*?)?$~', 'redirectexit_callback', $setLocation);
+		}
+		else {
+					$setLocation = preg_replace_callback('~^' . preg_quote($scripturl, '~') . '\?((?:board|topic)=[^#"]+?)(#[^"]*?)?$~', 'redirectexit_callback', $setLocation);
+		}
 	}
 
 	// Maybe integrations want to change where we are heading?
@@ -809,9 +904,12 @@ 
 
 	// We send a Refresh header only in special cases because Location looks better. (and is quicker...)
 	if ($refresh)
-		header('Refresh: 0; URL=' . strtr($setLocation, array(' ' => '%20')));
-	else
-		header('Location: ' . str_replace(' ', '%20', $setLocation));
+	{
+			header('Refresh: 0; URL=' . strtr($setLocation, array(' ' => '%20')));
+	}
+	else {
+			header('Location: ' . str_replace(' ', '%20', $setLocation));
+	}
 
 	// Debugging.
 	if ($db_show_debug === true)
@@ -837,10 +935,13 @@ 
 	global $scripturl;
 
 	if (defined('SID') && SID != '')
-		return $scripturl . '/' . strtr($matches[1], '&;=', '//,') . '.html?' . SID . (isset($matches[2]) ? $matches[2] : '');
-	else
-		return $scripturl . '/' . strtr($matches[1], '&;=', '//,') . '.html' . (isset($matches[2]) ? $matches[2] : '');
-}
+	{
+			return $scripturl . '/' . strtr($matches[1], '&;=', '//,') . '.html?' . SID . (isset($matches[2]) ? $matches[2] : '');
+	}
+	else {
+			return $scripturl . '/' . strtr($matches[1], '&;=', '//,') . '.html' . (isset($matches[2]) ? $matches[2] : '');
+	}
+	}
 
 /**
  * Ends execution.
@@ -868,10 +969,14 @@ 
 	// Attempt to prevent a recursive loop.
 	++$level;
 	if ($level > 1 && !$from_fatal_error && !$has_fatal_error)
-		exit;
+	{
+			exit;
+	}
 
 	if ($from_fatal_error)
-		$has_fatal_error = true;
+	{
+			$has_fatal_error = true;
+	}
 
 	// Clear out the stat cache.
 	trackStats();
@@ -880,19 +985,25 @@ 
 
 	// If we have mail to send, send it.
 	if (!empty($context['flush_mail']))
-		// @todo this relies on 'flush_mail' being only set in AddMailQueue itself... :\
+	{
+			// @todo this relies on 'flush_mail' being only set in AddMailQueue itself... :\
 		AddMailQueue(true);
+	}
 
 	$do_header = $header === null ? !$header_done : $header;
 	if ($do_footer === null)
-		$do_footer = $do_header;
+	{
+			$do_footer = $do_header;
+	}
 
 	// Has the template/header been done yet?
 	if ($do_header)
 	{
 		// Was the page title set last minute? Also update the HTML safe one.
 		if (!empty($context['page_title']) && empty($context['page_title_html_safe']))
-			$context['page_title_html_safe'] = Util::htmlspecialchars(un_htmlspecialchars($context['page_title'])) . (!empty($context['current_page']) ? ' - ' . $txt['page'] . ' ' . ($context['current_page'] + 1) : '');
+		{
+					$context['page_title_html_safe'] = Util::htmlspecialchars(un_htmlspecialchars($context['page_title'])) . (!empty($context['current_page']) ? ' - ' . $txt['page'] . ' ' . ($context['current_page'] + 1) : '');
+		}
 
 		// Start up the session URL fixer.
 		ob_start('ob_sessrewrite');
@@ -940,8 +1051,10 @@ 
 
 	// Don't exit if we're coming from index.php; that will pass through normally.
 	if (!$from_index)
-		exit;
-}
+	{
+			exit;
+	}
+	}
 
 function setOldUrl($index = 'old_url')
 {
@@ -978,19 +1091,28 @@ 
 {
 	// Set topic class depending on locked status and number of replies.
 	if ($topic_context['is_very_hot'])
-		$topic_context['class'] = 'veryhot';
+	{
+			$topic_context['class'] = 'veryhot';
+	}
 	elseif ($topic_context['is_hot'])
-		$topic_context['class'] = 'hot';
-	else
-		$topic_context['class'] = 'normal';
+	{
+			$topic_context['class'] = 'hot';
+	}
+	else {
+			$topic_context['class'] = 'normal';
+	}
 
 	$topic_context['class'] .= !empty($topic_context['is_poll']) ? '_poll' : '_post';
 
 	if ($topic_context['is_locked'])
-		$topic_context['class'] .= '_locked';
+	{
+			$topic_context['class'] .= '_locked';
+	}
 
 	if ($topic_context['is_sticky'])
-		$topic_context['class'] .= '_sticky';
+	{
+			$topic_context['class'] .= '_sticky';
+	}
 
 	// This is so old themes will still work.
 	// @deprecated since 1.0 do not rely on it
@@ -1180,16 +1302,24 @@ 
 		$ip_array = array();
 
 		if (count($ip_parts) != 8)
-			return array();
+		{
+					return array();
+		}
 
 		for ($i = 0; $i < 8; $i++)
 		{
 			if ($ip_parts[$i] == '*')
-				$ip_array[$i] = array('low' => '0', 'high' => hexdec('ffff'));
+			{
+							$ip_array[$i] = array('low' => '0', 'high' => hexdec('ffff'));
+			}
 			elseif (preg_match('/^([0-9A-Fa-f]{1,4})\-([0-9A-Fa-f]{1,4})$/', $ip_parts[$i], $range) == 1)
-				$ip_array[$i] = array('low' => hexdec($range[1]), 'high' => hexdec($range[2]));
+			{
+							$ip_array[$i] = array('low' => hexdec($range[1]), 'high' => hexdec($range[2]));
+			}
 			elseif (is_numeric(hexdec($ip_parts[$i])))
-				$ip_array[$i] = array('low' => hexdec($ip_parts[$i]), 'high' => hexdec($ip_parts[$i]));
+			{
+							$ip_array[$i] = array('low' => hexdec($ip_parts[$i]), 'high' => hexdec($ip_parts[$i]));
+			}
 		}
 
 		return $ip_array;
@@ -1197,22 +1327,32 @@ 
 
 	// Pretend that 'unknown' is 255.255.255.255. (since that can't be an IP anyway.)
 	if ($fullip == 'unknown')
-		$fullip = '255.255.255.255';
+	{
+			$fullip = '255.255.255.255';
+	}
 
 	$ip_parts = explode('.', $fullip);
 	$ip_array = array();
 
 	if (count($ip_parts) != 4)
-		return array();
+	{
+			return array();
+	}
 
 	for ($i = 0; $i < 4; $i++)
 	{
 		if ($ip_parts[$i] == '*')
-			$ip_array[$i] = array('low' => '0', 'high' => '255');
+		{
+					$ip_array[$i] = array('low' => '0', 'high' => '255');
+		}
 		elseif (preg_match('/^(\d{1,3})\-(\d{1,3})$/', $ip_parts[$i], $range) == 1)
-			$ip_array[$i] = array('low' => $range[1], 'high' => $range[2]);
+		{
+					$ip_array[$i] = array('low' => $range[1], 'high' => $range[2]);
+		}
 		elseif (is_numeric($ip_parts[$i]))
-			$ip_array[$i] = array('low' => $ip_parts[$i], 'high' => $ip_parts[$i]);
+		{
+					$ip_array[$i] = array('low' => $ip_parts[$i], 'high' => $ip_parts[$i]);
+		}
 	}
 
 	// Makes it simpler to work with.
@@ -1239,7 +1379,9 @@ 
 
 	$host = '';
 	if ($cache->getVar($host, 'hostlookup-' . $ip, 600) || empty($ip))
-		return $host;
+	{
+			return $host;
+	}
 
 	$t = microtime(true);
 
@@ -1294,11 +1436,15 @@ 
 
 	// This is the last try :/.
 	if (!isset($host) || $host === false)
-		$host = @gethostbyaddr($ip);
+	{
+			$host = @gethostbyaddr($ip);
+	}
 
 	// It took a long time, so let's cache it!
 	if (microtime(true) - $t > 0.5)
-		$cache->put('hostlookup-' . $ip, $host, 600);
+	{
+			$cache->put('hostlookup-' . $ip, $host, 600);
+	}
 
 	return $host;
 }
@@ -1338,7 +1484,9 @@ 
 
 				// Create an integer representation
 				for ($i = 0; $i < $max_chars; $i++)
-					$total += $possible_chars[ord($encrypted[$i])] * pow(63, $i);
+				{
+									$total += $possible_chars[ord($encrypted[$i])] * pow(63, $i);
+				}
 
 				// Return the value
 				$returned_ints[] = $max_chars == 4 ? min($total, 16777215) : $total;
@@ -1351,8 +1499,10 @@ 
 		// Trim characters before and after and add slashes for database insertion.
 		$returned_words = array();
 		foreach ($words as $word)
-			if (($word = trim($word, '-_\'')) !== '')
+		{
+					if (($word = trim($word, '-_\'')) !== '')
 				$returned_words[] = $max_chars === null ? $word : substr($word, 0, $max_chars);
+		}
 
 		// Filter out all words that occur more than once.
 		return array_unique($returned_words);
@@ -1378,15 +1528,22 @@ 
 
 	// Does the current loaded theme have this and we are not forcing the usage of this function?
 	if (function_exists('template_create_button') && !$force_use)
-		return template_create_button($name, $alt, $label = '', $custom = '');
+	{
+			return template_create_button($name, $alt, $label = '', $custom = '');
+	}
 
 	if (!$settings['use_image_buttons'])
-		return $txt[$alt];
+	{
+			return $txt[$alt];
+	}
 	elseif (!empty($settings['use_buttons']))
-		return '<img src="' . $settings['images_url'] . '/buttons/' . $name . '" alt="' . $txt[$alt] . '" ' . $custom . ' />' . ($label != '' ? '&nbsp;<strong>' . $txt[$label] . '</strong>' : '');
-	else
-		return '<img src="' . $settings['lang_images_url'] . '/' . $name . '" alt="' . $txt[$alt] . '" ' . $custom . ' />';
-}
+	{
+			return '<img src="' . $settings['images_url'] . '/buttons/' . $name . '" alt="' . $txt[$alt] . '" ' . $custom . ' />' . ($label != '' ? '&nbsp;<strong>' . $txt[$label] . '</strong>' : '');
+	}
+	else {
+			return '<img src="' . $settings['lang_images_url'] . '/' . $name . '" alt="' . $txt[$alt] . '" ' . $custom . ' />';
+	}
+	}
 
 /**
  * Sets up all of the top menu buttons
@@ -1413,8 +1570,10 @@ 
 
 	// Change the seed.
 	if (mt_rand(1, 250) == 69 || empty($modSettings['rand_seed']))
-		updateSettings(array('rand_seed' => mt_rand()));
-}
+	{
+			updateSettings(array('rand_seed' => mt_rand()));
+	}
+	}
 
 /**
  * Process functions of an integration hook.
@@ -1500,35 +1659,50 @@ 
 function replaceEntities__callback($matches)
 {
 	if (!isset($matches[2]))
-		return '';
+	{
+			return '';
+	}
 
 	$num = $matches[2][0] === 'x' ? hexdec(substr($matches[2], 1)) : (int) $matches[2];
 
 	// remove left to right / right to left overrides
 	if ($num === 0x202D || $num === 0x202E)
-		return '';
+	{
+			return '';
+	}
 
 	// Quote, Ampersand, Apostrophe, Less/Greater Than get html replaced
 	if (in_array($num, array(0x22, 0x26, 0x27, 0x3C, 0x3E)))
-		return '&#' . $num . ';';
+	{
+			return '&#' . $num . ';';
+	}
 
 	// <0x20 are control characters, 0x20 is a space, > 0x10FFFF is past the end of the utf8 character set
 	// 0xD800 >= $num <= 0xDFFF are surrogate markers (not valid for utf8 text)
 	if ($num < 0x20 || $num > 0x10FFFF || ($num >= 0xD800 && $num <= 0xDFFF))
-		return '';
+	{
+			return '';
+	}
 	// <0x80 (or less than 128) are standard ascii characters a-z A-Z 0-9 and punctuation
 	elseif ($num < 0x80)
-		return chr($num);
+	{
+			return chr($num);
+	}
 	// <0x800 (2048)
 	elseif ($num < 0x800)
-		return chr(($num >> 6) + 192) . chr(($num & 63) + 128);
+	{
+			return chr(($num >> 6) + 192) . chr(($num & 63) + 128);
+	}
 	// < 0x10000 (65536)
 	elseif ($num < 0x10000)
-		return chr(($num >> 12) + 224) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+	{
+			return chr(($num >> 12) + 224) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+	}
 	// <= 0x10FFFF (1114111)
-	else
-		return chr(($num >> 18) + 240) . chr((($num >> 12) & 63) + 128) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
-}
+	else {
+			return chr(($num >> 18) + 240) . chr((($num >> 12) & 63) + 128) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+	}
+	}
 
 /**
  * Converts html entities to utf8 equivalents
@@ -1546,27 +1720,38 @@ 
 function fixchar__callback($matches)
 {
 	if (!isset($matches[1]))
-		return '';
+	{
+			return '';
+	}
 
 	$num = $matches[1][0] === 'x' ? hexdec(substr($matches[1], 1)) : (int) $matches[1];
 
 	// <0x20 are control characters, > 0x10FFFF is past the end of the utf8 character set
 	// 0xD800 >= $num <= 0xDFFF are surrogate markers (not valid for utf8 text), 0x202D-E are left to right overrides
 	if ($num < 0x20 || $num > 0x10FFFF || ($num >= 0xD800 && $num <= 0xDFFF) || $num === 0x202D || $num === 0x202E)
-		return '';
+	{
+			return '';
+	}
 	// <0x80 (or less than 128) are standard ascii characters a-z A-Z 0-9 and punctuation
 	elseif ($num < 0x80)
-		return chr($num);
+	{
+			return chr($num);
+	}
 	// <0x800 (2048)
 	elseif ($num < 0x800)
-		return chr(($num >> 6) + 192) . chr(($num & 63) + 128);
+	{
+			return chr(($num >> 6) + 192) . chr(($num & 63) + 128);
+	}
 	// < 0x10000 (65536)
 	elseif ($num < 0x10000)
-		return chr(($num >> 12) + 224) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+	{
+			return chr(($num >> 12) + 224) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+	}
 	// <= 0x10FFFF (1114111)
-	else
-		return chr(($num >> 18) + 240) . chr((($num >> 12) & 63) + 128) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
-}
+	else {
+			return chr(($num >> 18) + 240) . chr((($num >> 12) & 63) + 128) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+	}
+	}
 
 /**
  * Strips out invalid html entities, replaces others with html style &#123; codes
@@ -1583,16 +1768,21 @@ 
 function entity_fix__callback($matches)
 {
 	if (!isset($matches[2]))
-		return '';
+	{
+			return '';
+	}
 
 	$num = $matches[2][0] === 'x' ? hexdec(substr($matches[2], 1)) : (int) $matches[2];
 
 	// We don't allow control characters, characters out of range, byte markers, etc
 	if ($num < 0x20 || $num > 0x10FFFF || ($num >= 0xD800 && $num <= 0xDFFF) || $num == 0x202D || $num == 0x202E)
-		return '';
-	else
-		return '&#' . $num . ';';
-}
+	{
+			return '';
+	}
+	else {
+			return '&#' . $num . ';';
+	}
+	}
 
 /**
  * Retrieve additional search engines, if there are any, as an array.
@@ -1608,7 +1798,9 @@ 
 	{
 		$search_engines = Util::unserialize($modSettings['additional_search_engines']);
 		foreach ($search_engines as $engine)
-			$engines[strtolower(preg_replace('~[^A-Za-z0-9 ]~', '', $engine['name']))] = $engine;
+		{
+					$engines[strtolower(preg_replace('~[^A-Za-z0-9 ]~', '', $engine['name']))] = $engine;
+		}
 	}
 
 	return $engines;
@@ -1634,11 +1826,15 @@ 
 
 	// Start from the beginning...
 	if ($reset)
-		return $db->data_seek($messages_request, 0);
+	{
+			return $db->data_seek($messages_request, 0);
+	}
 
 	// If the query has already returned false, get out of here
 	if (empty($messages_request))
-		return false;
+	{
+			return false;
+	}
 
 	// Attempt to get the next message.
 	$message = $db->fetch_assoc($messages_request);
@@ -1671,22 +1867,32 @@ 
 {
 	// Search for key names or values
 	if ($assoc)
-		$position = array_search($key, array_keys($input), $strict);
-	else
-		$position = array_search($key, $input, $strict);
+	{
+			$position = array_search($key, array_keys($input), $strict);
+	}
+	else {
+			$position = array_search($key, $input, $strict);
+	}
 
 	// If the key is not found, just insert it at the end
 	if ($position === false)
-		return array_merge($input, $insert);
+	{
+			return array_merge($input, $insert);
+	}
 
 	if ($where === 'after')
-		$position++;
+	{
+			$position++;
+	}
 
 	// Insert as first
 	if ($position === 0)
-		$input = array_merge($insert, $input);
-	else
-		$input = array_merge(array_slice($input, 0, $position), $insert, array_slice($input, $position));
+	{
+			$input = array_merge($insert, $input);
+	}
+	else {
+			$input = array_merge(array_slice($input, 0, $position), $insert, array_slice($input, $position));
+	}
 
 	return $input;
 }
@@ -1706,9 +1912,12 @@ 
 	global $modSettings;
 
 	if (!isset($modSettings['scheduleTaskImmediate']))
-		$scheduleTaskImmediate = array();
-	else
-		$scheduleTaskImmediate = Util::unserialize($modSettings['scheduleTaskImmediate']);
+	{
+			$scheduleTaskImmediate = array();
+	}
+	else {
+			$scheduleTaskImmediate = Util::unserialize($modSettings['scheduleTaskImmediate']);
+	}
 
 	// If it has not been scheduled, the do so now
 	if (!isset($scheduleTaskImmediate[$task]))
@@ -1739,9 +1948,12 @@ 
 
 	// Not on, bail
 	if (!isset($modSettings['scheduleTaskImmediate']))
-		return;
-	else
-		$scheduleTaskImmediate = Util::unserialize($modSettings['scheduleTaskImmediate']);
+	{
+			return;
+	}
+	else {
+			$scheduleTaskImmediate = Util::unserialize($modSettings['scheduleTaskImmediate']);
+	}
 
 	// Clear / remove the task if it was set
 	if (isset($scheduleTaskImmediate[$task]))
@@ -1845,10 +2057,13 @@ 
 	$time = trim(file_get_contents(BOARDDIR . '/db_last_error.txt'));
 
 	if (preg_match('~^\d{10}$~', $time) === 1)
-		return $time;
-	else
-		return 0;
-}
+	{
+			return $time;
+	}
+	else {
+			return 0;
+	}
+	}
 
 /**
  * This function has the only task to retrieve the correct prefix to be used
@@ -1867,7 +2082,9 @@ 
 	if ($response_prefix === null && (!$cache->getVar($response_prefix, 'response_prefix') || !$response_prefix))
 	{
 		if ($language === $user_info['language'])
-			$response_prefix = $txt['response_prefix'];
+		{
+					$response_prefix = $txt['response_prefix'];
+		}
 		else
 		{
 			loadLanguage('index', $language, false);
@@ -1895,10 +2112,13 @@ 
 {
 	$value = trim($value);
 	if (filter_var($value, FILTER_VALIDATE_EMAIL) && Util::strlen($value) < 255)
-		return $value;
-	else
-		return false;
-}
+	{
+			return $value;
+	}
+	else {
+			return false;
+	}
+	}
 
 /**
  * Adds a protocol (http/s, ftp/mailto) to the beginning of an url if missing
@@ -2042,7 +2262,9 @@ 
 
 	// Don't know any browser!
 	if (empty($context['browser']))
-		detectBrowser();
+	{
+			detectBrowser();
+	}
 
 	return !empty($context['browser'][$browser]) || !empty($context['browser']['is_' . $browser]) ? true : false;
 }
@@ -2114,7 +2336,9 @@ 
 	foreach ($button_strip as $key => $value)
 	{
 		if (!isset($value['test']) || !empty($context[$value['test']]))
-			return true;
+		{
+					return true;
+		}
 	}
 
 	return false;

Spacing +1 added lines, -1 removed lines

@@ -1918,7 +1918,7 @@
 	}
 	else
 	{
-		$pattern = '~^(' . implode('|', array_map(function ($val) {return preg_quote($val, '~');}, $protocols)) . ')~i';
+		$pattern = '~^(' . implode('|', array_map(function($val) {return preg_quote($val, '~'); }, $protocols)) . ')~i';
 	}
 
 	$found = false;

sources/ext/PasswordHash.php

Braces +82 added lines, -32 removed lines

@@ -25,7 +25,8 @@ 
 # requirements (there can be none), but merely suggestions.
 #
 
-class PasswordHash {
+class PasswordHash
+{
 	var $itoa64;
 	var $iteration_count_log2;
 	var $portable_hashes;
@@ -36,7 +37,9 @@ 
 		$this->itoa64 = './0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz';
 
 		if ($iteration_count_log2 < 4 || $iteration_count_log2 > 31)
-			$iteration_count_log2 = 8;
+		{
+					$iteration_count_log2 = 8;
+		}
 		$this->iteration_count_log2 = $iteration_count_log2;
 
 		$this->portable_hashes = $portable_hashes;
@@ -49,25 +52,30 @@ 
 		$output = '';
 
 		// PHP >= 7
-		if (is_callable('random_bytes')) {
+		if (is_callable('random_bytes'))
+		{
 			$output = random_bytes($count);
 		}
 		// *nix
 		elseif (@is_readable('/dev/urandom') &&
-			($fh = @fopen('/dev/urandom', 'rb'))) {
+			($fh = @fopen('/dev/urandom', 'rb')))
+		{
 			$output = fread($fh, $count);
 			fclose($fh);
 		}
 		// This is much to slow on windows php < 5.3.4
 		elseif (function_exists('openssl_random_pseudo_bytes') &&
-			(substr(PHP_OS, 0, 3) !== 'WIN' || version_compare(PHP_VERSION, '5.3.4', '>='))) {
+			(substr(PHP_OS, 0, 3) !== 'WIN' || version_compare(PHP_VERSION, '5.3.4', '>=')))
+		{
 			$output = openssl_random_pseudo_bytes($count);
 		}
 
 		// Do it ourselves then
-		if (strlen($output) < $count) {
+		if (strlen($output) < $count)
+		{
 			$output = '';
-			for ($i = 0; $i < $count; $i += 16) {
+			for ($i = 0; $i < $count; $i += 16)
+			{
 				$this->random_state =
 					md5(microtime() . $this->random_state);
 				$output .=
@@ -83,19 +91,28 @@ 
 	{
 		$output = '';
 		$i = 0;
-		do {
+		do
+		{
 			$value = ord($input[$i++]);
 			$output .= $this->itoa64[$value & 0x3f];
 			if ($i < $count)
-				$value |= ord($input[$i]) << 8;
+			{
+							$value |= ord($input[$i]) << 8;
+			}
 			$output .= $this->itoa64[($value >> 6) & 0x3f];
 			if ($i++ >= $count)
-				break;
+			{
+							break;
+			}
 			if ($i < $count)
-				$value |= ord($input[$i]) << 16;
+			{
+							$value |= ord($input[$i]) << 16;
+			}
 			$output .= $this->itoa64[($value >> 12) & 0x3f];
 			if ($i++ >= $count)
-				break;
+			{
+							break;
+			}
 			$output .= $this->itoa64[($value >> 18) & 0x3f];
 		} while ($i < $count);
 
@@ -116,22 +133,30 @@ 
 	{
 		$output = '*0';
 		if (substr($setting, 0, 2) == $output)
-			$output = '*1';
+		{
+					$output = '*1';
+		}
 
 		$id = substr($setting, 0, 3);
 		# We use "$P$", phpBB3 uses "$H$" for the same thing
 		if ($id != '$P$' && $id != '$H$')
-			return $output;
+		{
+					return $output;
+		}
 
 		$count_log2 = strpos($this->itoa64, $setting[3]);
 		if ($count_log2 < 7 || $count_log2 > 30)
-			return $output;
+		{
+					return $output;
+		}
 
 		$count = 1 << $count_log2;
 
 		$salt = substr($setting, 4, 8);
 		if (strlen($salt) != 8)
-			return $output;
+		{
+					return $output;
+		}
 
 		# We're kind of forced to use MD5 here since it's the only
 		# cryptographic primitive available in all versions of PHP
@@ -139,14 +164,19 @@ 
 		# in PHP would result in much worse performance and
 		# consequently in lower iteration counts and hashes that are
 		# quicker to crack (by non-PHP code).
-		if (PHP_VERSION >= '5') {
+		if (PHP_VERSION >= '5')
+		{
 			$hash = md5($salt . $password, TRUE);
-			do {
+			do
+			{
 				$hash = md5($hash . $password, TRUE);
 			} while (--$count);
-		} else {
+		}
+		else
+		{
 			$hash = pack('H*', md5($salt . $password));
-			do {
+			do
+			{
 				$hash = pack('H*', md5($hash . $password));
 			} while (--$count);
 		}
@@ -193,11 +223,13 @@ 
 		$output .= '$';
 
 		$i = 0;
-		do {
+		do
+		{
 			$c1 = ord($input[$i++]);
 			$output .= $itoa64[$c1 >> 2];
 			$c1 = ($c1 & 0x03) << 4;
-			if ($i >= 16) {
+			if ($i >= 16)
+			{
 				$output .= $itoa64[$c1];
 				break;
 			}
@@ -219,34 +251,48 @@ 
 	public function HashPassword($password)
 	{
 		if ( strlen( $password ) > 4096 )
-			return '*';
+		{
+					return '*';
+		}
 
 		$random = '';
 
-		if (CRYPT_BLOWFISH == 1 && !$this->portable_hashes) {
+		if (CRYPT_BLOWFISH == 1 && !$this->portable_hashes)
+		{
 			$random = $this->get_random_bytes(16);
 			$hash =
 				crypt($password, $this->gensalt_blowfish($random));
 			if (strlen($hash) == 60)
-				return $hash;
+			{
+							return $hash;
+			}
 		}
 
-		if (CRYPT_EXT_DES == 1 && !$this->portable_hashes) {
+		if (CRYPT_EXT_DES == 1 && !$this->portable_hashes)
+		{
 			if (strlen($random) < 3)
-				$random = $this->get_random_bytes(3);
+			{
+							$random = $this->get_random_bytes(3);
+			}
 			$hash =
 				crypt($password, $this->gensalt_extended($random));
 			if (strlen($hash) == 20)
-				return $hash;
+			{
+							return $hash;
+			}
 		}
 
 		if (strlen($random) < 6)
-			$random = $this->get_random_bytes(6);
+		{
+					$random = $this->get_random_bytes(6);
+		}
 		$hash =
 			$this->crypt_private($password,
 				$this->gensalt_private($random));
 		if (strlen($hash) == 34)
-			return $hash;
+		{
+					return $hash;
+		}
 
 		# Returning '*' on error is safe here, but would _not_ be safe
 		# in a crypt(3)-like function used _both_ for generating new
@@ -257,11 +303,15 @@ 
 	public function CheckPassword($password, $stored_hash)
 	{
 		if ( strlen( $password ) > 4096 )
-			return false;
+		{
+					return false;
+		}
 
 		$hash = $this->crypt_private($password, $stored_hash);
 		if ($hash[0] == '*')
-			$hash = crypt($password, $stored_hash);
+		{
+					$hash = crypt($password, $stored_hash);
+		}
 
 		return $this->_hash_equals($hash, $stored_hash);
 	}

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 
 		$this->portable_hashes = $portable_hashes;
 
-		$this->random_state = microtime() . uniqid(rand(), TRUE);
+		$this->random_state = microtime() . uniqid(rand(), true);
 	}
 
 	private function get_random_bytes($count)
@@ -140,9 +140,9 @@ 
 		# consequently in lower iteration counts and hashes that are
 		# quicker to crack (by non-PHP code).
 		if (PHP_VERSION >= '5') {
-			$hash = md5($salt . $password, TRUE);
+			$hash = md5($salt . $password, true);
 			do {
-				$hash = md5($hash . $password, TRUE);
+				$hash = md5($hash . $password, true);
 			} while (--$count);
 		} else {
 			$hash = pack('H*', md5($salt . $password));

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 	var $portable_hashes;
 	var $random_state;
 
-	public function __construct( $iteration_count_log2, $portable_hashes )
+	public function __construct($iteration_count_log2, $portable_hashes)
 	{
 		$this->itoa64 = './0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz';
 
@@ -218,7 +218,7 @@ 
 
 	public function HashPassword($password)
 	{
-		if ( strlen( $password ) > 4096 )
+		if (strlen($password) > 4096)
 			return '*';
 
 		$random = '';
@@ -256,7 +256,7 @@ 
 
 	public function CheckPassword($password, $stored_hash)
 	{
-		if ( strlen( $password ) > 4096 )
+		if (strlen($password) > 4096)
 			return false;
 
 		$hash = $this->crypt_private($password, $stored_hash);

sources/ext/bad-behavior/badbehavior-plugin.php

Braces +54 added lines, -19 removed lines

@@ -92,27 +92,42 @@ 
 	if (strpos($query, 'DATE_SUB') !== false)
 	{
 		if (!empty($modSettings['badbehavior_logging']))
-			$query = 'DELETE FROM {db_prefix}log_badbehavior WHERE date < ' . (bb2_db_date() - 7 * 86400);
-		else
-			return true;
+		{
+					$query = 'DELETE FROM {db_prefix}log_badbehavior WHERE date < ' . (bb2_db_date() - 7 * 86400);
+		}
+		else {
+					return true;
+		}
 	}
 
 	if (strpos($query, 'OPTIMIZE TABLE') !== false)
-		return true;
+	{
+			return true;
+	}
 	elseif (strpos($query, '@@session.wait_timeout') !== false)
-		return true;
+	{
+			return true;
+	}
 	elseif (empty($query))
-		return true;
+	{
+			return true;
+	}
 
 	// Run the query, return success, failure or the actual results
 	$result = $db->query('', $query, array());
 
 	if (!$result)
-		return false;
+	{
+			return false;
+	}
 	elseif ($result === true)
-		return (bb2_db_affected_rows() !== 0);
+	{
+			return (bb2_db_affected_rows() !== 0);
+	}
 	elseif (bb2_db_num_rows($result) === 0)
-		return false;
+	{
+			return false;
+	}
 
 	return bb2_db_rows($result);
 }
@@ -131,7 +146,9 @@ 
 
 	$temp = array();
 	while ($row = $db->fetch_assoc($result))
-		$temp[] = $row;
+	{
+			$temp[] = $row;
+	}
 	$db->free_result($result);
 
 	return $temp;
@@ -164,7 +181,9 @@ 
 
 	// Logging not enabled
 	if (!$settings['logging'])
-		return '';
+	{
+			return '';
+	}
 
 	// Clean the data that bb sent us
 	$ip = bb2_db_escape($package['ip']);
@@ -203,7 +222,9 @@ 
 		foreach ($package['request_entity'] as $h => $v)
 		{
 			if (is_array($v))
-				$v = bb2_multi_implode($v, ' | ');
+			{
+							$v = bb2_multi_implode($v, ' | ');
+			}
 
 			$request_entity .= bb2_db_escape("$h: $v\n");
 		}
@@ -214,7 +235,9 @@ 
 
 		// Make it safe for the db
 		while (preg_match('~[\'\\\\]$~', substr($request_entity, -1)) === 1)
-			$request_entity = substr($request_entity, 0, -1);
+		{
+					$request_entity = substr($request_entity, 0, -1);
+		}
 	}
 
 	// Add it
@@ -238,16 +261,22 @@ 
 function bb2_multi_implode($array, $glue = ',', $trim_all = false)
 {
 	if (!is_array($array))
-		$array = array($array);
+	{
+			$array = array($array);
+	}
 
 	foreach ($array as $key => $value)
 	{
 		if (is_array($value))
-			$array[$key] = bb2_multi_implode($value, $glue, $trim_all);
+		{
+					$array[$key] = bb2_multi_implode($value, $glue, $trim_all);
+		}
 	}
 
 	if ($trim_all)
-		$array = array_map('trim', $array);
+	{
+			$array = array_map('trim', $array);
+	}
 
 	return implode($glue, $array);
 }
@@ -276,7 +305,9 @@ 
 
 	// Nothing in the whitelist
 	if (empty($whitelist['badbehavior_ip_wl']) && empty($whitelist['badbehavior_useragent_wl']) && empty($whitelist['badbehavior_url_wl']))
-		return false;
+	{
+			return false;
+	}
 
 	// Build up the whitelist array so badbehavior can use it
 	return array_merge(
@@ -300,7 +331,9 @@ 
 
 	// Make sure that the proxy addresses are split into an array, and if it's empty - make sure reverse proxy is disabled
 	if (!empty($modSettings['badbehavior_reverse_proxy_addresses']))
-		$badbehavior_reverse_proxy_addresses = explode('|', trim($modSettings['badbehavior_reverse_proxy_addresses']));
+	{
+			$badbehavior_reverse_proxy_addresses = explode('|', trim($modSettings['badbehavior_reverse_proxy_addresses']));
+	}
 	else
 	{
 		$badbehavior_reverse_proxy_addresses = array();
@@ -368,7 +401,9 @@ 
 		}
 
 		if ($bb2_blocked !== false)
-			return sprintf($txt['badbehavior_blocked'], $bb2_blocked[0]['COUNT(*)']);
+		{
+					return sprintf($txt['badbehavior_blocked'], $bb2_blocked[0]['COUNT(*)']);
+		}
 	}
 }
 

Spacing +1 added lines, -1 removed lines

@@ -191,7 +191,7 @@
 			$check = $length + Util::strlen($h) + Util::strlen($v) + 2;
 			if ($check < 255)
 			{
-				$headers .= bb2_db_escape($h . ': ' .  $v . "\n");
+				$headers .= bb2_db_escape($h . ': ' . $v . "\n");
 				$length = $check;
 			}
 		}

sources/ext/bad-behavior/bad-behavior/roundtripdns.inc.php

Braces +16 added lines, -4 removed lines

@@ -1,4 +1,7 @@ 
-<?php if (!defined('BB2_CORE')) die("I said no cheating!");
+<?php if (!defined('BB2_CORE'))
+{
+	die("I said no cheating!");
+}
 
 # Round trip DNS verification
 
@@ -9,12 +12,21 @@ 
 # between no records and error condition
 function bb2_roundtripdns($ip,$domain)
 {
-	if (@is_ipv6($ip)) return $ip;
+	if (@is_ipv6($ip))
+	{
+		return $ip;
+	}
 
 	$host = gethostbyaddr($ip);
 	$host_result = strpos(strrev($host), strrev($domain));
-	if ($host_result === false || $host_result > 0) return false;
+	if ($host_result === false || $host_result > 0)
+	{
+		return false;
+	}
 	$addrs = gethostbynamel($host);
-	if (in_array($ip, $addrs)) return true;
+	if (in_array($ip, $addrs))
+	{
+		return true;
+	}
 	return false;
 }

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 # TODO: Not IPv6 safe
 # FIXME: Returns false on DNS server failure; PHP provides no distinction
 # between no records and error condition
-function bb2_roundtripdns($ip,$domain)
+function bb2_roundtripdns($ip, $domain)
 {
 	if (@is_ipv6($ip)) return $ip;
 

sources/ext/bad-behavior/bad-behavior/whitelist.inc.php

Braces +38 added lines, -14 removed lines

@@ -1,33 +1,57 @@
-<?php if (!defined('BB2_CORE')) die('I said no cheating!');
+<?php if (!defined('BB2_CORE'))
+{
+	die('I said no cheating!');
+}
 
 function bb2_run_whitelist($package)
 {
 	# FIXME: Transitional, until port maintainters implement bb2_read_whitelist
-	if (function_exists('bb2_read_whitelist')) {
+	if (function_exists('bb2_read_whitelist'))
+	{
 		$whitelists = bb2_read_whitelist();
-	} else {
+	}
+	else
+	{
 		$whitelists = @parse_ini_file(dirname(BB2_CORE) . "/whitelist.ini");
 	}
 
-	if (@!empty($whitelists['ip'])) {
-		foreach (array_filter($whitelists['ip']) as $range) {
-			if (match_cidr($package['ip'], $range)) return true;
+	if (@!empty($whitelists['ip']))
+	{
+		foreach (array_filter($whitelists['ip']) as $range)
+		{
+			if (match_cidr($package['ip'], $range))
+			{
+				return true;
+			}
 		}
 	}
-	if (@!empty($whitelists['useragent'])) {
-		foreach (array_filter($whitelists['useragent']) as $user_agent) {
-			if (!strcmp($package['headers_mixed']['User-Agent'], $user_agent)) return true;
+	if (@!empty($whitelists['useragent']))
+	{
+		foreach (array_filter($whitelists['useragent']) as $user_agent)
+		{
+			if (!strcmp($package['headers_mixed']['User-Agent'], $user_agent))
+			{
+				return true;
+			}
 		}
 	}
-	if (@!empty($whitelists['url'])) {
-		if (strpos($package['request_uri'], "?") === FALSE) {
+	if (@!empty($whitelists['url']))
+	{
+		if (strpos($package['request_uri'], "?") === FALSE)
+		{
 			$request_uri = $package['request_uri'];
-		} else {
+		}
+		else
+		{
 			$request_uri = substr($package['request_uri'], 0, strpos($package['request_uri'], "?"));
 		}
-		foreach (array_filter($whitelists['url']) as $url) {
+		foreach (array_filter($whitelists['url']) as $url)
+		{
 			$pos = strpos($request_uri, $url);
-			if ($pos !== false && $pos == 0) return true;
+			if ($pos !== false && $pos == 0)
+			{
+				return true;
+			}
 		}
 	}
 	return false;

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 		}
 	}
 	if (@!empty($whitelists['url'])) {
-		if (strpos($package['request_uri'], "?") === FALSE) {
+		if (strpos($package['request_uri'], "?") === false) {
 			$request_uri = $package['request_uri'];
 		} else {
 			$request_uri = substr($package['request_uri'], 0, strpos($package['request_uri'], "?"));