Completed
Pull Request — release-2.1 (#4398)
by Colin
07:12
created
Themes/default/ManageAttachments.template.php 1 patch
Braces   +20 added lines, -14 removed lines patch added patch discarded remove patch
@@ -122,9 +122,10 @@  discard block
 block discarded – undo
122 122
 		<h3 class="catbg">', $txt['attachment_transfer'], '</h3>
123 123
 	</div>';
124 124
 
125
-	if (!empty($context['results']))
126
-		echo '
125
+	if (!empty($context['results'])) {
126
+			echo '
127 127
 	<div class="noticebox">', $context['results'], '</div>';
128
+	}
128 129
 
129 130
 	echo '
130 131
 	<div class="windowbg2 noup">
@@ -136,9 +137,10 @@  discard block
 block discarded – undo
136 137
 					<select name="from">
137 138
 						<option value="0">', $txt['attachment_transfer_select'], '</option>';
138 139
 
139
-	foreach ($context['attach_dirs'] as $id => $dir)
140
-		echo '
140
+	foreach ($context['attach_dirs'] as $id => $dir) {
141
+			echo '
141 142
 						<option value="', $id, '">', $dir, '</option>';
143
+	}
142 144
 
143 145
 	echo '
144 146
 					</select>
@@ -149,13 +151,14 @@  discard block
 block discarded – undo
149 151
 						<option value="0">', $txt['attachment_transfer_auto_select'], '</option>
150 152
 						<option value="-1">', $txt['attachment_transfer_forum_root'], '</option>';
151 153
 
152
-	if (!empty($context['base_dirs']))
153
-		foreach ($context['base_dirs'] as $id => $dir)
154
+	if (!empty($context['base_dirs'])) {
155
+			foreach ($context['base_dirs'] as $id => $dir)
154 156
 			echo '
155 157
 						<option value="', $id, '">', $dir, '</option>';
156
-	else
157
-			echo '
158
+	} else {
159
+				echo '
158 160
 						<option value="0" disabled>', $txt['attachment_transfer_no_base'], '</option>';
161
+	}
159 162
 
160 163
 	echo '
161 164
 					</select>
@@ -165,18 +168,20 @@  discard block
 block discarded – undo
165 168
 					<select name="to">
166 169
 						<option value="0">', $txt['attachment_transfer_select'], '</option>';
167 170
 
168
-	foreach ($context['attach_dirs'] as $id => $dir)
169
-		echo '
171
+	foreach ($context['attach_dirs'] as $id => $dir) {
172
+			echo '
170 173
 						<option value="', $id, '">', $dir, '</option>';
174
+	}
171 175
 
172 176
 	echo '
173 177
 					</select>
174 178
 				</dd>';
175 179
 
176
-	if (!empty($modSettings['attachmentDirFileLimit']))
177
-		echo '
180
+	if (!empty($modSettings['attachmentDirFileLimit'])) {
181
+			echo '
178 182
 				<dt>', $txt['attachment_transfer_empty'], '</dt>
179 183
 				<dd><input type="checkbox" name="empty_it"', $context['checked'] ? ' checked' : '', '></dd>';
184
+	}
180 185
 
181 186
 	echo '
182 187
 			</dl>
@@ -253,10 +258,11 @@  discard block
 block discarded – undo
253 258
 		// Loop through each error reporting the status
254 259
 		foreach ($context['repair_errors'] as $error => $number)
255 260
 		{
256
-			if (!empty($number))
257
-				echo '
261
+			if (!empty($number)) {
262
+							echo '
258 263
 				<input type="checkbox" name="to_fix[]" id="', $error, '" value="', $error, '">
259 264
 				<label for="', $error, '">', sprintf($txt['attach_repair_' . $error], $number), '</label><br>';
265
+			}
260 266
 		}
261 267
 
262 268
 		echo '
Please login to merge, or discard this patch.
Themes/default/ManagePaid.template.php 1 patch
Braces   +61 added lines, -44 removed lines patch added patch discarded remove patch
@@ -24,11 +24,12 @@  discard block
 block discarded – undo
24 24
 				<h3 class="catbg">', $txt['paid_' . $context['action_type'] . '_subscription'], '</h3>
25 25
 			</div>';
26 26
 
27
-	if (!empty($context['disable_groups']))
28
-		echo '
27
+	if (!empty($context['disable_groups'])) {
28
+			echo '
29 29
 			<div class="information">
30 30
 				<span class="alert">', $txt['paid_mod_edit_note'], '</span>
31 31
 			</div>';
32
+	}
32 33
 
33 34
 	echo '
34 35
 			<div class="windowbg2">
@@ -69,9 +70,10 @@  discard block
 block discarded – undo
69 70
 							<option value="0"', $context['sub']['prim_group'] == 0 ? ' selected' : '', '>', $txt['paid_mod_no_group'], '</option>';
70 71
 
71 72
 	// Put each group into the box.
72
-	foreach ($context['groups'] as $id => $name)
73
-		echo '
73
+	foreach ($context['groups'] as $id => $name) {
74
+			echo '
74 75
 							<option value="', $id, '"', $context['sub']['prim_group'] == $id ? ' selected' : '', '>', $name, '</option>';
76
+	}
75 77
 
76 78
 	echo '
77 79
 						</select>
@@ -83,11 +85,12 @@  discard block
 block discarded – undo
83 85
 					<dd>';
84 86
 
85 87
 	// Put a checkbox in for each group
86
-	foreach ($context['groups'] as $id => $name)
87
-		echo '
88
+	foreach ($context['groups'] as $id => $name) {
89
+			echo '
88 90
 						<label for="addgroup_', $id, '">
89 91
 							<input type="checkbox" id="addgroup_', $id, '" name="addgroup[', $id, ']"', in_array($id, $context['sub']['add_groups']) ? ' checked' : '', !empty($context['disable_groups']) ? ' disabled' : '', '>&nbsp;<span class="smalltext">', $name, '</span>
90 92
 						</label><br>';
93
+	}
91 94
 
92 95
 	echo '
93 96
 					</dd>
@@ -141,8 +144,8 @@  discard block
 block discarded – undo
141 144
 					<fieldset>';
142 145
 
143 146
 	//!! Removed until implemented
144
-	if (!empty($sdflsdhglsdjgs))
145
-		echo '
147
+	if (!empty($sdflsdhglsdjgs)) {
148
+			echo '
146 149
 						<dl class="settings">
147 150
 							<dt>
148 151
 								<label for="allow_partial_check">', $txt['paid_mod_allow_partial'], '</label>:<br><span class="smalltext">', $txt['paid_mod_allow_partial_desc'], '</span>
@@ -151,6 +154,7 @@  discard block
 block discarded – undo
151 154
 								<input type="checkbox" name="allow_partial" id="allow_partial_check"', empty($context['sub']['allow_partial']) ? '' : ' checked', '>
152 155
 							</dd>
153 156
 						</dl>';
157
+	}
154 158
 
155 159
 	echo '
156 160
 						<div class="information">
@@ -251,8 +255,8 @@  discard block
 block discarded – undo
251 255
 				<dl class="settings">';
252 256
 
253 257
 	// Do we need a username?
254
-	if ($context['action_type'] == 'add')
255
-		echo '
258
+	if ($context['action_type'] == 'add') {
259
+			echo '
256 260
 					<dt>
257 261
 						<strong>', $txt['paid_username'], ':</strong><br>
258 262
 						<span class="smalltext">', $txt['one_username'], '</span>
@@ -260,6 +264,7 @@  discard block
 block discarded – undo
260 264
 					<dd>
261 265
 						<input type="text" name="name" id="name_control" value="', $context['sub']['username'], '" size="30">
262 266
 					</dd>';
267
+	}
263 268
 
264 269
 	echo '
265 270
 					<dt>
@@ -277,9 +282,10 @@  discard block
 block discarded – undo
277 282
 					<select name="year" id="year" onchange="generateDays();">';
278 283
 
279 284
 	// Show a list of all the years we allow...
280
-	for ($year = 2005; $year <= 2030; $year++)
281
-		echo '
285
+	for ($year = 2005; $year <= 2030; $year++) {
286
+			echo '
282 287
 						<option value="', $year, '"', $year == $context['sub']['start']['year'] ? ' selected' : '', '>', $year, '</option>';
288
+	}
283 289
 
284 290
 	echo '
285 291
 					</select>&nbsp;
@@ -287,9 +293,10 @@  discard block
 block discarded – undo
287 293
 					<select name="month" id="month" onchange="generateDays();">';
288 294
 
289 295
 	// There are 12 months per year - ensure that they all get listed.
290
-	for ($month = 1; $month <= 12; $month++)
291
-		echo '
296
+	for ($month = 1; $month <= 12; $month++) {
297
+			echo '
292 298
 						<option value="', $month, '"', $month == $context['sub']['start']['month'] ? ' selected' : '', '>', $txt['months'][$month], '</option>';
299
+	}
293 300
 
294 301
 	echo '
295 302
 					</select>&nbsp;
@@ -297,9 +304,10 @@  discard block
 block discarded – undo
297 304
 					<select name="day" id="day">';
298 305
 
299 306
 	// This prints out all the days in the current month - this changes dynamically as we switch months.
300
-	for ($day = 1; $day <= $context['sub']['start']['last_day']; $day++)
301
-		echo '
307
+	for ($day = 1; $day <= $context['sub']['start']['last_day']; $day++) {
308
+			echo '
302 309
 						<option value="', $day, '"', $day == $context['sub']['start']['day'] ? ' selected' : '', '>', $day, '</option>';
310
+	}
303 311
 
304 312
 	echo '
305 313
 					</select>
@@ -311,9 +319,10 @@  discard block
 block discarded – undo
311 319
 					<select name="yearend" id="yearend" onchange="generateDays(\'end\');">';
312 320
 
313 321
 	// Show a list of all the years we allow...
314
-	for ($year = 2005; $year <= 2030; $year++)
315
-		echo '
322
+	for ($year = 2005; $year <= 2030; $year++) {
323
+			echo '
316 324
 						<option value="', $year, '"', $year == $context['sub']['end']['year'] ? ' selected' : '', '>', $year, '</option>';
325
+	}
317 326
 
318 327
 	echo '
319 328
 					</select>&nbsp;
@@ -321,9 +330,10 @@  discard block
 block discarded – undo
321 330
 					<select name="monthend" id="monthend" onchange="generateDays(\'end\');">';
322 331
 
323 332
 	// There are 12 months per year - ensure that they all get listed.
324
-	for ($month = 1; $month <= 12; $month++)
325
-		echo '
333
+	for ($month = 1; $month <= 12; $month++) {
334
+			echo '
326 335
 						<option value="', $month, '"', $month == $context['sub']['end']['month'] ? ' selected' : '', '>', $txt['months'][$month], '</option>';
336
+	}
327 337
 
328 338
 	echo '
329 339
 					</select>&nbsp;
@@ -331,9 +341,10 @@  discard block
 block discarded – undo
331 341
 					<select name="dayend" id="dayend">';
332 342
 
333 343
 	// This prints out all the days in the current month - this changes dynamically as we switch months.
334
-	for ($day = 1; $day <= $context['sub']['end']['last_day']; $day++)
335
-		echo '
344
+	for ($day = 1; $day <= $context['sub']['end']['last_day']; $day++) {
345
+			echo '
336 346
 						<option value="', $day, '"', $day == $context['sub']['end']['day'] ? ' selected' : '', '>', $day, '</option>';
347
+	}
337 348
 
338 349
 	echo '
339 350
 					</select>
@@ -415,8 +426,7 @@  discard block
 block discarded – undo
415 426
 			<div class="information">
416 427
 				', $txt['paid_subs_none'], '
417 428
 			</div>';
418
-	}
419
-	else
429
+	} else
420 430
 	{
421 431
 		echo '
422 432
 			<div class="information">
@@ -428,8 +438,9 @@  discard block
 block discarded – undo
428 438
 		{
429 439
 
430 440
 			// Ignore the inactive ones...
431
-			if (empty($subscription['active']))
432
-				continue;
441
+			if (empty($subscription['active'])) {
442
+							continue;
443
+			}
433 444
 
434 445
 			echo '
435 446
 			<div class="cat_bar">
@@ -439,9 +450,10 @@  discard block
 block discarded – undo
439 450
 				<p><strong>', $subscription['name'], '</strong></p>
440 451
 				<p class="smalltext">', $subscription['desc'], '</p>';
441 452
 
442
-			if (!$subscription['flexible'])
443
-				echo '
453
+			if (!$subscription['flexible']) {
454
+							echo '
444 455
 				<div><strong>', $txt['paid_duration'], ':</strong> ', $subscription['length'], '</div>';
456
+			}
445 457
 
446 458
 			if ($context['user']['is_owner'])
447 459
 			{
@@ -454,24 +466,25 @@  discard block
 block discarded – undo
454 466
 				<select name="cur[', $subscription['id'], ']">';
455 467
 
456 468
 					// Print out the costs for this one.
457
-					foreach ($subscription['costs'] as $duration => $value)
458
-						echo '
469
+					foreach ($subscription['costs'] as $duration => $value) {
470
+											echo '
459 471
 					<option value="', $duration, '">', sprintf($modSettings['paid_currency_symbol'], $value), '/', $txt[$duration], '</option>';
472
+					}
460 473
 
461 474
 					echo '
462 475
 				</select>';
463
-				}
464
-				else
465
-					echo '
476
+				} else {
477
+									echo '
466 478
 				', sprintf($modSettings['paid_currency_symbol'], $subscription['costs']['fixed']);
479
+				}
467 480
 
468 481
 				echo '
469 482
 				<hr>
470 483
 				<input type="submit" name="sub_id[', $subscription['id'], ']" value="', $txt['paid_order'], '" class="button">';
471
-			}
472
-			else
473
-				echo '
484
+			} else {
485
+							echo '
474 486
 				<a href="', $scripturl, '?action=admin;area=paidsubscribe;sa=modifyuser;sid=', $subscription['id'], ';uid=', $context['member']['id'], (empty($context['current'][$subscription['id']]) ? '' : ';lid=' . $context['current'][$subscription['id']]['id']), '">', empty($context['current'][$subscription['id']]) ? $txt['paid_admin_add'] : $txt['paid_edit_subscription'], '</a>';
487
+			}
475 488
 
