fisharebest / webtrees

app/Module/BatchUpdateModule.php

Braces +3 added lines, -1 removed lines

@@ -235,7 +235,8 @@ 
 					<?php // Reset - otherwise we might "undo all changes", which refreshes the ?>
 					<?php // page, which makes them all again!  ?>
 					<script>reset_reload();</script>
-			<?php else: ?>
+			<?php else {
+	: ?>
 					<hr>
 					<div id="batch_update2" class="col-sm-12">
 						<?php if ($this->curr_xref): ?>
@@ -273,6 +274,7 @@ 
 		foreach (array_keys($this->all_xrefs) as $key) {
 			if ($key > $xref) {
 				$record = self::getLatestRecord($key, $this->all_xrefs[$key]);
+}
 				if ($this->PLUGIN->doesRecordNeedUpdate($key, $record)) {
 					return $key;
 				}

admin_site_upgrade.php

Spacing +2 added lines, -2 removed lines

@@ -29,10 +29,10 @@
 // Check for updates
 $latest_version_txt = Functions::fetchLatestVersion();
 if (preg_match('/^[0-9.]+\|[0-9.]+\|/', $latest_version_txt)) {
-	list($latest_version, , $download_url) = explode('|', $latest_version_txt);
+	list($latest_version,, $download_url) = explode('|', $latest_version_txt);
 } else {
 	// Cannot determine the latest version
-	list($latest_version, , $download_url) = explode('|', '||');
+	list($latest_version,, $download_url) = explode('|', '||');
 }
 
 $latest_version_html = '<span dir="ltr">' . $latest_version . '</span>';

app/Module.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -168,7 +168,7 @@
 					DebugBar::addThrowable($ex);
 
 					// The module has been deleted or is broken? Disable it.
-					Log::addConfigurationLog("Module {$module_name} is missing or broken - disabling it. " . $ex->getMessage());
+					Log::addConfigurationLog("module {$module_name} is missing or broken - disabling it. " . $ex->getMessage());
 					Database::prepare(
 						"UPDATE `##module` SET status = 'disabled' WHERE module_name = :module_name"
 					)->execute([

app/Database.php

Spacing +1 added lines, -2 removed lines

@@ -78,8 +78,7 @@
 		// Create the underlying PDO object
 		self::$pdo = new PDO(
 			(substr($config['dbhost'], 0, 1) === '/' ?
-				"mysql:unix_socket={$config['dbhost']};dbname={$config['dbname']}" :
-				"mysql:host={$config['dbhost']};dbname={$config['dbname']};port={$config['dbport']}"
+				"mysql:unix_socket={$config['dbhost']};dbname={$config['dbname']}" : "mysql:host={$config['dbhost']};dbname={$config['dbname']};port={$config['dbport']}"
 			),
 			$config['dbuser'], $config['dbpass'],
 			[

placelist.php

Braces +3 added lines, -1 removed lines

@@ -200,7 +200,8 @@ 
 			<div class="tab-pane fade show active" role="tabpanel" id="individuals">
 				<?php if (empty($myindilist)): ?>
 						<p><?= I18N::translate('No results found.') ?></p>
-				<?php else: ?>
+				<?php else {
+	: ?>
 						<?= FunctionsPrintLists::individualTable($myindilist) ?>
 					<?php endif ?>
 			</div>
@@ -216,6 +217,7 @@ 
 		<?php
 		}
 		echo '<h4><a href="placelist.php?display=list">', I18N::translate('Show all places in a list'), '</a></h4>';
+}
 
 		if ($gm_module && $gm_module->getPreference('GM_PLACE_HIERARCHY')) {
 			$gm_module->mapScripts($numfound, $level, $parent, $linklevels, $place_names);

resources/views/sidebars/individuals.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 <?php use Fisharebest\Webtrees\Theme; ?>
 <?php use Fisharebest\Webtrees\View; ?>
 
-<form method="post" action="<?= e(Html::url('module.php' ,['mod' => 'individuals', 'mod_action' => 'ajax'])) ?>" onsubmit="return false;">
+<form method="post" action="<?= e(Html::url('module.php', ['mod' => 'individuals', 'mod_action' => 'ajax'])) ?>" onsubmit="return false;">
 	<input type="search" name="sb_indi_name" id="sb_indi_name" placeholder="<?= I18N::translate('Search') ?>">
 	<p>
 		<?php foreach ($initials as $letter => $count): ?>

resources/views/sidebars/families.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 <?php use Fisharebest\Webtrees\Theme; ?>
 <?php use Fisharebest\Webtrees\View; ?>
 
-<form method="post" action="<?= e(Html::url('module.php' ,['mod' => 'families', 'mod_action' => 'ajax'])) ?>" onsubmit="return false;">
+<form method="post" action="<?= e(Html::url('module.php', ['mod' => 'families', 'mod_action' => 'ajax'])) ?>" onsubmit="return false;">
 	<input type="search" name="sb_fam_name" id="sb_fam_name" placeholder="<?= I18N::translate('Search') ?>">
 	<p>
 		<?php foreach ($initials as $letter => $count): ?>

resources/views/blocks/changes-list.php

Braces +5 added lines, -2 removed lines

@@ -3,9 +3,12 @@
 
 <?php if ($summary !== ''): ?>
 	<span class="details_label"><?= $summary ?></span>
-<?php else: ?>
+<?php else {
+	: ?>
 	<?php foreach ($facts as $fact): ?>
-		<?php $record = $fact->getParent(); ?>
+		<?php $record = $fact->getParent();
+}
+?>
 		<a href="<?= e($record->url()) ?>" class="list_item name2">
 			<?= $record->getFullName() ?>
 		</a>

app/Module/ChartsBlockModule.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 					]);
 					break;
 				case 'descendants':
-					$title           = I18N::translate('Descendants of %s', $person->getFullName());
+					$title = I18N::translate('Descendants of %s', $person->getFullName());
 					$chart_url = route('descendants-chart', [
 						'xref'        => $person->getXref(),
 						'ged'         => $person->getTree()->getName(),
@@ -98,7 +98,7 @@ 
 					]);
 					break;
 				case 'hourglass':
-					$title           = I18N::translate('Hourglass chart of %s', $person->getFullName());
+					$title = I18N::translate('Hourglass chart of %s', $person->getFullName());
 					$chart_url = route('hourglass-chart', [
 						'xref'        => $person->getXref(),
 						'ged'         => $person->getTree()->getName(),