Completed
Pull Request — master (#926)
by Mark
13:47
created
admin_module_tabs.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -98,8 +98,11 @@
 block discarded – undo
98 98
 				<td class="col-xs-1">
99 99
 					<?php if ($module instanceof ModuleConfigInterface): ?>
100 100
 					<a href="<?php echo $module->getConfigLink(); ?>"><?php echo $module->getTitle(); ?> <i class="fa fa-cogs"></i></a>
101
-					<?php else: ?>
102
-					<?php echo $module->getTitle(); ?>
101
+					<?php else {
102
+	: ?>
103
+					<?php echo $module->getTitle();
104
+}
105
+?>
103 106
 					<?php endif; ?>
104 107
 				</td>
105 108
 				<td class="col-xs-5"><?php echo $module->getDescription(); ?></td>
Please login to merge, or discard this patch.
admin_site_merge.php 1 patch
Braces   +20 added lines, -8 removed lines patch added patch discarded remove patch
@@ -232,9 +232,12 @@  discard block
 block discarded – undo
232 232
 				<?php endforeach; ?>
233 233
 				</tbody>
234 234
 			</table>
235
-			<?php else: ?>
235
+			<?php else {
236
+	: ?>
236 237
 			<p>
237
-				<?php echo I18N::translate('No matching facts found'); ?>
238
+				<?php echo I18N::translate('No matching facts found');
239
+}
240
+?>
238 241
 			</p>
239 242
 			<?php endif; ?>
240 243
 		</div>
@@ -279,9 +282,12 @@  discard block
 block discarded – undo
279 282
 							<?php endforeach; ?>
280 283
 							</tbody>
281 284
 						</table>
282
-					<?php else: ?>
285
+					<?php else {
286
+	: ?>
283 287
 						<p>
284
-							<?php echo I18N::translate('No matching facts found'); ?>
288
+							<?php echo I18N::translate('No matching facts found');
289
+}
290
+?>
285 291
 						</p>
286 292
 					<?php endif; ?>
287 293
 				</div>
@@ -325,9 +331,12 @@  discard block
 block discarded – undo
325 331
 							<?php endforeach; ?>
326 332
 							</tbody>
327 333
 						</table>
328
-					<?php else: ?>
334
+					<?php else {
335
+	: ?>
329 336
 						<p>
330
-							<?php echo I18N::translate('No matching facts found'); ?>
337
+							<?php echo I18N::translate('No matching facts found');
338
+}
339
+?>
331 340
 						</p>
332 341
 					<?php endif; ?>
333 342
 				</div>
@@ -341,10 +350,13 @@  discard block
 block discarded – undo
341 350
 	</button>
342 351
 </form>
343 352
 
344
-<?php else: ?>
353
+<?php else {
354
+	: ?>
345 355
 
346 356
 <form class="form form-horizontal">
347
-	<input type="hidden" name="ged" value="<?php echo $WT_TREE->getNameHtml(); ?>">
357
+	<input type="hidden" name="ged" value="<?php echo $WT_TREE->getNameHtml();
358
+}
359
+?>">
348 360
 	<p><?php echo /* I18N: Records are indviduals, sources, etc. */ I18N::translate('Select two records to merge.'); ?></p>
349 361
 
350 362
 	<div class="form-group">
Please login to merge, or discard this patch.
admin_trees_config.php 1 patch
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -554,18 +554,24 @@
 block discarded – undo
554 554
 					<div class="bg-danger text-danger">
555 555
 						<?php echo $resn->xref, ' — ', I18N::translate('this record does not exist'); ?>
556 556
 					</div>
557
-					<?php else: ?>
557
+					<?php else {
558
+	: ?>
558 559
 					<div class="text-muted">
559
-						<?php echo I18N::translate('All records'); ?>
560
+						<?php echo I18N::translate('All records');
561
+}
562
+?>
560 563
 					</div>
561 564
 					<?php endif; ?>
562 565
 				</td>
563 566
 				<td>
564 567
 					<?php if ($resn->tag_label): ?>
565 568
 					<?php echo $resn->tag_label; ?>
566
-					<?php else: ?>
569
+					<?php else {
570
+	: ?>
567 571
 					<div class="text-muted">
568
-						<?php echo I18N::translate('All facts and events'); ?>
572
+						<?php echo I18N::translate('All facts and events');
573
+}
574
+?>
569 575
 					</div>
570 576
 					<?php endif; ?>
571 577
 				</td>
Please login to merge, or discard this patch.
admin_trees_duplicates.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -177,8 +177,11 @@
 block discarded – undo
177 177
 	</li>
178 178
 <?php endforeach; ?>
179 179
 </ul>
180
-<?php else: ?>
181
-<p><?php echo I18N::translate('No duplicates have been found.'); ?></p>
180
+<?php else {
181
+	: ?>
182
+<p><?php echo I18N::translate('No duplicates have been found.');
183
+}
184
+?></p>
182 185
 <?php endif; ?>
183 186
 
184 187
 <?php endforeach; ?>
Please login to merge, or discard this patch.
admin_trees_manage.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -433,8 +433,11 @@
 block discarded – undo
433 433
 									<i class="fa fa-li fa-star"></i>
434 434
 									<?php if ($tree->getName() == Site::getPreference('DEFAULT_GEDCOM')): ?>
435 435
 										<?php echo I18N::translate('Default family tree'); ?>
436
-									<?php else: ?>
437
-										<a href="#" onclick="document.defaultform<?php echo $tree->getTreeId(); ?>.submit();">
436
+									<?php else {
437
+	: ?>
438
+										<a href="#" onclick="document.defaultform<?php echo $tree->getTreeId();
439
+}
440
+?>.submit();">
438 441
 											<?php echo I18N::translate('Set as default'); ?>
439 442
 											<span class="sr-only">
440 443
 										<?php echo $tree->getTitleHtml(); ?>
Please login to merge, or discard this patch.
admin_users.php 1 patch
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -791,8 +791,11 @@
 block discarded – undo
791 791
 		<p>
792 792
 		<?php if ($ucnt): ?>
793 793
 			<input type="submit" value="<?php echo I18N::translate('delete'); ?>">
794
-			<?php else: ?>
795
-			<?php echo I18N::translate('Nothing found to cleanup'); ?>
794
+			<?php else {
795
+	: ?>
796
+			<?php echo I18N::translate('Nothing found to cleanup');
797
+}
798
+?>
796 799
 			<?php endif; ?>
797 800
 		</p>
798 801
 	</form>
Please login to merge, or discard this patch.
app/Database.php 1 patch
Spacing   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -85,8 +85,7 @@
 block discarded – undo
85 85
 		// Create the underlying PDO object
86 86
 		self::$pdo = new PDO(
87 87
 			(substr($DBHOST, 0, 1) === '/' ?
88
-				"mysql:unix_socket={$DBHOST};dbname={$DBNAME}" :
89
-				"mysql:host={$DBHOST};dbname={$DBNAME};port={$DBPORT}"
88
+				"mysql:unix_socket={$DBHOST};dbname={$DBNAME}" : "mysql:host={$DBHOST};dbname={$DBNAME};port={$DBPORT}"
90 89
 			),
91 90
 			$DBUSER, $DBPASS,
92 91
 			array(
Please login to merge, or discard this patch.
app/File.php 1 patch
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
 		try {
48 48
 			$fp = fsockopen($scheme . $host, $port, $errno, $errstr, 5);
49 49
 
50
-			fputs($fp, "GET $path?$query HTTP/1.0\r\nHost: $host\r\nConnection: Close\r\n\r\n");
50
+			fputs($fp, "get $path?$query HTTP/1.0\r\nHost: $host\r\nConnection: Close\r\n\r\n");
51 51
 
52 52
 			// The first part of the response include the HTTP headers
53 53
 			$response = fread($fp, 65536);
Please login to merge, or discard this patch.
calendar.php 1 patch
Upper-Lower-Casing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -67,14 +67,14 @@
 block discarded – undo
67 67
 	if (strlen($match[1]) > strlen($match[2])) {
68 68
 		$match[2] = substr($match[1], 0, strlen($match[1]) - strlen($match[2])) . $match[2];
69 69
 	}
70
-	$ged_date = new Date("FROM {$cal} {$match[1]} TO {$cal} {$match[2]}");
70
+	$ged_date = new Date("from {$cal} {$match[1]} TO {$cal} {$match[2]}");
71 71
 	$view     = 'year';
72 72
 } else {
73 73
 	// advanced-year "decade/century wildcard"
74 74
 	if (preg_match('/^(\d+)(\?+)$/', $year, $match)) {
75 75
 		$y1       = $match[1] . str_replace('?', '0', $match[2]);
76 76
 		$y2       = $match[1] . str_replace('?', '9', $match[2]);
77
-		$ged_date = new Date("FROM {$cal} {$y1} TO {$cal} {$y2}");
77
+		$ged_date = new Date("from {$cal} {$y1} TO {$cal} {$y2}");
78 78
 		$view     = 'year';
79 79
 	} else {
80 80
 		if ($year < 0) {
Please login to merge, or discard this patch.