476 489
 			echo '
477 490
 			</div><!-- .windowbg -->';
@@ -498,19 +511,20 @@  discard block
 block discarded – undo
498 511
 			</thead>
499 512
 			<tbody>';
500 513
 
501
-	if (empty($context['current']))
502
-		echo '
514
+	if (empty($context['current'])) {
515
+			echo '
503 516
 				<tr class="windowbg">
504 517
 					<td colspan="4">
505 518
 						', $txt['paid_none_yet'], '
506 519
 					</td>
507 520
 				</tr>';
521
+	}
508 522
 
509 523
 	foreach ($context['current'] as $sub)
510 524
 	{
511 525
 
512
-		if (!$sub['hide'])
513
-			echo '
526
+		if (!$sub['hide']) {
527
+					echo '
514 528
 				<tr class="windowbg">
515 529
 					<td>
516 530
 						', (allowedTo('admin_forum') ? '<a href="' . $scripturl . '?action=admin;area=paidsubscribe;sa=modifyuser;lid=' . $sub['id'] . '">' . $sub['name'] . '</a>' : $sub['name']), '
@@ -521,6 +535,7 @@  discard block
 block discarded – undo
521 535
 					<td>', $sub['start'], '</td>
522 536
 					<td>', $sub['end'], '</td>
523 537
 				</tr>';
538
+		}
524 539
 	}
