fisharebest / webtrees

app/Media.php

Switch Indentation +68 added lines, -68 removed lines

@@ -190,17 +190,17 @@ 
 					if (FunctionsMedia::hasMemoryForImage($main_file)) {
 						try {
 							switch ($imgsize['mime']) {
-							case 'image/png':
-								$main_image = imagecreatefrompng($main_file);
-								break;
-							case 'image/gif':
-								$main_image = imagecreatefromgif($main_file);
-								break;
-							case 'image/jpeg':
-								$main_image = imagecreatefromjpeg($main_file);
-								break;
-							default:
-								return $file; // Nothing else we can do :-(
+								case 'image/png':
+									$main_image = imagecreatefrompng($main_file);
+									break;
+								case 'image/gif':
+									$main_image = imagecreatefromgif($main_file);
+									break;
+								case 'image/jpeg':
+									$main_image = imagecreatefromjpeg($main_file);
+									break;
+								default:
+									return $file; // Nothing else we can do :-(
 							}
 							if ($main_image) {
 								// How big should the thumbnail be?
@@ -213,15 +213,15 @@ 
 								// Shrink the image
 								imagecopyresampled($thumb_image, $main_image, 0, 0, 0, 0, $width, $height, $imgsize[0], $imgsize[1]);
 								switch ($imgsize['mime']) {
-								case 'image/png':
-									imagepng($thumb_image, $file);
-									break;
-								case 'image/gif':
-									imagegif($thumb_image, $file);
-									break;
-								case 'image/jpeg':
-									imagejpeg($thumb_image, $file);
-									break;
+									case 'image/png':
+										imagepng($thumb_image, $file);
+										break;
+									case 'image/gif':
+										imagegif($thumb_image, $file);
+										break;
+									case 'image/jpeg':
+										imagejpeg($thumb_image, $file);
+										break;
 								}
 								imagedestroy($main_image);
 								imagedestroy($thumb_image);
@@ -490,54 +490,54 @@ 
 	public function mimeType() {
 		// Themes contain icon definitions for some/all of these mime-types
 		switch ($this->extension()) {
-		case 'bmp':
-			return 'image/bmp';
-		case 'doc':
-			return 'application/msword';
-		case 'docx':
-			return 'application/msword';
-		case 'ged':
-			return 'text/x-gedcom';
-		case 'gif':
-			return 'image/gif';
-		case 'htm':
-			return 'text/html';
-		case 'html':
-			return 'text/html';
-		case 'jpeg':
-			return 'image/jpeg';
-		case 'jpg':
-			return 'image/jpeg';
-		case 'mov':
-			return 'video/quicktime';
-		case 'mp3':
-			return 'audio/mpeg';
-		case 'ogv':
-			return 'video/ogg';
-		case 'pdf':
-			return 'application/pdf';
-		case 'png':
-			return 'image/png';
-		case 'rar':
-			return 'application/x-rar-compressed';
-		case 'swf':
-			return 'application/x-shockwave-flash';
-		case 'svg':
-			return 'image/svg';
-		case 'tif':
-			return 'image/tiff';
-		case 'tiff':
-			return 'image/tiff';
-		case 'xls':
-			return 'application/vnd-ms-excel';
-		case 'xlsx':
-			return 'application/vnd-ms-excel';
-		case 'wmv':
-			return 'video/x-ms-wmv';
-		case 'zip':
-			return 'application/zip';
-		default:
-			return 'application/octet-stream';
+			case 'bmp':
+				return 'image/bmp';
+			case 'doc':
+				return 'application/msword';
+			case 'docx':
+				return 'application/msword';
+			case 'ged':
+				return 'text/x-gedcom';
+			case 'gif':
+				return 'image/gif';
+			case 'htm':
+				return 'text/html';
+			case 'html':
+				return 'text/html';
+			case 'jpeg':
+				return 'image/jpeg';
+			case 'jpg':
+				return 'image/jpeg';
+			case 'mov':
+				return 'video/quicktime';
+			case 'mp3':
+				return 'audio/mpeg';
+			case 'ogv':
+				return 'video/ogg';
+			case 'pdf':
+				return 'application/pdf';
+			case 'png':
+				return 'image/png';
+			case 'rar':
+				return 'application/x-rar-compressed';
+			case 'swf':
+				return 'application/x-shockwave-flash';
+			case 'svg':
+				return 'image/svg';
+			case 'tif':
+				return 'image/tiff';
+			case 'tiff':
+				return 'image/tiff';
+			case 'xls':
+				return 'application/vnd-ms-excel';
+			case 'xlsx':
+				return 'application/vnd-ms-excel';
+			case 'wmv':
+				return 'video/x-ms-wmv';
+			case 'zip':
+				return 'application/zip';
+			default:
+				return 'application/octet-stream';
 		}
 	}
 

app/I18N.php

Switch Indentation +32 added lines, -32 removed lines

@@ -141,13 +141,13 @@ 
 	public static function collation() {
 		$collation = self::$locale->collation();
 		switch ($collation) {
-		case 'croatian_ci':
-		case 'german2_ci':
-		case 'vietnamese_ci':
-			// Only available in MySQL 5.6
-			return 'utf8_unicode_ci';
-		default:
-			return 'utf8_' . $collation;
+			case 'croatian_ci':
+			case 'german2_ci':
+			case 'vietnamese_ci':
+				// Only available in MySQL 5.6
+				return 'utf8_unicode_ci';
+			default:
+				return 'utf8_' . $collation;
 		}
 	}
 
@@ -283,15 +283,15 @@ 
 	 */
 	public static function gedcomAge($string) {
 		switch ($string) {
-		case 'STILLBORN':
-			// I18N: Description of an individual’s age at an event. For example, Died 14 Jan 1900 (stillborn)
-			return self::translate('(stillborn)');
-		case 'INFANT':
-			// I18N: Description of an individual’s age at an event. For example, Died 14 Jan 1900 (in infancy)
-			return self::translate('(in infancy)');
-		case 'CHILD':
-			// I18N: Description of an individual’s age at an event. For example, Died 14 Jan 1900 (in childhood)
-			return self::translate('(in childhood)');
+			case 'STILLBORN':
+				// I18N: Description of an individual’s age at an event. For example, Died 14 Jan 1900 (stillborn)
+				return self::translate('(stillborn)');
+			case 'INFANT':
+				// I18N: Description of an individual’s age at an event. For example, Died 14 Jan 1900 (in infancy)
+				return self::translate('(in infancy)');
+			case 'CHILD':
+				// I18N: Description of an individual’s age at an event. For example, Died 14 Jan 1900 (in childhood)
+				return self::translate('(in childhood)');
 		}
 		$age = array();
 		if (preg_match('/(\d+)y/', $string, $match)) {
@@ -591,13 +591,13 @@ 
 	 */
 	public static function scriptDirection($script) {
 		switch ($script) {
-		case 'Arab':
-		case 'Hebr':
-		case 'Mong':
-		case 'Thaa':
-			return 'rtl';
-		default:
-			return 'ltr';
+			case 'Arab':
+			case 'Hebr':
+			case 'Mong':
+			case 'Thaa':
+				return 'rtl';
+			default:
+				return 'ltr';
 		}
 	}
 
@@ -878,15 +878,15 @@ 
 	 */
 	public static function defaultCalendar() {
 		switch (self::$locale->languageTag()) {
-		case 'ar':
-			return new ArabicCalendar;
-		case 'fa':
-			return new PersianCalendar;
-		case 'he':
-		case 'yi':
-			return new JewishCalendar;
-		default:
-			return new GregorianCalendar;
+			case 'ar':
+				return new ArabicCalendar;
+			case 'fa':
+				return new PersianCalendar;
+			case 'he':
+			case 'yi':
+				return new JewishCalendar;
+			default:
+				return new GregorianCalendar;
 		}
 	}
 }

app/Individual.php

Switch Indentation +23 added lines, -23 removed lines

@@ -879,33 +879,33 @@
 	public function getPrimaryChildFamily() {
 		$families = $this->getChildFamilies();
 		switch (count($families)) {
-		case 0:
-			return null;
-		case 1:
-			return reset($families);
-		default:
-			// If there is more than one FAMC record, choose the preferred parents:
-			// a) records with '2 _PRIMARY'
-			foreach ($families as $famid => $fam) {
-				if (preg_match("/\n1 FAMC @{$famid}@\n(?:[2-9].*\n)*(?:2 _PRIMARY Y)/", $this->getGedcom())) {
-					return $fam;
+			case 0:
+				return null;
+			case 1:
+				return reset($families);
+			default:
+				// If there is more than one FAMC record, choose the preferred parents:
+				// a) records with '2 _PRIMARY'
+				foreach ($families as $famid => $fam) {
+					if (preg_match("/\n1 FAMC @{$famid}@\n(?:[2-9].*\n)*(?:2 _PRIMARY Y)/", $this->getGedcom())) {
+						return $fam;
+					}
 				}
-			}
-			// b) records with '2 PEDI birt'
-			foreach ($families as $famid => $fam) {
-				if (preg_match("/\n1 FAMC @{$famid}@\n(?:[2-9].*\n)*(?:2 PEDI birth)/", $this->getGedcom())) {
-					return $fam;
+				// b) records with '2 PEDI birt'
+				foreach ($families as $famid => $fam) {
+					if (preg_match("/\n1 FAMC @{$famid}@\n(?:[2-9].*\n)*(?:2 PEDI birth)/", $this->getGedcom())) {
+						return $fam;
+					}
 				}
-			}
-			// c) records with no '2 PEDI'
-			foreach ($families as $famid => $fam) {
-				if (!preg_match("/\n1 FAMC @{$famid}@\n(?:[2-9].*\n)*(?:2 PEDI)/", $this->getGedcom())) {
-					return $fam;
+				// c) records with no '2 PEDI'
+				foreach ($families as $famid => $fam) {
+					if (!preg_match("/\n1 FAMC @{$famid}@\n(?:[2-9].*\n)*(?:2 PEDI)/", $this->getGedcom())) {
+						return $fam;
+					}
 				}
-			}
 
-			// d) any record
-			return reset($families);
+				// d) any record
+				return reset($families);
 		}
 	}
 

app/Census/CensusColumnSexMF.php

Switch Indentation +6 added lines, -6 removed lines

@@ -37,12 +37,12 @@
 	 */
 	public function generate(Individual $individual, Individual $head = null) {
 		switch ($individual->getSex()) {
-		case 'M':
-			return $this->male;
-		case 'F':
-			return $this->female;
-		default:
-			return '';
+			case 'M':
+				return $this->male;
+			case 'F':
+				return $this->female;
+			default:
+				return '';
 		}
 	}
 }

app/Theme/WebtreesTheme.php

Switch Indentation +4 added lines, -4 removed lines

@@ -41,10 +41,10 @@
 	protected function flashMessageContainer(\stdClass $message) {
 		// This theme uses jQueryUI markup.
 		switch ($message->status) {
-		case 'danger':
-			return '<p class="ui-state-error">' . $message->text . '</p>';
-		default:
-			return '<p class="ui-state-highlight">' . $message->text . '</p>';
+			case 'danger':
+				return '<p class="ui-state-error">' . $message->text . '</p>';
+			default:
+				return '<p class="ui-state-highlight">' . $message->text . '</p>';
 		}
 	}
 

app/Theme/AbstractTheme.php

Switch Indentation +20 added lines, -20 removed lines

@@ -265,12 +265,12 @@ 
 		$method = $user->getPreference('contactmethod');
 
 		switch ($method) {
-		case 'none':
-			return '';
-		case 'mailto':
-			return '<a href="mailto:' . Filter::escapeHtml($user->getEmail()) . '">' . $user->getRealNameHtml() . '</a>';
-		default:
-			return "<a href='#' onclick='message(\"" . Filter::escapeHtml($user->getUserName()) . "\", \"" . $method . "\", \"" . WT_BASE_URL . Filter::escapeHtml(Functions::getQueryUrl()) . "\", \"\");return false;'>" . $user->getRealNameHtml() . '</a>';
+			case 'none':
+				return '';
+			case 'mailto':
+				return '<a href="mailto:' . Filter::escapeHtml($user->getEmail()) . '">' . $user->getRealNameHtml() . '</a>';
+			default:
+				return "<a href='#' onclick='message(\"" . Filter::escapeHtml($user->getUserName()) . "\", \"" . $method . "\", \"" . WT_BASE_URL . Filter::escapeHtml(Functions::getQueryUrl()) . "\", \"\");return false;'>" . $user->getRealNameHtml() . '</a>';
 		}
 	}
 
@@ -1346,20 +1346,20 @@ 
 		$records  = array();
 		foreach ($favorites as $favorite) {
 			switch ($favorite['type']) {
-			case 'URL':
-				$submenus[] = new Menu($favorite['title'], $favorite['url']);
-				break;
-			case 'INDI':
-			case 'FAM':
-			case 'SOUR':
-			case 'OBJE':
-			case 'NOTE':
-				$record = GedcomRecord::getInstance($favorite['gid'], $this->tree);
-				if ($record && $record->canShowName()) {
-					$submenus[] = new Menu($record->getFullName(), $record->getHtmlUrl());
-					$records[]  = $record;
-				}
-				break;
+				case 'URL':
+					$submenus[] = new Menu($favorite['title'], $favorite['url']);
+					break;
+				case 'INDI':
+				case 'FAM':
+				case 'SOUR':
+				case 'OBJE':
+				case 'NOTE':
+					$record = GedcomRecord::getInstance($favorite['gid'], $this->tree);
+					if ($record && $record->canShowName()) {
+						$submenus[] = new Menu($record->getFullName(), $record->getHtmlUrl());
+						$records[]  = $record;
+					}
+					break;
 			}
 		}
 

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;
-				}
-				// -- compact the tree
-				if ($curgen < $this->generations) {
-					if ($i % 2 == 0) {
-						$yoffset = $yoffset - (($boxspacing / 2) * ($curgen - 1));
-					} else {
-						$yoffset = $yoffset + (($boxspacing / 2) * ($curgen - 1));
+				case self::PORTRAIT:
+					$xoffset = ($this->generations - $curgen) * (($this->getBoxDimensions()->width + $bxspacing) / 1.8);
+					if (!$i && $this->root->getSpouseFamilies()) {
+						$xoffset -= self::ARROW_SIZE;
 					}
-					$parent = (int) (($i - 1) / 2);
-					$pgen   = $curgen;
-					while ($parent > 0) {
-						if ($parent % 2 == 0) {
-							$yoffset = $yoffset - (($boxspacing / 2) * $pgen);
+					// -- compact the tree
+					if ($curgen < $this->generations) {
+						if ($i % 2 == 0) {
+							$yoffset = $yoffset - (($boxspacing / 2) * ($curgen - 1));
 						} else {
-							$yoffset = $yoffset + (($boxspacing / 2) * $pgen);
+							$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);
 						}
-						$pgen++;
-						if ($pgen > 3) {
+						if ($curgen > 3) {
 							$temp = 0;
-							for ($j = 1; $j < ($pgen - 2); $j++) {
+							for ($j = 1; $j < ($curgen - 2); $j++) {
 								$temp += (pow(2, $j) - 1);
 							}
-							if ($parent % 2 == 0) {
+							if ($i % 2 == 0) {
 								$yoffset = $yoffset - (($boxspacing / 2) * $temp);
 							} else {
 								$yoffset = $yoffset + (($boxspacing / 2) * $temp);
 							}
 						}
-						$parent = (int) (($parent - 1) / 2);
 					}
-					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);
-						}
+					$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;
 					}
-				}
-				$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;
+					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/Controller/IndividualController.php

Switch Indentation +52 added lines, -52 removed lines

@@ -182,22 +182,22 @@ 
 							$name = str_replace('/', '', $name);
 							$name = preg_replace('/(\S*)\*/', '<span class="starredname">\\1</span>', $name);
 							switch ($fact) {
-							case 'TYPE':
-								echo GedcomCodeName::getValue($name, $this->record);
-								break;
-							case 'SURN':
-								// The SURN field is not necessarily the surname.
-								// Where it is not a substring of the real surname, show it after the real surname.
-								$surname = Filter::escapeHtml($primary_name['surname']);
-								if (strpos($primary_name['surname'], str_replace(',', ' ', $nmatch[$i][2])) !== false) {
-									echo '<span dir="auto">' . $surname . '</span>';
-								} else {
-									echo I18N::translate('%1$s (%2$s)', '<span dir="auto">' . $surname . '</span>', '<span dir="auto">' . $name . '</span>');
-								}
-								break;
-							default:
-								echo '<span dir="auto">' . $name . '</span>';
-								break;
+								case 'TYPE':
+									echo GedcomCodeName::getValue($name, $this->record);
+									break;
+								case 'SURN':
+									// The SURN field is not necessarily the surname.
+									// Where it is not a substring of the real surname, show it after the real surname.
+									$surname = Filter::escapeHtml($primary_name['surname']);
+									if (strpos($primary_name['surname'], str_replace(',', ' ', $nmatch[$i][2])) !== false) {
+										echo '<span dir="auto">' . $surname . '</span>';
+									} else {
+										echo I18N::translate('%1$s (%2$s)', '<span dir="auto">' . $surname . '</span>', '<span dir="auto">' . $name . '</span>');
+									}
+									break;
+								default:
+									echo '<span dir="auto">' . $name . '</span>';
+									break;
 							}
 						}
 					echo '</dd>';
@@ -232,33 +232,33 @@ 
 			echo 'new ';
 		}
 		switch ($sex) {
-		case 'M':
-			echo 'male_gender"';
-			if ($event->canEdit()) {
-				echo ' title="', I18N::translate('Male'), ' - ', I18N::translate('Edit'), '"';
-				echo ' onclick="edit_record(\'' . $this->record->getXref() . '\', \'' . $event->getFactId() . '\'); return false;">';
-			 } else {
-				echo ' title="', I18N::translate('Male'), '">';
-			 }
-			break;
-		case 'F':
-			echo 'female_gender"';
-			if ($event->canEdit()) {
-				echo ' title="', I18N::translate('Female'), ' - ', I18N::translate('Edit'), '"';
-				echo ' onclick="edit_record(\'' . $this->record->getXref() . '\', \'' . $event->getFactId() . '\'); return false;">';
-			 } else {
-				echo ' title="', I18N::translate('Female'), '">';
-			 }
-			break;
-		default:
-			echo 'unknown_gender"';
-			if ($event->canEdit()) {
-				echo ' title="', I18N::translateContext('unknown gender', 'Unknown'), ' - ', I18N::translate('Edit'), '"';
-				echo ' onclick="edit_record(\'' . $this->record->getXref() . '\', \'' . $event->getFactId() . '\'); return false;">';
-			 } else {
-				echo ' title="', I18N::translateContext('unknown gender', 'Unknown'), '">';
-			 }
-			break;
+			case 'M':
+				echo 'male_gender"';
+				if ($event->canEdit()) {
+					echo ' title="', I18N::translate('Male'), ' - ', I18N::translate('Edit'), '"';
+					echo ' onclick="edit_record(\'' . $this->record->getXref() . '\', \'' . $event->getFactId() . '\'); return false;">';
+			 	} else {
+					echo ' title="', I18N::translate('Male'), '">';
+			 	}
+				break;
+			case 'F':
+				echo 'female_gender"';
+				if ($event->canEdit()) {
+					echo ' title="', I18N::translate('Female'), ' - ', I18N::translate('Edit'), '"';
+					echo ' onclick="edit_record(\'' . $this->record->getXref() . '\', \'' . $event->getFactId() . '\'); return false;">';
+			 	} else {
+					echo ' title="', I18N::translate('Female'), '">';
+			 	}
+				break;
+			default:
+				echo 'unknown_gender"';
+				if ($event->canEdit()) {
+					echo ' title="', I18N::translateContext('unknown gender', 'Unknown'), ' - ', I18N::translate('Edit'), '"';
+					echo ' onclick="edit_record(\'' . $this->record->getXref() . '\', \'' . $event->getFactId() . '\'); return false;">';
+			 	} else {
+					echo ' title="', I18N::translateContext('unknown gender', 'Unknown'), '">';
+			 	}
+				break;
 		}
 		echo '</span>';
 	}
@@ -324,15 +324,15 @@ 
 	 */
 	public function getPersonStyle($person) {
 		switch ($person->getSex()) {
-		case 'M':
-			$class = 'person_box';
-			break;
-		case 'F':
-			$class = 'person_boxF';
-			break;
-		default:
-			$class = 'person_boxNN';
-			break;
+			case 'M':
+				$class = 'person_box';
+				break;
+			case 'F':
+				$class = 'person_boxF';
+				break;
+			default:
+				$class = 'person_boxNN';
+				break;
 		}
 		if ($person->isPendingDeletion()) {
 			$class .= ' old';

app/Controller/FanchartController.php

Switch Indentation +19 added lines, -19 removed lines

@@ -247,15 +247,15 @@ 
 					$text .= "\n" . I18N::reverseText($person->getLifeSpan());
 
 					switch ($person->getSex()) {
-					case 'M':
-						$bg = $bgcolorM;
-						break;
-					case 'F':
-						$bg = $bgcolorF;
-						break;
-					default:
-						$bg = $bgcolor;
-						break;
+						case 'M':
+							$bg = $bgcolorM;
+							break;
+						case 'F':
+							$bg = $bgcolorF;
+							break;
+						default:
+							$bg = $bgcolor;
+							break;
 					}
 
 					imagefilledarc($image, $cx, $cy, $rx, $rx, $deg1, $deg2, $bg, IMG_ARC_PIE);
@@ -366,19 +366,19 @@ 
 		$imagemap .= '</map>';
 
 		switch ($what) {
-		case 'html':
-			return $html . $imagemap . '<div id="fan_chart_img"><img src="' . WT_SCRIPT_NAME . '?rootid=' . $this->root->getXref() . '&amp;fan_style=' . $this->fan_style . '&amp;generations=' . $this->generations . '&amp;fan_width=' . $this->fan_width . '&amp;img=1" width="' . $fanw . '" height="' . $fanh . '" alt="' . strip_tags($this->getPageTitle()) . '" usemap="#fanmap"></div>';
+			case 'html':
+				return $html . $imagemap . '<div id="fan_chart_img"><img src="' . WT_SCRIPT_NAME . '?rootid=' . $this->root->getXref() . '&amp;fan_style=' . $this->fan_style . '&amp;generations=' . $this->generations . '&amp;fan_width=' . $this->fan_width . '&amp;img=1" width="' . $fanw . '" height="' . $fanh . '" alt="' . strip_tags($this->getPageTitle()) . '" usemap="#fanmap"></div>';
 
-		case 'png':
-			imagestringup($image, 1, $fanw - 10, $fanh / 3, WT_BASE_URL, $color);
-			ob_start();
-			imagepng($image);
-			imagedestroy($image);
+			case 'png':
+				imagestringup($image, 1, $fanw - 10, $fanh / 3, WT_BASE_URL, $color);
+				ob_start();
+				imagepng($image);
+				imagedestroy($image);
 
-			return ob_get_clean();
+				return ob_get_clean();
 
-		default:
-			throw new \InvalidArgumentException(__METHOD__ . ' ' . $what);
+			default:
+				throw new \InvalidArgumentException(__METHOD__ . ' ' . $what);
 		}
 	}
 }