Passed
Push — master ( 5a5c41...fade3b )
by Cody
04:50 queued 10s
created
plugins/af_tumblr_1280/init.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -22,8 +22,9 @@
 block discarded – undo
22 22
 
23 23
 	public function hook_article_filter($article) {
24 24
 
25
-		if (!function_exists("curl_init") || ini_get("open_basedir"))
26
-			return $article;
25
+		if (!function_exists("curl_init") || ini_get("open_basedir")) {
26
+					return $article;
27
+		}
27 28
 
28 29
 		$doc = new DOMDocument();
29 30
 		$doc->loadHTML('<?xml encoding="UTF-8">' . $article["content"]);
Please login to merge, or discard this patch.
plugins/af_unburn/init.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -21,8 +21,9 @@  discard block
 block discarded – undo
21 21
 	public function hook_article_filter($article) {
22 22
 		$owner_uid = $article["owner_uid"];
23 23
 
24
-		if (defined('NO_CURL') || !function_exists("curl_init") || ini_get("open_basedir"))
25
-			return $article;
24
+		if (defined('NO_CURL') || !function_exists("curl_init") || ini_get("open_basedir")) {
25
+					return $article;
26
+		}
26 27
 
27 28
 		if ((strpos($article["link"], "feedproxy.google.com") !== FALSE ||
28 29
 		  		strpos($article["link"], "/~r/") !== FALSE ||
@@ -57,7 +58,9 @@  discard block
 block discarded – undo
57 58
 						parse_str($query, $args);
58 59
 
59 60
 						foreach (array("utm_source", "utm_medium", "utm_campaign") as $param) {
60
-							if (isset($args[$param])) unset($args[$param]);
61
+							if (isset($args[$param])) {
62
+								unset($args[$param]);
63
+							}
61 64
 						}
62 65
 
63 66
 						$new_query = http_build_query($args);
Please login to merge, or discard this patch.
plugins/af_redditimgur/init.php 1 patch
Braces   +13 added lines, -7 removed lines patch added patch discarded remove patch
@@ -22,7 +22,9 @@  discard block
 block discarded – undo
22 22
 	}
23 23
 
24 24
 	public function hook_prefs_tab($args) {
25
-		if ($args != "prefFeeds") return;
25
+		if ($args != "prefFeeds") {
26
+			return;
27
+		}
26 28
 
27 29
 		print "<div dojoType=\"dijit.layout.AccordionPane\"
28 30
 			title=\"<i class='material-icons'>extension</i> ".__('Reddit content settings (af_redditimgur)')."\">";
@@ -173,8 +175,7 @@  discard block
 block discarded – undo
173 175
 									if ($child["data"]["url"] == $matches[0]) {
174 176
 										try {
175 177
 											$source_stream = $child["data"]["media"]["reddit_video"]["fallback_url"];
176
-										}
177
-										catch (Exception $e) {
178
+										} catch (Exception $e) {
178 179
 										}
179 180
 										break 2;
180 181
 									}
@@ -230,8 +231,9 @@  discard block
 block discarded – undo
230 231
 
231 232
 					$source_stream = str_replace(".gifv", ".mp4", $entry->getAttribute("href"));
232 233
 
233
-					if (strpos($source_stream, "imgur.com") !== FALSE)
234
-						$poster_url = str_replace(".mp4", "h.jpg", $source_stream);
234
+					if (strpos($source_stream, "imgur.com") !== FALSE) {
235
+											$poster_url = str_replace(".mp4", "h.jpg", $source_stream);
236
+					}
235 237
 
236 238
 					$this->handle_as_video($doc, $entry, $source_stream, $poster_url);
237 239
 
@@ -426,7 +428,9 @@  discard block
 block discarded – undo
426 428
 					if ($row = $sth->fetch()) {
427 429
 						$num_found = $row['cid'];
428 430
 
429
-						if ($num_found > 0) $article["force_catchup"] = true;
431
+						if ($num_found > 0) {
432
+							$article["force_catchup"] = true;
433
+						}
430 434
 					}
431 435
 				}
432 436
 			}
@@ -458,7 +462,9 @@  discard block
 block discarded – undo
458 462
 		$video->setAttribute("controls", "1");
459 463
 		$video->setAttribute("loop", "1");
460 464
 
461
-		if ($poster_url) $video->setAttribute("poster", $poster_url);
465
+		if ($poster_url) {
466
+			$video->setAttribute("poster", $poster_url);
467
+		}
462 468
 
463 469
 		$source = $doc->createElement('source');
464 470
 		$source->setAttribute("src", $source_stream);
Please login to merge, or discard this patch.
plugins/mail/init.php 1 patch
Braces   +9 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,7 +30,9 @@  discard block
 block discarded – undo
30 30
 	}
31 31
 
32 32
 	public function hook_prefs_tab($args) {
33
-		if ($args != "prefPrefs") return;
33
+		if ($args != "prefPrefs") {
34
+			return;
35
+		}
34 36
 
35 37
 		print "<div dojoType=\"dijit.layout.AccordionPane\"
36 38
 			title=\"<i class='material-icons'>mail</i> ".__('Mail plugin')."\">";
@@ -94,7 +96,9 @@  discard block
 block discarded – undo
94 96
 			$user_name = htmlspecialchars($row['full_name']);
95 97
 		}
96 98
 
97
-		if (!$user_name) $user_name = $_SESSION['name'];
99
+		if (!$user_name) {
100
+			$user_name = $_SESSION['name'];
101
+		}
98 102
 
99 103
 		print_hidden("from_email", "$user_email");
100 104
 		print_hidden("from_name", "$user_name");
@@ -120,8 +124,9 @@  discard block
 block discarded – undo
120 124
 
121 125
 		while ($line = $sth->fetch()) {
122 126
 
123
-			if (!$subject)
124
-				$subject = __("[Forwarded]") . " " . htmlspecialchars($line["title"]);
127
+			if (!$subject) {
128
+							$subject = __("[Forwarded]") . " " . htmlspecialchars($line["title"]);
129
+			}
125 130
 
126 131
 			$tpl->setVariable('ARTICLE_TITLE', strip_tags($line["title"]));
127 132
 			$tnote = strip_tags($line["note"]);
Please login to merge, or discard this patch.
plugins/af_comics/init.php 1 patch
Braces   +25 added lines, -15 removed lines patch added patch discarded remove patch
@@ -45,7 +45,9 @@  discard block
 block discarded – undo
45 45
 	}
46 46
 
47 47
 	public function hook_prefs_tab($args) {
48
-		if ($args != "prefFeeds") return;
48
+		if ($args != "prefFeeds") {
49
+			return;
50
+		}
49 51
 
50 52
 		print "<div dojoType=\"dijit.layout.AccordionPane\"
51 53
 			title=\"<i class='material-icons'>photo</i> ".__('Feeds supported by af_comics')."\">";
@@ -77,8 +79,9 @@  discard block
 block discarded – undo
77 79
 
78 80
 	public function hook_article_filter($article) {
79 81
 		foreach ($this->filters as $f) {
80
-			if ($f->process($article))
81
-				break;
82
+			if ($f->process($article)) {
83
+							break;
84
+			}
82 85
 		}
83 86
 
84 87
 		return $article;
@@ -89,8 +92,9 @@  discard block
 block discarded – undo
89 92
 	 * @SuppressWarnings(PHPMD.UnusedFormalParameter)
90 93
 	 */
91 94
 	public function hook_fetch_feed($feed_data, $fetch_url, $owner_uid, $feed, $last_article_timestamp, $auth_login, $auth_pass) {
92
-		if ($auth_login || $auth_pass)
93
-			return $feed_data;
95
+		if ($auth_login || $auth_pass) {
96
+					return $feed_data;
97
+		}
94 98
 
95 99
 		if (preg_match('#^https?://(?:feeds\.feedburner\.com/uclick|www\.gocomics\.com)/([-a-z0-9]+)$#i', $fetch_url, $comic)) {
96 100
 			$site_url = 'https://www.gocomics.com/' . $comic[1];
@@ -153,29 +157,35 @@  discard block
 block discarded – undo
153 157
 
154 158
 			$tpl->addBlock('feed');
155 159
 
156
-			if ($tpl->generateOutputToString($tmp_data))
157
-				$feed_data = $tmp_data;
160
+			if ($tpl->generateOutputToString($tmp_data)) {
161
+							$feed_data = $tmp_data;
162
+			}
158 163
 		}
159 164
 
160 165
 		return $feed_data;
161 166
 	}
162 167
 
163 168
 	public function hook_subscribe_feed($contents, $url, $auth_login, $auth_pass) {
164
-		if ($auth_login || $auth_pass)
165
-			return $contents;
169
+		if ($auth_login || $auth_pass) {
170
+					return $contents;
171
+		}
166 172
 
167
-		if (preg_match('#^https?://www\.gocomics\.com/([-a-z0-9]+)$#i', $url))
168
-			return '<?xml version="1.0" encoding="utf-8"?>'; // Get is_html() to return false.
173
+		if (preg_match('#^https?://www\.gocomics\.com/([-a-z0-9]+)$#i', $url)) {
174
+					return '<?xml version="1.0" encoding="utf-8"?>';
175
+		}
176
+		// Get is_html() to return false.
169 177
 
170 178
 		return $contents;
171 179
 	}
172 180
 
173 181
 	public function hook_feed_basic_info($basic_info, $fetch_url, $owner_uid, $feed, $auth_login, $auth_pass) {
174
-		if ($auth_login || $auth_pass)
175
-			return $basic_info;
182
+		if ($auth_login || $auth_pass) {
183
+					return $basic_info;
184
+		}
176 185
 
177
-		if (preg_match('#^https?://www\.gocomics\.com/([-a-z0-9]+)$#i', $fetch_url, $matches))
178
-			$basic_info = array('title' => ucfirst($matches[1]), 'site_url' => $matches[0]);
186
+		if (preg_match('#^https?://www\.gocomics\.com/([-a-z0-9]+)$#i', $fetch_url, $matches)) {
187
+					$basic_info = array('title' => ucfirst($matches[1]), 'site_url' => $matches[0]);
188
+		}
179 189
 
180 190
 		return $basic_info;
181 191
 	}
Please login to merge, or discard this patch.
plugins/af_proxy_http/init.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -200,13 +200,17 @@
 block discarded – undo
200 200
 			}
201 201
 		}
202 202
 
203
-		if ($need_saving) $article["content"] = $doc->saveHTML();
203
+		if ($need_saving) {
204
+			$article["content"] = $doc->saveHTML();
205
+		}
204 206
 
205 207
 		return $article;
206 208
 	}