525 540
 	echo '
526 541
 			</tbody>
@@ -571,15 +586,17 @@  discard block
 block discarded – undo
571 586
 			', $gateway['desc'], '<br>
572 587
 			<form action="', $gateway['form'], '" method="post">';
573 588
 
574
-		if (!empty($gateway['javascript']))
575
-			echo '
589
+		if (!empty($gateway['javascript'])) {
590
+					echo '
576 591
 				<script>
577 592
 					', $gateway['javascript'], '
578 593
 				</script>';
594
+		}
579 595
 
580
-		foreach ($gateway['hidden'] as $name => $value)
581
-			echo '
596
+		foreach ($gateway['hidden'] as $name => $value) {
597
+					echo '
582 598
 				<input type="hidden" id="', $gateway['id'], '_', $name, '" name="', $name, '" value="', $value, '">';
599
+		}
583 600
 
584 601
 		echo '
585 602
 				<br>
Please login to merge, or discard this patch.
Themes/default/ManageNews.template.php 1 patch
Braces   +21 added lines, -14 removed lines patch added patch discarded remove patch
@@ -18,9 +18,10 @@  discard block
 block discarded – undo
18 18
 	global $context, $txt, $scripturl;
19 19
 
20 20
 	// Are we done sending the newsletter?
21
-	if (!empty($context['newsletter_sent']))
22
-		echo '
21
+	if (!empty($context['newsletter_sent'])) {
22
+			echo '
23 23
 	<div class="infobox">', $txt['admin_news_newsletter_' . $context['newsletter_sent']], '</div>';
24
+	}
24 25
 
25 26
 	echo '
26 27
 	<div id="admincenter">
@@ -39,9 +40,10 @@  discard block
 block discarded – undo
39 40
 					</dt>
40 41
 					<dd>';
41 42
 
42
-	foreach ($context['groups'] as $group)
43
-		echo '
43
+	foreach ($context['groups'] as $group) {
44
+			echo '
44 45
 						<label for="groups_', $group['id'], '"><input type="checkbox" name="groups[', $group['id'], ']" id="groups_', $group['id'], '" value="', $group['id'], '" checked> ', $group['name'], '</label> <em>(', $group['member_count'], ')</em><br>';
46
+	}
45 47
 
46 48
 	echo '
47 49
 						<br>
@@ -82,9 +84,10 @@  discard block
 block discarded – undo
82 84
 						</dt>
83 85
 						<dd>';
84 86
 
85
-	foreach ($context['groups'] as $group)
86
-		echo '
87
+	foreach ($context['groups'] as $group) {
88
+			echo '
87 89
 							<label for="exclude_groups_', $group['id'], '"><input type="checkbox" name="exclude_groups[', $group['id'], ']" id="exclude_groups_', $group['id'], '" value="', $group['id'], '"> ', $group['name'], '</label> <em>(', $group['member_count'], ')</em><br>';
90
+	}
88 91
 
89 92
 	echo '
90 93
 							<br>
@@ -230,9 +233,10 @@  discard block
 block discarded – undo
230 233
 				<div id="bbcBox_message"></div>';
231 234
 
232 235
 	// What about smileys?
233
-	if (!empty($context['smileys']['postform']) || !empty($context['smileys']['popup']))
234
-		echo '
236
+	if (!empty($context['smileys']['postform']) || !empty($context['smileys']['popup'])) {
237
+			echo '
235 238
 				<div id="smileyBox_message"></div>';
239
+	}
236 240
 
237 241
 	// Show BBC buttons, smileys and textbox.
238 242
 	echo '
@@ -253,9 +257,10 @@  discard block
 block discarded – undo
253 257
 			<input type="hidden" name="email_force" value="', $context['email_force'], '">
254 258
 			<input type="hidden" name="total_emails" value="', $context['total_emails'], '">';
255 259
 
256
-	foreach ($context['recipients'] as $key => $values)
257
-		echo '
260
+	foreach ($context['recipients'] as $key => $values) {
261
+			echo '
258 262
 			<input type="hidden" name="', $key, '" value="', implode(($key == 'emails' ? ';' : ','), $values), '">';
263
+	}
259 264
 
260 265
 	echo '
261 266
 			<script>';
@@ -411,9 +416,10 @@  discard block
 block discarded – undo
411 416
 				<input type="hidden" name="parse_html" value="', $context['parse_html'], '">';
412 417
 
413 418
 	// All the things we must remember!
414
-	foreach ($context['recipients'] as $key => $values)
415
-		echo '
419
+	foreach ($context['recipients'] as $key => $values) {
420
+			echo '
416 421
 				<input type="hidden" name="', $key, '" value="', implode(($key == 'emails' ? ';' : ','), $values), '">';
422
+	}
417 423
 
418 424
 	echo '
419 425
 			</div><!-- .windowbg -->
@@ -446,9 +452,10 @@  discard block
 block discarded – undo
446 452
 {
447 453
 	global $context, $txt;
448 454
 
449
-	if (!empty($context['saved_successful']))
450
-		echo '
455
+	if (!empty($context['saved_successful'])) {
456
+			echo '
451 457
 			<div class="infobox">', $txt['settings_saved'], '</div>';
458
+	}
452 459
 
453 460
 	template_show_list('news_lists');
454 461
 }
Please login to merge, or discard this patch.
Themes/default/Stats.template.php 1 patch
Braces   +26 added lines, -18 removed lines patch added patch discarded remove patch
@@ -44,10 +44,11 @@  discard block
 block discarded – undo
44 44
 				<dt>', $txt['users_online_today'], ':</dt>
45 45
 				<dd>', $context['online_today'], '</dd>';
46 46
 
47
-	if (!empty($modSettings['hitStats']))
48
-		echo '
47
+	if (!empty($modSettings['hitStats'])) {
48
+			echo '
49 49
 				<dt>', $txt['num_hits'], ':</dt>
50 50
 				<dd>', $context['num_hits'], '</dd>';
51
+	}
51 52
 
52 53
 	echo '
53 54
 			</dl>
@@ -71,17 +72,19 @@  discard block
 block discarded – undo
71 72
 				<dt>', $txt['gender_stats'], ':</dt>
72 73
 				<dd>';
73 74
 
