fisharebest / webtrees

app/Datatables.php

Indentation +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
 		return [
 			'data-language'     => json_encode($language),
-		  'data-length-menu' => json_encode([array_keys($length_menu), array_values($length_menu)])
+			'data-length-menu' => json_encode([array_keys($length_menu), array_values($length_menu)])
 		];
 	}
 

app/Theme/AbstractTheme.php

Indentation +3 added lines, -3 removed lines

@@ -101,9 +101,9 @@
 		'info'    => 'fa fa-info-circle',
 		'warning' => 'fa fa-exclamation-circle',
 
-	  // Icons for file types
-	  'mime-application-pdf' => '',
-	  'mime-text-html'       => '',
+		// Icons for file types
+		'mime-application-pdf' => '',
+		'mime-text-html'       => '',
 
 		// Other icons
 		'mail'   => 'fa fa-envelope-o',

Spacing +2 added lines, -2 removed lines

@@ -840,7 +840,7 @@ 
 			$icons = '<div class="icons">' .
 				'<span class="iconz icon-zoomin" title="' . I18N::translate('Zoom in/out on this box.') . '"></span>' .
 				'<div class="itr"><i class="icon-pedigree"></i><div class="popup">' .
-				'<ul class="' . $personBoxClass . '">' . implode('', array_map(function(Menu $menu) {
+				'<ul class="' . $personBoxClass . '">' . implode('', array_map(function (Menu $menu) {
 					return $menu->bootstrap4();
 				}, $this->individualBoxMenu($individual))) . '</ul>' .
 				'</div>' .
@@ -892,7 +892,7 @@ 
 			$icons = '<div class="icons">' .
 				'<span class="iconz icon-zoomin" title="' . I18N::translate('Zoom in/out on this box.') . '"></span>' .
 				'<div class="itr"><i class="icon-pedigree"></i><div class="popup">' .
-				'<ul class="' . $personBoxClass . '">' . implode('', array_map(function(Menu $menu) {
+				'<ul class="' . $personBoxClass . '">' . implode('', array_map(function (Menu $menu) {
 					return $menu->bootstrap4();
 				}, $this->individualBoxMenu($individual))) . '</ul>' .
 				'</div>' .

app/Functions/FunctionsCharts.php

Spacing +3 added lines, -3 removed lines

@@ -112,7 +112,7 @@ 
 		$hfam = $husb->getPrimaryChildFamily();
 		if ($hfam) {
 			// remove the|| test for $sosa
-			echo '<td rowspan="2"><img src="' . Theme::theme()->parameter('image-hline') . '"></td><td rowspan="2"><img  src="' . Theme::theme()->parameter('image-vline') . '" width="3" height="' . ($pbheight - 14 ) . '"></td>';
+			echo '<td rowspan="2"><img src="' . Theme::theme()->parameter('image-hline') . '"></td><td rowspan="2"><img  src="' . Theme::theme()->parameter('image-vline') . '" width="3" height="' . ($pbheight - 14) . '"></td>';
 			echo '<td><img class="linea1 lined1"  src="' . Theme::theme()->parameter('image-hline') . '"></td><td>';
 			// husband’s father
 			if ($hfam && $hfam->getHusband()) {
@@ -202,7 +202,7 @@ 
 		// wife’s parents
 		$hfam = $wife->getPrimaryChildFamily();
 		if ($hfam) {
-			echo '<td rowspan="2"><img src="' . Theme::theme()->parameter('image-hline') . '"></td><td rowspan="2"><img src="' . Theme::theme()->parameter('image-vline') . '" width="3" height="' . ($pbheight - 14 ) . '"></td>';
+			echo '<td rowspan="2"><img src="' . Theme::theme()->parameter('image-hline') . '"></td><td rowspan="2"><img src="' . Theme::theme()->parameter('image-vline') . '" width="3" height="' . ($pbheight - 14) . '"></td>';
 			echo '<td><img class="linea3 lined3" src="' . Theme::theme()->parameter('image-hline') . '"></td><td>';
 			// wife’s father
 			if ($hfam && $hfam->getHusband()) {
@@ -345,7 +345,7 @@ 
 							}
 
 							if ($f == $maxfam) {
-								echo '<img height="' . ((($bheight / 2)) ) . 'px"';
+								echo '<img height="' . ((($bheight / 2))) . 'px"';
 							} else {
 								echo '<img height="' . $pbheight . 'px"';
 							}

message.php

Braces +3 added lines, -1 removed lines

@@ -123,7 +123,8 @@ 
 				<div class="form-control"><?= Html::escape(Auth::user()->getRealName()) ?></div>
 			</div>
 		</div>
-	<?php else: ?>
+	<?php else {
+	: ?>
 		<div class="form-group row">
 			<label class="col-sm-3 col-form-label" for="from-name">
 				<?= I18N::translate('Your name') ?>
@@ -181,6 +182,7 @@ 
 function recipients($to) {
 	if ($to === 'all') {
 		$recipients = User::all();
+}
 	} elseif ($to === 'last_6mo') {
 		$recipients = array_filter(User::all(), function(User $user) {
 			return $user->getPreference('sessiontime') > 0 && WT_TIMESTAMP - $user->getPreference('sessiontime') > 60 * 60 * 24 * 30 * 6;

Spacing +3 added lines, -3 removed lines

@@ -95,7 +95,7 @@ 
 $controller->restrictAccess(!in_array($to, ['all', 'never_logged', 'last_6mo']) || Auth::isAdmin());
 $controller->pageHeader();
 
-$to_names = implode(I18N::$list_separator, array_map(function(User $user) {
+$to_names = implode(I18N::$list_separator, array_map(function (User $user) {
 	return $user->getRealName();
 }, recipients($to)));
 
@@ -184,11 +184,11 @@ 
 	if ($to === 'all') {
 		$recipients = User::all();
 	} elseif ($to === 'last_6mo') {
-		$recipients = array_filter(User::all(), function(User $user) {
+		$recipients = array_filter(User::all(), function (User $user) {
 			return $user->getPreference('sessiontime') > 0 && WT_TIMESTAMP - $user->getPreference('sessiontime') > 60 * 60 * 24 * 30 * 6;
 		});
 	} elseif ($to === 'never_logged') {
-		$recipients = array_filter(User::all(), function(User $user) {
+		$recipients = array_filter(User::all(), function (User $user) {
 			return $user->getPreference('verified_by_admin') && $user->getPreference('reg_timestamp') > $user->getPreference('sessiontime');
 		});
 	} else {

app/Controller/IndividualController.php

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
 	public function getTabs() {
 		$active_tabs = Module::getActiveTabs($this->record->getTree());
 
-		return array_filter($active_tabs, function(ModuleTabInterface $tab) {
+		return array_filter($active_tabs, function (ModuleTabInterface $tab) {
 			return $tab->hasTabContent();
 		});
 	}
@@ -247,7 +247,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 = '';
 		}

app/User.php

Spacing +7 added lines, -7 removed lines

@@ -222,7 +222,7 @@ 
 			" ORDER BY real_name"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}
@@ -241,7 +241,7 @@ 
 			" ORDER BY real_name"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}
@@ -281,7 +281,7 @@ 
 			" ORDER BY real_name"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}
@@ -300,7 +300,7 @@ 
 			" ORDER BY real_name"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}
@@ -318,7 +318,7 @@ 
 			" ORDER BY real_name"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}
@@ -336,7 +336,7 @@ 
 			" ORDER BY real_name"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}
@@ -353,7 +353,7 @@ 
 			" JOIN `##session` USING (user_id)"
 		)->fetchAll();
 
-		return array_map(function($row) {
+		return array_map(function ($row) {
 			return new static($row);
 		}, $rows);
 	}

app/Controller/AdminController.php

Spacing +1 added lines, -1 removed lines

@@ -670,7 +670,7 @@
 		)->fetchAssoc();
 
 		foreach ($modules as $module) {
-			$new_status = (bool) $request->get('status-' . $module->getName())  ? 'enabled' : 'disabled';
+			$new_status = (bool) $request->get('status-' . $module->getName()) ? 'enabled' : 'disabled';
 			$old_status = $module_status[$module->getName()];
 
 			if ($new_status !== $old_status) {

Doc Comments +8 added lines, -8 removed lines

@@ -834,7 +834,7 @@ 
 	 * @param string $component
 	 * @param string $route
 	 * @param string $component_title
-	 * @param string $page_title
+	 * @param string $title
 	 *
 	 * @return Response
 	 */
@@ -919,7 +919,7 @@ 
 	/**
 	 * Count the number of pending changes in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalChanges() {
 		return Database::prepare(
@@ -933,7 +933,7 @@ 
 	/**
 	 * Count the number of families in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalFamilies() {
 		return Database::prepare(
@@ -947,7 +947,7 @@ 
 	/**
 	 * Count the number of individuals in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalIndividuals() {
 		return Database::prepare(
@@ -961,7 +961,7 @@ 
 	/**
 	 * Count the number of media objects in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalMediaObjects() {
 		return Database::prepare(
@@ -975,7 +975,7 @@ 
 	/**
 	 * Count the number of notes in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalNotes() {
 		return Database::prepare(
@@ -990,7 +990,7 @@ 
 	/**
 	 * Count the number of repositorie in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalRepositories() {
 		return Database::prepare(
@@ -1004,7 +1004,7 @@ 
 	/**
 	 * Count the number of sources in each tree.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	private function totalSources() {
 		return Database::prepare(