fisharebest / webtrees

calendar.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -67,14 +67,14 @@
 	if (strlen($match[1]) > strlen($match[2])) {
 		$match[2] = substr($match[1], 0, strlen($match[1]) - strlen($match[2])) . $match[2];
 	}
-	$ged_date = new Date("FROM {$cal} {$match[1]} TO {$cal} {$match[2]}");
+	$ged_date = new Date("from {$cal} {$match[1]} TO {$cal} {$match[2]}");
 	$view     = 'year';
 } else {
 	// advanced-year "decade/century wildcard"
 	if (preg_match('/^(\d+)(\?+)$/', $year, $match)) {
 		$y1       = $match[1] . str_replace('?', '0', $match[2]);
 		$y2       = $match[1] . str_replace('?', '9', $match[2]);
-		$ged_date = new Date("FROM {$cal} {$y1} TO {$cal} {$y2}");
+		$ged_date = new Date("from {$cal} {$y1} TO {$cal} {$y2}");
 		$view     = 'year';
 	} else {
 		if ($year < 0) {

Switch Indentation +184 added lines, -184 removed lines

@@ -113,15 +113,15 @@ 
 $controller->setPageTitle(I18N::translate('Anniversary calendar'));
 
 switch ($view) {
-case 'day':
-	$controller->setPageTitle(I18N::translate('On this day…') . ' ' . $ged_date->display(false));
-	break;
-case 'month':
-	$controller->setPageTitle(I18N::translate('In this month…') . ' ' . $ged_date->display(false, '%F %Y'));
-	break;
-case 'year':
-	$controller->setPageTitle(I18N::translate('In this year…') . ' ' . $ged_date->display(false, '%Y'));
-	break;
+	case 'day':
+		$controller->setPageTitle(I18N::translate('On this day…') . ' ' . $ged_date->display(false));
+		break;
+	case 'month':
+		$controller->setPageTitle(I18N::translate('In this month…') . ' ' . $ged_date->display(false, '%F %Y'));
+		break;
+	case 'year':
+		$controller->setPageTitle(I18N::translate('In this year…') . ' ' . $ged_date->display(false, '%Y'));
+		break;
 }
 
 // Only generate the content for interactive users (not search robots).
@@ -323,38 +323,38 @@ 
 	$found_facts = [];
 
 	switch ($view) {
-	case 'day':
-		$found_facts = apply_filter(FunctionsDb::getAnniversaryEvents($cal_date->minJD, $filterev, $WT_TREE), $filterof, $filtersx);
-		break;
-	case 'month':
-		$cal_date->d = 0;
-		$cal_date->setJdFromYmd();
-		// Make a separate list for each day. Unspecified/invalid days go in day 0.
-		for ($d = 0; $d <= $days_in_month; ++$d) {
-			$found_facts[$d] = [];
-		}
-		// Fetch events for each day
-		for ($jd = $cal_date->minJD; $jd <= $cal_date->maxJD; ++$jd) {
-			foreach (apply_filter(FunctionsDb::getAnniversaryEvents($jd, $filterev, $WT_TREE), $filterof, $filtersx) as $fact) {
-				$tmp = $fact->getDate()->minimumDate();
-				if ($tmp->d >= 1 && $tmp->d <= $tmp->daysInMonth()) {
-					// If the day is valid (for its own calendar), display it in the
-					// anniversary day (for the display calendar).
-					$found_facts[$jd - $cal_date->minJD + 1][] = $fact;
-				} else {
-					// Otherwise, display it in the "Day not set" box.
-					$found_facts[0][] = $fact;
+		case 'day':
+			$found_facts = apply_filter(FunctionsDb::getAnniversaryEvents($cal_date->minJD, $filterev, $WT_TREE), $filterof, $filtersx);
+			break;
+		case 'month':
+			$cal_date->d = 0;
+			$cal_date->setJdFromYmd();
+			// Make a separate list for each day. Unspecified/invalid days go in day 0.
+			for ($d = 0; $d <= $days_in_month; ++$d) {
+				$found_facts[$d] = [];
+			}
+			// Fetch events for each day
+			for ($jd = $cal_date->minJD; $jd <= $cal_date->maxJD; ++$jd) {
+				foreach (apply_filter(FunctionsDb::getAnniversaryEvents($jd, $filterev, $WT_TREE), $filterof, $filtersx) as $fact) {
+					$tmp = $fact->getDate()->minimumDate();
+					if ($tmp->d >= 1 && $tmp->d <= $tmp->daysInMonth()) {
+						// If the day is valid (for its own calendar), display it in the
+						// anniversary day (for the display calendar).
+						$found_facts[$jd - $cal_date->minJD + 1][] = $fact;
+					} else {
+						// Otherwise, display it in the "Day not set" box.
+						$found_facts[0][] = $fact;
+					}
 				}
 			}
-		}
-		break;
-	case 'year':
-		$cal_date->m = 0;
-		$cal_date->setJdFromYmd();
-		$found_facts = apply_filter(FunctionsDb::getCalendarEvents($ged_date->minimumJulianDay(), $ged_date->maximumJulianDay(), $filterev, $WT_TREE), $filterof, $filtersx);
-		// Eliminate duplicates (e.g. BET JUL 1900 AND SEP 1900 will appear twice in 1900)
-		$found_facts = array_unique($found_facts);
-		break;
+			break;
+		case 'year':
+			$cal_date->m = 0;
+			$cal_date->setJdFromYmd();
+			$found_facts = apply_filter(FunctionsDb::getCalendarEvents($ged_date->minimumJulianDay(), $ged_date->maximumJulianDay(), $filterev, $WT_TREE), $filterof, $filtersx);
+			// Eliminate duplicates (e.g. BET JUL 1900 AND SEP 1900 will appear twice in 1900)
+			$found_facts = array_unique($found_facts);
+			break;
 	}
 
 	// Group the facts by family/individual
@@ -363,156 +363,156 @@ 
 	$cal_facts = [];
 
 	switch ($view) {
-	case 'year':
-	case 'day':
-		foreach ($found_facts as $fact) {
-			$record = $fact->getParent();
-			$xref   = $record->getXref();
-			if ($record instanceof Individual) {
-				if (empty($indis[$xref])) {
-					$indis[$xref] = calendar_fact_text($fact, true);
-				} else {
-					$indis[$xref] .= '<br>' . calendar_fact_text($fact, true);
-				}
-			} elseif ($record instanceof Family) {
-				if (empty($indis[$xref])) {
-					$fams[$xref] = calendar_fact_text($fact, true);
-				} else {
-					$fams[$xref] .= '<br>' . calendar_fact_text($fact, true);
+		case 'year':
+		case 'day':
+			foreach ($found_facts as $fact) {
+				$record = $fact->getParent();
+				$xref   = $record->getXref();
+				if ($record instanceof Individual) {
+					if (empty($indis[$xref])) {
+						$indis[$xref] = calendar_fact_text($fact, true);
+					} else {
+						$indis[$xref] .= '<br>' . calendar_fact_text($fact, true);
+					}
+				} elseif ($record instanceof Family) {
+					if (empty($indis[$xref])) {
+						$fams[$xref] = calendar_fact_text($fact, true);
+					} else {
+						$fams[$xref] .= '<br>' . calendar_fact_text($fact, true);
+					}
 				}
 			}
-		}
-		break;
-	case 'month':
-		foreach ($found_facts as $d => $facts) {
-			$cal_facts[$d] = [];
-			foreach ($facts as $fact) {
-				$xref = $fact->getParent()->getXref();
-				if (empty($cal_facts[$d][$xref])) {
-					$cal_facts[$d][$xref] = calendar_fact_text($fact, false);
-				} else {
-					$cal_facts[$d][$xref] .= '<br>' . calendar_fact_text($fact, false);
+			break;
+		case 'month':
+			foreach ($found_facts as $d => $facts) {
+				$cal_facts[$d] = [];
+				foreach ($facts as $fact) {
+					$xref = $fact->getParent()->getXref();
+					if (empty($cal_facts[$d][$xref])) {
+						$cal_facts[$d][$xref] = calendar_fact_text($fact, false);
+					} else {
+						$cal_facts[$d][$xref] .= '<br>' . calendar_fact_text($fact, false);
+					}
 				}
 			}
-		}
-		break;
+			break;
 	}
 
 	switch ($view) {
-	case 'year':
-	case 'day':
-		$males   = 0;
-		$females = 0;
-		echo '<table class="width100"><tr>';
-		echo '<td class="descriptionbox center width50"><i class="icon-indis"></i>', I18N::translate('Individuals'), '</td>';
-		echo '<td class="descriptionbox center width50"><i class="icon-cfamily"></i>', I18N::translate('Families'), '</td>';
-		echo '</tr><tr>';
-		echo '<td class="optionbox wrap">';
-
-		$content = calendar_list_text($indis, '<li>', '</li>', true);
-		if ($content) {
-			echo '<ul>', $content, '</ul>';
-		}
+		case 'year':
+		case 'day':
+			$males   = 0;
+			$females = 0;
+			echo '<table class="width100"><tr>';
+			echo '<td class="descriptionbox center width50"><i class="icon-indis"></i>', I18N::translate('Individuals'), '</td>';
+			echo '<td class="descriptionbox center width50"><i class="icon-cfamily"></i>', I18N::translate('Families'), '</td>';
+			echo '</tr><tr>';
+			echo '<td class="optionbox wrap">';
 
-		echo '</td>';
-		echo '<td class="optionbox wrap">';
+			$content = calendar_list_text($indis, '<li>', '</li>', true);
+			if ($content) {
+				echo '<ul>', $content, '</ul>';
+			}
 
-		$content = calendar_list_text($fams, '<li>', '</li>', true);
-		if ($content) {
-			echo '<ul>', $content, '</ul>';
-		}
+			echo '</td>';
+			echo '<td class="optionbox wrap">';
 
-		echo '</td>';
-		echo '</tr><tr>';
-		echo '<td class="descriptionbox">', I18N::translate('Total individuals: %s', count($indis));
-		echo '<br>';
-		echo '<i class="icon-sex_m_15x15" title="', I18N::translate('Males'), '"></i> ', $males, ' ';
-		echo '<i class="icon-sex_f_15x15" title="', I18N::translate('Females'), '"></i> ', $females, ' ';
-		if (count($indis) !== $males + $females) {
-			echo '<i class="icon-sex_u_15x15" title="', I18N::translate('All individuals'), '"></i> ', count($indis) - $males - $females;
-		}
-		echo '</td>';
-		echo '<td class="descriptionbox">', I18N::translate('Total families: %s', count($fams)), '</td>';
-		echo '</tr></table>';
+			$content = calendar_list_text($fams, '<li>', '</li>', true);
+			if ($content) {
+				echo '<ul>', $content, '</ul>';
+			}
 
-		break;
-	case 'month':
-// We use JD%7 = 0/Mon…6/Sun. Standard definitions use 0/Sun…6/Sat.
-		$week_start    = (I18N::firstDay() + 6) % 7;
-		$weekend_start = (I18N::weekendStart() + 6) % 7;
-		$weekend_end   = (I18N::weekendEnd() + 6) % 7;
-		// The french  calendar has a 10-day week, which starts on primidi
-		if ($days_in_week === 10) {
-			$week_start    = 0;
-			$weekend_start = -1;
-			$weekend_end   = -1;
-		}
-		echo '<table class="width100"><thead><tr>';
-		for ($week_day = 0; $week_day < $days_in_week; ++$week_day) {
-			$day_name = $cal_date->dayNames(($week_day + $week_start) % $days_in_week);
-			if ($week_day == $weekend_start || $week_day == $weekend_end) {
-				echo '<th class="descriptionbox weekend" width="' . (100 / $days_in_week) . '%">', $day_name, '</th>';
-			} else {
-				echo '<th class="descriptionbox" width="' . (100 / $days_in_week) . '%">', $day_name, '</th>';
+			echo '</td>';
+			echo '</tr><tr>';
+			echo '<td class="descriptionbox">', I18N::translate('Total individuals: %s', count($indis));
+			echo '<br>';
+			echo '<i class="icon-sex_m_15x15" title="', I18N::translate('Males'), '"></i> ', $males, ' ';
+			echo '<i class="icon-sex_f_15x15" title="', I18N::translate('Females'), '"></i> ', $females, ' ';
+			if (count($indis) !== $males + $females) {
+				echo '<i class="icon-sex_u_15x15" title="', I18N::translate('All individuals'), '"></i> ', count($indis) - $males - $females;
 			}
-		}
-		echo '</tr>';
-		echo '</thead>';
-		echo '<tbody>';
-		// Print days 1 to n of the month, but extend to cover "empty" days before/after the month to make whole weeks.
-		// e.g. instead of 1 -> 30 (=30 days), we might have -1 -> 33 (=35 days)
-		$start_d = 1 - ($cal_date->minJD - $week_start) % $days_in_week;
-		$end_d   = $days_in_month + ($days_in_week - ($cal_date->maxJD - $week_start + 1) % $days_in_week) % $days_in_week;
-		// Make sure that there is an empty box for any leap/missing days
-		if ($start_d === 1 && $end_d === $days_in_month && count($found_facts[0]) > 0) {
-			$end_d += $days_in_week;
-		}
-		for ($d = $start_d; $d <= $end_d; ++$d) {
-			if (($d + $cal_date->minJD - $week_start) % $days_in_week === 1) {
-				echo '<tr>';
+			echo '</td>';
+			echo '<td class="descriptionbox">', I18N::translate('Total families: %s', count($fams)), '</td>';
+			echo '</tr></table>';
+
+			break;
+		case 'month':
+	// We use JD%7 = 0/Mon…6/Sun. Standard definitions use 0/Sun…6/Sat.
+			$week_start    = (I18N::firstDay() + 6) % 7;
+			$weekend_start = (I18N::weekendStart() + 6) % 7;
+			$weekend_end   = (I18N::weekendEnd() + 6) % 7;
+			// The french  calendar has a 10-day week, which starts on primidi
+			if ($days_in_week === 10) {
+				$week_start    = 0;
+				$weekend_start = -1;
+				$weekend_end   = -1;
 			}
-			echo '<td class="optionbox wrap">';
-			if ($d < 1 || $d > $days_in_month) {
-				if (count($cal_facts[0]) > 0) {
-					echo '<span class="cal_day">', I18N::translate('Day not set'), '</span><br style="clear: both;">';
-					echo '<div class="details1" style="height: 180px; overflow: auto;">';
-					echo calendar_list_text($cal_facts[0], '', '', false);
-					echo '</div>';
-					$cal_facts[0] = [];
-				}
-			} else {
-				// Format the day number using the calendar
-				$tmp   = new Date($cal_date->format("%@ {$d} %O %E"));
-				$d_fmt = $tmp->minimumDate()->format('%j');
-				if ($d === $today->d && $cal_date->m === $today->m) {
-					echo '<span class="cal_day current_day">', $d_fmt, '</span>';
+			echo '<table class="width100"><thead><tr>';
+			for ($week_day = 0; $week_day < $days_in_week; ++$week_day) {
+				$day_name = $cal_date->dayNames(($week_day + $week_start) % $days_in_week);
+				if ($week_day == $weekend_start || $week_day == $weekend_end) {
+					echo '<th class="descriptionbox weekend" width="' . (100 / $days_in_week) . '%">', $day_name, '</th>';
 				} else {
-					echo '<span class="cal_day">', $d_fmt, '</span>';
+					echo '<th class="descriptionbox" width="' . (100 / $days_in_week) . '%">', $day_name, '</th>';
 				}
-				// Show a converted date
-				foreach (explode('_and_', $CALENDAR_FORMAT) as $convcal) {
-					switch ($convcal) {
-					case 'french':
-						$alt_date = new FrenchDate($cal_date->minJD + $d - 1);
-						break;
-					case 'gregorian':
-						$alt_date = new GregorianDate($cal_date->minJD + $d - 1);
-						break;
-					case 'jewish':
-						$alt_date = new JewishDate($cal_date->minJD + $d - 1);
-						break;
-					case 'julian':
-						$alt_date = new JulianDate($cal_date->minJD + $d - 1);
-						break;
-					case 'hijri':
-						$alt_date = new HijriDate($cal_date->minJD + $d - 1);
-						break;
-					case 'jalali':
-						$alt_date = new JalaliDate($cal_date->minJD + $d - 1);
-						break;
-					default:
-						break 2;
+			}
+			echo '</tr>';
+			echo '</thead>';
+			echo '<tbody>';
+			// Print days 1 to n of the month, but extend to cover "empty" days before/after the month to make whole weeks.
+			// e.g. instead of 1 -> 30 (=30 days), we might have -1 -> 33 (=35 days)
+			$start_d = 1 - ($cal_date->minJD - $week_start) % $days_in_week;
+			$end_d   = $days_in_month + ($days_in_week - ($cal_date->maxJD - $week_start + 1) % $days_in_week) % $days_in_week;
+			// Make sure that there is an empty box for any leap/missing days
+			if ($start_d === 1 && $end_d === $days_in_month && count($found_facts[0]) > 0) {
+				$end_d += $days_in_week;
+			}
+			for ($d = $start_d; $d <= $end_d; ++$d) {
+				if (($d + $cal_date->minJD - $week_start) % $days_in_week === 1) {
+					echo '<tr>';
+				}
+				echo '<td class="optionbox wrap">';
+				if ($d < 1 || $d > $days_in_month) {
+					if (count($cal_facts[0]) > 0) {
+						echo '<span class="cal_day">', I18N::translate('Day not set'), '</span><br style="clear: both;">';
+						echo '<div class="details1" style="height: 180px; overflow: auto;">';
+						echo calendar_list_text($cal_facts[0], '', '', false);
+						echo '</div>';
+						$cal_facts[0] = [];
+					}
+				} else {
+					// Format the day number using the calendar
+					$tmp   = new Date($cal_date->format("%@ {$d} %O %E"));
+					$d_fmt = $tmp->minimumDate()->format('%j');
+					if ($d === $today->d && $cal_date->m === $today->m) {
+						echo '<span class="cal_day current_day">', $d_fmt, '</span>';
+					} else {
+						echo '<span class="cal_day">', $d_fmt, '</span>';
 					}
+					// Show a converted date
+					foreach (explode('_and_', $CALENDAR_FORMAT) as $convcal) {
+						switch ($convcal) {
+							case 'french':
+								$alt_date = new FrenchDate($cal_date->minJD + $d - 1);
+								break;
+							case 'gregorian':
+								$alt_date = new GregorianDate($cal_date->minJD + $d - 1);
+								break;
+							case 'jewish':
+								$alt_date = new JewishDate($cal_date->minJD + $d - 1);
+								break;
+							case 'julian':
+								$alt_date = new JulianDate($cal_date->minJD + $d - 1);
+								break;
+							case 'hijri':
+								$alt_date = new HijriDate($cal_date->minJD + $d - 1);
+								break;
+							case 'jalali':
+								$alt_date = new JalaliDate($cal_date->minJD + $d - 1);
+								break;
+							default:
+								break 2;
+						}
 					if (get_class($alt_date) !== get_class($cal_date) && $alt_date->inValidRange()) {
 						echo '<span class="rtl_cal_day">' . $alt_date->format('%j %M') . '</span>';
 						// Just show the first conversion
@@ -643,17 +643,17 @@ 
 		$html .= $tag1 . '<a href="' . $tmp->getHtmlUrl() . '">' . $tmp->getFullName() . '</a> ';
 		if ($show_sex_symbols && $tmp instanceof Individual) {
 			switch ($tmp->getSex()) {
-			case 'M':
-				$html .= '<i class="icon-sex_m_9x9" title="' . I18N::translate('Male') . '"></i>';
-				++$males;
-				break;
-			case 'F':
-				$html .= '<i class="icon-sex_f_9x9" title="' . I18N::translate('Female') . '"></i>';
-				++$females;
-				break;
-			default:
-				$html .= '<i class="icon-sex_u_9x9" title="' . I18N::translateContext('unknown gender', 'Unknown') . '"></i>';
-				break;
+				case 'M':
+					$html .= '<i class="icon-sex_m_9x9" title="' . I18N::translate('Male') . '"></i>';
+					++$males;
+					break;
+				case 'F':
+					$html .= '<i class="icon-sex_f_9x9" title="' . I18N::translate('Female') . '"></i>';
+					++$females;
+					break;
+				default:
+					$html .= '<i class="icon-sex_u_9x9" title="' . I18N::translateContext('unknown gender', 'Unknown') . '"></i>';
+					break;
 			}
 		}
 		$html .= '<div class="indent">' . $facts . '</div>' . $tag2;

admin_users.php

Braces +3 added lines, -1 removed lines

@@ -770,13 +770,15 @@
 		<p>
 		<?php if ($ucnt): ?>
 			<input type="submit" value="<?= I18N::translate('delete') ?>">
-			<?php else: ?>
+			<?php else {
+	: ?>
 			<?= I18N::translate('Nothing found to cleanup') ?>
 			<?php endif ?>
 		</p>
 	</form>
 	<?php
 	break;
+}
 
 case 'cleanup2':
 	foreach (User::all() as $user) {

Switch Indentation +97 added lines, -97 removed lines

@@ -41,108 +41,108 @@ 
 
 // Form actions
 switch (Filter::post('action')) {
-case 'save':
-	if (Filter::checkCsrf()) {
-		$user_id        = Filter::postInteger('user_id');
-		$user           = User::find($user_id);
-		$username       = Filter::post('username');
-		$real_name      = Filter::post('real_name');
-		$email          = Filter::post('email');
-		$pass1          = Filter::post('pass1', WT_REGEX_PASSWORD);
-		$pass2          = Filter::post('pass2', WT_REGEX_PASSWORD);
-		$theme          = Filter::post('theme', implode('|', array_keys(Theme::themeNames())), '');
-		$language       = Filter::post('language');
-		$timezone       = Filter::post('timezone');
-		$contact_method = Filter::post('contact_method');
-		$comment        = Filter::post('comment');
-		$auto_accept    = Filter::postBool('auto_accept');
-		$canadmin       = Filter::postBool('canadmin');
-		$visible_online = Filter::postBool('visible_online');
-		$verified       = Filter::postBool('verified');
-		$approved       = Filter::postBool('approved');
-
-		if ($user_id === 0) {
-			// Create a new user
-			if (User::findByUserName($username)) {
-				FlashMessages::addMessage(I18N::translate('Duplicate username. A user with that username already exists. Please choose another username.'));
-			} elseif (User::findByEmail($email)) {
-				FlashMessages::addMessage(I18N::translate('Duplicate email address. A user with that email already exists.'));
-			} elseif ($pass1 !== $pass2) {
-				FlashMessages::addMessage(I18N::translate('The passwords do not match.'));
+	case 'save':
+		if (Filter::checkCsrf()) {
+			$user_id        = Filter::postInteger('user_id');
+			$user           = User::find($user_id);
+			$username       = Filter::post('username');
+			$real_name      = Filter::post('real_name');
+			$email          = Filter::post('email');
+			$pass1          = Filter::post('pass1', WT_REGEX_PASSWORD);
+			$pass2          = Filter::post('pass2', WT_REGEX_PASSWORD);
+			$theme          = Filter::post('theme', implode('|', array_keys(Theme::themeNames())), '');
+			$language       = Filter::post('language');
+			$timezone       = Filter::post('timezone');
+			$contact_method = Filter::post('contact_method');
+			$comment        = Filter::post('comment');
+			$auto_accept    = Filter::postBool('auto_accept');
+			$canadmin       = Filter::postBool('canadmin');
+			$visible_online = Filter::postBool('visible_online');
+			$verified       = Filter::postBool('verified');
+			$approved       = Filter::postBool('approved');
+
+			if ($user_id === 0) {
+				// Create a new user
+				if (User::findByUserName($username)) {
+					FlashMessages::addMessage(I18N::translate('Duplicate username. A user with that username already exists. Please choose another username.'));
+				} elseif (User::findByEmail($email)) {
+					FlashMessages::addMessage(I18N::translate('Duplicate email address. A user with that email already exists.'));
+				} elseif ($pass1 !== $pass2) {
+					FlashMessages::addMessage(I18N::translate('The passwords do not match.'));
+				} else {
+					$user = User::create($username, $real_name, $email, $pass1);
+					$user->setPreference('reg_timestamp', date('U'))->setPreference('sessiontime', '0');
+					Log::addAuthenticationLog('User ->' . $username . '<- created');
+				}
 			} else {
-				$user = User::create($username, $real_name, $email, $pass1);
-				$user->setPreference('reg_timestamp', date('U'))->setPreference('sessiontime', '0');
-				Log::addAuthenticationLog('User ->' . $username . '<- created');
-			}
-		} else {
-			$user = User::find($user_id);
-			if ($user && $username && $real_name) {
-				$user->setEmail($email);
-				$user->setUserName($username);
-				$user->setRealName($real_name);
-				if ($pass1 !== null && $pass1 === $pass2) {
-					$user->setPassword($pass1);
+				$user = User::find($user_id);
+				if ($user && $username && $real_name) {
+					$user->setEmail($email);
+					$user->setUserName($username);
+					$user->setRealName($real_name);
+					if ($pass1 !== null && $pass1 === $pass2) {
+						$user->setPassword($pass1);
+					}
 				}
 			}
-		}
 
-		if ($user) {
-			// Approving for the first time? Send a confirmation email
-			if ($approved && !$user->getPreference('verified_by_admin') && $user->getPreference('sessiontime') == 0) {
-				I18N::init($user->getPreference('language'));
-
-				// Create a dummy user, so we can send messages from the tree.
-				$sender = new User(
-					(object) [
-						'user_id'   => null,
-						'user_name' => '',
-						'real_name' => $WT_TREE->getTitle(),
-						'email'     => $WT_TREE->getPreference('WEBTREES_EMAIL'),
-					]
-				);
-
-				Mail::send(
-					$sender,
-					$user,
-					$sender,
-					I18N::translate('Approval of account at %s', WT_BASE_URL),
-					View::make('emails/approve-user-text', ['user' => $user]),
-					View::make('emails/approve-user-html', ['user' => $user])
-				);
-			}
+			if ($user) {
+				// Approving for the first time? Send a confirmation email
+				if ($approved && !$user->getPreference('verified_by_admin') && $user->getPreference('sessiontime') == 0) {
+					I18N::init($user->getPreference('language'));
+
+					// Create a dummy user, so we can send messages from the tree.
+					$sender = new User(
+						(object) [
+							'user_id'   => null,
+							'user_name' => '',
+							'real_name' => $WT_TREE->getTitle(),
+							'email'     => $WT_TREE->getPreference('WEBTREES_EMAIL'),
+						]
+					);
+
+					Mail::send(
+						$sender,
+						$user,
+						$sender,
+						I18N::translate('Approval of account at %s', WT_BASE_URL),
+						View::make('emails/approve-user-text', ['user' => $user]),
+						View::make('emails/approve-user-html', ['user' => $user])
+					);
+				}
 
-			$user
-				->setPreference('theme', $theme)
-				->setPreference('language', $language)
-				->setPreference('TIMEZONE', $timezone)
-				->setPreference('contactmethod', $contact_method)
-				->setPreference('comment', $comment)
-				->setPreference('auto_accept', $auto_accept ? '1' : '0')
-				->setPreference('visibleonline', $visible_online ? '1' : '0')
-				->setPreference('verified', $verified ? '1' : '0')
-				->setPreference('verified_by_admin', $approved ? '1' : '0');
-
-			// We cannot change our own admin status. Another admin will need to do it.
-			if ($user->getUserId() !== Auth::id()) {
-				$user->setPreference('canadmin', $canadmin ? '1' : '0');
-			}
+				$user
+					->setPreference('theme', $theme)
+					->setPreference('language', $language)
+					->setPreference('TIMEZONE', $timezone)
+					->setPreference('contactmethod', $contact_method)
+					->setPreference('comment', $comment)
+					->setPreference('auto_accept', $auto_accept ? '1' : '0')
+					->setPreference('visibleonline', $visible_online ? '1' : '0')
+					->setPreference('verified', $verified ? '1' : '0')
+					->setPreference('verified_by_admin', $approved ? '1' : '0');
+
+				// We cannot change our own admin status. Another admin will need to do it.
+				if ($user->getUserId() !== Auth::id()) {
+					$user->setPreference('canadmin', $canadmin ? '1' : '0');
+				}
 
-			foreach (Tree::getAll() as $tree) {
-				$tree->setUserPreference($user, 'gedcomid', Filter::post('gedcomid' . $tree->getTreeId(), WT_REGEX_XREF));
-				$tree->setUserPreference($user, 'canedit', Filter::post('canedit' . $tree->getTreeId(), implode('|', array_keys($ALL_EDIT_OPTIONS))));
-				if (Filter::post('gedcomid' . $tree->getTreeId(), WT_REGEX_XREF)) {
-					$tree->setUserPreference($user, 'RELATIONSHIP_PATH_LENGTH', Filter::postInteger('RELATIONSHIP_PATH_LENGTH' . $tree->getTreeId(), 0, 10, 0));
-				} else {
-					// Do not allow a path length to be set if the individual ID is not
-					$tree->setUserPreference($user, 'RELATIONSHIP_PATH_LENGTH', null);
+				foreach (Tree::getAll() as $tree) {
+					$tree->setUserPreference($user, 'gedcomid', Filter::post('gedcomid' . $tree->getTreeId(), WT_REGEX_XREF));
+					$tree->setUserPreference($user, 'canedit', Filter::post('canedit' . $tree->getTreeId(), implode('|', array_keys($ALL_EDIT_OPTIONS))));
+					if (Filter::post('gedcomid' . $tree->getTreeId(), WT_REGEX_XREF)) {
+						$tree->setUserPreference($user, 'RELATIONSHIP_PATH_LENGTH', Filter::postInteger('RELATIONSHIP_PATH_LENGTH' . $tree->getTreeId(), 0, 10, 0));
+					} else {
+						// Do not allow a path length to be set if the individual ID is not
+						$tree->setUserPreference($user, 'RELATIONSHIP_PATH_LENGTH', null);
+					}
 				}
 			}
 		}
-	}
 
-	header('Location: admin_users.php');
+		header('Location: admin_users.php');
 
-	return;
+		return;
 }
 
 switch (Filter::get('action')) {
@@ -182,12 +182,12 @@ 
 			// Datatables numbers columns 0, 1, 2
 			// MySQL numbers columns 1, 2, 3
 			switch ($value['dir']) {
-			case 'asc':
-				$sql_select .= (1 + $value['column']) . " ASC ";
-				break;
-			case 'desc':
-				$sql_select .= (1 + $value['column']) . " DESC ";
-				break;
+				case 'asc':
+					$sql_select .= (1 + $value['column']) . " ASC ";
+					break;
+				case 'desc':
+					$sql_select .= (1 + $value['column']) . " DESC ";
+					break;
 			}
 		}
 	} else {

app/Module/BatchUpdateModule.php

Braces +3 added lines, -1 removed lines

@@ -235,7 +235,8 @@ 
 					<?php // Reset - otherwise we might "undo all changes", which refreshes the ?>
 					<?php // page, which makes them all again!  ?>
 					<script>reset_reload();</script>
-			<?php else: ?>
+			<?php else {
+	: ?>
 					<hr>
 					<div id="batch_update2" class="col-sm-12">
 						<?php if ($this->curr_xref): ?>
@@ -273,6 +274,7 @@ 
 		foreach (array_keys($this->all_xrefs) as $key) {
 			if ($key > $xref) {
 				$record = self::getLatestRecord($key, $this->all_xrefs[$key]);
+}
 				if ($this->PLUGIN->doesRecordNeedUpdate($key, $record)) {
 					return $key;
 				}

Switch Indentation +66 added lines, -66 removed lines

@@ -95,13 +95,13 @@ 
 	 */
 	public function modAction($mod_action) {
 		switch ($mod_action) {
-		case 'admin_batch_update':
-			echo $this->main();
-			break;
+			case 'admin_batch_update':
+				echo $this->main();
+				break;
 
-		default:
-			http_response_code(404);
-			break;
+			default:
+				http_response_code(404);
+				break;
 		}
 	}
 
@@ -126,36 +126,36 @@ 
 			$this->getAllXrefs();
 
 			switch ($this->action) {
-			case 'update':
-				$record = self::getLatestRecord($this->xref, $this->all_xrefs[$this->xref]);
-				if ($this->PLUGIN->doesRecordNeedUpdate($this->xref, $record)) {
-					$newrecord = $this->PLUGIN->updateRecord($this->xref, $record);
-					if ($newrecord != $record) {
-						if ($newrecord) {
-							GedcomRecord::getInstance($this->xref, $WT_TREE)->updateRecord($newrecord, !$this->PLUGIN->chan);
-						} else {
-							GedcomRecord::getInstance($this->xref, $WT_TREE)->deleteRecord();
-						}
-					}
-				}
-				$this->xref = $this->findNextXref($this->xref);
-				break;
-			case 'update_all':
-				foreach ($this->all_xrefs as $xref => $type) {
-					$record = self::getLatestRecord($xref, $type);
-					if ($this->PLUGIN->doesRecordNeedUpdate($xref, $record)) {
-						$newrecord = $this->PLUGIN->updateRecord($xref, $record);
+				case 'update':
+					$record = self::getLatestRecord($this->xref, $this->all_xrefs[$this->xref]);
+					if ($this->PLUGIN->doesRecordNeedUpdate($this->xref, $record)) {
+						$newrecord = $this->PLUGIN->updateRecord($this->xref, $record);
 						if ($newrecord != $record) {
 							if ($newrecord) {
-								GedcomRecord::getInstance($xref, $WT_TREE)->updateRecord($newrecord, !$this->PLUGIN->chan);
+								GedcomRecord::getInstance($this->xref, $WT_TREE)->updateRecord($newrecord, !$this->PLUGIN->chan);
 							} else {
-								GedcomRecord::getInstance($xref, $WT_TREE)->deleteRecord();
+								GedcomRecord::getInstance($this->xref, $WT_TREE)->deleteRecord();
 							}
 						}
 					}
-				}
-				$this->xref = '';
-				break;
+					$this->xref = $this->findNextXref($this->xref);
+					break;
+				case 'update_all':
+					foreach ($this->all_xrefs as $xref => $type) {
+						$record = self::getLatestRecord($xref, $type);
+						if ($this->PLUGIN->doesRecordNeedUpdate($xref, $record)) {
+							$newrecord = $this->PLUGIN->updateRecord($xref, $record);
+							if ($newrecord != $record) {
+								if ($newrecord) {
+									GedcomRecord::getInstance($xref, $WT_TREE)->updateRecord($newrecord, !$this->PLUGIN->chan);
+								} else {
+									GedcomRecord::getInstance($xref, $WT_TREE)->deleteRecord();
+								}
+							}
+						}
+					}
+					$this->xref = '';
+					break;
 			}
 
 			// Make sure that our requested record really does need updating.
@@ -313,28 +313,28 @@ 
 		$vars = [];
 		foreach ($this->PLUGIN->getRecordTypesToUpdate() as $type) {
 			switch ($type) {
-			case 'INDI':
-				$sql[]  = "SELECT i_id, 'INDI' FROM `##individuals` WHERE i_file=?";
-				$vars[] = $WT_TREE->getTreeId();
-				break;
-			case 'FAM':
-				$sql[]  = "SELECT f_id, 'FAM' FROM `##families` WHERE f_file=?";
-				$vars[] = $WT_TREE->getTreeId();
-				break;
-			case 'SOUR':
-				$sql[]  = "SELECT s_id, 'SOUR' FROM `##sources` WHERE s_file=?";
-				$vars[] = $WT_TREE->getTreeId();
-				break;
-			case 'OBJE':
-				$sql[]  = "SELECT m_id, 'OBJE' FROM `##media` WHERE m_file=?";
-				$vars[] = $WT_TREE->getTreeId();
-				break;
-			default:
-				$sql[]  = "SELECT o_id, ? FROM `##other` WHERE o_type=? AND o_file=?";
-				$vars[] = $type;
-				$vars[] = $type;
-				$vars[] = $WT_TREE->getTreeId();
-				break;
+				case 'INDI':
+					$sql[]  = "SELECT i_id, 'INDI' FROM `##individuals` WHERE i_file=?";
+					$vars[] = $WT_TREE->getTreeId();
+					break;
+				case 'FAM':
+					$sql[]  = "SELECT f_id, 'FAM' FROM `##families` WHERE f_file=?";
+					$vars[] = $WT_TREE->getTreeId();
+					break;
+				case 'SOUR':
+					$sql[]  = "SELECT s_id, 'SOUR' FROM `##sources` WHERE s_file=?";
+					$vars[] = $WT_TREE->getTreeId();
+					break;
+				case 'OBJE':
+					$sql[]  = "SELECT m_id, 'OBJE' FROM `##media` WHERE m_file=?";
+					$vars[] = $WT_TREE->getTreeId();
+					break;
+				default:
+					$sql[]  = "SELECT o_id, ? FROM `##other` WHERE o_type=? AND o_file=?";
+					$vars[] = $type;
+					$vars[] = $type;
+					$vars[] = $WT_TREE->getTreeId();
+					break;
 			}
 		}
 		$this->all_xrefs = Database::prepare(implode(' UNION ', $sql) . ' ORDER BY 1 ASC')
@@ -410,20 +410,20 @@ 
 		global $WT_TREE;
 
 		switch ($type) {
-		case 'INDI':
-			return Individual::getInstance($xref, $WT_TREE)->getGedcom();
-		case 'FAM':
-			return Family::getInstance($xref, $WT_TREE)->getGedcom();
-		case 'SOUR':
-			return Source::getInstance($xref, $WT_TREE)->getGedcom();
-		case 'REPO':
-			return Repository::getInstance($xref, $WT_TREE)->getGedcom();
-		case 'OBJE':
-			return Media::getInstance($xref, $WT_TREE)->getGedcom();
-		case 'NOTE':
-			return Note::getInstance($xref, $WT_TREE)->getGedcom();
-		default:
-			return GedcomRecord::getInstance($xref, $WT_TREE)->getGedcom();
+			case 'INDI':
+				return Individual::getInstance($xref, $WT_TREE)->getGedcom();
+			case 'FAM':
+				return Family::getInstance($xref, $WT_TREE)->getGedcom();
+			case 'SOUR':
+				return Source::getInstance($xref, $WT_TREE)->getGedcom();
+			case 'REPO':
+				return Repository::getInstance($xref, $WT_TREE)->getGedcom();
+			case 'OBJE':
+				return Media::getInstance($xref, $WT_TREE)->getGedcom();
+			case 'NOTE':
+				return Note::getInstance($xref, $WT_TREE)->getGedcom();
+			default:
+				return GedcomRecord::getInstance($xref, $WT_TREE)->getGedcom();
 		}
 	}
 

app/Module/StoriesModule.php

Braces +3 added lines, -1 removed lines

@@ -381,7 +381,8 @@ 
 						<a href="<?= $individual->getHtmlUrl() ?>#tab-stories">
 							<?= $individual->getFullName() ?>
 						</a>
-						<?php else: ?>
+						<?php else {
+	: ?>
 							<?= $story->xref ?>
 						<?php endif ?>
 						</td>
@@ -410,6 +411,7 @@ 
 	 */
 	private function showList() {
 		global $controller, $WT_TREE;
+}
 
 		$controller = new PageController;
 		$controller

Switch Indentation +15 added lines, -15 removed lines

@@ -50,21 +50,21 @@
 	 */
 	public function modAction($mod_action) {
 		switch ($mod_action) {
-		case 'admin_edit':
-			$this->edit();
-			break;
-		case 'admin_delete':
-			$this->delete();
-			$this->config();
-			break;
-		case 'admin_config':
-			$this->config();
-			break;
-		case 'show_list':
-			$this->showList();
-			break;
-		default:
-			http_response_code(404);
+			case 'admin_edit':
+				$this->edit();
+				break;
+			case 'admin_delete':
+				$this->delete();
+				$this->config();
+				break;
+			case 'admin_config':
+				$this->config();
+				break;
+			case 'show_list':
+				$this->showList();
+				break;
+			default:
+				http_response_code(404);
 		}
 	}
 

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

Switch Indentation +129 added lines, -129 removed lines

@@ -46,181 +46,181 @@
 echo '<div id="place-hierarchy">';
 
 switch ($display) {
-case 'list':
-	echo '<h2>', $controller->getPageTitle(), '</h2>';
-	$list_places = Place::allPlaces($WT_TREE);
-	$numfound    = count($list_places);
+	case 'list':
+		echo '<h2>', $controller->getPageTitle(), '</h2>';
+		$list_places = Place::allPlaces($WT_TREE);
+		$numfound    = count($list_places);
 
-	$divisor = $numfound > 20 ? 3 : 2;
+		$divisor = $numfound > 20 ? 3 : 2;
 
-	if ($numfound === 0) {
-		echo '<b>', I18N::translate('No results found.'), '</b><br>';
-	} else {
-		$columns = array_chunk($list_places, ceil($numfound / $divisor));
-
-		$html = '<table class="list_table"><thead>';
-		$html .= '<tr><th class="list_label" colspan="' . $divisor . '">';
-		$html .= '<i class="icon-place"></i> ' . I18N::translate('Place list');
-		$html .= '</th></tr></thead>';
-		$html .= '<tbody><tr>';
-		foreach ($columns as $column) {
-			$html .= '<td class="list_value_wrap"><ul>';
-			foreach ($column as $item) {
-				$html .= '<li><a href="' . $item->getURL() . '">' . $item->getReverseName() . '</a></li>';
+		if ($numfound === 0) {
+			echo '<b>', I18N::translate('No results found.'), '</b><br>';
+		} else {
+			$columns = array_chunk($list_places, ceil($numfound / $divisor));
+
+			$html = '<table class="list_table"><thead>';
+			$html .= '<tr><th class="list_label" colspan="' . $divisor . '">';
+			$html .= '<i class="icon-place"></i> ' . I18N::translate('Place list');
+			$html .= '</th></tr></thead>';
+			$html .= '<tbody><tr>';
+			foreach ($columns as $column) {
+				$html .= '<td class="list_value_wrap"><ul>';
+				foreach ($column as $item) {
+					$html .= '<li><a href="' . $item->getURL() . '">' . $item->getReverseName() . '</a></li>';
+				}
+				$html .= '</ul></td>';
 			}
-			$html .= '</ul></td>';
+			$html .= '</tr></tbody></table>';
+			echo $html;
 		}
-		$html .= '</tr></tbody></table>';
-		echo $html;
-	}
-	echo '<h4><a href="placelist.php?display=hierarchy">', I18N::translate('Show places in hierarchy'), '</a></h4>';
-	break;
-case 'hierarchy':
-	$gm_module = Module::getModuleByName('googlemap');
+		echo '<h4><a href="placelist.php?display=hierarchy">', I18N::translate('Show places in hierarchy'), '</a></h4>';
+		break;
+	case 'hierarchy':
+		$gm_module = Module::getModuleByName('googlemap');
 
-	// Find this place and its ID
-	$place    = new Place(implode(', ', array_reverse($parent)), $WT_TREE);
-	$place_id = $place->getPlaceId();
+		// Find this place and its ID
+		$place    = new Place(implode(', ', array_reverse($parent)), $WT_TREE);
+		$place_id = $place->getPlaceId();
 
-	$child_places = $place->getChildPlaces();
+		$child_places = $place->getChildPlaces();
 
-	$numfound = count($child_places);
+		$numfound = count($child_places);
 
-	//-- if the number of places found is 0 then automatically redirect to search page
-	if ($numfound === 0) {
-		$action = 'show';
-	}
-
-	echo '<h2>', $controller->getPageTitle();
-	// Breadcrumbs
-	if ($place_id) {
-		$parent_place = $place->getParentPlace();
-		while ($parent_place->getPlaceId()) {
-			echo ', <a href="', $parent_place->getURL(), '" dir="auto">', $parent_place->getPlaceName(), '</a>';
-			$parent_place = $parent_place->getParentPlace();
+		//-- if the number of places found is 0 then automatically redirect to search page
+		if ($numfound === 0) {
+			$action = 'show';
 		}
-		echo ', <a href="placelist.php">', I18N::translate('Top level'), '</a>';
-	}
-	echo '</h2>';
 
-	if ($gm_module && $gm_module->getPreference('GM_PLACE_HIERARCHY')) {
-		$linklevels  = '';
-		$place_names = [];
-		for ($j = 0; $j < $level; $j++) {
-			$linklevels .= '&amp;parent[' . $j . ']=' . rawurlencode($parent[$j]);
+		echo '<h2>', $controller->getPageTitle();
+		// Breadcrumbs
+		if ($place_id) {
+			$parent_place = $place->getParentPlace();
+			while ($parent_place->getPlaceId()) {
+				echo ', <a href="', $parent_place->getURL(), '" dir="auto">', $parent_place->getPlaceName(), '</a>';
+				$parent_place = $parent_place->getParentPlace();
+			}
+			echo ', <a href="placelist.php">', I18N::translate('Top level'), '</a>';
 		}
+		echo '</h2>';
 
-		$gm_module->createMap();
-	} elseif (Module::getModuleByName('places_assistant')) {
-		// Places Assistant is a custom/add-on module that was once part of the core code.
-		\PlacesAssistantModule::display_map($level, $parent);
-	}
+		if ($gm_module && $gm_module->getPreference('GM_PLACE_HIERARCHY')) {
+			$linklevels  = '';
+			$place_names = [];
+			for ($j = 0; $j < $level; $j++) {
+				$linklevels .= '&amp;parent[' . $j . ']=' . rawurlencode($parent[$j]);
+			}
 
-	if ($numfound > 0) {
-		if ($numfound > 20) {
-			$divisor = 3;
-		} elseif ($numfound > 4) {
-			$divisor = 2;
-		} else {
-			$divisor = 1;
+			$gm_module->createMap();
+		} elseif (Module::getModuleByName('places_assistant')) {
+			// Places Assistant is a custom/add-on module that was once part of the core code.
+			\PlacesAssistantModule::display_map($level, $parent);
 		}
 
-		$columns = array_chunk($child_places, ceil($numfound / $divisor));
-		$html    = '<table id="place_hierarchy" class="list_table"><thead><tr><th class="list_label" colspan="' . $divisor . '">';
-		$html .= '<i class="icon-place"></i> ';
-		if ($place_id) {
-			$html .= I18N::translate('Places in %s', $place->getPlaceName());
-		} else {
-			$html .= I18N::translate('Place hierarchy');
-		}
-		$html .= '</th></tr></thead>';
-		$html .= '<tbody><tr>';
-		foreach ($columns as $column) {
-			$html .= '<td class="list_value"><ul>';
-			foreach ($column as $item) {
-				$html .= '<li><a href="' . $item->getURL() . '" class="list_item">' . $item->getPlaceName() . '</a></li>';
-				if ($gm_module && $gm_module->getPreference('GM_PLACE_HIERARCHY')) {
-					list($tmp)     = explode(', ', $item->getGedcomName(), 2);
-					$place_names[] = $tmp;
+		if ($numfound > 0) {
+			if ($numfound > 20) {
+				$divisor = 3;
+			} elseif ($numfound > 4) {
+				$divisor = 2;
+			} else {
+				$divisor = 1;
+			}
+
+			$columns = array_chunk($child_places, ceil($numfound / $divisor));
+			$html    = '<table id="place_hierarchy" class="list_table"><thead><tr><th class="list_label" colspan="' . $divisor . '">';
+			$html .= '<i class="icon-place"></i> ';
+			if ($place_id) {
+				$html .= I18N::translate('Places in %s', $place->getPlaceName());
+			} else {
+				$html .= I18N::translate('Place hierarchy');
+			}
+			$html .= '</th></tr></thead>';
+			$html .= '<tbody><tr>';
+			foreach ($columns as $column) {
+				$html .= '<td class="list_value"><ul>';
+				foreach ($column as $item) {
+					$html .= '<li><a href="' . $item->getURL() . '" class="list_item">' . $item->getPlaceName() . '</a></li>';
+					if ($gm_module && $gm_module->getPreference('GM_PLACE_HIERARCHY')) {
+						list($tmp)     = explode(', ', $item->getGedcomName(), 2);
+						$place_names[] = $tmp;
+					}
 				}
+				$html .= '</ul></td>';
 			}
-			$html .= '</ul></td>';
-		}
-		$html .= '</tr></tbody>';
-		if ($numfound > 0 && $action == 'find' && $place_id) {
-			$html .= '<tfoot><tr><td class="list_label" colspan="' . $divisor . '">';
-			$html .= I18N::translate('View all records found in this place');
-			$html .= '</td></tr><tr><td class="list_value" colspan="' . $divisor . '" style="text-align: center;">';
-			$html .= '<a href="' . $place->getURL() . '&amp;action=show" class="formField">' . $place->getPlaceName() . '</a>';
-			$html .= '</td></tr></tfoot>';
+			$html .= '</tr></tbody>';
+			if ($numfound > 0 && $action == 'find' && $place_id) {
+				$html .= '<tfoot><tr><td class="list_label" colspan="' . $divisor . '">';
+				$html .= I18N::translate('View all records found in this place');
+				$html .= '</td></tr><tr><td class="list_value" colspan="' . $divisor . '" style="text-align: center;">';
+				$html .= '<a href="' . $place->getURL() . '&amp;action=show" class="formField">' . $place->getPlaceName() . '</a>';
+				$html .= '</td></tr></tfoot>';
+			}
+			$html .= '</table>';
+			// -- echo the array
+			echo $html;
 		}
-		$html .= '</table>';
-		// -- echo the array
-		echo $html;
-	}
-	if ($place_id && $action == 'show') {
-		// -- array of names
-		$myindilist = [];
-		$myfamlist  = [];
-
-		$positions =
-			Database::prepare("SELECT DISTINCT pl_gid FROM `##placelinks` WHERE pl_p_id=? AND pl_file=?")
-			->execute([$place_id, $WT_TREE->getTreeId()])
-			->fetchOneColumn();
-
-		foreach ($positions as $position) {
-			$record = GedcomRecord::getInstance($position, $WT_TREE);
-			if ($record && $record->canShow()) {
-				if ($record instanceof Individual) {
-					$myindilist[] = $record;
-				}
-				if ($record instanceof Family) {
-					$myfamlist[] = $record;
+		if ($place_id && $action == 'show') {
+			// -- array of names
+			$myindilist = [];
+			$myfamlist  = [];
+
+			$positions =
+				Database::prepare("SELECT DISTINCT pl_gid FROM `##placelinks` WHERE pl_p_id=? AND pl_file=?")
+				->execute([$place_id, $WT_TREE->getTreeId()])
+				->fetchOneColumn();
+
+			foreach ($positions as $position) {
+				$record = GedcomRecord::getInstance($position, $WT_TREE);
+				if ($record && $record->canShow()) {
+					if ($record instanceof Individual) {
+						$myindilist[] = $record;
+					}
+					if ($record instanceof Family) {
+						$myfamlist[] = $record;
+					}
 				}
 			}
-		}
-		echo '<br>';
+			echo '<br>';
 
-		?>
+			?>
 		<ul class="nav nav-tabs" role="tablist">
 			<li class="nav-item">
 				<a class="nav-link active<?= empty($myindilist) ? ' text-muted' : '' ?>" data-toggle="tab" role="tab" href="#individuals">
 					<?= I18N::translate('Individuals') ?>
-					<?= Bootstrap4::badgeCount($myindilist) ?>
+						<?= Bootstrap4::badgeCount($myindilist) ?>
 				</a>
 			</li>
 			<li class="nav-item">
 				<a class="nav-link<?= empty($myfamlist) ? ' text-muted' : '' ?>" data-toggle="tab" role="tab" href="#families">
 					<?= I18N::translate('Families') ?>
-					<?= Bootstrap4::badgeCount($myfamlist) ?>
+						<?= Bootstrap4::badgeCount($myfamlist) ?>
 				</a>
 			</li>
 		</ul>
 		<div class="tab-content">
 			<div class="tab-pane fade show active" role="tabpanel" id="individuals">
 				<?php if (empty($myindilist)): ?>
-					<p><?= I18N::translate('No results found.') ?></p>
+						<p><?= I18N::translate('No results found.') ?></p>
 				<?php else: ?>
-					<?= FunctionsPrintLists::individualTable($myindilist) ?>
-				<?php endif ?>
+						<?= FunctionsPrintLists::individualTable($myindilist) ?>
+					<?php endif ?>
 			</div>
 
 			<div class="tab-pane fade" role="tabpanel" id="families">
 				<?php if (empty($myfamlist)): ?>
-					<p><?= I18N::translate('No results found.') ?></p>
+						<p><?= I18N::translate('No results found.') ?></p>
 				<?php else: ?>
-					<?= FunctionsPrintLists::familyTable($myfamlist) ?>
-				<?php endif ?>
+						<?= FunctionsPrintLists::familyTable($myfamlist) ?>
+					<?php endif ?>
 			</div>
 		</div>
 		<?php
-	}
-	echo '<h4><a href="placelist.php?display=list">', I18N::translate('Show all places in a list'), '</a></h4>';
+		}
+		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);
-	}
-	break;
+		if ($gm_module && $gm_module->getPreference('GM_PLACE_HIERARCHY')) {
+			$gm_module->mapScripts($numfound, $level, $parent, $linklevels, $place_names);
+		}
+		break;
 }
 
 echo '</div>'; // <div id="place-hierarchy">

admin_site_upgrade.php

Spacing +2 added lines, -2 removed lines

@@ -29,10 +29,10 @@
 // Check for updates
 $latest_version_txt = Functions::fetchLatestVersion();
 if (preg_match('/^[0-9.]+\|[0-9.]+\|/', $latest_version_txt)) {
-	list($latest_version, , $download_url) = explode('|', $latest_version_txt);
+	list($latest_version,, $download_url) = explode('|', $latest_version_txt);
 } else {
 	// Cannot determine the latest version
-	list($latest_version, , $download_url) = explode('|', '||');
+	list($latest_version,, $download_url) = explode('|', '||');
 }
 
 $latest_version_html = '<span dir="ltr">' . $latest_version . '</span>';

Switch Indentation +46 added lines, -46 removed lines

@@ -118,18 +118,18 @@ 
 foreach (Module::getInstalledModules('disabled') as $module) {
 	if (!in_array($module->getName(), Module::getCoreModuleNames())) {
 		switch ($modules_action) {
-		case 'disable':
-			Database::prepare(
-				"UPDATE `##module` SET status = 'disabled' WHERE module_name = ?"
-			)->execute([$module->getName()]);
-			break;
-		case 'ignore':
-			echo '<br>', I18N::translate('Custom module'), ' — ', WT_MODULES_DIR, $module->getName(), ' — ', $module->getTitle(), $icon_success;
-			break;
-		default:
-			echo '<br>', I18N::translate('Custom module'), ' — ', WT_MODULES_DIR, $module->getName(), ' — ', $module->getTitle(), $icon_failure;
-			$custom_modules = true;
-			break;
+			case 'disable':
+				Database::prepare(
+					"UPDATE `##module` SET status = 'disabled' WHERE module_name = ?"
+				)->execute([$module->getName()]);
+				break;
+			case 'ignore':
+				echo '<br>', I18N::translate('Custom module'), ' — ', WT_MODULES_DIR, $module->getName(), ' — ', $module->getTitle(), $icon_success;
+				break;
+			default:
+				echo '<br>', I18N::translate('Custom module'), ' — ', WT_MODULES_DIR, $module->getName(), ' — ', $module->getTitle(), $icon_failure;
+				$custom_modules = true;
+				break;
 		}
 	}
 }
@@ -158,40 +158,40 @@ 
 $custom_themes = false;
 foreach (Theme::themeNames() as $theme_id => $theme_name) {
 	switch ($theme_id) {
-	case 'clouds':
-	case 'colors':
-	case 'fab':
-	case 'minimal':
-	case 'webtrees':
-	case 'xenea':
-		break;
-	default:
-		$theme_used = Database::prepare(
-			"SELECT EXISTS (SELECT 1 FROM `##site_setting`   WHERE setting_name='THEME_DIR' AND setting_value=?)" .
-			" OR EXISTS (SELECT 1 FROM `##gedcom_setting` WHERE setting_name='THEME_DIR' AND setting_value=?)" .
-			" OR EXISTS (SELECT 1 FROM `##user_setting`   WHERE setting_name='theme'     AND setting_value=?)"
-		)->execute([$theme_id, $theme_id, $theme_id])->fetchOne();
-		if ($theme_used) {
-			switch ($themes_action) {
-			case 'disable':
-				Database::prepare(
-					"DELETE FROM `##site_setting`   WHERE setting_name = 'THEME_DIR' AND setting_value = ?"
-				)->execute([$theme_id]);
-				Database::prepare(
-					"DELETE FROM `##gedcom_setting` WHERE setting_name = 'THEME_DIR' AND setting_value = ?"
-				)->execute([$theme_id]);
-				Database::prepare(
-					"DELETE FROM `##user_setting`   WHERE setting_name = 'theme'     AND setting_value = ?"
-				)->execute([$theme_id]);
-				break;
-			case 'ignore':
-				echo '<br>', I18N::translate('Custom theme'), ' — ', $theme_id, ' — ', $theme_name, $icon_success;
-				break;
-			default:
-				echo '<br>', I18N::translate('Custom theme'), ' — ', $theme_id, ' — ', $theme_name, $icon_failure;
-				$custom_themes = true;
-				break;
-			}
+		case 'clouds':
+		case 'colors':
+		case 'fab':
+		case 'minimal':
+		case 'webtrees':
+		case 'xenea':
+			break;
+		default:
+			$theme_used = Database::prepare(
+				"SELECT EXISTS (SELECT 1 FROM `##site_setting`   WHERE setting_name='THEME_DIR' AND setting_value=?)" .
+				" OR EXISTS (SELECT 1 FROM `##gedcom_setting` WHERE setting_name='THEME_DIR' AND setting_value=?)" .
+				" OR EXISTS (SELECT 1 FROM `##user_setting`   WHERE setting_name='theme'     AND setting_value=?)"
+			)->execute([$theme_id, $theme_id, $theme_id])->fetchOne();
+			if ($theme_used) {
+				switch ($themes_action) {
+					case 'disable':
+						Database::prepare(
+							"DELETE FROM `##site_setting`   WHERE setting_name = 'THEME_DIR' AND setting_value = ?"
+						)->execute([$theme_id]);
+						Database::prepare(
+							"DELETE FROM `##gedcom_setting` WHERE setting_name = 'THEME_DIR' AND setting_value = ?"
+						)->execute([$theme_id]);
+						Database::prepare(
+							"DELETE FROM `##user_setting`   WHERE setting_name = 'theme'     AND setting_value = ?"
+						)->execute([$theme_id]);
+						break;
+					case 'ignore':
+						echo '<br>', I18N::translate('Custom theme'), ' — ', $theme_id, ' — ', $theme_name, $icon_success;
+						break;
+					default:
+						echo '<br>', I18N::translate('Custom theme'), ' — ', $theme_id, ' — ', $theme_name, $icon_failure;
+						$custom_themes = true;
+						break;
+				}
 		}
 		break;
 	}

search_advanced.php

Braces +6 added lines, -2 removed lines

@@ -146,7 +146,8 @@ 
 					value="<?= Html::escape($controller->getValue($i)) ?>">
 						<?php if (preg_match('/^NAME:/', $currentFieldSearch) > 0): ?>
 							<select class="form-control form-control-sm col-3" name="fields[<?= $i ?>]">
-								<option value="<?= $currentField ?>:EXACT" <?php if (preg_match('/:EXACT$/', $currentFieldSearch) > 0) echo 'selected' ?>>
+								<option value="<?= $currentField ?>:EXACT" <?php if (preg_match('/:EXACT$/', $currentFieldSearch) > 0) {
+	echo 'selected' ?>>
 									<?= I18N::translate('Exact') ?>
 								</option>
 								<option value="<?= $currentField ?>:BEGINS" <?php if (preg_match('/:BEGINS$/', $currentFieldSearch) > 0) echo 'selected' ?>>
@@ -161,13 +162,15 @@ 
 						<?php else: ?>
 							<input type="hidden" name="fields[<?= $i ?>]" value="<?= $controller->getField($i) ?>">
 						<?php endif;
+}
 						if (preg_match('/:DATE$/', $currentFieldSearch) > 0) {
 							?>
 							<select class="form-control form-control-sm col-3" name="plusminus[<?= $i ?>]">
 								<option value="">
 									<?= I18N::translate('Exact date') ?>
 								</option>
-								<option value="2" <?php if (!empty($controller->plusminus[$i]) && $controller->plusminus[$i] == 2) echo 'selected' ?>>
+								<option value="2" <?php if (!empty($controller->plusminus[$i]) && $controller->plusminus[$i] == 2) {
+	echo 'selected' ?>>
 									<?= I18N::plural('±%s year', '±%s years', 2, I18N::number(2)) ?>
 								</option>
 								<option value="5" <?php if (!empty($controller->plusminus[$i]) && $controller->plusminus[$i] == 5) echo 'selected' ?>>
@@ -183,6 +186,7 @@ 
 					//-- relative fields
 					if ($i == 0 && $fct > 4) {
 						$j = $fct;
+}
 						// Get the current options for Father’s and Mother’s name searches
 						$fatherGivnOption = 'SDX';
 						$fatherSurnOption = 'SDX';

Switch Indentation +12 added lines, -12 removed lines

@@ -192,18 +192,18 @@
 							$searchField  = $controller->getField($k);
 							$searchOption = substr($searchField, 20); // Assume we have something like "FAMC:HUSB:NAME:GIVN:foo"
 							switch (substr($searchField, 0, 20)) {
-							case 'FAMC:HUSB:NAME:GIVN:':
-								$fatherGivnOption = $searchOption;
-								break;
-							case 'FAMC:HUSB:NAME:SURN:':
-								$fatherSurnOption = $searchOption;
-								break;
-							case 'FAMC:WIFE:NAME:GIVN:':
-								$motherGivnOption = $searchOption;
-								break;
-							case 'FAMC:WIFE:NAME:SURN:':
-								$motherSurnOption = $searchOption;
-								break;
+								case 'FAMC:HUSB:NAME:GIVN:':
+									$fatherGivnOption = $searchOption;
+									break;
+								case 'FAMC:HUSB:NAME:SURN:':
+									$fatherSurnOption = $searchOption;
+									break;
+								case 'FAMC:WIFE:NAME:GIVN:':
+									$motherGivnOption = $searchOption;
+									break;
+								case 'FAMC:WIFE:NAME:SURN:':
+									$motherSurnOption = $searchOption;
+									break;
 							}
 						}
 					}

app/Module/FamilyNavigatorModule.php

Indentation +2 added lines, -2 removed lines

@@ -150,8 +150,8 @@
 				}
 
 				foreach ($family->getChildren() as $child) {
-          $icon = $controller->record === $child ? '<i class="icon-selected"></i>' : '';
-          $menu = new Menu($icon . Functions::getCloseRelationshipName($controller->record, $child));
+					$icon = $controller->record === $child ? '<i class="icon-selected"></i>' : '';
+					$menu = new Menu($icon . Functions::getCloseRelationshipName($controller->record, $child));
 					$menu->addSubmenu(new Menu($this->getFamily($child)));
 					?>
 					<tr class="text-center wt-child wt-gender-<?= $child->getSex() ?>">

Braces +6 added lines, -2 removed lines

@@ -141,7 +141,8 @@ 
 							<div class="small">
 								<?= $spouse->getLifeSpan() ?>
 							</div>
-							<?php else: ?>
+							<?php else {
+	: ?>
 								<?= $spouse->getFullName() ?>
 							<?php endif ?>
 						</td>
@@ -151,6 +152,7 @@ 
 
 				foreach ($family->getChildren() as $child) {
           $icon = $controller->record === $child ? '<i class="icon-selected"></i>' : '';
+}
           $menu = new Menu($icon . Functions::getCloseRelationshipName($controller->record, $child));
 					$menu->addSubmenu(new Menu($this->getFamily($child)));
 					?>
@@ -168,7 +170,8 @@ 
 							<div class="small">
 								<?= $child->getLifeSpan() ?>
 							</div>
-							<?php else: ?>
+							<?php else {
+	: ?>
 								<?= $child->getFullName() ?>
 							<?php endif ?>
 						</td>
@@ -192,6 +195,7 @@ 
 	private function getHTML($person, $showUnknown = false) {
 		if ($person instanceof Individual) {
 			return sprintf(self::LNK, $person->getHtmlUrl(), $person->getFullName());
+}
 		} elseif ($showUnknown) {
 			return sprintf(self::MSG, I18N::translate('unknown'));
 		} else {

app/Module/RelativesTabModule.php

Indentation +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
 					$row_class .= ' old';
 				}
 				$icon = $controller->record === $person ? '<i class="icon-selected"></i>' : '';
-        ?>
+				?>
 				<tr class="<?= $row_class ?>">
 					<th scope="row">
 					<?= $icon ?>