elkarte / Elkarte

sources/Security.php

Braces +4 added lines, -1 removed lines

@@ -1730,10 +1730,13 @@
 		if ($result[0] === 127 // Valid Response
 			&& ($result[3] & 3 || $result[3] & 5) // Listed as Suspicious + Harvester || Suspicious + Comment Spammer
 			&& $result[2] >= $modSettings['badbehavior_httpbl_threat'] // Level
-			&& $result[1] <= $modSettings['badbehavior_httpbl_maxage']) // Age
+			&& $result[1] <= $modSettings['badbehavior_httpbl_maxage'])
+		{
+			// Age
 		{
 			return true;
 		}
+		}
 	}
 
 	return false;

Spacing +5 added lines, -5 removed lines

@@ -348,7 +348,7 @@ 
 					(' . implode(' OR ', $ban_query) . ')',
 				$ban_query_vars
 			)->fetch_callback(
-				static function ($row) use ($restrictions, &$flag_is_activated) {
+				static function($row) use ($restrictions, &$flag_is_activated) {
 					// Store every type of ban that applies to you in your session.
 					foreach ($restrictions as $restriction)
 					{
@@ -410,7 +410,7 @@ 
 				'current_time' => time(),
 			]
 		)->fetch_callback(
-			static function ($row) {
+			static function($row) {
 				$_SESSION['ban']['cannot_access']['ids'][] = $row['id_ban'];
 				$_SESSION['ban']['cannot_access']['reason'] = $row['reason'];
 			}
@@ -704,7 +704,7 @@ 
 			'now' => time(),
 		]
 	)->fetch_callback(
-		static function ($row) use (&$ban_ids, &$ban_reason, $restriction) {
+		static function($row) use (&$ban_ids, &$ban_reason, $restriction) {
 			if (!empty($row['cannot_access']))
 			{
 				$_SESSION['ban']['cannot_access']['ids'][] = $row['id_ban'];
@@ -829,7 +829,7 @@ 
 			// If global cookies are on, trim to superdomain AFTER IDNA normalization.
 			if (!empty($modSettings['globalCookies']))
 			{
-				$trim = static function (string $h): string {
+				$trim = static function(string $h): string {
 					if (preg_match('~(?:[^.]+\.)?([^.]{3,}\..+)\z~i', $h, $parts) === 1)
 					{
 						return $parts[1];
@@ -1344,7 +1344,7 @@ 
 			'permissions' => $permissions,
 		]
 	)->fetch_callback(
-		static function ($row) use ($simple, &$deny_boards, &$boards) {
+		static function($row) use ($simple, &$deny_boards, &$boards) {
 			if ($simple)
 			{
 				if (empty($row['add_deny']))

sources/subs/PaidSubscriptions.subs.php

Braces +3 added lines, -1 removed lines

@@ -1377,11 +1377,13 @@
 		{
 			// If we revert to the old id-group we need to ensure it wasn't from a subscription.
 			foreach ($context['subscriptions'] as $id => $group)
-				// It was? Make them a regular member then!
+			{
+							// It was? Make them a regular member then!
 			{
 				if ($group['prim_group'] == $member['id_group'])
 				{
 					$member['id_group'] = 0;
+			}
 				}
 			}
 

Spacing +10 added lines, -10 removed lines

@@ -90,7 +90,7 @@ 
 			'guest' => $txt['guest'],
 		])
 	)->fetch_callback(
-		function ($row) use (&$subscribers) {
+		function($row) use (&$subscribers) {
 			global $txt;
 
 			$subscribers[] = [
@@ -149,7 +149,7 @@ 
 			'current_time' => time(),
 		]
 	)->fetch_callback(
-		function ($row) use (&$groups) {
+		function($row) use (&$groups) {
 			// Specific primary group?
 			if ($row['id_group'] != 0)
 			{
@@ -485,7 +485,7 @@ 
 		FROM {db_prefix}subscriptions',
 		[]
 	)->fetch_callback(
-		function ($row) {
+		function($row) {
 			global $context, $modSettings, $txt;
 
 			// Pick a cost.
@@ -555,7 +555,7 @@ 
 		GROUP BY id_subscribe, status',
 		[]
 	)->fetch_callback(
-		function ($row) {
+		function($row) {
 			global $context;
 
 			$ind = $row['status'] == 0 ? 'finished' : 'total';
@@ -575,7 +575,7 @@ 
 		GROUP BY id_subscribe',
 		[]
 	)->fetch_callback(
-		function ($row) {
+		function($row) {
 			global $context;
 
 			if (isset($context['subscriptions'][$row['id_subscribe']]))
@@ -609,7 +609,7 @@ 
 			'selected_member' => $memID,
 		]
 	)->fetch_callback(
-		function ($row) use (&$current, $active_subscriptions) {
+		function($row) use (&$current, $active_subscriptions) {
 			global $txt;
 
 			// The subscription must exist!
@@ -670,7 +670,7 @@ 
 			'guest' => $txt['guest'],
 		]
 	)->fetch_callback(
-		function ($row) use (&$members) {
+		function($row) use (&$members) {
 			$members[$row['id_member']] = $row;
 		}
 	);
@@ -883,7 +883,7 @@ 
 			'current_subscription' => $sub_id,
 		]
 	)->fetch_callback(
-		function ($row) use (&$subscription) {
+		function($row) use (&$subscription) {
 			// Sort the date.
 			preg_match('~(\d*)(\w)~', $row['length'], $match);
 			if (isset($match[2]))
@@ -1111,7 +1111,7 @@ 
 			'subscription_list' => $toDelete,
 		]
 	)->fetch_callback(
-		function ($row) use (&$delete) {
+		function($row) use (&$delete) {
 			$delete[$row['id_subscribe']] = $row['id_member'];
 		}
 	);
@@ -1321,7 +1321,7 @@ 
 			'is_active' => 1,
 		]
 	)->fetch_callback(
-		function ($row) use (&$removals, &$allowed, &$member, &$new_id_group, $id_subscribe) {
+		function($row) use (&$removals, &$allowed, &$member, &$new_id_group, $id_subscribe) {
 			global $context;
 
 			if (!isset($context['subscriptions'][$row['id_subscribe']]))

sources/subs/Post.subs.php

Braces +3 added lines, -1 removed lines

@@ -735,10 +735,12 @@
 		}
 		// If this is being unapproved, and it's equal to the id_last_msg we need to find a new one!
 		elseif (!$approve)
-			// Default to the first message and then we'll override in a bit ;)
+		{
+					// Default to the first message and then we'll override in a bit ;)
 		{
 			$topic_changes[$row['id_topic']]['id_last_msg'] = $row['id_first_msg'];
 		}
+		}
 
 		$topic_changes[$row['id_topic']]['unapproved_posts'] += $approve ? -1 : 1;
 		$board_changes[$row['id_board']]['unapproved_posts'] += $approve ? -1 : 1;

Spacing +2 added lines, -2 removed lines

@@ -785,7 +785,7 @@ 
 				'approved' => 1,
 			]
 		)->fetch_callback(
-			function ($row) use (&$topic_changes) {
+			function($row) use (&$topic_changes) {
 				$topic_changes[$row['id_topic']]['id_last_msg'] = $row['id_last_msg'];
 			}
 		);
@@ -947,7 +947,7 @@ 
 				'approved' => 1,
 			]
 		)->fetch_callback(
-			function ($row) use (&$lastMsg) {
+			function($row) use (&$lastMsg) {
 				$lastMsg[$row['id_board']] = $row['id_msg'];
 			}
 		);

sources/ElkArte/AdminController/ManageFeatures.php

Braces +1 added lines, -2 removed lines

@@ -306,8 +306,7 @@
 					class_exists($modSettings['front_page'])
 					&& in_array('validateFrontPageOptions', get_class_methods($modSettings['front_page']))
 					&& !$front_page::validateFrontPageOptions($this->_req->post)
-				)
-				{
+				) {
 					$this->_req->post->front_page = '';
 				}
 			}

Spacing +5 added lines, -5 removed lines

@@ -1128,7 +1128,7 @@ 
 						'class' => 'centertext',
 					],
 					'data' => [
-						'function' => static function ($rowData) {
+						'function' => static function($rowData) {
 							$isChecked = $rowData['disabled'] ? '' : ' checked="checked"';
 							$onClickHandler = $rowData['can_show_register'] ? sprintf('onclick="document.getElementById(\'reg_%1$s\').disabled = !this.checked;"', $rowData['id']) : '';
 
@@ -1144,7 +1144,7 @@ 
 						'class' => 'centertext',
 					],
 					'data' => [
-						'function' => static function ($rowData) {
+						'function' => static function($rowData) {
 							$isChecked = $rowData['on_register'] && !$rowData['disabled'] ? ' checked="checked"' : '';
 							$isDisabled = $rowData['can_show_register'] ? '' : ' disabled="disabled"';
 
@@ -1220,7 +1220,7 @@ 
 						'value' => $txt['custom_profile_fieldtype'],
 					],
 					'data' => [
-						'function' => static function ($rowData) {
+						'function' => static function($rowData) {
 							global $txt;
 
 							$textKey = sprintf('custom_profile_type_%1$s', $rowData['field_type']);
@@ -1240,7 +1240,7 @@ 
 						'class' => 'centertext',
 					],
 					'data' => [
-						'function' => static function ($rowData) {
+						'function' => static function($rowData) {
 							$isChecked = $rowData['active'] === '1' ? ' checked="checked"' : '';
 
 							return sprintf('<input type="checkbox" name="cust[]" id="cust_%1$s" value="%1$s" class="input_check"%2$s />', $rowData['id_field'], $isChecked);
@@ -1258,7 +1258,7 @@ 
 						'value' => $txt['custom_profile_placement'],
 					],
 					'data' => [
-						'function' => static function ($rowData) {
+						'function' => static function($rowData) {
 							global $txt;
 
 							$placement = 'custom_profile_placement_';

sources/ElkArte/Errors/ErrorHandler.php

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

@@ -213,7 +213,7 @@
 				<<<'MSG'
 PHP Fatal error:  Uncaught exception '%s' with message '%s' in %s:%s<br />
 Stack trace:<br />%s<br />  thrown in %s on line %s
-MSG;
+msg;
 
 			// write trace lines into main template
 			return sprintf(

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
 		// Register the class handlers to the PHP handler functions
 		set_error_handler(fn($error_level, $error_string, $file, $line) => $this->error_handler($error_level, $error_string, $file, $line));
-		set_exception_handler(function (Throwable $e) {
+		set_exception_handler(function(Throwable $e) {
 			$this->exception_handler($e);
 		});
 	}

themes/index.php

Braces +2 added lines, -1 removed lines

@@ -12,5 +12,6 @@
 	header('Location: ' . $boardurl);
 }
 // Can't find it... just forget it.
-else
+else {
 	exit;
+}

themes/default/css/index.php

Braces +5 added lines, -2 removed lines

@@ -2,6 +2,9 @@
 
 // Try to handle it with the upper level index.php. (it should know what to do.)
 if (file_exists(dirname(dirname(__FILE__)) . '/index.php'))
+{
 	include (dirname(dirname(__FILE__)) . '/index.php');
-else
-	exit;
\ No newline at end of file
+}
+else {
+	exit;
+}

themes/default/images/post/index.php

Braces +5 added lines, -2 removed lines

@@ -2,6 +2,9 @@
 
 // Try to handle it with the upper level index.php.  (it should know what to do.)
 if (file_exists(dirname(dirname(__FILE__)) . '/index.php'))
+{
 	include (dirname(dirname(__FILE__)) . '/index.php');
-else
-	exit;
\ No newline at end of file
+}
+else {
+	exit;
+}

ssi_examples.php

Braces +4 added lines, -2 removed lines

@@ -135,7 +135,7 @@ 
 			</li>
 		</ul>
 		<?php if (User::$info->is_admin)
-		{
+{
 			?>
 			<h3>Advanced Functions <i class="helpicon i-help" title="Functions that require additional tweaking, not just copy and paste."></i></h3>
 			<ul>
@@ -949,10 +949,12 @@ 
 	$topics = ssi_recentTopics(8, null, null, 'array');
 
 	foreach ($topics as $topic)
-		echo '
+	{
+			echo '
 			<li>
 				<a href="', $topic['href'], '">', $topic['subject'], '</a> ', $txt['by'], ' ', $topic['poster']['link'], '
 			</li>';
+	}
 
 	unset($topics);