207 209
 
208 210
 	public function hook_prefs_tab($args) {
209
-		if ($args != "prefFeeds") return;
211
+		if ($args != "prefFeeds") {
212
+			return;
213
+		}
210 214
 
211 215
 		print "<div dojoType=\"dijit.layout.AccordionPane\"
212 216
 			title=\"<i class='material-icons'>extension</i> ".__('Image proxy settings (af_proxy_http)')."\">";
Please login to merge, or discard this patch.
plugins/search_sphinx/init.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -53,10 +53,11 @@
 block discarded – undo
53 53
 
54 54
 		$ids = join(",", $ids);
55 55
 
56
-		if ($ids)
57
-			return array("ref_id IN ($ids)", array());
58
-		else
59
-			return array("ref_id = -1", array());
56
+		if ($ids) {
57
+					return array("ref_id IN ($ids)", array());
58
+		} else {
59
+					return array("ref_id = -1", array());
60
+		}
60 61
 	}
61 62
 
62 63
 	public function api_version() {
Please login to merge, or discard this patch.
plugins/search_sphinx/sphinxapi.php 1 patch
Braces   +241 added lines, -134 removed lines patch added patch discarded remove patch
@@ -133,14 +133,16 @@  discard block
 block discarded – undo
133 133
 	}
134 134
 
