albertlast / SMF2.1

Sources/Class-Punycode.php

Braces +24 added lines, -12 removed lines

@@ -157,12 +157,14 @@ 
 		}
 
 		$parts = explode('.', $preprocessed);
-		foreach ($parts as $p => &$part) {
+		foreach ($parts as $p => &$part)
+		{
 			$part = $this->encodePart($part);
 
 			$validation_status = $this->validateLabel($part, true);
 
-			switch ($validation_status) {
+			switch ($validation_status)
+			{
 				case self::IDNA_ERROR_LABEL_TOO_LONG:
 				case self::IDNA_ERROR_LEADING_HYPHEN:
 				case self::IDNA_ERROR_TRAILING_HYPHEN:
@@ -215,13 +217,16 @@ 
 		$h = $b = count($codePoints['basic']);
 
 		$output = '';
-		foreach ($codePoints['basic'] as $code) {
+		foreach ($codePoints['basic'] as $code)
+		{
 			$output .= $this->codePointToChar($code);
 		}
-		if ($input === $output) {
+		if ($input === $output)
+		{
 			return $output;
 		}
-		if ($b > 0) {
+		if ($b > 0)
+		{
 			$output .= static::DELIMITER;
 		}
 
@@ -230,20 +235,26 @@ 
 
 		$i = 0;
 		$length = mb_strlen($input, $this->encoding);
-		while ($h < $length) {
+		while ($h < $length)
+		{
 			$m = $codePoints['nonBasic'][$i++];
 			$delta = $delta + ($m - $n) * ($h + 1);
 			$n = $m;
 
-			foreach ($codePoints['all'] as $c) {
-				if ($c < $n || $c < static::INITIAL_N) {
+			foreach ($codePoints['all'] as $c)
+			{
+				if ($c < $n || $c < static::INITIAL_N)
+				{
 					$delta++;
 				}
-				if ($c === $n) {
+				if ($c === $n)
+				{
 					$q = $delta;
-					for ($k = static::BASE;; $k += static::BASE) {
+					for ($k = static::BASE;; $k += static::BASE)
+					{
 						$t = $this->calculateThreshold($k, $bias);
-						if ($q < $t) {
+						if ($q < $t)
+						{
 							break;
 						}
 
@@ -530,7 +541,8 @@ 
 
 		$labels = explode('.', $domain);
 
-		foreach ($labels as $l => $label) {
+		foreach ($labels as $l => $label)
+		{
 			$label = preg_replace('/[' . $regexes['ignored'] . ']/u', '', $label);
 
 			$label = utf8_normalize_c(strtr($label, $maps));

Sources/MessageIndex.php

Braces +4 added lines, -2 removed lines

@@ -1330,8 +1330,10 @@
 		'calendar_updated' => time(),
 	));
 	
-	if (!empty($cache_enable) && $cache_enable >= 3) {
-		foreach ($affectedBoards as $b => $junk) {
+	if (!empty($cache_enable) && $cache_enable >= 3)
+	{
+		foreach ($affectedBoards as $b => $junk)
+		{
 			cache_put_data('board-' . $b, null);
 		}
 	}

Sources/RemoveTopic.php

Braces +4 added lines, -2 removed lines

@@ -569,9 +569,11 @@
 	updateSettings(array(
 		'calendar_updated' => time(),
 	));
-	if (!empty($cache_enable) && $cache_enable >= 3) {
+	if (!empty($cache_enable) && $cache_enable >= 3)
+	{
 		cache_put_data('board-' . $recycle_board, null);
-		foreach ($adjustBoards as $stats) {
+		foreach ($adjustBoards as $stats)
+		{
 			cache_put_data('board-' . $stats['id_board'], null);
 		}
 	}

Sources/tasks/CreatePost-Notify.php

Braces +10 added lines, -5 removed lines

@@ -170,11 +170,14 @@ 
 
 			// If this user subscribes both to the topic and the board there will be two records returned.
 			// Copy board/topic data to the new record or it will be lost.
-			if (!empty($this->members['watching'][$row['id_member']])) {
-				if ($this->members['watching'][$row['id_member']]['id_board'] > 0) {
+			if (!empty($this->members['watching'][$row['id_member']]))
+			{
+				if ($this->members['watching'][$row['id_member']]['id_board'] > 0)
+				{
 					$row['id_board'] = $this->members['watching'][$row['id_member']]['id_board'];
 				}
-				if ($this->members['watching'][$row['id_member']]['id_topic'] > 0) {
+				if ($this->members['watching'][$row['id_member']]['id_topic'] > 0)
+				{
 					$row['id_topic'] = $this->members['watching'][$row['id_member']]['id_topic'];
 				}
 			}
@@ -191,7 +194,8 @@ 
 				foreach ($this->members[$member_type] as $member_id => $member_data)
 				{
 					// The member receiving the alert has ignored the member mentioning them.
-					if (!empty($member_data['mentioned_by']['ignored'])) {
+					if (!empty($member_data['mentioned_by']['ignored']))
+					{
 						unset($this->members[$member_type][$member_id], $msgOptions[$member_type . '_members'][$member_id]);
 					}
 
@@ -202,7 +206,8 @@ 
 			}
 		}
 
-		$unnotified = array_filter($this->members['watching'], function ($member) {
+		$unnotified = array_filter($this->members['watching'], function ($member)
+		{
 			return empty($member['sent']);
 		});
 

Sources/tasks/UpdateUnicode.php

Braces +2 added lines, -1 removed lines

@@ -416,7 +416,8 @@
 		if (!@touch($this->temp_dir . DIRECTORY_SEPARATOR . 'lock'))
 			return true;
 
-		register_shutdown_function(function () {
+		register_shutdown_function(function ()
+		{
 			if (file_exists($this->temp_dir . DIRECTORY_SEPARATOR . 'lock'))
 				unlink($this->temp_dir . DIRECTORY_SEPARATOR . 'lock');
 		});

Sources/Errors.php

Braces +2 added lines, -1 removed lines

@@ -348,7 +348,8 @@
 
 	$message = $txt[$e->getMessage()] ?? $e->getMessage();
 
-	if (!empty($modSettings['enableErrorLogging'])) {
+	if (!empty($modSettings['enableErrorLogging']))
+	{
 		log_error($message, 'general', $e->getFile(), $e->getLine());
 	}
 

Sources/MoveTopic.php

Braces +4 added lines, -2 removed lines

@@ -708,9 +708,11 @@
 	updateSettings(array(
 		'calendar_updated' => time(),
 	));
-	if (!empty($cache_enable) && $cache_enable >= 3) {
+	if (!empty($cache_enable) && $cache_enable >= 3)
+	{
 		cache_put_data('board-' . $toBoard, null);
-		foreach ($fromBoards as $stats) {
+		foreach ($fromBoards as $stats)
+		{
 			cache_put_data('board-' . $stats['id_board'], null);
 		}
 	}

Sources/ScheduledTasks.php

Braces +14 added lines, -7 removed lines

@@ -740,7 +740,8 @@ 
 	$smcFunc['db_free_result']($request);
 
 	// Random emails from the queue..
-	if (!empty($ids)) {
+	if (!empty($ids))
+	{
 		$request = $smcFunc['db_query']('', '
 			SELECT id_mail, recipient, body, subject, headers, send_html, time_sent, private, priority
 			FROM {db_prefix}mail_queue
@@ -811,22 +812,27 @@ 
 	{
 		// First, figure out when the next send attempt should happen based on the current priority.
 		$next_send_time = $email['time_sent'];
-		if ($email['priority'] >= $priority_offset) {
-			for ($i = 0; $i < $email['priority']; $i++) {
+		if ($email['priority'] >= $priority_offset)
+		{
+			for ($i = 0; $i < $email['priority']; $i++)
+			{
 				$next_send_time += 20 * max(0, $email['priority'] - $priority_offset);
 			}
 		}
 
 		// If the email is too old, discard it.
-		if ($next_send_time > $email['time_sent'] + $smtp_expire) {
+		if ($next_send_time > $email['time_sent'] + $smtp_expire)
+		{
 			continue;
 		}
 
 		++$email['priority'];
 
 		// Don't send if it's too soon.
-		if (time() < $next_send_time) {
-			if ($email['priority'] < $max_priority) {
+		if (time() < $next_send_time)
+		{
+			if ($email['priority'] < $max_priority)
+			{
 				$failed_emails[] = array($email['to'], $email['body'], $email['subject'], $email['headers'], $email['send_html'], $email['time_sent'], $email['private'], $email['priority']);
 			}
 
@@ -855,7 +861,8 @@ 
 			$result = smtp_mail(array($email['to']), $email['subject'], $email['body'], $email['headers']);
 
 		// If we failed, and we haven't already hit the limit, schedule this for another attempt.
-		if (empty($result) && $email['priority'] < $max_priority) {
+		if (empty($result) && $email['priority'] < $max_priority)
+		{
 			$failed_emails[] = array($email['to'], $email['body'], $email['subject'], $email['headers'], $email['send_html'], $email['time_sent'], $email['private'], $email['priority']);
 		}
 	}

cron.php

Braces +2 added lines, -2 removed lines

@@ -97,7 +97,6 @@ 
 	if ($ts <= 0 || $ts % 15 != 0 || time() - $ts < 0 || time() - $ts > 20)
 		obExit_cron();
 }
-
 else
 	$_SERVER['SERVER_PROTOCOL'] = 'HTTP/1.0';
 
@@ -323,7 +322,8 @@ 
 
 	$message = $txt[$e->getMessage()] ?? $e->getMessage();
 
-	if (!empty($modSettings['enableErrorLogging'])) {
+	if (!empty($modSettings['enableErrorLogging']))
+	{
 		log_error($message, 'cron', $e->getFile(), $e->getLine());
 	}
 }