74
-		foreach ($context['gender'] as $g => $n)
75
-			echo $g, ': ', $n, '<br>';
75
+		foreach ($context['gender'] as $g => $n) {
76
+					echo $g, ': ', $n, '<br>';
77
+		}
76 78
 
77 79
 		echo '
78 80
 				</dd>';
79 81
 	}
80 82
 
81
-	if (!empty($modSettings['hitStats']))
82
-		echo '
83
+	if (!empty($modSettings['hitStats'])) {
84
+			echo '
83 85
 				<dt>', $txt['average_hits'], ':</dt>
84 86
 				<dd>', $context['average_hits'], '</dd>';
87
+	}
85 88
 
86 89
 	echo '
87 90
 			</dl>';
@@ -105,14 +108,15 @@  discard block
 block discarded – undo
105 108
 					</dt>
106 109
 					<dd class="statsbar">';
107 110
 
108
-			if (!empty($item['percent']))
109
-				echo '
111
+			if (!empty($item['percent'])) {
112
+							echo '
110 113
 						<div class="bar" style="width: ', $item['percent'], '%;">
111 114
 							<span class="righttext">', $item['num'], '</span>
112 115
 						</div>';
113
-			else
114
-				echo '
116
+			} else {
117
+							echo '
115 118
 						<div class="bar empty"><span class="righttext">', $item['num'], '</span></div>';
119
+			}
116 120
 
117 121
 			echo '
118 122
 					</dd>';
@@ -144,9 +148,10 @@  discard block
 block discarded – undo
144 148
 					<th>', $txt['stats_new_members'], '</th>
145 149
 					<th>', $txt['most_online'], '</th>';
146 150
 
147
-		if (!empty($modSettings['hitStats']))
148
-			echo '
151
+		if (!empty($modSettings['hitStats'])) {
152
+					echo '
149 153
 					<th>', $txt['page_views'], '</th>';
154
+		}
150 155
 
151 156
 		echo '
152 157
 				</tr>
@@ -165,9 +170,10 @@  discard block
 block discarded – undo
165 170
 					<th>', $year['new_members'], '</th>
166 171
 					<th>', $year['most_members_online'], '</th>';
167 172
 
168
-			if (!empty($modSettings['hitStats']))
169
-				echo '
173
+			if (!empty($modSettings['hitStats'])) {
174
+							echo '
170 175
 					<th>', $year['hits'], '</th>';
176
+			}
171 177
 
172 178
 			echo '
173 179
 				</tr>';
@@ -184,9 +190,10 @@  discard block
 block discarded – undo
184 190
 					<th>', $month['new_members'], '</th>
185 191
 					<th>', $month['most_members_online'], '</th>';
186 192
 
187
-				if (!empty($modSettings['hitStats']))
188
-					echo '
193
+				if (!empty($modSettings['hitStats'])) {
194
+									echo '
189 195
 					<th>', $month['hits'], '</th>';
196
+				}
190 197
 
191 198
 				echo '
192 199
 				</tr>';
@@ -203,9 +210,10 @@  discard block
 block discarded – undo
203 210
 					<td>', $day['new_members'], '</td>
204 211
 					<td>', $day['most_members_online'], '</td>';
205 212
 
206
-						if (!empty($modSettings['hitStats']))
207
-							echo '
213
+						if (!empty($modSettings['hitStats'])) {
214
+													echo '
208 215
 					<td>', $day['hits'], '</td>';
216
+						}
209 217
 
210 218
 						echo '
211 219
 				</tr>';
Please login to merge, or discard this patch.
Themes/default/ModerationCenter.template.php 1 patch
Braces   +56 added lines, -39 removed lines patch added patch discarded remove patch
@@ -55,18 +55,20 @@  discard block
 block discarded – undo
55 55
 			<div class="modbox">
56 56
 				<ul>';
57 57
 
58
-		foreach ($context['group_requests'] as $request)
59
-			echo '
58
+		foreach ($context['group_requests'] as $request) {
59
+					echo '
60 60
 					<li class="smalltext">
61 61
 						<a href="', $request['request_href'], '">', $request['group']['name'], '</a> ', $txt['mc_groupr_by'], ' ', $request['member']['link'], '
62 62
 					</li>';
63
+		}
63 64
 
64 65
 		// Don't have any watched users right now?
65
-		if (empty($context['group_requests']))
66
-			echo '
66
+		if (empty($context['group_requests'])) {
67
+					echo '
67 68
 					<li>
68 69
 						<strong class="smalltext">', $txt['mc_group_requests_none'], '</strong>
69 70
 					</li>';
71
+		}
70 72
 
71 73
 		echo '
72 74
 				</ul>
@@ -124,18 +126,20 @@  discard block
 block discarded – undo
124 126
 			<div class="modbox">
125 127
 				<ul>';
126 128
 
127
-		foreach ($context['watched_users'] as $user)
128
-			echo '
129
+		foreach ($context['watched_users'] as $user) {
130
+					echo '
129 131
 					<li>
130 132
 						<span class="smalltext">', sprintf(!empty($user['last_login']) ? $txt['mc_seen'] : $txt['mc_seen_never'], $user['link'], $user['last_login']), '</span>
131 133
 					</li>';
134
+		}
132 135
 
133 136
 		// Don't have any watched users right now?
134
-		if (empty($context['watched_users']))
135
-			echo '
137
+		if (empty($context['watched_users'])) {
138
+					echo '
136 139
 					<li>
137 140
 						<strong class="smalltext">', $txt['mc_watched_users_none'], '</strong>
138 141
 					</li>';
142
+		}
139 143
 
140 144
 		echo '
141 145
 				</ul>
@@ -193,18 +197,20 @@  discard block
 block discarded – undo
193 197
 			<div class="modbox">
194 198
 				<ul>';
195 199
 
196
-		foreach ($context['reported_posts'] as $post)
197
-			echo '
200
+		foreach ($context['reported_posts'] as $post) {
201
+					echo '
198 202
 					<li>
199 203
 						<span class="smalltext">', sprintf($txt['mc_post_report'], $post['report_link'], $post['author']['link']), '</span>
200 204
 					</li>';
205
+		}
201 206
 
202 207
 		// Don't have any watched users right now?
203
-		if (empty($context['reported_posts']))
204
-			echo '
208
+		if (empty($context['reported_posts'])) {
209
+					echo '
205 210
 					<li>
206 211
 						<strong class="smalltext">', $txt['mc_recent_reports_none'], '</strong>
207 212
 					</li>';
213
+		}
208 214
 
209 215
 		echo '
210 216
 				</ul>
@@ -262,18 +268,20 @@  discard block
 block discarded – undo
262 268
 			<div class="modbox">
263 269
 				<ul>';
264 270
 
265
-		foreach ($context['reported_users'] as $user)
266
-			echo '
271
+		foreach ($context['reported_users'] as $user) {
272
+					echo '
267 273
 					<li>
268 274
 						<span class="smalltext">', $user['user']['link'], '</span>
269 275
 					</li>';
276
+		}
270 277
 
271 278
 		// Don't have any watched users right now?
272
-		if (empty($context['reported_users']))
273
-			echo '
279
+		if (empty($context['reported_users'])) {
280
+					echo '
274 281
 					<li>
275 282
 						<strong class="smalltext">', $txt['mc_reported_users_none'], '</strong>
276 283
 					</li>';
284
+		}
277 285
 