135 135
 	// x32, int
136
-	if ( is_int($v) )
137
-		return pack ( "NN", $v < 0 ? -1 : 0, $v );
136
+	if ( is_int($v) ) {
137
+			return pack ( "NN", $v < 0 ? -1 : 0, $v );
138
+	}
138 139
 
139 140
 	// x32, bcmath
140 141
 	if ( function_exists("bcmul") )
141 142
 	{
142
-		if ( bccomp ( $v, 0 ) == -1 )
143
-			$v = bcadd ( "18446744073709551616", $v );
143
+		if ( bccomp ( $v, 0 ) == -1 ) {
144
+					$v = bcadd ( "18446744073709551616", $v );
145
+		}
144 146
 		$h = bcdiv ( $v, "4294967296", 0 );
145 147
 		$l = bcmod ( $v, "4294967296" );
146 148
 		return pack ( "NN", (float)$h, (float)$l ); // conversion to float is intentional; int would lose 31st bit
@@ -158,9 +160,9 @@  discard block
 block discarded – undo
158 160
 
159 161
 	if ( $v<0 )
160 162
 	{
161
-		if ( $l==0 )
162
-			$h = 4294967296.0 - $h;
163
-		else
163
+		if ( $l==0 ) {
164
+					$h = 4294967296.0 - $h;
165
+		} else
164 166
 		{
165 167
 			$h = 4294967295.0 - $h;
166 168
 			$l = 4294967296.0 - $l;
@@ -180,8 +182,9 @@  discard block
 block discarded – undo
180 182
 		assert ( $v>=0 );
181 183
 
182 184
 		// x64, int
183
-		if ( is_int($v) )
184
-			return pack ( "NN", $v>>32, $v&0xFFFFFFFF );
185
+		if ( is_int($v) ) {
186
+					return pack ( "NN", $v>>32, $v&0xFFFFFFFF );
187
+		}
185 188
 
186 189
 		// x64, bcmath
187 190
 		if ( function_exists("bcmul") )
@@ -204,8 +207,9 @@  discard block
 block discarded – undo
204 207
 	}
205 208
 
206 209
 	// x32, int
207
-	if ( is_int($v) )
208
-		return pack ( "NN", 0, $v );
210
+	if ( is_int($v) ) {
211
+			return pack ( "NN", 0, $v );
212
+	}
209 213
 
210 214
 	// x32, bcmath
211 215
 	if ( function_exists("bcmul") )
@@ -235,16 +239,23 @@  discard block
 block discarded – undo
235 239
 
236 240
 	if ( PHP_INT_SIZE>=8 )
237 241
 	{
238
-		if ( $hi<0 ) $hi += (1<<32); // because php 5.2.2 to 5.2.5 is totally fucked up again
239
-		if ( $lo<0 ) $lo += (1<<32);
242
+		if ( $hi<0 ) {
243
+			$hi += (1<<32);
244
+		}
245
+		// because php 5.2.2 to 5.2.5 is totally fucked up again
246
+		if ( $lo<0 ) {
247
+			$lo += (1<<32);
248
+		}
240 249
 
241 250
 		// x64, int
242
-		if ( $hi<=2147483647 )
243
-			return ($hi<<32) + $lo;
251
+		if ( $hi<=2147483647 ) {
252
+					return ($hi<<32) + $lo;
253
+		}
244 254
 
245 255
 		// x64, bcmath
246
-		if ( function_exists("bcmul") )
247
-			return bcadd ( $lo, bcmul ( $hi, "4294967296" ) );
256
+		if ( function_exists("bcmul") ) {
257
+					return bcadd ( $lo, bcmul ( $hi, "4294967296" ) );
258
+		}
248 259
 
249 260
 		// x64, no-bcmath
250 261
 		$C = 100000;
@@ -256,16 +267,18 @@  discard block
 block discarded – undo
256 267
 			$l  = $l % $C;
257 268
 		}
258 269
 
259
-		if ( $h==0 )
260
-			return $l;
270
+		if ( $h==0 ) {
271
+					return $l;
272
+		}
261 273
 		return sprintf ( "%d%05d", $h, $l );
262 274
 	}
263 275
 
264 276
 	// x32, int
265 277
 	if ( $hi==0 )
266 278
 	{
267
-		if ( $lo>0 )
268
-			return $lo;
279
+		if ( $lo>0 ) {
280
+					return $lo;
281
+		}
269 282
 		return sprintf ( "%u", $lo );
270 283
 	}
271 284
 
@@ -273,8 +286,9 @@  discard block
 block discarded – undo
273 286
 	$lo = sprintf ( "%u", $lo );
274 287
 
275 288
 	// x32, bcmath
276
-	if ( function_exists("bcmul") )
277
-		return bcadd ( $lo, bcmul ( $hi, "4294967296" ) );
289
+	if ( function_exists("bcmul") ) {
290
+			return bcadd ( $lo, bcmul ( $hi, "4294967296" ) );
291
+	}
278 292
 
279 293
 	// x32, no-bcmath
280 294
 	$hi = (float)$hi;
@@ -289,8 +303,9 @@  discard block
 block discarded – undo
289 303
 
290 304
 	$h = sprintf ( "%.0f", $h );
291 305
 	$l = sprintf ( "%07.0f", $l );
292
-	if ( $h=="0" )
293
-		return sprintf( "%.0f", (float)$l );
306
+	if ( $h=="0" ) {
307
+			return sprintf( "%.0f", (float)$l );
308
+	}
294 309
 	return $h . $l;
295 310
 }
296 311
 
@@ -302,8 +317,13 @@  discard block
 block discarded – undo
302 317
 	// x64
303 318
 	if ( PHP_INT_SIZE>=8 )
304 319
 	{
305
-		if ( $hi<0 ) $hi += (1<<32); // because php 5.2.2 to 5.2.5 is totally fucked up again
306
-		if ( $lo<0 ) $lo += (1<<32);
320
+		if ( $hi<0 ) {
321
+			$hi += (1<<32);
322
+		}
323
+		// because php 5.2.2 to 5.2.5 is totally fucked up again
324
+		if ( $lo<0 ) {
325
+			$lo += (1<<32);
326
+		}
307 327
 
308 328
 		return ($hi<<32) + $lo;
309 329
 	}
