fisharebest / webtrees

app/Report/ReportParserSetup.php

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 namespace Fisharebest\Webtrees\Report;
 
 use Fisharebest\Webtrees\Auth;
-use Fisharebest\Webtrees\Date;
 use Fisharebest\Webtrees\I18N;
 
 /**

app/Controller/LifespanController.php

Switch Indentation +17 added lines, -17 removed lines

@@ -506,23 +506,23 @@
 	 */
 	public function getCalendarDate($date) {
 		switch ($this->calendar) {
-			case 'julian':
-				$caldate = new JulianDate($date);
-				break;
-			case 'french':
-				$caldate = new FrenchDate($date);
-				break;
-			case 'jewish':
-				$caldate = new JewishDate($date);
-				break;
-			case 'hijri':
-				$caldate = new HijriDate($date);
-				break;
-			case 'jalali':
-				$caldate = new JalaliDate($date);
-				break;
-			default:
-				$caldate = new GregorianDate($date);
+		case 'julian':
+			$caldate = new JulianDate($date);
+			break;
+		case 'french':
+			$caldate = new FrenchDate($date);
+			break;
+		case 'jewish':
+			$caldate = new JewishDate($date);
+			break;
+		case 'hijri':
+			$caldate = new HijriDate($date);
+			break;
+		case 'jalali':
+			$caldate = new JalaliDate($date);
+			break;
+		default:
+			$caldate = new GregorianDate($date);
 		}
 
 		return $caldate;

app/Controller/PedigreeController.php

Switch Indentation +74 added lines, -74 removed lines

@@ -102,26 +102,26 @@ 
 
 		$this->arrows = new \stdClass();
 		switch ($this->orientation) {
-			case self::PORTRAIT:
-				//drop through
-			case self::LANDSCAPE:
-				$this->arrows->prevGen = I18N::direction() === 'rtl' ? 'icon-larrow' : 'icon-rarrow';
-				$this->arrows->menu    = I18N::direction() === 'rtl' ? 'icon-rarrow' : 'icon-larrow';
-				$addoffset['x']        = $this->chartHasAncestors ? self::ARROW_SIZE : 0;
-				$addoffset['y']        = 0;
-				break;
-			case self::OLDEST_AT_TOP:
-				$this->arrows->prevGen = 'icon-uarrow';
-				$this->arrows->menu    = 'icon-darrow';
-				$addoffset['x']        = 0;
-				$addoffset['y']        = $this->root->getSpouseFamilies() ? self::ARROW_SIZE : 0;
-				break;
-			case self::OLDEST_AT_BOTTOM:
-				$this->arrows->prevGen = 'icon-darrow';
-				$this->arrows->menu    = 'icon-uarrow';
-				$addoffset['x']        = 0;
-				$addoffset['y']        = $this->chartHasAncestors ? self::ARROW_SIZE : 0;
-				break;
+		case self::PORTRAIT:
+			//drop through
+		case self::LANDSCAPE:
+			$this->arrows->prevGen = I18N::direction() === 'rtl' ? 'icon-larrow' : 'icon-rarrow';
+			$this->arrows->menu    = I18N::direction() === 'rtl' ? 'icon-rarrow' : 'icon-larrow';
+			$addoffset['x']        = $this->chartHasAncestors ? self::ARROW_SIZE : 0;
+			$addoffset['y']        = 0;
+			break;
+		case self::OLDEST_AT_TOP:
+			$this->arrows->prevGen = 'icon-uarrow';
+			$this->arrows->menu    = 'icon-darrow';
+			$addoffset['x']        = 0;
+			$addoffset['y']        = $this->root->getSpouseFamilies() ? self::ARROW_SIZE : 0;
+			break;
+		case self::OLDEST_AT_BOTTOM:
+			$this->arrows->prevGen = 'icon-darrow';
+			$this->arrows->menu    = 'icon-uarrow';
+			$addoffset['x']        = 0;
+			$addoffset['y']        = $this->chartHasAncestors ? self::ARROW_SIZE : 0;
+			break;
 		}
 
 		// -- this next section will create and position the DIV layers for the pedigree tree
@@ -150,73 +150,73 @@ 
 
 			// -- calculate the xoffset
 			switch ($this->orientation) {
-				case self::PORTRAIT:
-					$xoffset = ($this->generations - $curgen) * (($this->getBoxDimensions()->width + $bxspacing) / 1.8);
-					if(!$i && $this->root->getSpouseFamilies()) {
-						$xoffset -= self::ARROW_SIZE;
+			case self::PORTRAIT:
+				$xoffset = ($this->generations - $curgen) * (($this->getBoxDimensions()->width + $bxspacing) / 1.8);
+				if(!$i && $this->root->getSpouseFamilies()) {
+					$xoffset -= self::ARROW_SIZE;
+				}
+				// -- compact the tree
+				if ($curgen < $this->generations) {
+					if ($i % 2 == 0) {
+						$yoffset = $yoffset - (($boxspacing / 2) * ($curgen - 1));
+					} else {
+						$yoffset = $yoffset + (($boxspacing / 2) * ($curgen - 1));
 					}
-					// -- compact the tree
-					if ($curgen < $this->generations) {
-						if ($i % 2 == 0) {
-							$yoffset = $yoffset - (($boxspacing / 2) * ($curgen - 1));
+					$parent = (int) (($i - 1) / 2);
+					$pgen   = $curgen;
+					while ($parent > 0) {
+						if ($parent % 2 == 0) {
+							$yoffset = $yoffset - (($boxspacing / 2) * $pgen);
 						} else {
-							$yoffset = $yoffset + (($boxspacing / 2) * ($curgen - 1));
-						}
-						$parent = (int) (($i - 1) / 2);
-						$pgen   = $curgen;
-						while ($parent > 0) {
-							if ($parent % 2 == 0) {
-								$yoffset = $yoffset - (($boxspacing / 2) * $pgen);
-							} else {
-								$yoffset = $yoffset + (($boxspacing / 2) * $pgen);
-							}
-							$pgen++;
-							if ($pgen > 3) {
-								$temp = 0;
-								for ($j = 1; $j < ($pgen - 2); $j++) {
-									$temp += (pow(2, $j) - 1);
-								}
-								if ($parent % 2 == 0) {
-									$yoffset = $yoffset - (($boxspacing / 2) * $temp);
-								} else {
-									$yoffset = $yoffset + (($boxspacing / 2) * $temp);
-								}
-							}
-							$parent = (int) (($parent - 1) / 2);
+							$yoffset = $yoffset + (($boxspacing / 2) * $pgen);
 						}
-						if ($curgen > 3) {
+						$pgen++;
+						if ($pgen > 3) {
 							$temp = 0;
-							for ($j = 1; $j < ($curgen - 2); $j++) {
+							for ($j = 1; $j < ($pgen - 2); $j++) {
 								$temp += (pow(2, $j) - 1);
 							}
-							if ($i % 2 == 0) {
+							if ($parent % 2 == 0) {
 								$yoffset = $yoffset - (($boxspacing / 2) * $temp);
 							} else {
 								$yoffset = $yoffset + (($boxspacing / 2) * $temp);
 							}
 						}
+						$parent = (int) (($parent - 1) / 2);
 					}
-					$yoffset -= (($boxspacing / 2) * pow(2, ($this->generations - 2)) - ($boxspacing / 2));
-					break;
-				case self::LANDSCAPE:
-					$xoffset = ($this->generations - $curgen) * ($this->getBoxDimensions()->width + $bxspacing);
-					if ($curgen == 1) {
-						$xoffset += 10;
-					}
-					break;
-				case self::OLDEST_AT_TOP:
-					//swap x & y offsets as chart is rotated
-					$xoffset = $yoffset;
-					$yoffset = $curgen  * ($this->getBoxDimensions()->height + ($byspacing * 4));
-					break;
-				case self::OLDEST_AT_BOTTOM:
-					//swap x & y offsets as chart is rotated
-					$xoffset = $yoffset;
-					$yoffset = ($this->generations - $curgen) * ($this->getBoxDimensions()->height + ($byspacing * 2));
-					if ($i && $this->root->getSpouseFamilies()) {
-						$yoffset += self::ARROW_SIZE;
+					if ($curgen > 3) {
+						$temp = 0;
+						for ($j = 1; $j < ($curgen - 2); $j++) {
+							$temp += (pow(2, $j) - 1);
+						}
+						if ($i % 2 == 0) {
+							$yoffset = $yoffset - (($boxspacing / 2) * $temp);
+						} else {
+							$yoffset = $yoffset + (($boxspacing / 2) * $temp);
+						}
 					}
-					break;
+				}
+				$yoffset -= (($boxspacing / 2) * pow(2, ($this->generations - 2)) - ($boxspacing / 2));
+				break;
+			case self::LANDSCAPE:
+				$xoffset = ($this->generations - $curgen) * ($this->getBoxDimensions()->width + $bxspacing);
+				if ($curgen == 1) {
+					$xoffset += 10;
+				}
+				break;
+			case self::OLDEST_AT_TOP:
+				//swap x & y offsets as chart is rotated
+				$xoffset = $yoffset;
+				$yoffset = $curgen  * ($this->getBoxDimensions()->height + ($byspacing * 4));
+				break;
+			case self::OLDEST_AT_BOTTOM:
+				//swap x & y offsets as chart is rotated
+				$xoffset = $yoffset;
+				$yoffset = ($this->generations - $curgen) * ($this->getBoxDimensions()->height + ($byspacing * 2));
+				if ($i && $this->root->getSpouseFamilies()) {
+					$yoffset += self::ARROW_SIZE;
+				}
+				break;
 			}
 			$this->nodes[$i]["x"] = (int) $xoffset;
 			$this->nodes[$i]["y"] = (int) $yoffset;

app/Functions/FunctionsDate.php

Switch Indentation +60 added lines, -60 removed lines

@@ -32,32 +32,32 @@ 
 	 */
 	public static function getAgeAtEvent($age_string, $show_years) {
 		switch (strtoupper($age_string)) {
-			case 'CHILD':
-				return I18N::translate('Child');
-			case 'INFANT':
-				return I18N::translate('Infant');
-			case 'STILLBORN':
-				return I18N::translate('Stillborn');
-			default:
-				return preg_replace_callback(
-					array(
-						'/(\d+)([ymwd])/',
-					),
-					function ($match) use ($age_string, $show_years) {
-						switch ($match[2]) {
-							case 'y':
-								if ($show_years || preg_match('/[dm]/', $age_string)) {
-									return I18N::plural('%s year', '%s years', $match[1], I18N::digits($match[1]));
-								} else {
-									return I18N::digits($match[1]);
-								}
-							case 'm':
-								return I18N::plural('%s month', '%s months', $match[1], I18N::digits($match[1]));
-							case 'w':
-								return I18N::plural('%s week', '%s weeks', $match[1], I18N::digits($match[1]));
-							case 'd':
-								return I18N::plural('%s day', '%s days', $match[1], I18N::digits($match[1]));
-						}
+		case 'CHILD':
+			return I18N::translate('Child');
+		case 'INFANT':
+			return I18N::translate('Infant');
+		case 'STILLBORN':
+			return I18N::translate('Stillborn');
+		default:
+			return preg_replace_callback(
+				array(
+					'/(\d+)([ymwd])/',
+				),
+				function ($match) use ($age_string, $show_years) {
+					switch ($match[2]) {
+					case 'y':
+					if ($show_years || preg_match('/[dm]/', $age_string)) {
+					return I18N::plural('%s year', '%s years', $match[1], I18N::digits($match[1]));
+					} else {
+					return I18N::digits($match[1]);
+					}
+					case 'm':
+					return I18N::plural('%s month', '%s months', $match[1], I18N::digits($match[1]));
+					case 'w':
+					return I18N::plural('%s week', '%s weeks', $match[1], I18N::digits($match[1]));
+					case 'd':
+					return I18N::plural('%s day', '%s days', $match[1], I18N::digits($match[1]));
+					}
 					},
 					$age_string
 				);
@@ -79,40 +79,40 @@ 
 		preg_match_all('/%[^%]/', $time_fmt, $matches);
 		foreach ($matches[0] as $match) {
 			switch ($match) {
-				case '%a':
-					$t = gmdate('His', $time);
-					if ($t == '000000') {
-						$time_fmt = str_replace($match, /* I18N: time format “%a” - exactly 00:00:00 */
-							I18N::translate('midnight'), $time_fmt);
-					} elseif ($t < '120000') {
-						$time_fmt = str_replace($match, /* I18N: time format “%a” - between 00:00:01 and 11:59:59 */
-							I18N::translate('a.m.'), $time_fmt);
-					} elseif ($t == '120000') {
-						$time_fmt = str_replace($match, /* I18N: time format “%a” - exactly 12:00:00 */
-							I18N::translate('noon'), $time_fmt);
-					} else {
-						$time_fmt = str_replace($match, /* I18N: time format “%a” - between 12:00:01 and 23:59:59 */
-							I18N::translate('p.m.'), $time_fmt);
-					}
-					break;
-				case '%A':
-					$t = gmdate('His', $time);
-					if ($t == '000000') {
-						$time_fmt = str_replace($match, /* I18N: time format “%A” - exactly 00:00:00 */
-							I18N::translate('Midnight'), $time_fmt);
-					} elseif ($t < '120000') {
-						$time_fmt = str_replace($match, /* I18N: time format “%A” - between 00:00:01 and 11:59:59 */
-							I18N::translate('A.M.'), $time_fmt);
-					} elseif ($t == '120000') {
-						$time_fmt = str_replace($match, /* I18N: time format “%A” - exactly 12:00:00 */
-							I18N::translate('Noon'), $time_fmt);
-					} else {
-						$time_fmt = str_replace($match, /* I18N: time format “%A” - between 12:00:01 and 23:59:59 */
-							I18N::translate('P.M.'), $time_fmt);
-					}
-					break;
-				default:
-					$time_fmt = str_replace($match, I18N::digits(gmdate(substr($match, -1), $time)), $time_fmt);
+			case '%a':
+				$t = gmdate('His', $time);
+				if ($t == '000000') {
+					$time_fmt = str_replace($match, /* I18N: time format “%a” - exactly 00:00:00 */
+						I18N::translate('midnight'), $time_fmt);
+				} elseif ($t < '120000') {
+					$time_fmt = str_replace($match, /* I18N: time format “%a” - between 00:00:01 and 11:59:59 */
+						I18N::translate('a.m.'), $time_fmt);
+				} elseif ($t == '120000') {
+					$time_fmt = str_replace($match, /* I18N: time format “%a” - exactly 12:00:00 */
+						I18N::translate('noon'), $time_fmt);
+				} else {
+					$time_fmt = str_replace($match, /* I18N: time format “%a” - between 12:00:01 and 23:59:59 */
+						I18N::translate('p.m.'), $time_fmt);
+				}
+				break;
+			case '%A':
+				$t = gmdate('His', $time);
+				if ($t == '000000') {
+					$time_fmt = str_replace($match, /* I18N: time format “%A” - exactly 00:00:00 */
+						I18N::translate('Midnight'), $time_fmt);
+				} elseif ($t < '120000') {
+					$time_fmt = str_replace($match, /* I18N: time format “%A” - between 00:00:01 and 11:59:59 */
+						I18N::translate('A.M.'), $time_fmt);
+				} elseif ($t == '120000') {
+					$time_fmt = str_replace($match, /* I18N: time format “%A” - exactly 12:00:00 */
+						I18N::translate('Noon'), $time_fmt);
+				} else {
+					$time_fmt = str_replace($match, /* I18N: time format “%A” - between 12:00:01 and 23:59:59 */
+						I18N::translate('P.M.'), $time_fmt);
+				}
+				break;
+			default:
+				$time_fmt = str_replace($match, I18N::digits(gmdate(substr($match, -1), $time)), $time_fmt);
 			}
 		}
 

app/Functions/FunctionsDb.php

Switch Indentation +100 added lines, -100 removed lines

@@ -248,18 +248,18 @@ 
 	 */
 	public static function searchIndividualsPhonetic($soundex, $lastname, $firstname, $place, array $trees) {
 		switch ($soundex) {
-			case 'Russell':
-				$givn_sdx = Soundex::russell($firstname);
-				$surn_sdx = Soundex::russell($lastname);
-				$plac_sdx = Soundex::russell($place);
-				break;
-			case 'DaitchM':
-				$givn_sdx = Soundex::daitchMokotoff($firstname);
-				$surn_sdx = Soundex::daitchMokotoff($lastname);
-				$plac_sdx = Soundex::daitchMokotoff($place);
-				break;
-			default:
-				throw new \DomainException('soundex: ' . $soundex);
+		case 'Russell':
+			$givn_sdx = Soundex::russell($firstname);
+			$surn_sdx = Soundex::russell($lastname);
+			$plac_sdx = Soundex::russell($place);
+			break;
+		case 'DaitchM':
+			$givn_sdx = Soundex::daitchMokotoff($firstname);
+			$surn_sdx = Soundex::daitchMokotoff($lastname);
+			$plac_sdx = Soundex::daitchMokotoff($place);
+			break;
+		default:
+			throw new \DomainException('soundex: ' . $soundex);
 		}
 
 		// Nothing to search for?  Return nothing.
@@ -291,12 +291,12 @@ 
 			foreach ($givn_sdx as $n => $sdx) {
 				$sql .= $n ? " OR " : "";
 				switch ($soundex) {
-					case 'Russell':
-						$sql .= "n_soundex_givn_std LIKE CONCAT('%', :given_name_" . $n . ", '%')";
-						break;
-					case 'DaitchM':
-						$sql .= "n_soundex_givn_dm LIKE CONCAT('%', :given_name_" . $n . ", '%')";
-						break;
+				case 'Russell':
+					$sql .= "n_soundex_givn_std LIKE CONCAT('%', :given_name_" . $n . ", '%')";
+					break;
+				case 'DaitchM':
+					$sql .= "n_soundex_givn_dm LIKE CONCAT('%', :given_name_" . $n . ", '%')";
+					break;
 				}
 				$args['given_name_' . $n] = $sdx;
 			}
@@ -309,12 +309,12 @@ 
 			foreach ($surn_sdx as $n => $sdx) {
 				$sql .= $n ? " OR " : "";
 				switch ($soundex) {
-					case 'Russell':
-						$sql .= "n_soundex_surn_std LIKE CONCAT('%', :surname_" . $n . ", '%')";
-						break;
-					case 'DaitchM':
-						$sql .= "n_soundex_surn_dm LIKE CONCAT('%', :surname_" . $n . ", '%')";
-						break;
+				case 'Russell':
+					$sql .= "n_soundex_surn_std LIKE CONCAT('%', :surname_" . $n . ", '%')";
+					break;
+				case 'DaitchM':
+					$sql .= "n_soundex_surn_dm LIKE CONCAT('%', :surname_" . $n . ", '%')";
+					break;
 				}
 				$args['surname_' . $n] = $sdx;
 			}
@@ -327,12 +327,12 @@ 
 			foreach ($plac_sdx as $n => $sdx) {
 				$sql .= $n ? " OR " : "";
 				switch ($soundex) {
-					case 'Russell':
-						$sql .= "p_std_soundex LIKE CONCAT('%', :place_" . $n . ", '%')";
-						break;
-					case 'DaitchM':
-						$sql .= "p_dm_soundex LIKE CONCAT('%', :place_" . $n . ", '%')";
-						break;
+				case 'Russell':
+					$sql .= "p_std_soundex LIKE CONCAT('%', :place_" . $n . ", '%')";
+					break;
+				case 'DaitchM':
+					$sql .= "p_dm_soundex LIKE CONCAT('%', :place_" . $n . ", '%')";
+					break;
 				}
 				$args['place_' . $n] = $sdx;
 			}
@@ -790,82 +790,82 @@ 
 			} else {
 				// SPECIAL CASES:
 				switch ($anniv->m) {
-					case 2:
-						// 29 CSH does not include 30 CSH (but would include an invalid 31 CSH if there were no 30 CSH)
-						if ($anniv->d === 1) {
-							$where .= " AND d_day <= 1 AND d_mon = 2";
-						} elseif ($anniv->d === 30) {
-							$where .= " AND d_day >= 30 AND d_mon = 2";
-						} elseif ($anniv->d === 29 && $anniv->daysInMonth() === 29) {
-							$where .= " AND (d_day = 29 OR d_day > 30) AND d_mon = 2";
-						} else {
-							$where .= " AND d_day = :day AND d_mon = 2";
-							$args['day'] = $anniv->d;
-						}
-						break;
-					case 3:
-						// 1 KSL includes 30 CSH (if this year didn’t have 30 CSH)
-						// 29 KSL does not include 30 KSL (but would include an invalid 31 KSL if there were no 30 KSL)
-						if ($anniv->d === 1) {
-							$tmp = new JewishDate(array($anniv->y, 'CSH', 1));
-							if ($tmp->daysInMonth() === 29) {
-								$where .= " AND (d_day <= 1 AND d_mon = 3 OR d_day = 30 AND d_mon = 2)";
-							} else {
-								$where .= " AND d_day <= 1 AND d_mon = 3";
-							}
-						} elseif ($anniv->d === 30) {
-							$where .= " AND d_day >= 30 AND d_mon = 3";
-						} elseif ($anniv->d == 29 && $anniv->daysInMonth() === 29) {
-							$where .= " AND (d_day = 29 OR d_day > 30) AND d_mon = 3";
-						} else {
-							$where .= " AND d_day = :day AND d_mon = 3";
-							$args['day'] = $anniv->d;
-						}
-						break;
-					case 4:
-						// 1 TVT includes 30 KSL (if this year didn’t have 30 KSL)
-						if ($anniv->d === 1) {
-							$tmp = new JewishDate(array($anniv->y, 'KSL', 1));
-							if ($tmp->daysInMonth() === 29) {
-								$where .= " AND (d_day <=1 AND d_mon = 4 OR d_day = 30 AND d_mon = 3)";
-							} else {
-								$where .= " AND d_day <= 1 AND d_mon = 4";
-							}
-						} elseif ($anniv->d === $anniv->daysInMonth()) {
-							$where .= " AND d_day >= :day AND d_mon=4";
-							$args['day'] = $anniv->d;
+				case 2:
+					// 29 CSH does not include 30 CSH (but would include an invalid 31 CSH if there were no 30 CSH)
+					if ($anniv->d === 1) {
+						$where .= " AND d_day <= 1 AND d_mon = 2";
+					} elseif ($anniv->d === 30) {
+						$where .= " AND d_day >= 30 AND d_mon = 2";
+					} elseif ($anniv->d === 29 && $anniv->daysInMonth() === 29) {
+						$where .= " AND (d_day = 29 OR d_day > 30) AND d_mon = 2";
+					} else {
+						$where .= " AND d_day = :day AND d_mon = 2";
+						$args['day'] = $anniv->d;
+					}
+					break;
+				case 3:
+					// 1 KSL includes 30 CSH (if this year didn’t have 30 CSH)
+					// 29 KSL does not include 30 KSL (but would include an invalid 31 KSL if there were no 30 KSL)
+					if ($anniv->d === 1) {
+						$tmp = new JewishDate(array($anniv->y, 'CSH', 1));
+						if ($tmp->daysInMonth() === 29) {
+							$where .= " AND (d_day <= 1 AND d_mon = 3 OR d_day = 30 AND d_mon = 2)";
 						} else {
-							$where .= " AND d_day = :day AND d_mon=4";
-							$args['day'] = $anniv->d;
+							$where .= " AND d_day <= 1 AND d_mon = 3";
 						}
-						break;
-					case 7: // ADS includes ADR (non-leap)
-						if ($anniv->d === 1) {
-							$where .= " AND d_day <= 1";
-						} elseif ($anniv->d === $anniv->daysInMonth()) {
-							$where .= " AND d_day >= :day";
-							$args['day'] = $anniv->d;
+					} elseif ($anniv->d === 30) {
+						$where .= " AND d_day >= 30 AND d_mon = 3";
+					} elseif ($anniv->d == 29 && $anniv->daysInMonth() === 29) {
+						$where .= " AND (d_day = 29 OR d_day > 30) AND d_mon = 3";
+					} else {
+						$where .= " AND d_day = :day AND d_mon = 3";
+						$args['day'] = $anniv->d;
+					}
+					break;
+				case 4:
+					// 1 TVT includes 30 KSL (if this year didn’t have 30 KSL)
+					if ($anniv->d === 1) {
+						$tmp = new JewishDate(array($anniv->y, 'KSL', 1));
+						if ($tmp->daysInMonth() === 29) {
+							$where .= " AND (d_day <=1 AND d_mon = 4 OR d_day = 30 AND d_mon = 3)";
 						} else {
-							$where .= " AND d_day = :day";
-							$args['day'] = $anniv->d;
+							$where .= " AND d_day <= 1 AND d_mon = 4";
 						}
-						$where .= " AND (d_mon = 6 AND MOD(7 * d_year + 1, 19) >= 7 OR d_mon = 7)";
-						break;
-					case 8: // 1 NSN includes 30 ADR, if this year is non-leap
-						if ($anniv->d === 1) {
-							if ($anniv->isLeapYear()) {
-								$where .= " AND d_day <= 1 AND d_mon = 8";
-							} else {
-								$where .= " AND (d_day <= 1 AND d_mon = 8 OR d_day = 30 AND d_mon = 6)";
-							}
-						} elseif ($anniv->d === $anniv->daysInMonth()) {
-							$where .= " AND d_day >= :day AND d_mon = 8";
-							$args['day'] = $anniv->d;
+					} elseif ($anniv->d === $anniv->daysInMonth()) {
+						$where .= " AND d_day >= :day AND d_mon=4";
+						$args['day'] = $anniv->d;
+					} else {
+						$where .= " AND d_day = :day AND d_mon=4";
+						$args['day'] = $anniv->d;
+					}
+					break;
+				case 7: // ADS includes ADR (non-leap)
+					if ($anniv->d === 1) {
+						$where .= " AND d_day <= 1";
+					} elseif ($anniv->d === $anniv->daysInMonth()) {
+						$where .= " AND d_day >= :day";
+						$args['day'] = $anniv->d;
+					} else {
+						$where .= " AND d_day = :day";
+						$args['day'] = $anniv->d;
+					}
+					$where .= " AND (d_mon = 6 AND MOD(7 * d_year + 1, 19) >= 7 OR d_mon = 7)";
+					break;
+				case 8: // 1 NSN includes 30 ADR, if this year is non-leap
+					if ($anniv->d === 1) {
+						if ($anniv->isLeapYear()) {
+							$where .= " AND d_day <= 1 AND d_mon = 8";
 						} else {
-							$where .= " AND d_day = :day AND d_mon = 8";
-							$args['day'] = $anniv->d;
+							$where .= " AND (d_day <= 1 AND d_mon = 8 OR d_day = 30 AND d_mon = 6)";
 						}
-						break;
+					} elseif ($anniv->d === $anniv->daysInMonth()) {
+						$where .= " AND d_day >= :day AND d_mon = 8";
+						$args['day'] = $anniv->d;
+					} else {
+						$where .= " AND d_day = :day AND d_mon = 8";
+						$args['day'] = $anniv->d;
+					}
+					break;
 				}
 			}
 			// Only events in the past (includes dates without a year)

app/Functions/FunctionsExport.php

Switch Indentation +21 added lines, -21 removed lines

@@ -183,18 +183,18 @@ 
 	 */
 	public static function exportGedcom(Tree $tree, $gedout, $exportOptions) {
 		switch ($exportOptions['privatize']) {
-			case 'gedadmin':
-				$access_level = Auth::PRIV_NONE;
-				break;
-			case 'user':
-				$access_level = Auth::PRIV_USER;
-				break;
-			case 'visitor':
-				$access_level = Auth::PRIV_PRIVATE;
-				break;
-			case 'none':
-				$access_level = Auth::PRIV_HIDE;
-				break;
+		case 'gedadmin':
+			$access_level = Auth::PRIV_NONE;
+			break;
+		case 'user':
+			$access_level = Auth::PRIV_USER;
+			break;
+		case 'visitor':
+			$access_level = Auth::PRIV_PRIVATE;
+			break;
+		case 'none':
+			$access_level = Auth::PRIV_HIDE;
+			break;
 		}
 
 		$head = self::gedcomHeader($tree);
@@ -251,15 +251,15 @@ 
 
 		foreach ($rows as $row) {
 			switch ($row->type) {
-				case 'NOTE':
-					$record = Note::getInstance($row->xref, $tree, $row->gedcom);
-					break;
-				case 'REPO':
-					$record = Repository::getInstance($row->xref, $tree, $row->gedcom);
-					break;
-				default:
-					$record = GedcomRecord::getInstance($row->xref, $tree, $row->gedcom);
-					break;
+			case 'NOTE':
+				$record = Note::getInstance($row->xref, $tree, $row->gedcom);
+				break;
+			case 'REPO':
+				$record = Repository::getInstance($row->xref, $tree, $row->gedcom);
+				break;
+			default:
+				$record = GedcomRecord::getInstance($row->xref, $tree, $row->gedcom);
+				break;
 			}
 
 			$rec = $record->privatizeGedcom($access_level);

app/Functions/FunctionsMedia.php

Switch Indentation +11 added lines, -11 removed lines

@@ -34,17 +34,17 @@
 			$val = -1;
 		}
 		switch (substr($val, -1)) {
-			case 'g':
-			case 'G':
-				return (int) $val * 1024 * 1024 * 1024;
-			case 'm':
-			case 'M':
-				return (int) $val * 1024 * 1024;
-			case 'k':
-			case 'K':
-				return (int) $val * 1024;
-			default:
-				return (int) $val;
+		case 'g':
+		case 'G':
+			return (int) $val * 1024 * 1024 * 1024;
+		case 'm':
+		case 'M':
+			return (int) $val * 1024 * 1024;
+		case 'k':
+		case 'K':
+			return (int) $val * 1024;
+		default:
+			return (int) $val;
 		}
 	}
 

app/Functions/FunctionsPrint.php

Switch Indentation +60 added lines, -60 removed lines

@@ -50,20 +50,20 @@ 
 	public static function printPedigreePerson(Individual $person = null, $show_full = 1) {
 
 		switch ($show_full) {
-			case 0:
-				if ($person) {
-					echo Theme::theme()->individualBoxSmall($person);
-				} else {
-					echo Theme::theme()->individualBoxSmallEmpty();
-				}
-				break;
-			case 1:
-				if ($person) {
-					echo Theme::theme()->individualBox($person);
-				} else {
-					echo Theme::theme()->individualBoxEmpty();
-				}
-				break;
+		case 0:
+			if ($person) {
+				echo Theme::theme()->individualBoxSmall($person);
+			} else {
+				echo Theme::theme()->individualBoxSmallEmpty();
+			}
+			break;
+		case 1:
+			if ($person) {
+				echo Theme::theme()->individualBox($person);
+			} else {
+				echo Theme::theme()->individualBoxEmpty();
+			}
+			break;
 		}
 	}
 
@@ -259,25 +259,25 @@ 
 					$age      = Date::getAge($parent->getBirthDate(), $birth_date, 2);
 					$deatdate = $parent->getDeathDate();
 					switch ($parent->getSex()) {
-						case 'F':
-							// Highlight mothers who die in childbirth or shortly afterwards
-							if ($deatdate->isOK() && $deatdate->maximumJulianDay() < $birth_date->minimumJulianDay() + 90) {
-								$html .= ' <span title="' . GedcomTag::getLabel('_DEAT_PARE', $parent) . '" class="parentdeath">' . $sex . $age . '</span>';
-							} else {
-								$html .= ' <span title="' . I18N::translate('Mother’s age') . '">' . $sex . $age . '</span>';
-							}
-							break;
-						case 'M':
-							// Highlight fathers who die before the birth
-							if ($deatdate->isOK() && $deatdate->maximumJulianDay() < $birth_date->minimumJulianDay()) {
-								$html .= ' <span title="' . GedcomTag::getLabel('_DEAT_PARE', $parent) . '" class="parentdeath">' . $sex . $age . '</span>';
-							} else {
-								$html .= ' <span title="' . I18N::translate('Father’s age') . '">' . $sex . $age . '</span>';
-							}
-							break;
-						default:
-							$html .= ' <span title="' . I18N::translate('Parent’s age') . '">' . $sex . $age . '</span>';
-							break;
+					case 'F':
+						// Highlight mothers who die in childbirth or shortly afterwards
+						if ($deatdate->isOK() && $deatdate->maximumJulianDay() < $birth_date->minimumJulianDay() + 90) {
+							$html .= ' <span title="' . GedcomTag::getLabel('_DEAT_PARE', $parent) . '" class="parentdeath">' . $sex . $age . '</span>';
+						} else {
+							$html .= ' <span title="' . I18N::translate('Mother’s age') . '">' . $sex . $age . '</span>';
+						}
+						break;
+					case 'M':
+						// Highlight fathers who die before the birth
+						if ($deatdate->isOK() && $deatdate->maximumJulianDay() < $birth_date->minimumJulianDay()) {
+							$html .= ' <span title="' . GedcomTag::getLabel('_DEAT_PARE', $parent) . '" class="parentdeath">' . $sex . $age . '</span>';
+						} else {
+							$html .= ' <span title="' . I18N::translate('Father’s age') . '">' . $sex . $age . '</span>';
+						}
+						break;
+					default:
+						$html .= ' <span title="' . I18N::translate('Parent’s age') . '">' . $sex . $age . '</span>';
+						break;
 					}
 				}
 			}
@@ -578,33 +578,33 @@ 
 
 		// -- Add from pick list
 		switch ($type) {
-			case "INDI":
-				$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('INDI_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
-				$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('INDI_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
-				$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('INDI_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
-				break;
-			case "FAM":
-				$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('FAM_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
-				$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('FAM_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
-				$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('FAM_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
-				break;
-			case "SOUR":
-				$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('SOUR_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
-				$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('SOUR_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
-				$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('SOUR_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
-				break;
-			case "NOTE":
-				$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('NOTE_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
-				$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('NOTE_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
-				$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('NOTE_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
-				break;
-			case "REPO":
-				$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('REPO_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
-				$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('REPO_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
-				$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('REPO_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
-				break;
-			default:
-				return;
+		case "INDI":
+			$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('INDI_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
+			$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('INDI_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
+			$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('INDI_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
+			break;
+		case "FAM":
+			$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('FAM_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
+			$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('FAM_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
+			$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('FAM_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
+			break;
+		case "SOUR":
+			$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('SOUR_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
+			$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('SOUR_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
+			$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('SOUR_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
+			break;
+		case "NOTE":
+			$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('NOTE_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
+			$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('NOTE_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
+			$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('NOTE_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
+			break;
+		case "REPO":
+			$addfacts    = preg_split("/[, ;:]+/", $WT_TREE->getPreference('REPO_FACTS_ADD'), -1, PREG_SPLIT_NO_EMPTY);
+			$uniquefacts = preg_split("/[, ;:]+/", $WT_TREE->getPreference('REPO_FACTS_UNIQUE'), -1, PREG_SPLIT_NO_EMPTY);
+			$quickfacts  = preg_split("/[, ;:]+/", $WT_TREE->getPreference('REPO_FACTS_QUICK'), -1, PREG_SPLIT_NO_EMPTY);
+			break;
+		default:
+			return;
 		}
 		$addfacts            = array_merge(self::checkFactUnique($uniquefacts, $usedfacts, $type), $addfacts);
 		$quickfacts          = array_intersect($quickfacts, $addfacts);

app/Functions/FunctionsPrintLists.php

Switch Indentation +76 added lines, -76 removed lines

@@ -1687,37 +1687,37 @@ 
 
 		}
 		switch ($style) {
-			case 1:
-				return '<ul><li>' . implode('</li><li>', $html) . '</li></ul>';
-			case 2:
-				return implode(I18N::$list_separator, $html);
-			case 3:
-				$i     = 0;
-				$count = count($html);
-				if ($count > 36) {
-					$col = 4;
-				} elseif ($count > 18) {
-					$col = 3;
-				} elseif ($count > 6) {
-					$col = 2;
-				} else {
-					$col = 1;
-				}
-				$newcol = ceil($count / $col);
-				$html2  = '<table class="list_table"><tr>';
-				$html2 .= '<td class="list_value" style="padding: 14px;">';
-
-				foreach ($html as $surns) {
-					$html2 .= $surns . '<br>';
-					$i++;
-					if ($i == $newcol && $i < $count) {
-						$html2 .= '</td><td class="list_value" style="padding: 14px;">';
-						$newcol = $i + ceil($count / $col);
-					}
+		case 1:
+			return '<ul><li>' . implode('</li><li>', $html) . '</li></ul>';
+		case 2:
+			return implode(I18N::$list_separator, $html);
+		case 3:
+			$i     = 0;
+			$count = count($html);
+			if ($count > 36) {
+				$col = 4;
+			} elseif ($count > 18) {
+				$col = 3;
+			} elseif ($count > 6) {
+				$col = 2;
+			} else {
+				$col = 1;
+			}
+			$newcol = ceil($count / $col);
+			$html2  = '<table class="list_table"><tr>';
+			$html2 .= '<td class="list_value" style="padding: 14px;">';
+
+			foreach ($html as $surns) {
+				$html2 .= $surns . '<br>';
+				$i++;
+				if ($i == $newcol && $i < $count) {
+					$html2 .= '</td><td class="list_value" style="padding: 14px;">';
+					$newcol = $i + ceil($count / $col);
 				}
-				$html2 .= '</td></tr></table>';
+			}
+			$html2 .= '</td></tr></table>';
 
-				return $html2;
+			return $html2;
 		}
 	}
 
@@ -1747,15 +1747,15 @@ 
 		}
 
 		switch ($sort) {
-			case 'name':
-				uasort($arr, '\Fisharebest\Webtrees\Functions\Functions::eventSortName');
-				break;
-			case 'date_asc':
-				uasort($arr, '\Fisharebest\Webtrees\Functions\Functions::eventSort');
-				$arr = array_reverse($arr);
-				break;
-			case 'date_desc':
-				uasort($arr, '\Fisharebest\Webtrees\Functions\Functions::eventSort');
+		case 'name':
+			uasort($arr, '\Fisharebest\Webtrees\Functions\Functions::eventSortName');
+			break;
+		case 'date_asc':
+			uasort($arr, '\Fisharebest\Webtrees\Functions\Functions::eventSort');
+			$arr = array_reverse($arr);
+			break;
+		case 'date_desc':
+			uasort($arr, '\Fisharebest\Webtrees\Functions\Functions::eventSort');
 		}
 		$html = '';
 		foreach ($arr as $value) {
@@ -1788,15 +1788,15 @@ 
 		$n        = 0;
 		$table_id = 'table-chan-' . Uuid::uuid4(); // lists requires a unique ID in case there are multiple lists per page
 		switch ($sort) {
-			case 'name':        //name
-				$aaSorting = "[5,'asc'], [4,'desc']";
-				break;
-			case 'date_asc':    //date ascending
-				$aaSorting = "[4,'asc'], [5,'asc']";
-				break;
-			case 'date_desc':   //date descending
-				$aaSorting = "[4,'desc'], [5,'asc']";
-				break;
+		case 'name':        //name
+			$aaSorting = "[5,'asc'], [4,'desc']";
+			break;
+		case 'date_asc':    //date ascending
+			$aaSorting = "[4,'asc'], [5,'asc']";
+			break;
+		case 'date_desc':   //date descending
+			$aaSorting = "[4,'desc'], [5,'asc']";
+			break;
 		}
 		$html = '';
 		$controller
@@ -1843,27 +1843,27 @@ 
 			}
 			$html .= '<tr><td>';
 			switch ($record::RECORD_TYPE) {
-				case 'INDI':
-					$icon = $record->getSexImage('small');
-					break;
-				case 'FAM':
-					$icon = '<i class="icon-button_family"></i>';
-					break;
-				case 'OBJE':
-					$icon = '<i class="icon-button_media"></i>';
-					break;
-				case 'NOTE':
-					$icon = '<i class="icon-button_note"></i>';
-					break;
-				case 'SOUR':
-					$icon = '<i class="icon-button_source"></i>';
-					break;
-				case 'REPO':
-					$icon = '<i class="icon-button_repository"></i>';
-					break;
-				default:
-					$icon = '&nbsp;';
-					break;
+			case 'INDI':
+				$icon = $record->getSexImage('small');
+				break;
+			case 'FAM':
+				$icon = '<i class="icon-button_family"></i>';
+				break;
+			case 'OBJE':
+				$icon = '<i class="icon-button_media"></i>';
+				break;
+			case 'NOTE':
+				$icon = '<i class="icon-button_note"></i>';
+				break;
+			case 'SOUR':
+				$icon = '<i class="icon-button_source"></i>';
+				break;
+			case 'REPO':
+				$icon = '<i class="icon-button_repository"></i>';
+				break;
+			default:
+				$icon = '&nbsp;';
+				break;
 			}
 			$html .= '<a href="' . $record->getHtmlUrl() . '">' . $icon . '</a>';
 			$html .= '</td>';
@@ -2092,14 +2092,14 @@ 
 
 		// Now we've filtered the list, we can sort by event, if required
 		switch ($sort_by) {
-			case 'anniv':
-				// Data is already sorted by anniversary date
-				break;
-			case 'alpha':
-				uasort($filtered_events, function (Fact $x, Fact $y) {
-					return GedcomRecord::compare($x->getParent(), $y->getParent());
-				});
-				break;
+		case 'anniv':
+			// Data is already sorted by anniversary date
+			break;
+		case 'alpha':
+			uasort($filtered_events, function (Fact $x, Fact $y) {
+				return GedcomRecord::compare($x->getParent(), $y->getParent());
+			});
+			break;
 		}
 
 		foreach ($filtered_events as $fact) {