fisharebest / webtrees

app/Http/Controllers/AdminSiteController.php

Spacing +25 added lines, -25 removed lines

@@ -255,35 +255,35 @@ 
 
 		$args = [];
 		if ($search) {
-			$sql            .= " AND log_message LIKE CONCAT('%', :search, '%')";
+			$sql .= " AND log_message LIKE CONCAT('%', :search, '%')";
 			$args['search'] = $search;
 		}
 		if ($from) {
-			$sql          .= " AND log_time >= :from";
+			$sql .= " AND log_time >= :from";
 			$args['from'] = $from;
 		}
 		if ($to) {
-			$sql        .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
+			$sql .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
 			$args['to'] = $to;
 		}
 		if ($type) {
-			$sql          .= " AND log_type = :type";
+			$sql .= " AND log_type = :type";
 			$args['type'] = $type;
 		}
 		if ($text) {
-			$sql          .= " AND log_message LIKE CONCAT('%', :text, '%')";
+			$sql .= " AND log_message LIKE CONCAT('%', :text, '%')";
 			$args['text'] = $text;
 		}
 		if ($ip) {
-			$sql        .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
+			$sql .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
 			$args['ip'] = $ip;
 		}
 		if ($username) {
-			$sql          .= " AND user_name LIKE CONCAT('%', :user, '%')";
+			$sql .= " AND user_name LIKE CONCAT('%', :user, '%')";
 			$args['user'] = $username;
 		}
 		if ($gedc) {
-			$sql          .= " AND gedcom_name = :gedc";
+			$sql .= " AND gedcom_name = :gedc";
 			$args['gedc'] = $gedc;
 		}
 
@@ -309,7 +309,7 @@ 
 		}
 
 		if ($length) {
-			$sql            .= " LIMIT :limit OFFSET :offset";
+			$sql .= " LIMIT :limit OFFSET :offset";
 			$args['limit']  = $length;
 			$args['offset'] = $start;
 		}
@@ -361,35 +361,35 @@ 
 
 		$args = [];
 		if ($search) {
-			$sql            .= " AND log_message LIKE CONCAT('%', :search, '%')";
+			$sql .= " AND log_message LIKE CONCAT('%', :search, '%')";
 			$args['search'] = $search;
 		}
 		if ($from) {
-			$sql          .= " AND log_time >= :from";
+			$sql .= " AND log_time >= :from";
 			$args['from'] = $from;
 		}
 		if ($to) {
-			$sql        .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
+			$sql .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
 			$args['to'] = $to;
 		}
 		if ($type) {
-			$sql          .= " AND log_type = :type";
+			$sql .= " AND log_type = :type";
 			$args['type'] = $type;
 		}
 		if ($text) {
-			$sql          .= " AND log_message LIKE CONCAT('%', :text, '%')";
+			$sql .= " AND log_message LIKE CONCAT('%', :text, '%')";
 			$args['text'] = $text;
 		}
 		if ($ip) {
-			$sql        .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
+			$sql .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
 			$args['ip'] = $ip;
 		}
 		if ($username) {
-			$sql          .= " AND user_name LIKE CONCAT('%', :user, '%')";
+			$sql .= " AND user_name LIKE CONCAT('%', :user, '%')";
 			$args['user'] = $username;
 		}
 		if ($gedc) {
-			$sql          .= " AND gedcom_name = :gedc";
+			$sql .= " AND gedcom_name = :gedc";
 			$args['gedc'] = $gedc;
 		}
 
@@ -421,37 +421,37 @@ 
 
 		$args = [];
 		if ($from) {
-			$sql          .= " AND log_time >= :from";
+			$sql .= " AND log_time >= :from";
 			$args['from'] = $from;
 		}
 		if ($to) {
-			$sql        .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
+			$sql .= " AND log_time < TIMESTAMPADD(DAY, 1 , :to)"; // before end of the day
 			$args['to'] = $to;
 		}
 		if ($type) {
-			$sql          .= " AND log_type = :type";
+			$sql .= " AND log_type = :type";
 			$args['type'] = $type;
 		}
 		if ($text) {
-			$sql          .= " AND log_message LIKE CONCAT('%', :text, '%')";
+			$sql .= " AND log_message LIKE CONCAT('%', :text, '%')";
 			$args['text'] = $text;
 		}
 		if ($ip) {
-			$sql        .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
+			$sql .= " AND ip_address LIKE CONCAT('%', :ip, '%')";
 			$args['ip'] = $ip;
 		}
 		if ($username) {
-			$sql          .= " AND user_name LIKE CONCAT('%', :user, '%')";
+			$sql .= " AND user_name LIKE CONCAT('%', :user, '%')";
 			$args['user'] = $username;
 		}
 		if ($gedc) {
-			$sql          .= " AND gedcom_name = :gedc";
+			$sql .= " AND gedcom_name = :gedc";
 			$args['gedc'] = $gedc;
 		}
 
 		$sql .= " ORDER BY log_id";
 
-		$rows = Database::prepare($sql )->execute($args)->fetchAll();
+		$rows = Database::prepare($sql)->execute($args)->fetchAll();
 
 		$data = '';
 