@@ -311,15 +331,17 @@  discard block
 block discarded – undo
311 331
 	// x32, int
312 332
 	if ( $hi==0 )
313 333
 	{
314
-		if ( $lo>0 )
315
-			return $lo;
334
+		if ( $lo>0 ) {
335
+					return $lo;
336
+		}
316 337
 		return sprintf ( "%u", $lo );
317 338
 	}
318 339
 	// x32, int
319 340
 	elseif ( $hi==-1 )
320 341
 	{
321
-		if ( $lo<0 )
322
-			return $lo;
342
+		if ( $lo<0 ) {
343
+					return $lo;
344
+		}
323 345
 		return sprintf ( "%.0f", $lo - 4294967296.0 );
324 346
 	}
325 347
 
@@ -337,8 +359,9 @@  discard block
 block discarded – undo
337 359
 	$lo = sprintf ( "%u", $lo );
338 360
 
339 361
 	// x32, bcmath
340
-	if ( function_exists("bcmul") )
341
-		return $neg . bcadd ( bcadd ( $lo, bcmul ( $hi, "4294967296" ) ), $c );
362
+	if ( function_exists("bcmul") ) {
363
+			return $neg . bcadd ( bcadd ( $lo, bcmul ( $hi, "4294967296" ) ), $c );
364
+	}
342 365
 
343 366
 	// x32, no-bcmath
344 367
 	$hi = (float)$hi;
@@ -358,8 +381,9 @@  discard block
 block discarded – undo
358 381
 
359 382
 	$h = sprintf ( "%.0f", $h );
360 383
 	$l = sprintf ( "%07.0f", $l );
361
-	if ( $h=="0" )
362
-		return $neg . sprintf( "%.0f", (float)$l );
384
+	if ( $h=="0" ) {
385
+			return $neg . sprintf( "%.0f", (float)$l );
386
+	}
363 387
 	return $neg . $h . $l;
364 388
 }
365 389
 
@@ -369,10 +393,11 @@  discard block
 block discarded – undo
369 393
 	if ( PHP_INT_SIZE>=8 )
370 394
 	{
371 395
 		// x64 route, workaround broken unpack() in 5.2.2+
372
-		if ( $value<0 ) $value += (1<<32);
396
+		if ( $value<0 ) {
397
+			$value += (1<<32);
398
+		}
373 399
 		return $value;
374
-	}
375
-	else
400
+	} else
376 401
 	{
377 402
 		// x32 route, workaround php signed/unsigned braindamage
378 403
 		return sprintf ( "%u", $value );
@@ -470,8 +495,9 @@  discard block
 block discarded – undo
470 495
 
471 496
 	public function __destruct()
472 497
 	{
473
-		if ( $this->_socket !== false )
474
-			fclose ( $this->_socket );
498
+		if ( $this->_socket !== false ) {
499
+					fclose ( $this->_socket );
500
+		}
475 501
 	}
476 502
 
477 503
 	/// get last error message (string)
@@ -549,8 +575,9 @@  discard block
 block discarded – undo
549 575
 	/// leave mbstring workaround mode
550 576
 	public function _MBPop ()
551 577
 	{
552
-		if ( $this->_mbenc )
553
-			mb_internal_encoding ( $this->_mbenc );
578
+		if ( $this->_mbenc ) {
579
+					mb_internal_encoding ( $this->_mbenc );
580
+		}
554 581
 	}
555 582
 
556 583
 	/// connect to searchd server
@@ -560,8 +587,9 @@  discard block
 block discarded – undo
560 587
 		{
561 588
 			// we are in persistent connection mode, so we have a socket
562 589
 			// however, need to check whether it's still alive
563
-			if ( !@feof ( $this->_socket ) )
564
-				return $this->_socket;
590
+			if ( !@feof ( $this->_socket ) ) {
591
+							return $this->_socket;
592
+			}
565 593
 
566 594
 			// force reopen
567 595
 			$this->_socket = false;
@@ -575,24 +603,25 @@  discard block
 block discarded – undo
575 603
 		{
576 604
 			$host = $this->_path;
577 605
 			$port = 0;
578
-		}
579
-		else
606
+		} else
580 607
 		{
581 608
 			$host = $this->_host;
582 609
 			$port = $this->_port;
583 610
 		}
584 611
 
585
-		if ( $this->_timeout<=0 )
586
-			$fp = @fsockopen ( $host, $port, $errno, $errstr );
587
-		else
588
-			$fp = @fsockopen ( $host, $port, $errno, $errstr, $this->_timeout );
612
+		if ( $this->_timeout<=0 ) {
613
+					$fp = @fsockopen ( $host, $port, $errno, $errstr );
614
+		} else {
615
+					$fp = @fsockopen ( $host, $port, $errno, $errstr, $this->_timeout );
616
+		}
589 617
 
590 618
 		if ( !$fp )
591 619
 		{
592
-			if ( $this->_path )
593
-				$location = $this->_path;
594
-			else
595
-				$location = "{$this->_host}:{$this->_port}";
620
+			if ( $this->_path ) {
621
+							$location = $this->_path;
622
+			} else {
623
+							$location = "{$this->_host}:{$this->_port}";
624
+			}
596 625
 
597 626
 			$errstr = trim ( $errstr );
598 627
 			$this->_error = "connection to $location failed (errno=$errno, msg=$errstr)";
@@ -645,8 +674,9 @@  discard block
 block discarded – undo
645 674
 				}
646 675
 			}
647 676
 		}
648
-		if ( $this->_socket === false )
649
-			fclose ( $fp );
677
+		if ( $this->_socket === false ) {
678
+					fclose ( $fp );
679
+		}
650 680
 
651 681
 		// check response
652 682
 		$read = strlen ( $response );
@@ -706,10 +736,12 @@  discard block
 block discarded – undo
