fisharebest / webtrees

resources/views/admin/trees.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 				</div>
 				<div id="card-tree-content-<?= $managed_tree->getTreeId() ?>" class="collapse<?= $managed_tree == $tree || $managed_tree->getPreference('imported') === '0' ? ' show' : '' ?>" role="tabpanel" aria-labelledby="panel-tree-header-<?= $managed_tree->getTreeId() ?>">
 					<div class="card-body">
-						<?php $importing = Database::prepare( "SELECT 1 FROM `##gedcom_chunk` WHERE gedcom_id = ? AND imported = '0' LIMIT 1" )->execute([$managed_tree->getTreeId()])->fetchOne() ?>
+						<?php $importing = Database::prepare("SELECT 1 FROM `##gedcom_chunk` WHERE gedcom_id = ? AND imported = '0' LIMIT 1")->execute([$managed_tree->getTreeId()])->fetchOne() ?>
 						<?php if ($importing): ?>
 							<div id="import<?= $managed_tree->getTreeId() ?>" class="col-xs-12">
 								<div class="progress">

resources/views/admin/tree-import.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 							<select name="tree_name" class="form-control" id="import-server-file">
 								<option value=""></option>
 								<?php foreach ($gedcom_files as $gedcom_file): ?>
-									<option value="<?= e($gedcom_file) ?>" <?= $gedcom_file === $default_gedcom_file? 'selected' : '' ?>>
+									<option value="<?= e($gedcom_file) ?>" <?= $gedcom_file === $default_gedcom_file ? 'selected' : '' ?>>
 										<?= e($gedcom_file) ?>
 									</option>
 								<?php endforeach ?>

app/Module/OpenStreetMapModule.php

Spacing +52 added lines, -52 removed lines

@@ -296,7 +296,7 @@ 
 						$color            = self::LINE_COLORS[log($id, 2) % $color_count];
 						$icon['color']    = $color; //make icon color the same as the line
 						$sosa_points[$id] = $event->getLatLonJSArray();
-						$sosa_parent      = (int)floor($id / 2);
+						$sosa_parent      = (int) floor($id / 2);
 						if (array_key_exists($sosa_parent, $sosa_points)) {
 							// Would like to use a GeometryCollection to hold LineStrings
 							// rather than generate polylines but the MarkerCluster library
@@ -328,7 +328,7 @@ 
 								'modules/openstreetmap/event-sidebar',
 								$event->shortSummary($mapType, $id)
 							),
-							'zoom'     => (int)$event->getZoom(),
+							'zoom'     => (int) $event->getZoom(),
 						],
 					];
 				}
@@ -398,7 +398,7 @@ 
 							'stats'    => $placeStats,
 						]
 					),
-					'zoom'    => (int)($location->getZoom() ?? 2),
+					'zoom'    => (int) ($location->getZoom() ?? 2),
 				],
 			];
 		}
@@ -450,7 +450,7 @@ 
 		$xref        = $request->get('reference');
 		$tree        = $request->attributes->get('tree');
 		$individual  = Individual::getInstance($xref, $tree);
-		$generations = (int)$request->get(
+		$generations = (int) $request->get(
 			'generations',
 			$tree->getPreference('DEFAULT_PEDIGREE_GENERATIONS')
 		);
@@ -499,14 +499,14 @@ 
 						function ($item) {
 							return preg_replace('/[^a-z\d]/i', '', strtolower($item));
 						},
-						(array)$provider->styles
+						(array) $provider->styles
 					);
 
-					$key = preg_replace('/[^a-z\d]/i', '', strtolower((string)$provider->name));
+					$key = preg_replace('/[^a-z\d]/i', '', strtolower((string) $provider->name));
 
 					self::$map_providers[$key] = [
-						'name'   => (string)$provider->name,
-						'styles' => array_combine($style_keys, (array)$provider->styles),
+						'name'   => (string) $provider->name,
+						'styles' => array_combine($style_keys, (array) $provider->styles),
 					];
 				}
 			} catch (\Exception $ex) {
@@ -515,7 +515,7 @@ 
 					'provider' => 'openstreetmap',
 					'style'    => 'mapnik',
 				];
