fisharebest / webtrees

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);

admin_trees_config.php

Spacing +1 added lines, -1 removed lines

@@ -1505,7 +1505,7 @@
 						'0' => I18N::translateContext('Show the [first/last] [N] parts of a place name.', 'first'),
 						'1' => I18N::translateContext('Show the [first/last] [N] parts of a place name.', 'last'),
 					], $WT_TREE->getPreference('SHOW_PEDIGREE_PLACES_SUFFIX', ['name' => 'SHOW_PEDIGREE_PLACES_SUFFIX'])),
-					Bootstrap4::select(FunctionsEdit::numericOptions(range(1,9)), $WT_TREE->getPreference('SHOW_PEDIGREE_PLACES'), ['name' => 'SHOW_PEDIGREE_PLACES'])
+					Bootstrap4::select(FunctionsEdit::numericOptions(range(1, 9)), $WT_TREE->getPreference('SHOW_PEDIGREE_PLACES'), ['name' => 'SHOW_PEDIGREE_PLACES'])
 				) ?>
 				<p class="small text-muted">
 					<?= /* I18N: Help text for the “Abbreviate place names” configuration setting */ I18N::translate('Place names are frequently too long to fit on charts, lists, etc. They can be abbreviated by showing just the first few parts of the name, such as <i>village, county</i>, or the last few part of it, such as <i>region, country</i>.') ?>

app/Module.php

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

@@ -135,7 +135,7 @@
 					}
 				} catch (\Exception $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([

relationship.php

Braces +3 added lines, -1 removed lines

@@ -82,7 +82,8 @@ 
 				<?php if ($ancestors_only === '1'): ?>
 					<input type="hidden" name="ancestors" value="1">
 					<?= I18N::translate('Find relationships via ancestors') ?>
-				<?php else: ?>
+				<?php else {
+	: ?>
 					<?= Bootstrap4::radioButtons('ancestors', ['0' => I18N::translate('Find any relationship'), '1' => I18N::translate('Find relationships via ancestors')], $ancestors, false) ?>
 			<?php endif ?>
 			</div>
@@ -115,6 +116,7 @@ 
 if ($person1 && $person2) {
 	if (I18N::direction() === 'ltr') {
 		$diagonal1        = Theme::theme()->parameter('image-dline');
+}
 		$diagonal2        = Theme::theme()->parameter('image-dline2');
 	} else {
 		$diagonal1        = Theme::theme()->parameter('image-dline2');

app/Controller/IndividualController.php

Spacing +4 added lines, -4 removed lines

@@ -96,7 +96,7 @@ 
 	public function getTabs() {
 		$active_tabs = Module::getActiveTabs($this->record->getTree());
 
-		return array_filter($active_tabs, function(ModuleTabInterface $tab) { return $tab->hasTabContent(); });
+		return array_filter($active_tabs, function (ModuleTabInterface $tab) { return $tab->hasTabContent(); });
 	}
 
 	/**
@@ -141,7 +141,7 @@ 
 
 		if ($n === 0) {
 			$content_class = 'collapse show';
-			$aria  = 'true';
+			$aria = 'true';
 		}
 		if ($fact->isPendingDeletion()) {
 			$container_class .= ' old';
@@ -243,7 +243,7 @@ 
 		}
 
 		if ($individual->canEdit() && !$fact->isPendingDeletion()) {
-			$edit_links = FontAwesome::linkIcon('edit', I18N::translate('Edit the gender'), ['class' => 'btn btn-link', 'href' => 'edit_interface.php?action=edit&xref=' . $individual->getXref() . '&fact_id=' . $fact->getFactId() . '&ged=' . $individual->getTree()->getNameHtml() ]);
+			$edit_links = FontAwesome::linkIcon('edit', I18N::translate('Edit the gender'), ['class' => 'btn btn-link', 'href' => 'edit_interface.php?action=edit&xref=' . $individual->getXref() . '&fact_id=' . $fact->getFactId() . '&ged=' . $individual->getTree()->getNameHtml()]);
 
 		} else {
 			$edit_links = '';
@@ -253,7 +253,7 @@ 
 		<div class="' . $container_class . '">
 			<div class="card-header" role="tab" id="name-header-add">
 				<div class="card-title mb-0">
-					<b>' . I18N::translate('Gender') . '</b> ' .  $sex . $edit_links . '
+					<b>' . I18N::translate('Gender') . '</b> ' . $sex . $edit_links . '
 				</div>
 			</div>
 		</div>';

app/Select2.php

Spacing +3 added lines, -3 removed lines

@@ -176,7 +176,7 @@ 
 
 		$flag_files = [];
 		foreach ($it as $file) {
-			$file_path = substr($file->getPathname() , strlen($directory));
+			$file_path = substr($file->getPathname(), strlen($directory));
 			if ($file->getExtension() === 'png' && stripos($file_path, $query) !== false) {
 				if ($offset > 0) {
 					// Skip results
@@ -453,7 +453,7 @@ 
 			if ($place_name === $query) {
 				$found = true;
 			}
-			$results[]  = [
+			$results[] = [
 				'id'   => $place_name,
 				'text' => $place_name,
 			];
@@ -488,7 +488,7 @@ 
 					if ($place_name === $query) {
 						$found = true;
 					}
-					$results[]  = [
+					$results[] = [
 						'id'   => $place_name,
 						'text' => $place_name,
 					];

app/Filter.php

Doc Comments +1 added lines, -1 removed lines

@@ -334,7 +334,7 @@
 	 * @param string|null $regexp
 	 * @param string      $default
 	 *
-	 * @return string[]|string[][]
+	 * @return string[]
 	 */
 	public static function postArray($variable, $regexp = null, $default = '') {
 		return self::inputArray(INPUT_POST, $variable, $regexp, $default);

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 
 use Fisharebest\Webtrees\CommonMark\CensusTableExtension;
 use Fisharebest\Webtrees\CommonMark\XrefExtension;
-use Fisharebest\Webtrees\CommonMark\XrefParser;
 use League\CommonMark\Block\Renderer\DocumentRenderer;
 use League\CommonMark\Block\Renderer\ParagraphRenderer;
 use League\CommonMark\Converter;

app/Census/CensusColumnAgeFemale5Years.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 				$years -= $years % 5;
 			}
 
-			return (string)$years;
+			return (string) $years;
 		}
 	}
 }

app/Census/CensusColumnYearsMarried.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 		if ($marriage_date === null) {
 			return '';
 		} else {
-			return (string)Date::getAge($marriage_date, $this->date(), 0);
+			return (string) Date::getAge($marriage_date, $this->date(), 0);
 		}
 	}
 }