706 736
 		assert ( $max>=0 );
707 737
 		$this->_offset = $offset;
708 738
 		$this->_limit = $limit;
709
-		if ( $max>0 )
710
-			$this->_maxmatches = $max;
711
-		if ( $cutoff>0 )
712
-			$this->_cutoff = $cutoff;
739
+		if ( $max>0 ) {
740
+					$this->_maxmatches = $max;
741
+		}
742
+		if ( $cutoff>0 ) {
743
+					$this->_cutoff = $cutoff;
744
+		}
713 745
 	}
714 746
 
715 747
 	/// set maximum query time, in milliseconds, per-index
@@ -763,8 +795,9 @@  discard block
 block discarded – undo
763 795
 	public function SetWeights ( $weights )
764 796
 	{
765 797
 		assert ( is_array($weights) );
766
-		foreach ( $weights as $weight )
767
-			assert ( is_int($weight) );
798
+		foreach ( $weights as $weight ) {
799
+					assert ( is_int($weight) );
800
+		}
768 801
 
769 802
 		$this->_weights = $weights;
770 803
 	}
@@ -814,8 +847,9 @@  discard block
 block discarded – undo
814 847
 
815 848
 		if ( is_array($values) && count($values) )
816 849
 		{
817
-			foreach ( $values as $value )
818
-				assert ( is_numeric($value) );
850
+			foreach ( $values as $value ) {
851
+							assert ( is_numeric($value) );
852
+			}
819 853
 
820 854
 			$this->_filters[] = array ( "type"=>SPH_FILTER_VALUES, "attr"=>$attribute, "exclude"=>$exclude, "values"=>$values );
821 855
 		}
@@ -954,15 +988,18 @@  discard block
 block discarded – undo
954 988
 		$results = $this->RunQueries ();
955 989
 		$this->_reqs = array (); // just in case it failed too early
956 990
 
957
-		if ( !is_array($results) )
958
-			return false; // probably network error; error message should be already filled
991
+		if ( !is_array($results) ) {
992
+					return false;
993
+		}
994
+		// probably network error; error message should be already filled
959 995
 
960 996
 		$this->_error = $results[0]["error"];
961 997
 		$this->_warning = $results[0]["warning"];
962
-		if ( $results[0]["status"]==SEARCHD_ERROR )
963
-			return false;
964
-		else
965
-			return $results[0];
998
+		if ( $results[0]["status"]==SEARCHD_ERROR ) {
999
+					return false;
1000
+		} else {
1001
+					return $results[0];
1002
+		}
966 1003
 	}
967 1004
 
968 1005
 	/// helper to pack floats in network byte order
@@ -989,8 +1026,9 @@  discard block
 block discarded – undo
989 1026
 		$req .= pack ( "N", strlen($this->_sortby) ) . $this->_sortby;
990 1027
 		$req .= pack ( "N", strlen($query) ) . $query; // query itself
991 1028
 		$req .= pack ( "N", count($this->_weights) ); // weights
992
-		foreach ( $this->_weights as $weight )
993
-			$req .= pack ( "N", (int)$weight );
1029
+		foreach ( $this->_weights as $weight ) {
1030
+					$req .= pack ( "N", (int)$weight );
1031
+		}
994 1032
 		$req .= pack ( "N", strlen($index) ) . $index; // indexes
995 1033
 		$req .= pack ( "N", 1 ); // id64 range marker
996 1034
 		$req .= sphPackU64 ( $this->_min_id ) . sphPackU64 ( $this->_max_id ); // id64 range
@@ -1005,8 +1043,9 @@  discard block
 block discarded – undo
1005 1043
 			{
1006 1044
 				case SPH_FILTER_VALUES:
1007 1045
 					$req .= pack ( "N", count($filter["values"]) );
1008
-					foreach ( $filter["values"] as $value )
1009
-						$req .= sphPackI64 ( $value );
1046
+					foreach ( $filter["values"] as $value ) {
1047
+											$req .= sphPackI64 ( $value );
1048
+					}
1010 1049
 					break;
1011 1050
 
1012 1051
 				case SPH_FILTER_RANGE:
@@ -1045,16 +1084,18 @@  discard block
 block discarded – undo
1045 1084
 
1046 1085
 		// per-index weights
1047 1086
 		$req .= pack ( "N", count($this->_indexweights) );
1048
-		foreach ( $this->_indexweights as $idx=>$weight )
1049
-			$req .= pack ( "N", strlen($idx) ) . $idx . pack ( "N", $weight );
1087
+		foreach ( $this->_indexweights as $idx=>$weight ) {
1088
+					$req .= pack ( "N", strlen($idx) ) . $idx . pack ( "N", $weight );
1089
+		}
1050 1090
 
1051 1091
 		// max query time
1052 1092
 		$req .= pack ( "N", $this->_maxquerytime );
1053 1093
 
1054 1094
 		// per-field weights
1055 1095
 		$req .= pack ( "N", count($this->_fieldweights) );
1056
-		foreach ( $this->_fieldweights as $field=>$weight )
1057
-			$req .= pack ( "N", strlen($field) ) . $field . pack ( "N", $weight );
1096
+		foreach ( $this->_fieldweights as $field=>$weight ) {
1097
+					$req .= pack ( "N", strlen($field) ) . $field . pack ( "N", $weight );
1098
+		}
1058 1099
 
1059 1100
 		// comment
1060 1101
 		$req .= pack ( "N", strlen($comment) ) . $comment;
@@ -1200,8 +1241,7 @@  discard block
 block discarded – undo
1200 1241
 				{
1201 1242
 					$doc = sphUnpackU64 ( substr ( $response, $p, 8 ) ); $p += 8;
1202 1243
 					list(,$weight) = unpack ( "N*", substr ( $response, $p, 4 ) ); $p += 4;
1203
-				}
1204
-				else
1244
+				} else
1205 1245
 				{
1206 1246
 					list ( $doc, $weight ) = array_values ( unpack ( "N*N*",
1207 1247
 						substr ( $response, $p, 8 ) ) );
@@ -1211,10 +1251,11 @@  discard block
 block discarded – undo
1211 1251
 				$weight = sprintf ( "%u", $weight );
1212 1252
 
1213 1253
 				// create match entry
1214
-				if ( $this->_arrayresult )
1215
-					$result["matches"][$idx] = array ( "id"=>$doc, "weight"=>$weight );
1216
-				else
1217
-					$result["matches"][$doc]["weight"] = $weight;
1254
+				if ( $this->_arrayresult ) {
1255
+									$result["matches"][$idx] = array ( "id"=>$doc, "weight"=>$weight );
1256
+				} else {
1257
+									$result["matches"][$doc]["weight"] = $weight;
1258
+				}
1218 1259
 
1219 1260
 				// parse and create attributes
1220 1261
 				$attrvals = array ();
@@ -1257,10 +1298,11 @@  discard block
 block discarded – undo
1257 1298
 					}