278 286
 		echo '
279 287
 				</ul>
@@ -344,11 +352,12 @@  discard block
 block discarded – undo
344 352
 						<ul class="moderation_notes">';
345 353
 
346 354
 		// Cycle through the notes.
347
-		foreach ($context['notes'] as $note)
348
-			echo '
355
+		foreach ($context['notes'] as $note) {
356
+					echo '
349 357
 							<li class="smalltext">
350 358
 								', ($note['can_delete'] ? '<a href="' . $note['delete_href'] . ';' . $context['mod-modnote-del_token_var'] . '=' . $context['mod-modnote-del_token'] . '" data-confirm="' . $txt['mc_reportedp_delete_confirm'] . '" class="you_sure"><span class="generic_icons delete"></span></a>' : ''), $note['time'], ' <strong>', $note['author']['link'], ':</strong> ', $note['text'], '
351 359
 							</li>';
360
+		}
352 361
 
353 362
 		echo '
354 363
 						</ul>
@@ -390,18 +399,19 @@  discard block
 block discarded – undo
390 399
 	$remove_button = create_button('delete', 'remove_message', 'remove');
391 400
 
392 401
 	// No posts?
393
-	if (empty($context['unapproved_items']))
394
-		echo '
402
+	if (empty($context['unapproved_items'])) {
403
+			echo '
395 404
 			<div class="windowbg2">
396 405
 				<p class="centertext">
397 406
 					', $txt['mc_unapproved_' . $context['current_view'] . '_none_found'], '
398 407
 				</p>
399 408
 			</div>';
400
-	else
401
-		echo '
409
+	} else {
410
+			echo '
402 411
 			<div class="pagesection floatleft">
403 412
 				', $context['page_index'], '
404 413
 			</div>';
414
+	}
405 415
 
406 416
 	foreach ($context['unapproved_items'] as $item)
407 417
 	{
@@ -420,14 +430,16 @@  discard block
 block discarded – undo
420 430
 				<span class="floatright">
421 431
 					<a href="', $scripturl, '?action=moderate;area=postmod;sa=', $context['current_view'], ';start=', $context['start'], ';', $context['session_var'], '=', $context['session_id'], ';approve=', $item['id'], '">', $approve_button, '</a>';
422 432
 
423
-		if ($item['can_delete'])
424
-			echo '
433
+		if ($item['can_delete']) {
434
+					echo '
425 435
 					', $context['menu_separator'], '
426 436
 					<a href="', $scripturl, '?action=moderate;area=postmod;sa=', $context['current_view'], ';start=', $context['start'], ';', $context['session_var'], '=', $context['session_id'], ';delete=', $item['id'], '">', $remove_button, '</a>';
437
+		}
427 438
 
428
-		if (!empty($options['display_quick_mod']) && $options['display_quick_mod'] == 1)
429
-			echo '
439
+		if (!empty($options['display_quick_mod']) && $options['display_quick_mod'] == 1) {
440
+					echo '
430 441
 					<input type="checkbox" name="item[]" value="', $item['id'], '" checked> ';
442
+		}
431 443
 
432 444
 		echo '
433 445
 				</span>
@@ -437,8 +449,8 @@  discard block
 block discarded – undo
437 449
 	echo '
438 450
 			<div class="pagesection">';
439 451
 
440
-	if (!empty($options['display_quick_mod']) && $options['display_quick_mod'] == 1)
441
-		echo '
452
+	if (!empty($options['display_quick_mod']) && $options['display_quick_mod'] == 1) {
453
+			echo '
442 454
 				<div class="floatright">
443 455
 					<select name="do" onchange="if (this.value != 0 &amp;&amp; confirm(\'', $txt['mc_unapproved_sure'], '\')) submit();">
444 456
 						<option value="0">', $txt['with_selected'], ':</option>
@@ -450,12 +462,14 @@  discard block
 block discarded – undo
450 462
 						<input type="submit" name="mc_go" value="', $txt['go'], '" class="button">
451 463
 					</noscript>
452 464
 				</div>';
465
+	}
453 466
 
454
-	if (!empty($context['unapproved_items']))
455
-		echo '
467
+	if (!empty($context['unapproved_items'])) {
468
+			echo '
456 469
 				<div class="floatleft">
457 470
 					<div class="pagelinks">', $context['page_index'], '</div>
458 471
 				</div>';
472
+	}
459 473
 
460 474
 	echo '
461 475
 			</div><!-- .pagesection -->
@@ -475,8 +489,9 @@  discard block
 block discarded – undo
475 489
 	global $scripturl, $context, $txt, $delete_button;
476 490
 
477 491
 	// We'll have a delete please bob.
478
-	if (empty($delete_button))
479
-		$delete_button = create_button('delete', 'remove_message', 'remove', 'class="centericon"');
492
+	if (empty($delete_button)) {
493
+			$delete_button = create_button('delete', 'remove_message', 'remove', 'class="centericon"');
494
+	}
480 495
 
481 496
 	$output_html = '
482 497
 					<div>
@@ -485,10 +500,11 @@  discard block
 block discarded – undo
485 500
 						</div>
486 501
 						<div class="floatright">';
487 502
 
488
-	if ($post['can_delete'])
489
-		$output_html .= '
503
+	if ($post['can_delete']) {
504
+			$output_html .= '
490 505
 							<a href="' . $scripturl . '?action=moderate;area=userwatch;sa=post;delete=' . $post['id'] . ';start=' . $context['start'] . ';' . $context['session_var'] . '=' . $context['session_id'] . '" data-confirm="' . $txt['mc_watched_users_delete_post'] . '" class="you_sure">' . $delete_button . '</a>
491 506
 							<input type="checkbox" name="delete[]" value="' . $post['id'] . '">';
507
+	}
492 508
 
493 509
 	$output_html .= '
494 510
 						</div>
@@ -531,12 +547,12 @@  discard block
 block discarded – undo
531 547
 				<input type="submit" name="save" value="', $txt['save'], '" class="button">
532 548
 			</div>
533 549
 		</form>';
534
-	}
535
-	else
536
-		echo '
550
+	} else {
551
+			echo '
537 552
 		<div class="windowbg">
538 553
 			<div class="centertext">', $txt['mc_no_settings'], '</div>
539 554
 		</div>';
555
+	}
540 556
 
541 557
 	echo '
542 558
 	</div><!-- #modcenter -->';
@@ -632,13 +648,14 @@  discard block
 block discarded – undo
632 648
 					</dd>
633 649
 				</dl>';
634 650
 
635
-	if ($context['template_data']['can_edit_personal'])
636
-		echo '
651
+	if ($context['template_data']['can_edit_personal']) {
652
+			echo '
637 653
 				<input type="checkbox" name="make_personal" id="make_personal"', $context['template_data']['personal'] ? ' checked' : '', '>
638 654
 					<label for="make_personal">
639 655
 						<strong>', $txt['mc_warning_template_personal'], '</strong>
640 656
 					</label><br>
641 657
 					<span class="smalltext">', $txt['mc_warning_template_personal_desc'], '</span><br>';
658
+	}
642 659
 
643 660
 	echo '
644 661
 				<hr>
Please login to merge, or discard this patch.
Themes/default/ManageCalendar.template.php 1 patch
Braces   +14 added lines, -10 removed lines patch added patch discarded remove patch
@@ -40,9 +40,10 @@  discard block
 block discarded – undo