app/Module/SiteMapModule.php

Spacing +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
 	public function getFileAction(Request $request): Response {
 		$file = $request->get('file', '');
 
-		if (!preg_match('/^(\d+)-([imnrs])-(\d+)$/', $file, $match))		 {
+		if (!preg_match('/^(\d+)-([imnrs])-(\d+)$/', $file, $match)) {
 			throw new NotFoundHttpException('Bad sitemap file');
 		}
 

Braces +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
 	public function getFileAction(Request $request): Response {
 		$file = $request->get('file', '');
 
-		if (!preg_match('/^(\d+)-([imnrs])-(\d+)$/', $file, $match))		 {
+		if (!preg_match('/^(\d+)-([imnrs])-(\d+)$/', $file, $match)) {
 			throw new NotFoundHttpException('Bad sitemap file');
 		}
 

app/Http/Controllers/EditIndividualController.php

Spacing +1 added lines, -1 removed lines

@@ -223,7 +223,7 @@
 		/** @var Tree $tree */
 		$tree = $request->attributes->get('tree');
 
-		$xref   = $request->get('xref', '');
+		$xref = $request->get('xref', '');
 
 		$individual = Individual::getInstance($xref, $tree);
 

app/Http/Controllers/DescendantsChartController.php

Spacing +3 added lines, -3 removed lines

@@ -101,7 +101,7 @@ 
 
 		$this->checkModuleIsActive($tree, 'descendancy_chart');
 
-		$xref         = $request->get('xref');
+		$xref = $request->get('xref');
 		$individual = Individual::getInstance($xref, $tree);
 
 		$this->checkIndividualAccess($individual);
@@ -118,7 +118,7 @@ 
 
 		$descendants = $this->descendants($individual, $generations, []);
 
-		switch($chart_style) {
+		switch ($chart_style) {
 			case self::CHART_STYLE_LIST:
 			default:
 				return $this->descendantsList($individual, $generations);
@@ -315,7 +315,7 @@ 
 	 * @return Response
 	 */
 	private function descendantsFamilies(Tree $tree, array $descendants): Response {
-		$families  = [];
+		$families = [];
 		foreach ($descendants as $individual) {
 			foreach ($individual->getChildFamilies() as $family) {
 				$families[$family->getXref()] = $family;

resources/views/lists/notes-table.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 $count_media       = Database::prepare(
 	"SELECT l_to, COUNT(*) FROM `##media` JOIN `##link` ON l_from = m_id AND l_file = m_file AND l_type = 'NOTE' AND l_file = :tree_id GROUP BY l_to"
 )->execute(['tree_id' => $tree->getTreeId()])->fetchAssoc();
-$count_sources       = Database::prepare(
+$count_sources = Database::prepare(
 	"SELECT l_to, COUNT(*) FROM `##sources` JOIN `##link` ON l_from = s_id AND l_file = s_file AND l_type = 'NOTE' AND l_file = :tree_id GROUP BY l_to"
 )->execute(['tree_id' => $tree->getTreeId()])->fetchAssoc();
 ?>

resources/views/lists/chart-by-decade.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 }
 $chart_url .= '|1:||' . rawurlencode(I18N::percentage($vmax / $count)); // y axis
 $chart_url .= '|2:||';
-$step      = $vmax;
+$step = $vmax;
 for ($d = $vmax; $d > 0; $d--) {
 	if ($vmax < ($d * 10 + 1) && ($vmax % $d) == 0) {
 		$step = $d;

resources/views/lists/surnames-table.php

Spacing +2 added lines, -2 removed lines

@@ -43,14 +43,14 @@
 					<?php endforeach ?>
 				</td>
 
-				<td class="text-center" data-sort="<?= array_sum(array_map(function(array $x) { return count($x); }, $surns)) ?>">
+				<td class="text-center" data-sort="<?= array_sum(array_map(function (array $x) { return count($x); }, $surns)) ?>">
 					<?php foreach ($surns as $indis): ?>
 						<?= I18N::number(count($indis)) ?>
 						<br>
 					<?php endforeach ?>
 
 					<?php if (count($surns) > 1): ?>
-						<?= I18N::number(array_sum(array_map(function(array $x) { return count($x); }, $surns))) ?>
+						<?= I18N::number(array_sum(array_map(function (array $x) { return count($x); }, $surns))) ?>
 					<?php endif ?>
 				</td>
 			</tr>

Braces +5 added lines, -2 removed lines

@@ -12,7 +12,8 @@ 
 			<th>
 				<?php if ($route == 'family-list'):?>
 					<?= I18N::translate('Spouses') ?>
-				<?php else: ?>
+				<?php else {
+	: ?>
 					<?= I18N::translate('Individuals') ?>
 				<?php endif ?>
 			</th>
@@ -43,7 +44,9 @@ 
 					<?php endforeach ?>
 				</td>
 
-				<td class="text-center" data-sort="<?= array_sum(array_map(function(array $x) { return count($x); }, $surns)) ?>">
+				<td class="text-center" data-sort="<?= array_sum(array_map(function(array $x) { return count($x);
+}
+}, $surns)) ?>">
 					<?php foreach ($surns as $indis): ?>
 						<?= I18N::number(count($indis)) ?>
 						<br>

resources/views/lists/individuals-table.php

Braces +5 added lines, -2 removed lines

@@ -319,8 +319,11 @@
 					<?php if (!isset($unique_indis[$individual->getXref()]) && $age_at_death >= 0 && $age_at_death <= $max_age): ?>
 						<?php $deat_by_age[$age_at_death] .= $individual->getSex(); ?>
 					<?php endif ?>
-				<?php else: ?>
-					<?php $age_at_death = ''; ?>
+				<?php else {
+	: ?>
+					<?php $age_at_death = '';
+}
+?>
 					<?php $age_at_death_sort = PHP_INT_MAX; ?>
 				<?php endif ?>
 				<td class="center" data-sort="<?= e($age_at_death_sort) ?>">

resources/views/source-page.php

Spacing +2 added lines, -2 removed lines

@@ -8,13 +8,13 @@
 
 <?php if ($source->isPendingDeletion()): ?>
 	<?php if (Auth::isModerator($source->getTree())): ?>
-		<?= view('alerts/warning-dissmissible', ['alert' => /* I18N: %1$s is “accept”, %2$s is “reject”. These are links. */ I18N::translate( 'This source has been deleted. You should review the deletion and then %1$s or %2$s it.', '<a href="#" class="alert-link" onclick="accept_changes(\'' . e($source->getXref()) . '\', \'' . e($source->getTree()->getName()) . '\');">' . I18N::translateContext('You should review the deletion and then accept or reject it.', 'accept') . '</a>', '<a href="#" class="alert-link" onclick="reject_changes(\'' . e($source->getXref()) . '\', \'' . e($source->getTree()->getName()) . '\');">' . I18N::translateContext('You should review the deletion and then accept or reject it.', 'reject') . '</a>') . ' ' . FunctionsPrint::helpLink('pending_changes')]) ?>
+		<?= view('alerts/warning-dissmissible', ['alert' => /* I18N: %1$s is “accept”, %2$s is “reject”. These are links. */ I18N::translate('This source has been deleted. You should review the deletion and then %1$s or %2$s it.', '<a href="#" class="alert-link" onclick="accept_changes(\'' . e($source->getXref()) . '\', \'' . e($source->getTree()->getName()) . '\');">' . I18N::translateContext('You should review the deletion and then accept or reject it.', 'accept') . '</a>', '<a href="#" class="alert-link" onclick="reject_changes(\'' . e($source->getXref()) . '\', \'' . e($source->getTree()->getName()) . '\');">' . I18N::translateContext('You should review the deletion and then accept or reject it.', 'reject') . '</a>') . ' ' . FunctionsPrint::helpLink('pending_changes')]) ?>
 	<?php elseif (Auth::isEditor($source->getTree())): ?>
 		<?= view('alerts/warning-dissmissible', ['alert' => I18N::translate('This source has been deleted. The deletion will need to be reviewed by a moderator.') . ' ' . FunctionsPrint::helpLink('pending_changes')]) ?>
 	<?php endif ?>
 <?php elseif ($source->isPendingAddition()): ?>
 	<?php if (Auth::isModerator($source->getTree())): ?>
-		<?= view('alerts/warning-dissmissible', ['alert' => /* I18N: %1$s is “accept”, %2$s is “reject”. These are links. */ I18N::translate( 'This source has been edited. You should review the changes and then %1$s or %2$s them.', '<a href="#" class="alert-link" onclick="accept_changes(\'' . e($source->getXref()) . '\', \'' . e($source->getTree()->getName()) . '\');">' . I18N::translateContext('You should review the changes and then accept or reject them.', 'accept') . '</a>', '<a href="#" class="alert-link" onclick="reject_changes(\'' . $source->getXref() . '\');">' . I18N::translateContext('You should review the changes and then accept or reject them.', 'reject') . '</a>' ) . ' ' . FunctionsPrint::helpLink('pending_changes')]) ?>
+		<?= view('alerts/warning-dissmissible', ['alert' => /* I18N: %1$s is “accept”, %2$s is “reject”. These are links. */ I18N::translate('This source has been edited. You should review the changes and then %1$s or %2$s them.', '<a href="#" class="alert-link" onclick="accept_changes(\'' . e($source->getXref()) . '\', \'' . e($source->getTree()->getName()) . '\');">' . I18N::translateContext('You should review the changes and then accept or reject them.', 'accept') . '</a>', '<a href="#" class="alert-link" onclick="reject_changes(\'' . $source->getXref() . '\');">' . I18N::translateContext('You should review the changes and then accept or reject them.', 'reject') . '</a>') . ' ' . FunctionsPrint::helpLink('pending_changes')]) ?>
 	<?php elseif (Auth::isEditor($source->getTree())): ?>
 		<?= view('alerts/warning-dissmissible', ['alert' => I18N::translate('This source has been edited. The changes need to be reviewed by a moderator.') . ' ' . FunctionsPrint::helpLink('pending_changes')]) ?>
 	<?php endif ?>