1258 1299
 				}
1259 1300
 
1260
-				if ( $this->_arrayresult )
1261
-					$result["matches"][$idx]["attrs"] = $attrvals;
1262
-				else
1263
-					$result["matches"][$doc]["attrs"] = $attrvals;
1301
+				if ( $this->_arrayresult ) {
1302
+									$result["matches"][$idx]["attrs"] = $attrvals;
1303
+				} else {
1304
+									$result["matches"][$doc]["attrs"] = $attrvals;
1305
+				}
1264 1306
 			}
1265 1307
 
1266 1308
 			list ( $total, $total_found, $msecs, $words ) =
@@ -1311,25 +1353,63 @@  discard block
 block discarded – undo
1311 1353
 		// fixup options
1312 1354
 		/////////////////
1313 1355
 
1314
-		if ( !isset($opts["before_match"]) )		$opts["before_match"] = "<b>";
1315
-		if ( !isset($opts["after_match"]) )			$opts["after_match"] = "</b>";
1316
-		if ( !isset($opts["chunk_separator"]) )		$opts["chunk_separator"] = " ... ";
1317
-		if ( !isset($opts["limit"]) )				$opts["limit"] = 256;
1318
-		if ( !isset($opts["limit_passages"]) )		$opts["limit_passages"] = 0;
1319
-		if ( !isset($opts["limit_words"]) )			$opts["limit_words"] = 0;
1320
-		if ( !isset($opts["around"]) )				$opts["around"] = 5;
1321
-		if ( !isset($opts["exact_phrase"]) )		$opts["exact_phrase"] = false;
1322
-		if ( !isset($opts["single_passage"]) )		$opts["single_passage"] = false;
1323
-		if ( !isset($opts["use_boundaries"]) )		$opts["use_boundaries"] = false;
1324
-		if ( !isset($opts["weight_order"]) )		$opts["weight_order"] = false;
1325
-		if ( !isset($opts["query_mode"]) )			$opts["query_mode"] = false;
1326
-		if ( !isset($opts["force_all_words"]) )		$opts["force_all_words"] = false;
1327
-		if ( !isset($opts["start_passage_id"]) )	$opts["start_passage_id"] = 1;
1328
-		if ( !isset($opts["load_files"]) )			$opts["load_files"] = false;
1329
-		if ( !isset($opts["html_strip_mode"]) )		$opts["html_strip_mode"] = "index";
1330
-		if ( !isset($opts["allow_empty"]) )			$opts["allow_empty"] = false;
1331
-		if ( !isset($opts["passage_boundary"]) )	$opts["passage_boundary"] = "none";
1332
-		if ( !isset($opts["emit_zones"]) )			$opts["emit_zones"] = false;
1356
+		if ( !isset($opts["before_match"]) ) {
1357
+			$opts["before_match"] = "<b>";
1358
+		}
1359
+		if ( !isset($opts["after_match"]) ) {
1360
+			$opts["after_match"] = "</b>";
1361
+		}
1362
+		if ( !isset($opts["chunk_separator"]) ) {
1363
+			$opts["chunk_separator"] = " ... ";
1364
+		}
1365
+		if ( !isset($opts["limit"]) ) {
1366
+			$opts["limit"] = 256;
1367
+		}
1368
+		if ( !isset($opts["limit_passages"]) ) {
1369
+			$opts["limit_passages"] = 0;
1370
+		}
1371
+		if ( !isset($opts["limit_words"]) ) {
1372
+			$opts["limit_words"] = 0;
1373
+		}
1374
+		if ( !isset($opts["around"]) ) {
1375
+			$opts["around"] = 5;
1376
+		}
1377
+		if ( !isset($opts["exact_phrase"]) ) {
1378
+			$opts["exact_phrase"] = false;
1379
+		}
1380
+		if ( !isset($opts["single_passage"]) ) {
1381
+			$opts["single_passage"] = false;
1382
+		}
1383
+		if ( !isset($opts["use_boundaries"]) ) {
1384
+			$opts["use_boundaries"] = false;
1385
+		}
1386
+		if ( !isset($opts["weight_order"]) ) {
1387
+			$opts["weight_order"] = false;
1388
+		}
1389
+		if ( !isset($opts["query_mode"]) ) {
1390
+			$opts["query_mode"] = false;
1391
+		}
1392
+		if ( !isset($opts["force_all_words"]) ) {
1393
+			$opts["force_all_words"] = false;
1394
+		}
1395
+		if ( !isset($opts["start_passage_id"]) ) {
1396
+			$opts["start_passage_id"] = 1;
1397
+		}
1398
+		if ( !isset($opts["load_files"]) ) {
1399
+			$opts["load_files"] = false;
1400
+		}
1401
+		if ( !isset($opts["html_strip_mode"]) ) {
1402
+			$opts["html_strip_mode"] = "index";
1403
+		}
1404
+		if ( !isset($opts["allow_empty"]) ) {
1405
+			$opts["allow_empty"] = false;
1406
+		}
1407
+		if ( !isset($opts["passage_boundary"]) ) {
1408
+			$opts["passage_boundary"] = "none";
1409
+		}
1410
+		if ( !isset($opts["emit_zones"]) ) {
1411
+			$opts["emit_zones"] = false;
1412
+		}
1333 1413
 
