fisharebest / webtrees

app/Functions/FunctionsPrintLists.php

Spacing +3 added lines, -3 removed lines

@@ -1080,7 +1080,7 @@ 
 			"SELECT CONCAT(l_to, '@', l_file), COUNT(*) FROM `##sources` JOIN `##link` ON l_from = s_id AND l_file = s_file AND l_type = 'NOTE' GROUP BY l_to, l_file"
 		)->fetchAssoc();
 
-		$html     = '';
+		$html = '';
 		$html .= '<table ' . Datatables::noteTableAttributes() . '><thead><tr>';
 		$html .= '<th>' . I18N::translate('Title') . '</th>';
 		$html .= '<th>' . I18N::translate('Individuals') . '</th>';
@@ -1146,7 +1146,7 @@ 
 		)->fetchAssoc();
 
 		$html = '';
-		$html .= '<table ' . Datatables::repositoryTableAttributes() . '><thead><tr>';		$html .= '<th>' . I18N::translate('Repository name') . '</th>';
+		$html .= '<table ' . Datatables::repositoryTableAttributes() . '><thead><tr>'; $html .= '<th>' . I18N::translate('Repository name') . '</th>';
 		$html .= '<th>' . I18N::translate('Sources') . '</th>';
 		$html .= '<th>' . I18N::translate('Last change') . '</th>';
 		$html .= '<th>' . I18N::translate('Delete') . '</th>';
@@ -1573,7 +1573,7 @@ 
 					if ($endjd === $startjd) {
 						$html .= I18N::translate('No events exist for tomorrow.');
 					} else {
-						$html .=  /* I18N: translation for %s==1 is unused; it is translated separately as “tomorrow” */ I18N::plural('No events exist for the next %s day.', 'No events exist for the next %s days.', $endjd - $startjd + 1, I18N::number($endjd - $startjd + 1));
+						$html .= /* I18N: translation for %s==1 is unused; it is translated separately as “tomorrow” */ I18N::plural('No events exist for the next %s day.', 'No events exist for the next %s days.', $endjd - $startjd + 1, I18N::number($endjd - $startjd + 1));
 					}
 				} else {
 					if ($endjd === $startjd) {

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/Controller/IndividualController.php

Doc Comments -2 removed lines

@@ -118,7 +118,6 @@ 
 	/**
 	 * Format a name record
 	 *
-	 * @param int  $primary
 	 * @param Fact $fact
 	 *
 	 * @return string
@@ -216,7 +215,6 @@ 
 	/**
 	 * print information for a sex record
 	 *
-	 * @param Fact $event the Event object
 	 *
 	 * @return string
 	 */

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

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

editnews.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 
 switch ($action) {
 case 'delete':
-	Database::prepare("DELETE FROM `##news` WHERE news_id = :news_id")->execute(['news_id'   => $news_id,]);
+	Database::prepare("DELETE FROM `##news` WHERE news_id = :news_id")->execute(['news_id'   => $news_id, ]);
 
 	header('Location: ' . WT_BASE_URL . 'index.php?ctype=' . $ctype . '&ged=' . $controller->tree()->getNameUrl());
 

action.php

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
 
 	// Create a media object, and return parameters needed by Select2
 	header('Content-type: application/json');
-	$filename   = 'eek.jpg';
+	$filename = 'eek.jpg';
 	$auto   = Filter::post('auto');
 	$folder = Filter::post('folder');
 	$note   = Filter::post('note');
@@ -145,7 +145,7 @@ 
 	}
 
 
-	$gedcom = "0 @new@ OBJE\n1 FILE " . $folder . $file . "\n2 FORM " .  $format;
+	$gedcom = "0 @new@ OBJE\n1 FILE " . $folder . $file . "\n2 FORM " . $format;
 	if ($type !== '') {
 		$gedcom .= "\n3 TYPE " . $type;
 	}

index_edit.php

Spacing +2 added lines, -2 removed lines

@@ -127,7 +127,7 @@ 
 			}
 		}
 	}
-	if ($user_id < 0 || $gedcom_id < 0 ) {
+	if ($user_id < 0 || $gedcom_id < 0) {
 		header('Location: ' . WT_BASE_URL . 'admin.php');
 	} elseif ($user_id > 0) {
 		header('Location: ' . WT_BASE_URL . 'index.php?ctype=user&ged=' . $WT_TREE->getNameUrl());
@@ -371,7 +371,7 @@ 
 						<?= FontAwesome::decorativeIcon('save') ?>
 						<?= I18N::translate('save') ?>
 					</button>
-					<?php if ($user_id < 0 || $gedcom_id < 0 ): ?>
+					<?php if ($user_id < 0 || $gedcom_id < 0): ?>
 						<a class="btn btn-secondary" href="admin.php">
 					<?php elseif ($user_id > 0): ?>
 						<a class="btn btn-secondary" href="index.php?ctype=user&amp;ged=<?= $WT_TREE->getNameHtml() ?>">

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,
 					];