-				self::$map_providers  = [
+				self::$map_providers = [
 					'openstreetmap' => [
 						'name'   => 'OpenStreetMap',
 						'styles' => ['mapnik' => 'Mapnik'],
@@ -568,7 +568,7 @@ 
 		$maxgenerations = $tree->getPreference('MAX_PEDIGREE_GENERATIONS');
 		$generations    = $request->get('generations', $tree->getPreference('DEFAULT_PEDIGREE_GENERATIONS'));
 
-		return (object)[
+		return (object) [
 			'name' => 'modules/openstreetmap/pedigreemap',
 			'data' => [
 				'assets'         => $this->assets(),
@@ -631,7 +631,7 @@ 
 	 * @return object
 	 */
 	public function getAdminConfigAction(Request $request) {
-		return (object)[
+		return (object) [
 			'name' => 'modules/openstreetmap/admin-config',
 			'data' => [
 				'title'        => I18N::translate('Open Street Maps (Configuration)'),
@@ -687,8 +687,8 @@ 
 	 * @throws \Exception
 	 */
 	public function getAdminPlacesAction(Request $request) {
-		$parent_id   = (int)$request->get('parent_id', 0);
-		$inactive    = (bool)$request->get('inactive');
+		$parent_id   = (int) $request->get('parent_id', 0);
+		$inactive    = (bool) $request->get('inactive');
 		$hierarchy   = $this->gethierarchy($parent_id);
 		$title       = I18N::translate('Open Street Maps (Geographic data)');
 		$breadcrumbs = [
@@ -711,7 +711,7 @@ 
 		}
 		$breadcrumbs[] = array_pop($breadcrumbs);
 
-		return (object)[
+		return (object) [
 			'name' => 'modules/openstreetmap/admin-places',
 			'data' => [
 				'title'       => $title,
@@ -732,9 +732,9 @@ 
 	 * @throws \Exception
 	 */
 	public function getAdminPlaceEditAction(Request $request) {
-		$parent_id = (int)$request->get('parent_id', 0);
-		$place_id  = (int)$request->get('place_id');
-		$inactive  = (int)$request->get('inactive');
+		$parent_id = (int) $request->get('parent_id', 0);
+		$place_id  = (int) $request->get('place_id');
+		$inactive  = (int) $request->get('inactive');
 		$hierarchy = $this->gethierarchy($place_id);
 		$fqpn      = empty($hierarchy) ? '' : $hierarchy[0]->fqpn;
 		$location  = new Location($fqpn);
@@ -770,7 +770,7 @@ 
 		}
 		$breadcrumbs[] = $place_id === 0 ? I18N::translate('Add') : I18N::translate('Edit');
 
-		return (object)[
+		return (object) [
 			'name' => 'modules/openstreetmap/admin-place-edit',
 			'data' => [
 				'module'      => $this->getName(),
@@ -796,9 +796,9 @@ 
 	 */
 	public function postAdminSaveAction(Request $request): RedirectResponse {
 		if (Filter::checkCsrf()) {
-			$parent_id = (int)$request->get('parent_id');
-			$place_id  = (int)$request->get('place_id');
-			$inactive  = (int)$request->get('inactive');
+			$parent_id = (int) $request->get('parent_id');
+			$place_id  = (int) $request->get('place_id');
+			$inactive  = (int) $request->get('inactive');
 			$lat       = round($request->get('new_place_lati'), 5); // 5 decimal places (locate to within about 1 metre)
 			$lat       = ($lat < 0 ? 'S' : 'N') . abs($lat);
 			$lng       = round($request->get('new_place_long'), 5);
@@ -817,7 +817,7 @@ 
 						  VALUES (:id, :parent, :level, :place, :lng, :lat, :zoom, :icon)"
 				)->execute(
 					[
-						'id'     => (int)Database::prepare("SELECT MAX(pl_id)+1 FROM `##placelocation`")->fetchOne(),
+						'id'     => (int) Database::prepare("SELECT MAX(pl_id)+1 FROM `##placelocation`")->fetchOne(),
 						'parent' => $parent_id,
 						'level'  => $level,
 						'place'  => $request->get('new_place_name'),
@@ -836,7 +836,7 @@ 
 						'place' => $request->get('new_place_name'),
 						'lat'   => $request->get('lati_control') . $lat,
 						'lng'   => $request->get('long_control') . $lng,
-						'zoom'  => (int)$request->get('new_zoom_factor'),
+						'zoom'  => (int) $request->get('new_zoom_factor'),
 						'icon'  => $icon,
 					]
 				);
@@ -869,9 +869,9 @@ 
 	 */
 	public function postAdminDeleteRecordAction(Request $request): RedirectResponse {
 		if (Filter::checkCsrf()) {
-			$place_id  = (int)$request->get('place_id');
-			$parent_id = (int)$request->get('parent_id');
-			$inactive  = (int)$request->get('inactive');
+			$place_id  = (int) $request->get('place_id');
+			$parent_id = (int) $request->get('parent_id');
+			$inactive  = (int) $request->get('inactive');
 
 			try {
 				Database::prepare(
@@ -890,7 +890,7 @@ 
 				);
 			}
 			// If after deleting there are no more places at this level then go up a level
-			$children = (int)Database::prepare(
+			$children = (int) Database::prepare(
 				"SELECT COUNT(pl_id) FROM `##placelocation` WHERE pl_parent_id = :parent_id"
 			)
 				->execute(['parent_id' => $parent_id])
@@ -923,9 +923,9 @@ 
 	 * @throws \Exception
 	 */
 	public function getAdminExportAction(Request $request): RedirectResponse {
-		$parent_id = (int)$request->get('parent_id');
+		$parent_id = (int) $request->get('parent_id');
 		$format    = $request->get('format', 'csv');
-		$maxlevel  = (int)Database::prepare("SELECT max(pl_level) FROM `##placelocation`")->execute()->fetchOne();
+		$maxlevel  = (int) Database::prepare("SELECT max(pl_level) FROM `##placelocation`")->execute()->fetchOne();
 		$startfqpn = [];
 		$hierarchy = $this->gethierarchy($parent_id);
 		$geojson   = [];
@@ -977,8 +977,8 @@ 
 							)
 						)
 					);
-					$long = (float)strtr($place['pl_long'], ['E' => '', 'W' => '-', ',' => '.']);
-					$lati = (float)strtr($place['pl_lati'], ['N' => '', 'S' => '-', ',' => '.']);
+					$long = (float) strtr($place['pl_long'], ['E' => '', 'W' => '-', ',' => '.']);
+					$lati = (float) strtr($place['pl_lati'], ['N' => '', 'S' => '-', ',' => '.']);
 
 					$geojson['features'][] = [
 						'type'       => 'Feature',
@@ -1014,8 +1014,8 @@ 
 	 * @return object
 	 */
 	public function getAdminImportFormAction(Request $request) {
-		$parent_id   = (int)$request->get('parent_id');
-		$inactive    = (int)$request->get('inactive');
+		$parent_id   = (int) $request->get('parent_id');
+		$inactive    = (int) $request->get('inactive');
 		$breadcrumbs = [
 			route('admin-control-panel') => I18N::translate('Control panel'),
 			route('admin-modules')       => I18N::translate('Module administration'),
@@ -1030,7 +1030,7 @@ 
 			)                            => $this->getTitle() . ' (' . I18N::translate('Geographic data') . ')',
 			I18N::translate('Import file'),
 		];
-		$files       = $this->findFiles(WT_MODULES_DIR . $this->getName() . '/extra', ['csv', 'geojson', 'json']);
+		$files = $this->findFiles(WT_MODULES_DIR . $this->getName() . '/extra', ['csv', 'geojson', 'json']);
 		uasort(
 			$files,
 			function ($a, $b) {
@@ -1041,7 +1041,7 @@ 
 			}
 		);
 
-		return (object)[
+		return (object) [
 			'name' => 'modules/openstreetmap/admin-import-form',
 			'data' => [
 				'title'       => I18N::translate('Import geographic data'),
@@ -1105,8 +1105,8 @@ 
 					$row    = explode($delimiter, $input_array[0]);
 					$fields = count($row);
 					if ($fields >= 6 &&
-					    (bool)preg_match("/[SN][0-9]*\.?[0-9]*/", $row[$fields - 3]) &&
-					    (bool)preg_match("/[EW][0-9]*\.?[0-9]*/", $row[$fields - 4])) {
+					    (bool) preg_match("/[SN][0-9]*\.?[0-9]*/", $row[$fields - 3]) &&
+					    (bool) preg_match("/[EW][0-9]*\.?[0-9]*/", $row[$fields - 4])) {
 						$filetype = 'csv';
 					}
 				}
@@ -1146,7 +1146,7 @@ 
 							}
 						);
 						// convert separate place fields into a comma separated placename
-						$row[]    = implode(
+						$row[] = implode(
 							Place::GEDCOM_SEPARATOR,
 							array_filter(
 								array_reverse(
@@ -1162,7 +1162,7 @@ 
 			}
 
 			if ($filetype !== '?') {
-				if ((bool)$request->get('cleardatabase')) {
+				if ((bool) $request->get('cleardatabase')) {
 					Database::exec("TRUNCATE TABLE `##placelocation`");
 				}
 				//process places
@@ -1173,10 +1173,10 @@ 
 				usort(
 					$places,
 					function (array $a, array $b) {
-						if ((int)$a['pl_level'] === (int)$b['pl_level']) {
+						if ((int) $a['pl_level'] === (int) $b['pl_level']) {
 							return I18N::strcasecmp($a['fqpn'], $b['fqpn']);
 						} else {
-							return (int)$a['pl_level'] - (int)$b['pl_level'];
+							return (int) $a['pl_level'] - (int) $b['pl_level'];
 						}
 					}
 				);
@@ -1196,7 +1196,7 @@ 
 						)) {
 
 						// overwrite
-						$location->update((object)$place);
+						$location->update((object) $place);
 						$updated++;
 					} elseif (!$valid && $options !== 'update') {
 						//add
@@ -1266,7 +1266,7 @@ 
 	 */
 	public function postAdminImportPlacesAction(Request $request): RedirectResponse {
 		$gedcomName = $request->get('ged');
-		$inactive   = (int)$request->get('inactive');
+		$inactive   = (int) $request->get('inactive');
 		$tree       = Tree::findByName($gedcomName);
 
 		// Get all the places from the places table ...
@@ -1322,7 +1322,7 @@ 
 				"INSERT INTO `##placelocation` (pl_id, pl_parent_id, pl_level, pl_place)" .
 				" VALUES (:id, :parent, :level, :place)"
 			);
-			$checkRecordQry  = Database::prepare(
+			$checkRecordQry = Database::prepare(
 				"SELECT pl1.pl_id" .
 				" FROM	  `##placelocation` AS pl1" .
 				" LEFT JOIN `##placelocation` AS pl2 ON (pl1.pl_parent_id = pl2.pl_id)" .
@@ -1485,7 +1485,7 @@ 
 		$list = [];
 		/** @var array $rows */
 		foreach ($rows as $row) {
-			if ((bool)$row['inactive'] && !$show_inactive) {
+			if ((bool) $row['inactive'] && !$show_inactive) {
 				continue;
 			}
 
@@ -1496,20 +1496,20 @@ 
 				]
 			)->fetchOneRow();
 
-			if ((bool)$row['inactive']) {
+			if ((bool) $row['inactive']) {
 				$badge = 'danger';
-			} elseif ((int)$children->no_coord > 0) {
+			} elseif ((int) $children->no_coord > 0) {
 				$badge = 'warning';
-			} elseif ((int)$children->child_count > 0) {
+			} elseif ((int) $children->child_count > 0) {
 				$badge = 'info';
 			} else {
 				$badge = 'secondary';
 			}
 
-			$list[] = (object)array_merge(
+			$list[] = (object) array_merge(
 				$row,
 				[
-					'child_count' => (int)$children->child_count,
+					'child_count' => (int) $children->child_count,
 					'badge'       => $badge,
 				]
 			);
@@ -1531,7 +1531,7 @@ 
 			$row       = $statement->execute(['id' => $id])->fetchOneRow();
 			$fqpn[]    = $row->pl_place;
 			$row->fqpn = implode(Place::GEDCOM_SEPARATOR, $fqpn);
-			$id        = (int)$row->pl_parent_id;
+			$id        = (int) $row->pl_parent_id;
 			$arr[]     = $row;
 		}
 
@@ -1546,7 +1546,7 @@ 
 	 */
 	private function buildLevel($parent_id, $placename, &$places) {
 		$level = array_search('', $placename);
-		$rows  = (array)Database::prepare(
+		$rows  = (array) Database::prepare(
 			"SELECT pl_level, pl_id, pl_place, pl_long, pl_lati, pl_zoom, pl_icon FROM `##placelocation` WHERE pl_parent_id=? ORDER BY pl_place"
 		)
 			->execute([$parent_id])

app/Http/Controllers/ErrorController.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 			return new RedirectResponse(route('admin-trees'));
 		}
 
-		return $this->viewResponse('errors/no-tree-access', [ 'title' => '' ]);
+		return $this->viewResponse('errors/no-tree-access', ['title' => '']);
 	}
 
 	/**

app/Http/Controllers/AdminUsersController.php

Spacing +2 added lines, -2 removed lines

@@ -169,7 +169,7 @@ 
 		$args = [];
 
 		if ($search) {
-			$sql_select       .= " AND (user_name LIKE CONCAT('%', :search_1, '%') OR real_name LIKE CONCAT('%', :search_2, '%') OR email LIKE CONCAT('%', :search_3, '%'))";
+			$sql_select .= " AND (user_name LIKE CONCAT('%', :search_1, '%') OR real_name LIKE CONCAT('%', :search_2, '%') OR email LIKE CONCAT('%', :search_3, '%'))";
 			$args['search_1'] = $search;
 			$args['search_2'] = $search;
 			$args['search_3'] = $search;
@@ -198,7 +198,7 @@ 
 
 		if ($length) {
 			$user->setPreference('admin_users_page_size', $length);
-			$sql_select     .= " LIMIT :limit OFFSET :offset";
+			$sql_select .= " LIMIT :limit OFFSET :offset";
 			$args['limit']  = $length;
 			$args['offset'] = $start;
 		}

resources/views/admin/media.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 <br>
 <br>
 
-<table class="table table-bordered table-sm" id="media-table-<?= e($table_id) ?>" data-ajax="<?= e(route('admin-media-data', ['files' => $files, 'media_folder' => $media_folder, 'media_path' => $media_path, 'subfolders' => $subfolders ])) ?>">
+<table class="table table-bordered table-sm" id="media-table-<?= e($table_id) ?>" data-ajax="<?= e(route('admin-media-data', ['files' => $files, 'media_folder' => $media_folder, 'media_path' => $media_path, 'subfolders' => $subfolders])) ?>">
 	<thead>
 		<tr>
 			<th><?= I18N::translate('Media file') ?></th>

app/Http/Controllers/AdminMediaController.php

Spacing +6 added lines, -6 removed lines

@@ -152,7 +152,7 @@ 
 					"  OR   descriptive_title LIKE CONCAT('%', :search_2, '%'))" .
 					" AND   multimedia_file_refn NOT LIKE 'http://%'" .
 					" AND   multimedia_file_refn NOT LIKE 'https://%'";
-				$ARGS1   = [
+				$ARGS1 = [
 					'media_path_1' => $media_path,
 					'media_folder' => $media_folder,
 					'media_path_2' => Database::escapeLike($media_path),
@@ -169,7 +169,7 @@ 
 					" AND   multimedia_file_refn LIKE CONCAT(:media_path_3, '%')" .
 					" AND   multimedia_file_refn NOT LIKE 'http://%'" .
 					" AND   multimedia_file_refn NOT LIKE 'https://%'";
-				$ARGS2   = [
+				$ARGS2 = [
 					'media_folder' => $media_folder,
 					'media_path_3' => $media_path,
 				];
@@ -181,7 +181,7 @@ 
 					$ARGS2['media_path_4'] = Database::escapeLike($media_path);
 				}
 
-				$order   = $request->get('order', []);
+				$order = $request->get('order', []);
 				$SELECT1 .= " ORDER BY ";
 				if ($order) {
 					foreach ($order as $key => $value) {
@@ -205,7 +205,7 @@ 
 				}
 
 				if ($length > 0) {
-					$SELECT1         .= " LIMIT :length OFFSET :start";
+					$SELECT1 .= " LIMIT :length OFFSET :start";
 					$ARGS1['length'] = $length;
 					$ARGS1['start']  = $start;
 				}
@@ -238,7 +238,7 @@ 
 					" JOIN  `##media_file` USING (m_id, m_file)" .
 					" WHERE (multimedia_file_refn LIKE 'http://%' OR multimedia_file_refn LIKE 'https://%')" .
 					" AND   (multimedia_file_refn LIKE CONCAT('%', :search_1, '%') OR descriptive_title LIKE CONCAT('%', :search_2, '%'))";
-				$ARGS1   = [
+				$ARGS1 = [
 					'search_1' => Database::escapeLike($search),
 					'search_2' => Database::escapeLike($search),
 				];
@@ -274,7 +274,7 @@ 
 				}
 
 				if ($length > 0) {
-					$SELECT1         .= " LIMIT :length OFFSET :start";
+					$SELECT1 .= " LIMIT :length OFFSET :start";
 					$ARGS1['length'] = $length;
 					$ARGS1['start']  = $start;
 				}

app/Http/Controllers/AdminTreesController.php

Spacing +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
 				$download_filename . '.zip'
 			);
 		} else {
-			$response = new StreamedResponse(function() use ($tree, $exportOptions) {
+			$response = new StreamedResponse(function () use ($tree, $exportOptions) {
 				$stream = fopen('php://output', 'w');
 				FunctionsExport::exportGedcom($tree, $stream, $exportOptions);
 				fclose($stream);

app/Module/OpenStreetMap/Schema/Migration3.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 			" DROP COLUMN sv_long, " .
 			" DROP COLUMN sv_lati, " .
 			" DROP COLUMN sv_bearing, " .
-			" DROP COLUMN sv_elevation, ".
+			" DROP COLUMN sv_elevation, " .
 			" DROP COLUMN sv_zoom, " .
 			" DROP INDEX ix2, " .
 			" DROP INDEX ix3, " .
@@ -52,7 +52,7 @@ 
 			"DELETE FROM `##module_setting` WHERE module_name='openstreetmap' AND setting_name=''"
 			];
 
-		foreach($queries as $query) {
+		foreach ($queries as $query) {
 			try {
 				Database::exec($query);
 			} catch (PDOException $ex) {