1334 1414
 		/////////////////
1335 1415
 		// build request
@@ -1337,15 +1417,33 @@  discard block
 block discarded – undo
1337 1417
 
1338 1418
 		// v.1.2 req
1339 1419
 		$flags = 1; // remove spaces
1340
-		if ( $opts["exact_phrase"] )	$flags |= 2;
1341
-		if ( $opts["single_passage"] )	$flags |= 4;
1342
-		if ( $opts["use_boundaries"] )	$flags |= 8;
1343
-		if ( $opts["weight_order"] )	$flags |= 16;
1344
-		if ( $opts["query_mode"] )		$flags |= 32;
1345
-		if ( $opts["force_all_words"] )	$flags |= 64;
1346
-		if ( $opts["load_files"] )		$flags |= 128;
1347
-		if ( $opts["allow_empty"] )		$flags |= 256;
1348
-		if ( $opts["emit_zones"] )		$flags |= 512;
1420
+		if ( $opts["exact_phrase"] ) {
1421
+			$flags |= 2;
1422
+		}
1423
+		if ( $opts["single_passage"] ) {
1424
+			$flags |= 4;
1425
+		}
1426
+		if ( $opts["use_boundaries"] ) {
1427
+			$flags |= 8;
1428
+		}
1429
+		if ( $opts["weight_order"] ) {
1430
+			$flags |= 16;
1431
+		}
1432
+		if ( $opts["query_mode"] ) {
1433
+			$flags |= 32;
1434
+		}
1435
+		if ( $opts["force_all_words"] ) {
1436
+			$flags |= 64;
1437
+		}
1438
+		if ( $opts["load_files"] ) {
1439
+			$flags |= 128;
1440
+		}
1441
+		if ( $opts["allow_empty"] ) {
1442
+			$flags |= 256;
1443
+		}
1444
+		if ( $opts["emit_zones"] ) {
1445
+			$flags |= 512;
1446
+		}
1349 1447
 		$req = pack ( "NN", 0, $flags ); // mode=0, flags=$flags
1350 1448
 		$req .= pack ( "N", strlen($index) ) . $index; // req index
1351 1449
 		$req .= pack ( "N", strlen($words) ) . $words; // req words
@@ -1512,8 +1610,9 @@  discard block
 block discarded – undo
1512 1610
 		assert ( is_bool($mva) );
1513 1611
 
1514 1612
 		assert ( is_array($attrs) );
1515
-		foreach ( $attrs as $attr )
1516
-			assert ( is_string($attr) );
1613
+		foreach ( $attrs as $attr ) {
1614
+					assert ( is_string($attr) );
1615
+		}
1517 1616
 
1518 1617
 		assert ( is_array($values) );
1519 1618
 		foreach ( $values as $id=>$entry )
@@ -1526,10 +1625,12 @@  discard block
 block discarded – undo
1526 1625
 				if ( $mva )
1527 1626
 				{
1528 1627
 					assert ( is_array($v) );
1529
-					foreach ( $v as $vv )
1530
-						assert ( is_int($vv) );
1531
-				} else
1532
-					assert ( is_int($v) );
1628
+					foreach ( $v as $vv ) {
1629
+											assert ( is_int($vv) );
1630
+					}
1631
+				} else {
1632
+									assert ( is_int($v) );
1633
+				}
1533 1634
 			}
1534 1635
 		}
1535 1636
 
@@ -1551,9 +1652,10 @@  discard block
 block discarded – undo
1551 1652
 			foreach ( $entry as $v )
1552 1653
 			{
1553 1654
 				$req .= pack ( "N", $mva ? count($v) : $v );
1554
-				if ( $mva )
1555
-					foreach ( $v as $vv )
1655
+				if ( $mva ) {
1656
+									foreach ( $v as $vv )
1556 1657
 						$req .= pack ( "N", $vv );
1658
+				}
1557 1659
 			}
1558 1660
 		}
1559 1661
 
@@ -1595,13 +1697,15 @@  discard block
 block discarded – undo
1595 1697
 			$this->_error = 'already connected';
1596 1698
 			return false;
1597 1699
 		}
1598
-		if ( !$fp = $this->_Connect() )
1599
-			return false;
1700
+		if ( !$fp = $this->_Connect() ) {
1701
+					return false;
1702
+		}
1600 1703
 
1601 1704
 		// command, command version = 0, body length = 4, body = 1
1602 1705
 		$req = pack ( "nnNN", SEARCHD_COMMAND_PERSIST, 0, 4, 1 );
1603
-		if ( !$this->_Send ( $fp, $req, 12 ) )
1604
-			return false;
1706
+		if ( !$this->_Send ( $fp, $req, 12 ) ) {
1707
+					return false;
1708
+		}
1605 1709
 
1606 1710
 		$this->_socket = $fp;
1607 1711
 		return true;
@@ -1647,8 +1751,10 @@  discard block
 block discarded – undo
1647 1751
 		list ( $rows, $cols ) = array_values ( unpack ( "N*N*", substr ( $response, $p, 8 ) ) ); $p += 8;
1648 1752
 
1649 1753
 		$res = array();
1650
-		for ( $i=0; $i<$rows; $i++ )
1651
-			for ( $j=0; $j<$cols; $j++ )
1754
+		for ( $i=0; $i<$rows; $i++ ) {
1755
+					for ( $j=0;
1756
+		}
1757
+		$j<$cols; $j++ )
1652 1758
 		{
1653 1759
 			list(,$len) = unpack ( "N*", substr ( $response, $p, 4 ) ); $p += 4;
1654 1760
 			$res[$i][] = substr ( $response, $p, $len ); $p += $len;
@@ -1680,10 +1786,11 @@  discard block
 block discarded – undo
1680 1786
 		}
1681 1787
 
1682 1788
 		$tag = -1;
