Passed
Pull Request — release-2.1 (#6274)
by Mathias
05:20
created
Sources/Subs-Db-postgresql.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -78,7 +78,8 @@  discard block
 block discarded – undo
78 78
 	$db_passwd = str_replace(array('\\','\''), array('\\\\','\\\''), $db_passwd);
79 79
 
80 80
 	// Since pg_connect doesn't feed error info to pg_last_error, we have to catch issues with a try/catch.
81
-	set_error_handler(function($errno, $errstr) {
81
+	set_error_handler(function($errno, $errstr)
82
+	{
82 83
 		throw new ErrorException($errstr, $errno);});
83 84
 	try
84 85
 	{
@@ -939,7 +940,8 @@  discard block
 block discarded – undo
939 940
 		$error_array[2] = null;
940 941
 
941 942
 	if(empty($db_persist))
942
-	{ // without pooling
943
+	{
944
+// without pooling
943 945
 		if (empty($pg_error_data_prep))
944 946
 			$pg_error_data_prep = pg_prepare($db_connection, 'smf_log_errors',
945 947
 				'INSERT INTO ' . $db_prefix . 'log_errors
@@ -950,7 +952,8 @@  discard block
 block discarded – undo
950 952
 		pg_execute($db_connection, 'smf_log_errors', $error_array);
951 953
 	}
952 954
 	else
953
-	{ //with pooling
955
+	{
956
+//with pooling
954 957
 		$pg_error_data_prep = pg_prepare($db_connection, '',
955 958
 			'INSERT INTO ' . $db_prefix . 'log_errors
956 959
 				(id_member, log_time, ip, url, message, session, error_type, file, line, backtrace)
Please login to merge, or discard this patch.
Sources/random_compat/random_bytes_dev_urandom.php 1 patch
Braces   +35 added lines, -17 removed lines patch added patch discarded remove patch
@@ -26,11 +26,13 @@  discard block
 block discarded – undo
26 26
  * SOFTWARE.
27 27
  */
28 28
 
29
-if (!defined('RANDOM_COMPAT_READ_BUFFER')) {
29
+if (!defined('RANDOM_COMPAT_READ_BUFFER'))
30
+{
30 31
     define('RANDOM_COMPAT_READ_BUFFER', 8);
31 32
 }
32 33
 
33
-if (!is_callable('random_bytes')) {
34
+if (!is_callable('random_bytes'))
35
+{
34 36
     /**
35 37
      * Unless open_basedir is enabled, use /dev/urandom for
36 38
      * random numbers in accordance with best practices
@@ -53,7 +55,8 @@  discard block
 block discarded – undo
53 55
         /**
54 56
          * This block should only be run once
55 57
          */
56
-        if (empty($fp)) {
58
+        if (empty($fp))
59
+        {
57 60
             /**
58 61
              * We don't want to ever read C:\dev\random, only /dev/urandom on
59 62
              * Unix-like operating systems. While we guard against this
@@ -64,8 +67,10 @@  discard block
 block discarded – undo
64 67
              * like operating system (which means the directory separator is set
65 68
              * to "/" not "\".
66 69
              */
67
-            if (DIRECTORY_SEPARATOR === '/') {
68
-                if (!is_readable('/dev/urandom')) {
70
+            if (DIRECTORY_SEPARATOR === '/')
71
+            {
72
+                if (!is_readable('/dev/urandom'))
73
+                {
69 74
                     throw new Exception(
70 75
                         'Environment misconfiguration: ' .
71 76
                         '/dev/urandom cannot be read.'
@@ -77,17 +82,20 @@  discard block
 block discarded – undo
77 82
                  */
78 83
                 /** @var resource|bool $fp */
79 84
                 $fp = fopen('/dev/urandom', 'rb');
80
-                if (is_resource($fp)) {
85
+                if (is_resource($fp))
86
+                {
81 87
                     /** @var array<string, int> $st */
82 88
                     $st = fstat($fp);
83
-                    if (($st['mode'] & 0170000) !== 020000) {
89
+                    if (($st['mode'] & 0170000) !== 020000)
90
+                    {
84 91
                         fclose($fp);
85 92
                         $fp = false;
86 93
                     }
87 94
                 }
88 95
             }
89 96
 
90
-            if (is_resource($fp)) {
97
+            if (is_resource($fp))
98
+            {
91 99
                 /**
92 100
                  * stream_set_read_buffer() does not exist in HHVM
93 101
                  *
@@ -96,25 +104,31 @@  discard block
 block discarded – undo
96 104
                  *
97 105
                  * stream_set_read_buffer returns 0 on success
98 106
                  */
99
-                if (is_callable('stream_set_read_buffer')) {
107
+                if (is_callable('stream_set_read_buffer'))
108
+                {
100 109
                     stream_set_read_buffer($fp, RANDOM_COMPAT_READ_BUFFER);
101 110
                 }
102
-                if (is_callable('stream_set_chunk_size')) {
111
+                if (is_callable('stream_set_chunk_size'))
112
+                {
103 113
                     stream_set_chunk_size($fp, RANDOM_COMPAT_READ_BUFFER);
104 114
                 }
105 115
             }
106 116
         }
107 117
 
108
-        try {
118
+        try
119
+        {
109 120
             /** @var int $bytes */
110 121
             $bytes = RandomCompat_intval($bytes);
111
-        } catch (TypeError $ex) {
122
+        }
123
+        catch (TypeError $ex)
124
+        {
112 125
             throw new TypeError(
113 126
                 'random_bytes(): $bytes must be an integer'
114 127
             );
115 128
         }
116 129
 
117
-        if ($bytes < 1) {
130
+        if ($bytes < 1)
131
+        {
118 132
             throw new Error(
119 133
                 'Length must be greater than 0'
120 134
             );
@@ -127,7 +141,8 @@  discard block
 block discarded – undo
127 141
          * if (empty($fp)) line is logic that should only be run once per
128 142
          * page load.
129 143
          */
130
-        if (is_resource($fp)) {
144
+        if (is_resource($fp))
145
+        {
131 146
             /**
132 147
              * @var int
133 148
              */
@@ -146,7 +161,8 @@  discard block
 block discarded – undo
146 161
                  * @var string|bool
147 162
                  */
148 163
                 $read = fread($fp, $remaining);
149
-                if (!is_string($read)) {
164
+                if (!is_string($read))
165
+                {
150 166
                     /**
151 167
                      * We cannot safely read from the file. Exit the
152 168
                      * do-while loop and trigger the exception condition
@@ -170,8 +186,10 @@  discard block
 block discarded – undo
170 186
              * Is our result valid?
171 187
              * @var string|bool $buf
172 188
              */
173
-            if (is_string($buf)) {
174
-                if (RandomCompat_strlen($buf) === $bytes) {
189
+            if (is_string($buf))
190
+            {
191
+                if (RandomCompat_strlen($buf) === $bytes)
192
+                {
175 193
                     /**
176 194
                      * Return our random entropy buffer here:
177 195
                      */
Please login to merge, or discard this patch.
Sources/Subs-Admin.php 1 patch
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1219,7 +1219,8 @@  discard block
 block discarded – undo
1219 1219
 	}
1220 1220
 
1221 1221
 	// It's important to do the numbered ones before the named ones, or messes happen.
1222
-	uksort($substitutions, function($a, $b) {
1222
+	uksort($substitutions, function($a, $b)
1223
+	{
1223 1224
 		if (is_int($a) && is_int($b))
1224 1225
 			return $a > $b ? 1 : ($a < $b ? -1 : 0);
1225 1226
 		elseif (is_int($a))
@@ -1758,8 +1759,12 @@  discard block
 block discarded – undo
1758 1759
 		unset($mtime, $settingsFile, $settingsText);
1759 1760
 		$defined_vars = get_defined_vars();
1760 1761
 	}
1761
-	catch (Throwable $e) {}
1762
-	catch (ErrorException $e) {}
1762
+	catch (Throwable $e)
1763
+	{
1764
+}
1765
+	catch (ErrorException $e)
1766
+	{
1767
+}
1763 1768
 	if (isset($e))
1764 1769
 		return false;
1765 1770
 
@@ -1915,7 +1920,8 @@  discard block
 block discarded – undo
1915 1920
 	// For the same reason, replace literal returns and newlines with "\r" and "\n"
1916 1921
 	elseif (is_string($var) && (strpos($var, "\n") !== false || strpos($var, "\r") !== false))
1917 1922
 	{
1918
-		return strtr(preg_replace_callback('/[\r\n]+/', function($m) {
1923
+		return strtr(preg_replace_callback('/[\r\n]+/', function($m)
1924
+		{
1919 1925
 			return '\' . "' . strtr($m[0], array("\r" => '\r', "\n" => '\n')) . '" . \'';
1920 1926
 		}, $var), array("'' . " => '', " . ''" => ''));
1921 1927
 	}
Please login to merge, or discard this patch.
Sources/Subs-BoardIndex.php 1 patch
Braces   +1 added lines, -4 removed lines patch added patch discarded remove patch
@@ -280,7 +280,6 @@  discard block
 block discarded – undo
280 280
 					$this_category[$row_board['id_board']]['board_class'] = 'on';
281 281
 					$this_category[$row_board['id_board']]['board_tooltip'] = $txt['new_posts'];
282 282
 				}
283
-
284 283
 				else
285 284
 				{
286 285
 					$this_category[$row_board['id_board']]['board_tooltip'] = $txt['old_posts'];
@@ -415,7 +414,6 @@  discard block
 block discarded – undo
415 414
 			$this_last_post['href'] = $scripturl . '?topic=' . $row_board['id_topic'] . '.msg' . ($user_info['is_guest'] ? $row_board['id_msg'] : $row_board['new_from']) . (empty($row_board['is_read']) ? ';boardseen' : '') . '#new';
416 415
 			$this_last_post['link'] = '<a href="' . $this_last_post['href'] . '" title="' . $row_board['subject'] . '">' . $row_board['short_subject'] . '</a>';
417 416
 		}
418
-
419 417
 		else
420 418
 		{
421 419
 			$this_last_post['href'] = '';
@@ -510,8 +508,7 @@  discard block
 block discarded – undo
510 508
 					$board['last_post']['last_post_message'] = sprintf($txt['last_post_message'], $board['last_post']['member']['link'], $board['last_post']['link'], $board['last_post']['time'] > 0 ? timeformat($board['last_post']['time']) : $txt['not_applicable']);
511 509
 			}
512 510
 		}
513
-
514
-	else
511
+		else
515 512
 		foreach ($this_category as &$board)
516 513
 		{
517 514
 			if (isset($boards_parsed_data_by_cat_id[$board['id_cat']][$board['id']]))
Please login to merge, or discard this patch.
Sources/Subs.php 1 patch
Braces   +7 added lines, -10 removed lines patch added patch discarded remove patch
@@ -279,7 +279,6 @@  discard block
 block discarded – undo
279 279
 		$condition = 'id_member IN ({array_int:members})';
280 280
 		$parameters['members'] = $members;
281 281
 	}
282
-
283 282
 	elseif ($members === null)
284 283
 		$condition = '1=1';
285 284
 
@@ -382,11 +381,9 @@  discard block
 block discarded – undo
382 381
 				$val = $val . ' END';
383 382
 				$type = 'raw';
384 383
 			}
385
-
386 384
 			else
387 385
 				$val = alert_count($members, true);
388 386
 		}
389
-
390 387
 		elseif ($type == 'int' && ($val === '+' || $val === '-'))
391 388
 		{
392 389
 			$val = $var . ' ' . $val . ' 1';
@@ -2202,12 +2199,12 @@  discard block
 block discarded – undo
2202 2199
 			'tag' => 'cowsay',
2203 2200
 			'parameters' => array(
2204 2201
 				'e' => array('optional' => true, 'quoted' => true, 'match' => '(.*?)', 'default' => 'oo', 'validate' => function ($eyes) use ($smcFunc)
2205
-					{
2202
+				{
2206 2203
 						return $smcFunc['substr']($eyes . 'oo', 0, 2);
2207 2204
 					},
2208 2205
 				),
2209 2206
 				't' => array('optional' => true, 'quoted' => true, 'match' => '(.*?)', 'default' => '  ', 'validate' => function ($tongue) use ($smcFunc)
2210
-					{
2207
+				{
2211 2208
 						return $smcFunc['substr']($tongue . '  ', 0, 2);
2212 2209
 					},
2213 2210
 				),
@@ -3314,7 +3311,7 @@  discard block
 block discarded – undo
3314 3311
 
3315 3312
 	// Replace away!
3316 3313
 	$message = preg_replace_callback($smileyPregSearch, function($matches) use ($smileyPregReplacements)
3317
-		{
3314
+	{
3318 3315
 			return $smileyPregReplacements[$matches[1]];
3319 3316
 		}, $message);
3320 3317
 }
@@ -4112,7 +4109,6 @@  discard block
 block discarded – undo
4112 4109
 				if (!isset($minSeed) && isset($js_file['options']['seed']))
4113 4110
 					$minSeed = $js_file['options']['seed'];
4114 4111
 			}
4115
-
4116 4112
 			else
4117 4113
 			{
4118 4114
 				echo '
@@ -6029,7 +6025,8 @@  discard block
 block discarded – undo
6029 6025
 			$zones[$tzkey]['tzid'] = $tzid;
6030 6026
 			$zones[$tzkey]['dst_type'] = count($tzinfo) > 1 ? 1 : ($tzinfo[0]['isdst'] ? 2 : 0);
6031 6027
 
6032
-			foreach ($tzinfo as $transition) {
6028
+			foreach ($tzinfo as $transition)
6029
+			{
6033 6030
 				$zones[$tzkey]['abbrs'][] = $transition['abbr'];
6034 6031
 			}
6035 6032
 
@@ -6114,7 +6111,8 @@  discard block
 block discarded – undo
6114 6111
 			$desc = implode(', ', array_slice(array_unique($tzvalue['locations']), 0, 5)) . (count($tzvalue['locations']) > 5 ? ', ' . $txt['etc'] : '');
6115 6112
 
6116 6113
 		// We don't want abbreviations like '+03' or '-11'.
6117
-		$abbrs = array_filter($tzvalue['abbrs'], function ($abbr) {
6114
+		$abbrs = array_filter($tzvalue['abbrs'], function ($abbr)
6115
+		{
6118 6116
 			return !strspn($abbr, '+-');
6119 6117
 		});
6120 6118
 		$abbrs = count($abbrs) == count($tzvalue['abbrs']) ? array_unique($abbrs) : array();
@@ -6462,7 +6460,6 @@  discard block
 block discarded – undo
6462 6460
 			$isWritable = true;
6463 6461
 			break;
6464 6462
 		}
6465
-
6466 6463
 		else
6467 6464
 			@chmod($file, $val);
6468 6465
 	}
Please login to merge, or discard this patch.
Sources/Profile-View.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -359,7 +359,8 @@
 block discarded – undo
359 359
 	call_integration_hook('integrate_fetch_alerts', array(&$alerts, &$formats));
360 360
 
361 361
 	// Substitute $scripturl into the link formats. (Done here to make life easier for hooked mods.)
362
-	$formats = array_map(function ($format) use ($scripturl) {
362
+	$formats = array_map(function ($format) use ($scripturl)
363
+	{
363 364
 		$format['link'] = str_replace('{scripturl}', $scripturl, $format['link']);
364 365
 		$format['text'] = str_replace('{scripturl}', $scripturl, $format['text']);
365 366
 
Please login to merge, or discard this patch.