fisharebest / webtrees

app/Http/Controllers/SearchController.php

Spacing +39 added lines, -39 removed lines

@@ -603,7 +603,7 @@ 
 
 		foreach ($search_terms as $n => $q) {
 			$queryregex[]          = preg_quote(I18N::strtoupper($q), '/');
-			$sql                   .= " AND f_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
+			$sql .= " AND f_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
 			$args['collate_' . $n] = I18N::collation();
 			$args['query_' . $n]   = Database::escapeLike($q);
 		}
@@ -650,7 +650,7 @@ 
 	 * @return Family[]
 	 */
 	private function searchFamilyNames(array $search_terms, array $search_trees): array {
-		$sql  =
+		$sql =
 			"SELECT DISTINCT f_id AS xref, f_file AS gedcom_id, f_gedcom AS gedcom" .
 			" FROM `##families`" .
 			" LEFT JOIN `##name` husb ON f_husb = husb.n_id AND f_file = husb.n_file" .
@@ -659,7 +659,7 @@ 
 		$args = [];
 
 		foreach ($search_terms as $n => $q) {
-			$sql                        .= " AND (husb.n_full COLLATE :husb_collate_" . $n . " LIKE CONCAT('%', :husb_query_" . $n . ", '%') OR wife.n_full COLLATE :wife_collate_" . $n . " LIKE CONCAT('%', :wife_query_" . $n . ", '%'))";
+			$sql .= " AND (husb.n_full COLLATE :husb_collate_" . $n . " LIKE CONCAT('%', :husb_query_" . $n . ", '%') OR wife.n_full COLLATE :wife_collate_" . $n . " LIKE CONCAT('%', :wife_query_" . $n . ", '%'))";
 			$args['husb_collate_' . $n] = I18N::collation();
 			$args['husb_query_' . $n]   = Database::escapeLike($q);
 			$args['wife_collate_' . $n] = I18N::collation();
@@ -710,7 +710,7 @@ 
 
 		foreach ($search_terms as $n => $q) {
 			$queryregex[]          = preg_quote(I18N::strtoupper($q), '/');
-			$sql                   .= " AND i_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
+			$sql .= " AND i_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
 			$args['collate_' . $n] = I18N::collation();
 			$args['query_' . $n]   = Database::escapeLike($q);
 		}
@@ -855,7 +855,7 @@ 
 		}
 
 		// Add the where clause
-		$sql    .= " WHERE ind.i_file=?";
+		$sql .= " WHERE ind.i_file=?";
 		$bind[] = $this->tree()->getTreeId();
 
 		foreach ($fields as $field_name => $field_value) {
@@ -866,15 +866,15 @@ 
 					case 'GIVN':
 						switch ($modifiers[$field_name]) {
 							case 'EXACT':
-								$sql    .= " AND i_n.n_givn=?";
+								$sql .= " AND i_n.n_givn=?";
 								$bind[] = $field_value;
 								break;
 							case 'BEGINS':
-								$sql    .= " AND i_n.n_givn LIKE CONCAT(?, '%')";
+								$sql .= " AND i_n.n_givn LIKE CONCAT(?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'CONTAINS':
-								$sql    .= " AND i_n.n_givn LIKE CONCAT('%', ?, '%')";
+								$sql .= " AND i_n.n_givn LIKE CONCAT('%', ?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'SDX_STD':
@@ -888,7 +888,7 @@ 
 									$sql .= ' AND (' . implode(' OR ', $sdx) . ')';
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND i_n.n_givn LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND i_n.n_givn LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 								break;
@@ -904,7 +904,7 @@ 
 									$sql .= ' AND (' . implode(' OR ', $sdx) . ')';
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND i_n.n_givn LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND i_n.n_givn LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 								break;
@@ -913,15 +913,15 @@ 
 					case 'SURN':
 						switch ($modifiers[$field_name]) {
 							case 'EXACT':
-								$sql    .= " AND i_n.n_surname=?";
+								$sql .= " AND i_n.n_surname=?";
 								$bind[] = $field_value;
 								break;
 							case 'BEGINS':
-								$sql    .= " AND i_n.n_surname LIKE CONCAT(?, '%')";
+								$sql .= " AND i_n.n_surname LIKE CONCAT(?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'CONTAINS':
-								$sql    .= " AND i_n.n_surname LIKE CONCAT('%', ?, '%')";
+								$sql .= " AND i_n.n_surname LIKE CONCAT('%', ?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'SDX_STD':
@@ -935,7 +935,7 @@ 
 									$sql .= " AND (" . implode(' OR ', $sdx) . ")";
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND i_n.n_surn LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND i_n.n_surn LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 								break;
@@ -952,7 +952,7 @@ 
 									break;
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND i_n.n_surn LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND i_n.n_surn LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 						}
@@ -961,7 +961,7 @@ 
 					case '_MARNM':
 					case '_HEB':
 					case '_AKA':
-						$sql    .= " AND i_n.n_type=? AND i_n.n_full LIKE CONCAT('%', ?, '%')";
+						$sql .= " AND i_n.n_type=? AND i_n.n_full LIKE CONCAT('%', ?, '%')";
 						$bind[] = $parts[1];
 						$bind[] = $field_value;
 						break;
@@ -971,7 +971,7 @@ 
 				$date = new Date($field_value);
 				if ($date->isOK()) {
 					$delta  = 365 * ($modifiers[$field_name] ?? 0);
-					$sql    .= " AND i_d.d_fact=? AND i_d.d_julianday1>=? AND i_d.d_julianday2<=?";
+					$sql .= " AND i_d.d_fact=? AND i_d.d_julianday1>=? AND i_d.d_julianday2<=?";
 					$bind[] = $parts[0];
 					$bind[] = $date->minimumJulianDay() - $delta;
 					$bind[] = $date->maximumJulianDay() + $delta;
@@ -981,7 +981,7 @@ 
 				$date = new Date($field_value);
 				if ($date->isOK()) {
 					$delta  = 365 * $modifiers[$field_name];
-					$sql    .= " AND f_d.d_fact=? AND f_d.d_julianday1>=? AND f_d.d_julianday2<=?";
+					$sql .= " AND f_d.d_fact=? AND f_d.d_julianday1>=? AND f_d.d_julianday2<=?";
 					$bind[] = $parts[1];
 					$bind[] = $date->minimumJulianDay() - $delta;
 					$bind[] = $date->maximumJulianDay() + $delta;
@@ -989,12 +989,12 @@ 
 			} elseif ($parts[1] === 'PLAC') {
 				// *:PLAC
 				// SQL can only link a place to a person/family, not to an event.
-				$sql    .= " AND i_p.place LIKE CONCAT('%', ?, '%')";
+				$sql .= " AND i_p.place LIKE CONCAT('%', ?, '%')";
 				$bind[] = $field_value;
 			} elseif ($parts[0] === 'FAMS' && $parts[2] === 'PLAC') {
 				// FAMS:*:PLAC
 				// SQL can only link a place to a person/family, not to an event.
-				$sql    .= " AND f_p.place LIKE CONCAT('%', ?, '%')";
+				$sql .= " AND f_p.place LIKE CONCAT('%', ?, '%')";
 				$bind[] = $field_value;
 			} elseif ($parts[0] === 'FAMC' && $parts[2] === 'NAME') {
 				$table = $parts[1] === 'HUSB' ? 'f_n' : 'm_n';
@@ -1003,15 +1003,15 @@ 
 					case 'GIVN':
 						switch ($modifiers[$field_name]) {
 							case 'EXACT':
-								$sql    .= " AND {$table}.n_givn=?";
+								$sql .= " AND {$table}.n_givn=?";
 								$bind[] = $field_value;
 								break;
 							case 'BEGINS':
-								$sql    .= " AND {$table}.n_givn LIKE CONCAT(?, '%')";
+								$sql .= " AND {$table}.n_givn LIKE CONCAT(?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'CONTAINS':
-								$sql    .= " AND {$table}.n_givn LIKE CONCAT('%', ?, '%')";
+								$sql .= " AND {$table}.n_givn LIKE CONCAT('%', ?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'SDX_STD':
@@ -1025,7 +1025,7 @@ 
 									$sql .= ' AND (' . implode(' OR ', $sdx) . ')';
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND {$table}.n_givn = LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND {$table}.n_givn = LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 								break;
@@ -1042,7 +1042,7 @@ 
 									break;
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND {$table}.n_givn = LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND {$table}.n_givn = LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 						}
@@ -1050,15 +1050,15 @@ 
 					case 'SURN':
 						switch ($modifiers[$field_name]) {
 							case 'EXACT':
-								$sql    .= " AND {$table}.n_surname=?";
+								$sql .= " AND {$table}.n_surname=?";
 								$bind[] = $field_value;
 								break;
 							case 'BEGINS':
-								$sql    .= " AND {$table}.n_surname LIKE CONCAT(?, '%')";
+								$sql .= " AND {$table}.n_surname LIKE CONCAT(?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'CONTAINS':
-								$sql    .= " AND {$table}.n_surname LIKE CONCAT('%', ?, '%')";
+								$sql .= " AND {$table}.n_surname LIKE CONCAT('%', ?, '%')";
 								$bind[] = $field_value;
 								break;
 							case 'SDX_STD':
@@ -1072,7 +1072,7 @@ 
 									$sql .= ' AND (' . implode(' OR ', $sdx) . ')';
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND {$table}.n_surn = LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND {$table}.n_surn = LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 								break;
@@ -1088,7 +1088,7 @@ 
 									$sql .= ' AND (' . implode(' OR ', $sdx) . ')';
 								} else {
 									// No phonetic content? Use a substring match
-									$sql    .= " AND {$table}.n_surn = LIKE CONCAT('%', ?, '%')";
+									$sql .= " AND {$table}.n_surn = LIKE CONCAT('%', ?, '%')";
 									$bind[] = $field_value;
 								}
 								break;
@@ -1097,17 +1097,17 @@ 
 				}
 			} elseif ($parts[0] === 'FAMS') {
 				// e.g. searches for occupation, religion, note, etc.
-				$sql    .= " AND fam.f_gedcom REGEXP CONCAT('\n[0-9] ', ?, '(.*\n[0-9] CONT)* [^\n]*', ?)";
+				$sql .= " AND fam.f_gedcom REGEXP CONCAT('\n[0-9] ', ?, '(.*\n[0-9] CONT)* [^\n]*', ?)";
 				$bind[] = $parts[1];
 				$bind[] = $field_value;
 			} elseif ($parts[1] === 'TYPE') {
 				// e.g. FACT:TYPE or EVEN:TYPE
-				$sql    .= " AND ind.i_gedcom REGEXP CONCAT('\n1 ', ?, '.*(\n[2-9] .*)*\n2 TYPE .*', ?)";
+				$sql .= " AND ind.i_gedcom REGEXP CONCAT('\n1 ', ?, '.*(\n[2-9] .*)*\n2 TYPE .*', ?)";
 				$bind[] = $parts[0];
 				$bind[] = $field_value;
 			} else {
 				// e.g. searches for occupation, religion, note, etc.
-				$sql    .= " AND ind.i_gedcom REGEXP CONCAT('\n[0-9] ', ?, '(.*\n[0-9] CONT)* [^\n]*', ?)";
+				$sql .= " AND ind.i_gedcom REGEXP CONCAT('\n[0-9] ', ?, '(.*\n[0-9] CONT)* [^\n]*', ?)";
 				$bind[] = $parts[0];
 				$bind[] = $field_value;
 			}
@@ -1185,7 +1185,7 @@ 
 		$sql .= ")";
 
 		if ($givn_sdx !== '') {
-			$sql      .= " AND (";
+			$sql .= " AND (";
 			$givn_sdx = explode(':', $givn_sdx);
 			foreach ($givn_sdx as $n => $sdx) {
 				$sql .= $n > 0 ? " OR " : "";
@@ -1203,7 +1203,7 @@ 
 		}
 
 		if ($surn_sdx !== '') {
-			$sql      .= " AND (";
+			$sql .= " AND (";
 			$surn_sdx = explode(':', $surn_sdx);
 			foreach ($surn_sdx as $n => $sdx) {
 				$sql .= $n ? " OR " : "";
@@ -1221,7 +1221,7 @@ 
 		}
 
 		if ($plac_sdx !== '') {
-			$sql      .= " AND (";
+			$sql .= " AND (";
 			$plac_sdx = explode(':', $plac_sdx);
 			foreach ($plac_sdx as $n => $sdx) {
 				$sql .= $n ? " OR " : "";
@@ -1267,7 +1267,7 @@ 
 
 		foreach ($search_terms as $n => $q) {
 			$queryregex[]          = preg_quote(I18N::strtoupper($q), '/');
-			$sql                   .= " AND o_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
+			$sql .= " AND o_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
 			$args['collate_' . $n] = I18N::collation();
 			$args['query_' . $n]   = Database::escapeLike($q);
 		}
@@ -1322,7 +1322,7 @@ 
 
 		foreach ($search_terms as $n => $q) {
 			$queryregex[]          = preg_quote(I18N::strtoupper($q), '/');
-			$sql                   .= " AND o_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
+			$sql .= " AND o_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
 			$args['collate_' . $n] = I18N::collation();
 			$args['query_' . $n]   = Database::escapeLike($q);
 		}
@@ -1377,7 +1377,7 @@ 
 
 		foreach ($search_terms as $n => $q) {
 			$queryregex[]          = preg_quote(I18N::strtoupper($q), '/');
-			$sql                   .= " AND s_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
+			$sql .= " AND s_gedcom COLLATE :collate_" . $n . " LIKE CONCAT('%', :query_" . $n . ", '%')";
 			$args['collate_' . $n] = I18N::collation();
 			$args['query_' . $n]   = Database::escapeLike($q);
 		}

app/Http/Controllers/PlaceHierarchyController.php

Spacing +7 added lines, -7 removed lines

@@ -50,14 +50,14 @@ 
 		$osm_module = Module::getModuleByName(self::MAP_MODULE);
 		$method     = 'assets';
 		$showmap    = $osm_module &&
-			(bool)$osm_module->getPreference('place_hierarchy') &&
+			(bool) $osm_module->getPreference('place_hierarchy') &&
 			method_exists($osm_module, $method) &&
 			strpos($action, 'hierarchy') === 0;
 		$data       = null;
 		$note       = false;
 
 		if ($showmap) {
-			$note    = true;
+			$note = true;
 			$content .= view('modules/openstreetmap/map',
 				[
 					'assets' => $osm_module->$method(),
@@ -70,14 +70,14 @@ 
 
 		switch ($action) {
 			case 'list':
-				$nextaction      = ['hierarchy' => I18N::translate('Show place hierarchy')];
-				$content         .= view('place-list', $this->getList($tree));
+				$nextaction = ['hierarchy' => I18N::translate('Show place hierarchy')];
+				$content .= view('place-list', $this->getList($tree));
 				break;
 			case 'hierarchy':
 			case 'hierarchy-e':
 				$nextaction      = ['list' => I18N::translate('Show all places in a list')];
 				$data            = $this->getHierarchy($tree, $place, $parent);
-				$content         .= (null === $data || $showmap) ? '' : view('place-hierarchy', $data);
+				$content .= (null === $data || $showmap) ? '' : view('place-hierarchy', $data);
 				if (null === $data || $action === 'hierarchy-e') {
 					$content .= view('place-events', $this->getEvents($tree, $place));
 				}
@@ -117,7 +117,7 @@ 
 
 		return
 			[
-				'columns' => array_chunk($list_places, (int)ceil($numfound / $divisor)),
+				'columns' => array_chunk($list_places, (int) ceil($numfound / $divisor)),
 			];
 	}
 
@@ -139,7 +139,7 @@ 
 				[
 					'tree'      => $tree,
 					'col_class' => "w-" . ($divisor === 2 ? "25" : "50"),
-					'columns'   => array_chunk($child_places, (int)ceil($numfound / $divisor)),
+					'columns'   => array_chunk($child_places, (int) ceil($numfound / $divisor)),
 					'place'     => $place,
 					'parent'    => $parent,
 				];