1683
-		if ( strlen($response)==4 )
1684
-			list(,$tag) = unpack ( "N*", $response );
1685
-		else
1686
-			$this->_error = "unexpected response length";
1789
+		if ( strlen($response)==4 ) {
1790
+					list(,$tag) = unpack ( "N*", $response );
1791
+		} else {
1792
+					$this->_error = "unexpected response length";
1793
+		}
1687 1794
 
1688 1795
 		$this->_MBPop ();
1689 1796
 		return $tag;
Please login to merge, or discard this patch.
plugins/af_psql_trgm/init.php 1 patch
Braces   +42 added lines, -14 removed lines patch added patch discarded remove patch
@@ -15,10 +15,16 @@  discard block
 block discarded – undo
15 15
 		$min_title_length = (int) $_POST["min_title_length"];
16 16
 		$enable_globally = checkbox_to_sql_bool($_POST["enable_globally"]);
17 17
 
18
-		if ($similarity < 0) $similarity = 0;
19
-		if ($similarity > 1) $similarity = 1;
18
+		if ($similarity < 0) {
19
+			$similarity = 0;
20
+		}
21
+		if ($similarity > 1) {
22
+			$similarity = 1;
23
+		}
20 24
 
21
-		if ($min_title_length < 0) $min_title_length = 0;
25
+		if ($min_title_length < 0) {
26
+			$min_title_length = 0;
27
+		}
22 28
 
23 29
 		$similarity = sprintf("%.2f", $similarity);
24 30
 
@@ -121,7 +127,9 @@  discard block
 block discarded – undo
121 127
 	}
122 128
 
123 129
 	public function hook_prefs_tab($args) {
124
-		if ($args != "prefFeeds") return;
130
+		if ($args != "prefFeeds") {
131
+			return;
132
+		}
125 133
 
126 134
 		print "<div dojoType=\"dijit.layout.AccordionPane\"
127 135
 			title=\"<i class='material-icons'>extension</i> ".__('Mark similar articles as read')."\">";
@@ -140,8 +148,12 @@  discard block
 block discarded – undo
140 148
 			$min_title_length = $this->host->get($this, "min_title_length");
141 149
 			$enable_globally = $this->host->get($this, "enable_globally");
142 150
 
143
-			if (!$similarity) $similarity = '0.75';
144
-			if (!$min_title_length) $min_title_length = '32';
151
+			if (!$similarity) {
152
+				$similarity = '0.75';
153
+			}
154
+			if (!$min_title_length) {
155
+				$min_title_length = '32';
156
+			}
145 157
 
146 158
 			print "<form dojoType=\"dijit.form.Form\">";
147 159
 
@@ -198,7 +210,9 @@  discard block
 block discarded – undo
198 210
 			print "</form>";
199 211
 
200 212
 			$enabled_feeds = $this->host->get($this, "enabled_feeds");
201
-			if (!array($enabled_feeds)) $enabled_feeds = array();
213
+			if (!array($enabled_feeds)) {
214
+				$enabled_feeds = array();
215
+			}
202 216
 
203 217
 			$enabled_feeds = $this->filter_unknown_feeds($enabled_feeds);
204 218
 			$this->host->set($this, "enabled_feeds", $enabled_feeds);
@@ -225,7 +239,9 @@  discard block
 block discarded – undo
225 239
 		print "<section>";
226 240
 
227 241
 		$enabled_feeds = $this->host->get($this, "enabled_feeds");
228
-		if (!array($enabled_feeds)) $enabled_feeds = array();
242
+		if (!array($enabled_feeds)) {
243
+			$enabled_feeds = array();
244
+		}
229 245
 
230 246
 		$key = array_search($feed_id, $enabled_feeds);
231 247
 		$checked = $key !== FALSE ? "checked" : "";
@@ -242,7 +258,9 @@  discard block
 block discarded – undo
242 258
 
243 259
 	public function hook_prefs_save_feed($feed_id) {
244 260
 		$enabled_feeds = $this->host->get($this, "enabled_feeds");
245
-		if (!is_array($enabled_feeds)) $enabled_feeds = array();
261
+		if (!is_array($enabled_feeds)) {
262
+			$enabled_feeds = array();
263
+		}
246 264
 
247 265
 		$enable = checkbox_to_sql_bool($_POST["trgm_similarity_enabled"]);
248 266
 		$key = array_search($feed_id, $enabled_feeds);
@@ -262,24 +280,34 @@  discard block
 block discarded – undo
262 280
 
263 281
 	public function hook_article_filter($article) {
264 282
 
265
-		if (DB_TYPE != "pgsql") return $article;
283
+		if (DB_TYPE != "pgsql") {
284
+			return $article;
285
+		}
266 286
 
267 287
 		$res = $this->pdo->query("select 'similarity'::regproc");
268
-		if (!$res->fetch()) return $article;
288
+		if (!$res->fetch()) {
289
+			return $article;
290
+		}
269 291
 
270 292
 		$enable_globally = $this->host->get($this, "enable_globally");
271 293
 
272 294
 		if (!$enable_globally) {
273 295
 			$enabled_feeds = $this->host->get($this, "enabled_feeds");
274 296
 			$key = array_search($article["feed"]["id"], $enabled_feeds);
275
-			if ($key === FALSE) return $article;
297
+			if ($key === FALSE) {
298
+				return $article;
299
+			}
276 300
 		}
277 301
 
278 302
 		$similarity = (float) $this->host->get($this, "similarity");
279
-		if ($similarity < 0.01) return $article;
303
+		if ($similarity < 0.01) {
304
+			return $article;
305
+		}
280 306
 
281 307
 		$min_title_length = (int) $this->host->get($this, "min_title_length");
282
-		if (mb_strlen($article["title"]) < $min_title_length) return $article;
308
+		if (mb_strlen($article["title"]) < $min_title_length) {
309
+			return $article;
310
+		}
283 311
 
284 312
 		$owner_uid = $article["owner_uid"];
285 313
 		$entry_guid = $article["guid_hashed"];
Please login to merge, or discard this patch.