40 40
 							<option value="0000"', $context['holiday']['year'] == '0000' ? ' selected' : '', '>', $txt['every_year'], '</option>';
41 41
 
42 42
 	// Show a list of all the years we allow...
43
-	for ($year = $modSettings['cal_minyear']; $year <= $modSettings['cal_maxyear']; $year++)
44
-		echo '
43
+	for ($year = $modSettings['cal_minyear']; $year <= $modSettings['cal_maxyear']; $year++) {
44
+			echo '
45 45
 							<option value="', $year, '"', $year == $context['holiday']['year'] ? ' selected' : '', '>', $year, '</option>';
46
+	}
46 47
 
47 48
 	echo '
48 49
 						</select>&nbsp;
@@ -50,9 +51,10 @@  discard block
 block discarded – undo
50 51
 						<select name="month" id="month" onchange="generateDays();">';
51 52
 
52 53
 	// There are 12 months per year - ensure that they all get listed.
53
-	for ($month = 1; $month <= 12; $month++)
54
-		echo '
54
+	for ($month = 1; $month <= 12; $month++) {
55
+			echo '
55 56
 							<option value="', $month, '"', $month == $context['holiday']['month'] ? ' selected' : '', '>', $txt['months'][$month], '</option>';
57
+	}
56 58
 
57 59
 	echo '
58 60
 						</select>&nbsp;
@@ -60,23 +62,25 @@  discard block
 block discarded – undo
60 62
 						<select name="day" id="day" onchange="generateDays();">';
61 63
 
62 64
 	// This prints out all the days in the current month - this changes dynamically as we switch months.
63
-	for ($day = 1; $day <= $context['holiday']['last_day']; $day++)
64
-		echo '
65
+	for ($day = 1; $day <= $context['holiday']['last_day']; $day++) {
66
+			echo '
65 67
 							<option value="', $day, '"', $day == $context['holiday']['day'] ? ' selected' : '', '>', $day, '</option>';
68
+	}
66 69
 
67 70
 	echo '
68 71
 						</select>
69 72
 					</dd>
70 73
 				</dl>';
71 74
 
72
-	if ($context['is_new'])
73
-		echo '
75
+	if ($context['is_new']) {
76
+			echo '
74 77
 				<input type="submit" value="', $txt['holidays_button_add'], '" class="button">';
75
-	else
76
-		echo '
78
+	} else {
79
+			echo '
77 80
 				<input type="submit" name="edit" value="', $txt['holidays_button_edit'], '" class="button">
78 81
 				<input type="submit" name="delete" value="', $txt['holidays_button_remove'], '" class="button">
79 82
 				<input type="hidden" name="holiday" value="', $context['holiday']['id'], '">';
83
+	}
80 84
 	echo '
81 85
 				<input type="hidden" name="', $context['session_var'], '" value="', $context['session_id'], '">
82 86
 			</div><!-- .windowbg2 -->
Please login to merge, or discard this patch.
Themes/default/ReportedContent.template.php 1 patch
Braces   +54 added lines, -36 removed lines patch added patch discarded remove patch
@@ -55,8 +55,9 @@  discard block
 block discarded – undo
55 55
 		// Prepare the comments...
56 56
 		$comments = array();
57 57
 
58
-		foreach ($report['comments'] as $comment)
59
-			$comments[$comment['member']['id']] = $comment['member']['link'];
58
+		foreach ($report['comments'] as $comment) {
59
+					$comments[$comment['member']['id']] = $comment['member']['link'];
60
+		}
60 61
 
61 62
 		echo '
62 63
 				', $txt['mc_reportedp_reported_by'], ': ', implode(', ', $comments), '
@@ -72,18 +73,21 @@  discard block
 block discarded – undo
72 73
 				<li><a href="', $scripturl, '?action=moderate;area=reportedposts;sa=handle;closed=', (int) !$report['closed'], ';rid=', $report['id'], ';start=', $context['start'], ';', $context['session_var'], '=', $context['session_id'], ';', $context['mod-report-closed_token_var'], '=', $context['mod-report-closed_token'], '">', $close_button, '</a></li>';
73 74
 
74 75
 		// Delete message button.
75
-		if (!$report['closed'] && (is_array($context['report_remove_any_boards']) && in_array($report['topic']['id_board'], $context['report_remove_any_boards'])))
76
-			echo '
76
+		if (!$report['closed'] && (is_array($context['report_remove_any_boards']) && in_array($report['topic']['id_board'], $context['report_remove_any_boards']))) {
77
+					echo '
77 78
 				<li><a href="', $scripturl, '?action=deletemsg;topic=', $report['topic']['id'], '.0;msg=', $report['topic']['id_msg'], ';modcenter;', $context['session_var'], '=', $context['session_id'], '" data-confirm="', $txt['mc_reportedp_delete_confirm'], '" class="you_sure">', $delete_button, '</a></li>';
79
+		}
78 80
 
79 81
 		// Ban this user button.
80
-		if (!$report['closed'] && !empty($context['report_manage_bans']))
81
-			echo '
82
+		if (!$report['closed'] && !empty($context['report_manage_bans'])) {
83
+					echo '
82 84
 				<li><a href="', $scripturl, '?action=admin;area=ban;sa=add', (!empty($report['author']['id']) ? ';u=' . $report['author']['id'] : ';msg=' . $report['topic']['id_msg']), ';', $context['session_var'], '=', $context['session_id'], '">', $ban_button, '</a></li>';
85
+		}
83 86
 
84
-		if (!$context['view_closed'])
85
-			echo '
87
+		if (!$context['view_closed']) {
88
+					echo '
86 89
 				<li><input type="checkbox" name="close[]" value="' . $report['id'] . '"></li>';
90
+		}
87 91
 
88 92
 			echo '
89 93
 			</ul>
@@ -91,18 +95,20 @@  discard block
 block discarded – undo
91 95
 	}
92 96
 
93 97
 	// Were none found?
94
-	if (empty($context['reports']))
95
-		echo '
98
+	if (empty($context['reports'])) {
99
+			echo '
96 100
 		<div class="windowbg2">
97 101
 			<p class="centertext">', $txt['mc_reportedp_none_found'], '</p>
98 102
 		</div>';
103
+	}
99 104
 
100 105
 	echo '
101 106
 		<div class="pagesection">';
102 107
 	
103
-	if (!empty($context['total_reports']) && $context['total_reports'] >= $context['reports_how_many'])
104
-		echo '
108
+	if (!empty($context['total_reports']) && $context['total_reports'] >= $context['reports_how_many']) {
109
+			echo '
105 110
 			<div class="pagelinks floatleft">' . $context['page_index'] . '</div>';
111
+	}
106 112
 
107 113
 	echo '
108 114
 			<div class="floatright">', !$context['view_closed'] ? '
@@ -133,18 +139,20 @@  discard block
 block discarded – undo
133 139
 			<div class="modbox">
134 140
 				<ul>';
135 141
 
136
-	foreach ($context['reported_posts'] as $report)
137
-		echo '
142
+	foreach ($context['reported_posts'] as $report) {
143
+			echo '
138 144
 					<li class="smalltext">
139 145
 						<a href="', $report['report_href'], '">', $report['subject'], '</a> ', $txt['mc_reportedp_by'], ' ', $report['author']['link'], '
140 146
 					</li>';
147
+	}
141 148
 
142 149
 	// Don't have any watched users right now?
143
-	if (empty($context['reported_posts']))
144
-		echo '
150
+	if (empty($context['reported_posts'])) {
151
+			echo '
145 152
 					<li>
146 153
 						<strong class="smalltext">', $txt['mc_recent_reports_none'], '</strong>
147 154
 					</li>';
155
+	}
148 156
 
149 157
 	echo '
150 158
 				</ul>
@@ -234,14 +242,15 @@  discard block
 block discarded – undo
234 242
 				<h3 class="catbg">', $txt['mc_modreport_whoreported_title'], '</h3>
235 243
 			</div>';
236 244
 
237
-	foreach ($context['report']['comments'] as $comment)
238
-		echo '
245
+	foreach ($context['report']['comments'] as $comment) {
246
+			echo '
239 247
 			<div class="windowbg">
240 248
 				<p class="smalltext">
241 249
 					', sprintf($txt['mc_modreport_whoreported_data'], $comment['member']['link'] . (empty($comment['member']['id']) && !empty($comment['member']['ip']) ? ' (' . $comment['member']['ip'] . ')' : ''), $comment['time']), '
242 250
 				</p>
243 251
 				<p>', $comment['message'], '</p>
244 252
 			</div>';
253
+	}
245 254
 
246 255
 	echo '
247 256
 			<br>
@@ -250,11 +259,12 @@  discard block
 block discarded – undo
250 259
 			</div>
251 260
 			<div>';
252 261
 
253
-	if (empty($context['report']['mod_comments']))
254
-		echo '
262
+	if (empty($context['report']['mod_comments'])) {
263
+			echo '
255 264
 				<div class="information">
256 265
 					<p class="centertext">', $txt['mc_modreport_no_mod_comment'], '</p>
257 266
 				</div>';
267
+	}
258 268
 
259 269
 	foreach ($context['report']['mod_comments'] as $comment)
260 270
 	{
@@ -346,18 +356,20 @@  discard block
 block discarded – undo
346 356
 			<div class="modbox">
347 357
 				<ul>';
348 358
 
349
-	foreach ($context['reported_members'] as $report)
350
-		echo '
359
+	foreach ($context['reported_members'] as $report) {
360
+			echo '
351 361
 					<li class="smalltext">
352 362
 						<a href="', $report['report_href'], '">', $report['user_name'], '</a>
353 363
 					</li>';
364
+	}
354 365
 
355 366
 	// Don't have any reported members right now?
356
-	if (empty($context['reported_members']))
357
-		echo '
367
+	if (empty($context['reported_members'])) {
368
+			echo '
358 369
 					<li>
359 370
 						<strong class="smalltext">', $txt['mc_recent_reports_none'], '</strong>
360 371
 					</li>';
372
+	}
361 373
 
362 374
 	echo '
363 375
 				</ul>
@@ -443,8 +455,9 @@  discard block
 block discarded – undo
443 455
 
444 456
 		// Prepare the comments...
445 457
 		$comments = array();
446
-		foreach ($report['comments'] as $comment)
447
-			$comments[$comment['member']['id']] = $comment['member']['link'];
458
+		foreach ($report['comments'] as $comment) {
459
+					$comments[$comment['member']['id']] = $comment['member']['link'];
460
+		}
448 461
 
449 462
 		echo '
450 463
 				', $txt['mc_reportedp_reported_by'], ': ', implode(', ', $comments), '
@@ -456,13 +469,15 @@  discard block
 block discarded – undo
456 469
 				<li><a href="', $scripturl, '?action=moderate;area=reportedmembers;sa=handle;closed=', (int) !$report['closed'], ';rid=', $report['id'], ';start=', $context['start'], ';', $context['session_var'], '=', $context['session_id'], ';', $context['mod-report-closed_token_var'], '=', $context['mod-report-closed_token'], '">', $close_button, '</a></li>';
457 470
 
458 471
 		// Ban this user button.
459
-		if (!$report['closed'] && !empty($context['report_manage_bans']) && !empty($report['user']['id']))
460
-			echo '
472
+		if (!$report['closed'] && !empty($context['report_manage_bans']) && !empty($report['user']['id'])) {
473
+					echo '
461 474
 				<li><a href="', $scripturl, '?action=admin;area=ban;sa=add;u=', $report['user']['id'], ';', $context['session_var'], '=', $context['session_id'], '">', $ban_button, '</a></li>';
475
+		}
462 476
 
463
-		if (!$context['view_closed'])
464
-			echo '
477
+		if (!$context['view_closed']) {
478
+					echo '
465 479
 				<li><input type="checkbox" name="close[]" value="' . $report['id'] . '"></li>';
480
+		}
466 481
 
467 482
 			echo '
468 483
 			</ul>
@@ -470,11 +485,12 @@  discard block
 block discarded – undo
470 485
 	}
471 486
 
472 487
 	// Were none found?
473
-	if (empty($context['reports']))
474
-		echo '
488
+	if (empty($context['reports'])) {
489
+			echo '
475 490
 		<div class="windowbg2">
476 491
 			<p class="centertext">', $txt['mc_reportedp_none_found'], '</p>
477 492
 		</div>';
493
+	}
478 494
 
479 495
 	echo '
480 496
 		<div class="pagesection">
@@ -534,14 +550,15 @@  discard block
 block discarded – undo
534 550
 				<h3 class="catbg">', $txt['mc_memberreport_whoreported_title'], '</h3>
535 551
 			</div>';
536 552
 
537
-	foreach ($context['report']['comments'] as $comment)
538
-		echo '
553
+	foreach ($context['report']['comments'] as $comment) {
554
+			echo '
539 555
 			<div class="windowbg">
540 556
 				<p class="smalltext">
541 557
 					', sprintf($txt['mc_modreport_whoreported_data'], $comment['member']['link'] . (empty($comment['member']['id']) && !empty($comment['member']['ip']) ? ' (' . $comment['member']['ip'] . ')' : ''), $comment['time']), '
542 558
 				</p>
543 559
 				<p>', $comment['message'], '</p>
544 560
 			</div>';
561
+	}
545 562
 
546 563
 	echo '
547 564
 			<br>
@@ -550,11 +567,12 @@  discard block
 block discarded – undo
550 567
 			</div>
551 568
 			<div>';
552 569
 
553
-	if (empty($context['report']['mod_comments']))
554
-		echo '
570
+	if (empty($context['report']['mod_comments'])) {
571
+			echo '
555 572
 				<div class="information">
556 573
 					<p class="centertext">', $txt['mc_modreport_no_mod_comment'], '</p>
557 574
 				</div>';
575
+	}
558 576
 
559 577
 	foreach ($context['report']['mod_comments'] as $comment)
560 578
 	{
Please login to merge, or discard this patch.
Themes/default/ReportToMod.template.php 1 patch
Braces   +6 added lines, -5 removed lines patch added patch discarded remove patch
@@ -62,16 +62,17 @@
 block discarded – undo
62 62
 				<div id="error_box" class="errorbox">
63 63
 					<ul id="error_list">';
64 64
 
65
-		foreach ($context['post_errors'] as $key => $error)
66
-			echo '
65
+		foreach ($context['post_errors'] as $key => $error) {
66
+					echo '
67 67
 						<li id="error_', $key, '" class="error">', $error, '</li>';
68
+		}
68 69
 
69 70
 		echo '
70 71
 					</ul>';
71
-	}
72
-	else
73
-		echo '
72
+	} else {
73
+			echo '
74 74
 					<div style="display:none" id="error_box" class="errorbox">';
75
+	}
75 76
 
76 77
 	echo '
77 78
 					</div>';
Please login to merge, or discard this patch.
Themes/default/ManageMaintenance.template.php 1 patch
Braces   +36 added lines, -25 removed lines patch added patch discarded remove patch
@@ -18,11 +18,12 @@  discard block
 block discarded – undo
18 18
 	global $context, $txt, $scripturl, $modSettings;
19 19
 
20 20
 	// If maintenance has finished tell the user.
21
-	if (!empty($context['maintenance_finished']))
22
-		echo '
21
+	if (!empty($context['maintenance_finished'])) {
22
+			echo '
23 23
 	<div class="infobox">
24 24
 		', sprintf($txt['maintain_done'], $context['maintenance_finished']), '
25 25
 	</div>';
26
+	}
26 27
 
27 28
 	echo '
28 29
 	<div id="manage_maintenance">
@@ -107,11 +108,12 @@  discard block
 block discarded – undo
107 108
 	<div id="manage_maintenance">';
108 109
 
109 110
 	// If maintenance has finished tell the user.
110
-	if (!empty($context['maintenance_finished']))
111
-		echo '
111
+	if (!empty($context['maintenance_finished'])) {
112
+			echo '
112 113
 		<div class="infobox">
113 114
 			', sprintf($txt['maintain_done'], $context['maintenance_finished']), '
114 115
 		</div>';
116
+	}
115 117
 
116 118
 	echo '
117 119
 		<div class="cat_bar">
@@ -243,11 +245,12 @@  discard block
 block discarded – undo
243 245
 	<div id="manage_maintenance">';
244 246
 
245 247
 	// If maintenance has finished tell the user.
246
-	if (!empty($context['maintenance_finished']))
247
-	echo '
248
+	if (!empty($context['maintenance_finished'])) {
249
+		echo '
248 250
 		<div class="infobox">
249 251
 			', sprintf($txt['maintain_done'], $context['maintenance_finished']), '
250 252
 		</div>';
253
+	}
251 254
 
252 255
 	echo '
253 256
 		<div class="cat_bar">
@@ -311,9 +314,10 @@  discard block
 block discarded – undo
311 314
 				</p>
312 315
 				<div style="display: none; padding: 3px" id="membersPanel">';
313 316
 
314
-	foreach ($context['membergroups'] as $group)
315
-		echo '
317
+	foreach ($context['membergroups'] as $group) {
318
+			echo '
316 319
 					<label for="groups', $group['id'], '"><input type="checkbox" name="groups[', $group['id'], ']" id="groups', $group['id'], '" checked> ', $group['name'], '</label><br>';
320
+	}
317 321
 
318 322
 	echo '
319 323
 				</div>
@@ -357,11 +361,12 @@  discard block
 block discarded – undo
357 361
 	global $scripturl, $txt, $context, $settings, $modSettings;
358 362
 
359 363
 	// If maintenance has finished tell the user.
360
-	if (!empty($context['maintenance_finished']))
361
-		echo '
364
+	if (!empty($context['maintenance_finished'])) {
365
+			echo '
362 366
 	<div class="infobox">
363 367
 		', sprintf($txt['maintain_done'], $context['maintenance_finished']), '
364 368
 	</div>';
369
+	}
365 370
 
366 371
 	// Bit of javascript for showing which boards to prune in an otherwise hidden list.
367 372
 	echo '
@@ -429,21 +434,23 @@  discard block
 block discarded – undo
429 434
 								<ul>';
430 435
 
431 436
 		// Display a checkbox with every board.
432
-		foreach ($category['boards'] as $board)
433
-			echo '
437
+		foreach ($category['boards'] as $board) {
438
+					echo '
434 439
 									<li style="margin-', $context['right_to_left'] ? 'right' : 'left', ': ', $board['child_level'] * 1.5, 'em;">
435 440
 										<label for="boards_', $board['id'], '"><input type="checkbox" name="boards[', $board['id'], ']" id="boards_', $board['id'], '" checked>', $board['name'], '</label>
436 441
 									</li>';
442
+		}
437 443
 
438 444
 		echo '
439 445
 								</ul>
440 446
 							</fieldset>';
441 447
 
442 448
 		// Increase $i, and check if we're at the middle yet.
443
-		if (++$i == $middle)
444
-			echo '
449
+		if (++$i == $middle) {
450
+					echo '
445 451
 						</div><!-- .floatleft -->
446 452
 						<div class="floatright" style="width: 49%;">';
453
+		}
447 454
 	}
448 455
 
449 456
 	echo '
@@ -485,9 +492,10 @@  discard block
 block discarded – undo
485 492
 		echo '
486 493
 						<optgroup label="', $category['name'], '">';
487 494
 
488
-		foreach ($category['boards'] as $board)
489
-			echo '
495
+		foreach ($category['boards'] as $board) {
496
+					echo '
490 497
 							<option value="', $board['id'], '"> ', str_repeat('==', $board['child_level']), '=&gt;&nbsp;', $board['name'], '</option>';
498
+		}
491 499
 
492 500
 		echo '
493 501
 						</optgroup>';
@@ -505,9 +513,10 @@  discard block
 block discarded – undo
505 513
 		echo '
506 514
 						<optgroup label="', $category['name'], '">';
507 515
 
508
-		foreach ($category['boards'] as $board)
509
-			echo '
516
+		foreach ($category['boards'] as $board) {
517
+					echo '
510 518
 							<option value="', $board['id'], '"> ', str_repeat('==', $board['child_level']), '=&gt;&nbsp;', $board['name'], '</option>';
519
+		}
511 520
 
512 521
 		echo '
513 522
 						</optgroup>';
@@ -548,9 +557,10 @@  discard block
 block discarded – undo
548 557
 				', $txt['database_optimize_attempt'], '<br>';
549 558
 
550 559
 	// List each table being optimized...
551
-	foreach ($context['optimized_tables'] as $table)
552
-		echo '
560
+	foreach ($context['optimized_tables'] as $table) {
561
+			echo '
553 562
 				', sprintf($txt['database_optimizing'], $table['name'], $table['data_freed']), '<br>';
563
+	}
554 564
 
555 565
 	// How did we go?
556 566
 	echo '
@@ -610,13 +620,14 @@  discard block
 block discarded – undo
610 620
 				</li>
611 621
 			</ul>';
612 622
 
613
-		if (!empty($context['exceeding_messages_morethan']))
614
-			echo '
623
+		if (!empty($context['exceeding_messages_morethan'])) {
624
+					echo '
615 625
 			<p>', $context['exceeding_messages_morethan'], '</p>';
616
-	}
617
-	else
618
-		echo '
626
+		}
627
+	} else {
628
+			echo '
619 629
 			<p class="infobox">', $txt['convert_to_text'], '</p>';
630
+	}
620 631
 
621 632
 	echo '
622 633
 			<form action="', $scripturl, '?action=admin;area=maintain;sa=database;activity=convertmsgbody" method="post" accept-charset="', $context['character_set'], '">
Please login to merge, or discard this patch.