fisharebest / webtrees

app/Module/YahrzeitModule.php

Indentation +209 added lines, -209 removed lines

@@ -32,111 +32,111 @@ 
  * Class YahrzeitModule
  */
 class YahrzeitModule extends AbstractModule implements ModuleBlockInterface {
-	/** {@inheritdoc} */
-	public function getTitle() {
-		return /* I18N: Name of a module. Yahrzeiten (the plural of Yahrzeit) are special anniversaries of deaths in the Hebrew faith/calendar. */ I18N::translate('Yahrzeiten');
-	}
+    /** {@inheritdoc} */
+    public function getTitle() {
+        return /* I18N: Name of a module. Yahrzeiten (the plural of Yahrzeit) are special anniversaries of deaths in the Hebrew faith/calendar. */ I18N::translate('Yahrzeiten');
+    }
 
-	/** {@inheritdoc} */
-	public function getDescription() {
-		return /* I18N: Description of the “Yahrzeiten” module. A “Hebrew death” is a death where the date is recorded in the Hebrew calendar. */ I18N::translate('A list of the Hebrew death anniversaries that will occur in the near future.');
-	}
+    /** {@inheritdoc} */
+    public function getDescription() {
+        return /* I18N: Description of the “Yahrzeiten” module. A “Hebrew death” is a death where the date is recorded in the Hebrew calendar. */ I18N::translate('A list of the Hebrew death anniversaries that will occur in the near future.');
+    }
 
-	/**
-	 * Generate the HTML content of this block.
-	 *
-	 * @param int      $block_id
-	 * @param bool     $template
-	 * @param string[] $cfg
-	 *
-	 * @return string
-	 */
-	public function getBlock($block_id, $template = true, $cfg = array()) {
-		global $ctype, $controller, $WT_TREE;
+    /**
+     * Generate the HTML content of this block.
+     *
+     * @param int      $block_id
+     * @param bool     $template
+     * @param string[] $cfg
+     *
+     * @return string
+     */
+    public function getBlock($block_id, $template = true, $cfg = array()) {
+        global $ctype, $controller, $WT_TREE;
 
-		$days      = $this->getBlockSetting($block_id, 'days', '7');
-		$infoStyle = $this->getBlockSetting($block_id, 'infoStyle', 'table');
-		$calendar  = $this->getBlockSetting($block_id, 'calendar', 'jewish');
-		$block     = $this->getBlockSetting($block_id, 'block', '1');
+        $days      = $this->getBlockSetting($block_id, 'days', '7');
+        $infoStyle = $this->getBlockSetting($block_id, 'infoStyle', 'table');
+        $calendar  = $this->getBlockSetting($block_id, 'calendar', 'jewish');
+        $block     = $this->getBlockSetting($block_id, 'block', '1');
 
-		foreach (array('days', 'infoStyle', 'block') as $name) {
-			if (array_key_exists($name, $cfg)) {
-				$$name = $cfg[$name];
-			}
-		}
+        foreach (array('days', 'infoStyle', 'block') as $name) {
+            if (array_key_exists($name, $cfg)) {
+                $$name = $cfg[$name];
+            }
+        }
 
-		$startjd = WT_CLIENT_JD;
-		$endjd   = WT_CLIENT_JD + $days - 1;
+        $startjd = WT_CLIENT_JD;
+        $endjd   = WT_CLIENT_JD + $days - 1;
 
-		$id    = $this->getName() . $block_id;
-		$class = $this->getName() . '_block';
-		if ($ctype === 'gedcom' && Auth::isManager($WT_TREE) || $ctype === 'user' && Auth::check()) {
-			$title = '<a class="icon-admin" title="' . I18N::translate('Preferences') . '" href="block_edit.php?block_id=' . $block_id . '&amp;ged=' . $WT_TREE->getNameHtml() . '&amp;ctype=' . $ctype . '"></a>';
-		} else {
-			$title = '';
-		}
-		$title .= $this->getTitle();
+        $id    = $this->getName() . $block_id;
+        $class = $this->getName() . '_block';
+        if ($ctype === 'gedcom' && Auth::isManager($WT_TREE) || $ctype === 'user' && Auth::check()) {
+            $title = '<a class="icon-admin" title="' . I18N::translate('Preferences') . '" href="block_edit.php?block_id=' . $block_id . '&amp;ged=' . $WT_TREE->getNameHtml() . '&amp;ctype=' . $ctype . '"></a>';
+        } else {
+            $title = '';
+        }
+        $title .= $this->getTitle();
 
-		$content = '';
-		// The standard anniversary rules cover most of the Yahrzeit rules, we just
-		// need to handle a few special cases.
-		// Fetch normal anniversaries...
-		$yahrzeits = array();
-		for ($jd = $startjd - 1; $jd <= $endjd + $days; ++$jd) {
-			foreach (FunctionsDb::getAnniversaryEvents($jd, 'DEAT _YART', $WT_TREE) as $fact) {
-				// Exact hebrew dates only
-				$date = $fact->getDate();
-				if ($date->minimumDate() instanceof JewishDate && $date->minimumJulianDay() === $date->maximumJulianDay()) {
-					$fact->jd    = $jd;
-					$yahrzeits[] = $fact;
-				}
-			}
-		}
+        $content = '';
+        // The standard anniversary rules cover most of the Yahrzeit rules, we just
+        // need to handle a few special cases.
+        // Fetch normal anniversaries...
+        $yahrzeits = array();
+        for ($jd = $startjd - 1; $jd <= $endjd + $days; ++$jd) {
+            foreach (FunctionsDb::getAnniversaryEvents($jd, 'DEAT _YART', $WT_TREE) as $fact) {
+                // Exact hebrew dates only
+                $date = $fact->getDate();
+                if ($date->minimumDate() instanceof JewishDate && $date->minimumJulianDay() === $date->maximumJulianDay()) {
+                    $fact->jd    = $jd;
+                    $yahrzeits[] = $fact;
+                }
+            }
+        }
 
-		// ...then adjust dates
-		$jewish_calendar = new JewishCalendar;
+        // ...then adjust dates
+        $jewish_calendar = new JewishCalendar;
 
-		foreach ($yahrzeits as $yahrzeit) {
-			if ($yahrzeit->getTag() === 'DEAT') {
-				$today = new JewishDate($yahrzeit->jd);
-				$hd    = $yahrzeit->getDate()->minimumDate();
-				$hd1   = new JewishDate($hd);
-				$hd1->y += 1;
-				$hd1->setJdFromYmd();
-				// Special rules. See http://www.hebcal.com/help/anniv.html
-				// Everything else is taken care of by our standard anniversary rules.
-				if ($hd->d == 30 && $hd->m == 2 && $hd->y != 0 && $hd1->daysInMonth() < 30) {
-					// 30 CSH - Last day in CSH
-					$yahrzeit->jd = $jewish_calendar->ymdToJd($today->y, 3, 1) - 1;
-				} elseif ($hd->d == 30 && $hd->m == 3 && $hd->y != 0 && $hd1->daysInMonth() < 30) {
-					// 30 KSL - Last day in KSL
-					$yahrzeit->jd = $jewish_calendar->ymdToJd($today->y, 4, 1) - 1;
-				} elseif ($hd->d == 30 && $hd->m == 6 && $hd->y != 0 && $today->daysInMonth() < 30 && !$today->isLeapYear()) {
-					// 30 ADR - Last day in SHV
-					$yahrzeit->jd = $jewish_calendar->ymdToJd($today->y, 6, 1) - 1;
-				}
-			}
-		}
+        foreach ($yahrzeits as $yahrzeit) {
+            if ($yahrzeit->getTag() === 'DEAT') {
+                $today = new JewishDate($yahrzeit->jd);
+                $hd    = $yahrzeit->getDate()->minimumDate();
+                $hd1   = new JewishDate($hd);
+                $hd1->y += 1;
+                $hd1->setJdFromYmd();
+                // Special rules. See http://www.hebcal.com/help/anniv.html
+                // Everything else is taken care of by our standard anniversary rules.
+                if ($hd->d == 30 && $hd->m == 2 && $hd->y != 0 && $hd1->daysInMonth() < 30) {
+                    // 30 CSH - Last day in CSH
+                    $yahrzeit->jd = $jewish_calendar->ymdToJd($today->y, 3, 1) - 1;
+                } elseif ($hd->d == 30 && $hd->m == 3 && $hd->y != 0 && $hd1->daysInMonth() < 30) {
+                    // 30 KSL - Last day in KSL
+                    $yahrzeit->jd = $jewish_calendar->ymdToJd($today->y, 4, 1) - 1;
+                } elseif ($hd->d == 30 && $hd->m == 6 && $hd->y != 0 && $today->daysInMonth() < 30 && !$today->isLeapYear()) {
+                    // 30 ADR - Last day in SHV
+                    $yahrzeit->jd = $jewish_calendar->ymdToJd($today->y, 6, 1) - 1;
+                }
+            }
+        }
 
-		switch ($infoStyle) {
-		case 'list':
-			foreach ($yahrzeits as $yahrzeit) {
-				if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
-					$ind = $yahrzeit->getParent();
-					$content .= "<a href=\"" . $ind->getHtmlUrl() . "\" class=\"list_item name2\">" . $ind->getFullName() . "</a>" . $ind->getSexImage();
-					$content .= "<div class=\"indent\">";
-					$content .= $yahrzeit->getDate()->display(true);
-					$content .= ', ' . I18N::translate('%s year anniversary', $yahrzeit->anniv);
-					$content .= "</div>";
-				}
-			}
-			break;
-		case 'table':
-		default:
-			$table_id = Uuid::uuid4(); // table requires a unique ID
-			$controller
-				->addExternalJavascript(WT_JQUERY_DATATABLES_JS_URL)
-				->addInlineJavascript('
+        switch ($infoStyle) {
+        case 'list':
+            foreach ($yahrzeits as $yahrzeit) {
+                if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
+                    $ind = $yahrzeit->getParent();
+                    $content .= "<a href=\"" . $ind->getHtmlUrl() . "\" class=\"list_item name2\">" . $ind->getFullName() . "</a>" . $ind->getSexImage();
+                    $content .= "<div class=\"indent\">";
+                    $content .= $yahrzeit->getDate()->display(true);
+                    $content .= ', ' . I18N::translate('%s year anniversary', $yahrzeit->anniv);
+                    $content .= "</div>";
+                }
+            }
+            break;
+        case 'table':
+        default:
+            $table_id = Uuid::uuid4(); // table requires a unique ID
+            $controller
+                ->addExternalJavascript(WT_JQUERY_DATATABLES_JS_URL)
+                ->addInlineJavascript('
 					jQuery("#' . $table_id . '").dataTable({
 						dom: \'t\',
 						' . I18N::datatablesI18N() . ',
@@ -160,135 +160,135 @@ 
 					jQuery("#' . $table_id . '").css("visibility", "visible");
 					jQuery(".loading-image").css("display", "none");
 				');
-			$content = '';
-			$content .= '<div class="loading-image">&nbsp;</div>';
-			$content .= '<table id="' . $table_id . '" class="width100" style="visibility:hidden;">';
-			$content .= '<thead><tr>';
-			$content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
-			$content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
-			$content .= '<th>' . GedcomTag::getLabel('DEAT') . '</th>';
-			$content .= '<th>DEAT</th>';
-			$content .= '<th><i class="icon-reminder" title="' . I18N::translate('Anniversary') . '"></i></th>';
-			$content .= '<th>' . GedcomTag::getLabel('_YART') . '</th>';
-			$content .= '<th>_YART</th>';
-			$content .= '</tr></thead><tbody>';
+            $content = '';
+            $content .= '<div class="loading-image">&nbsp;</div>';
+            $content .= '<table id="' . $table_id . '" class="width100" style="visibility:hidden;">';
+            $content .= '<thead><tr>';
+            $content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
+            $content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
+            $content .= '<th>' . GedcomTag::getLabel('DEAT') . '</th>';
+            $content .= '<th>DEAT</th>';
+            $content .= '<th><i class="icon-reminder" title="' . I18N::translate('Anniversary') . '"></i></th>';
+            $content .= '<th>' . GedcomTag::getLabel('_YART') . '</th>';
+            $content .= '<th>_YART</th>';
+            $content .= '</tr></thead><tbody>';
 
-			foreach ($yahrzeits as $yahrzeit) {
-				if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
-					$content .= '<tr>';
-					$ind = $yahrzeit->getParent();
-					// Individual name(s)
-					$name = $ind->getFullName();
-					$url  = $ind->getHtmlUrl();
-					$content .= '<td>';
-					$content .= '<a href="' . $url . '">' . $name . '</a>';
-					$content .= $ind->getSexImage();
-					$addname = $ind->getAddName();
-					if ($addname) {
-						$content .= '<br><a href="' . $url . '">' . $addname . '</a>';
-					}
-					$content .= '</td>';
-					$content .= '<td>' . $ind->getSortName() . '</td>';
+            foreach ($yahrzeits as $yahrzeit) {
+                if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
+                    $content .= '<tr>';
+                    $ind = $yahrzeit->getParent();
+                    // Individual name(s)
+                    $name = $ind->getFullName();
+                    $url  = $ind->getHtmlUrl();
+                    $content .= '<td>';
+                    $content .= '<a href="' . $url . '">' . $name . '</a>';
+                    $content .= $ind->getSexImage();
+                    $addname = $ind->getAddName();
+                    if ($addname) {
+                        $content .= '<br><a href="' . $url . '">' . $addname . '</a>';
+                    }
+                    $content .= '</td>';
+                    $content .= '<td>' . $ind->getSortName() . '</td>';
 
-					// death/yahrzeit event date
-					$content .= '<td>' . $yahrzeit->getDate()->display() . '</td>';
-					$content .= '<td>' . $yahrzeit->getDate()->julianDay() . '</td>'; // sortable date
+                    // death/yahrzeit event date
+                    $content .= '<td>' . $yahrzeit->getDate()->display() . '</td>';
+                    $content .= '<td>' . $yahrzeit->getDate()->julianDay() . '</td>'; // sortable date
 
-					// Anniversary
-					$content .= '<td>' . $yahrzeit->anniv . '</td>';
+                    // Anniversary
+                    $content .= '<td>' . $yahrzeit->anniv . '</td>';
 
-					// upcomming yahrzeit dates
-					switch ($calendar) {
-					case 'gregorian':
-						$today = new GregorianDate($yahrzeit->jd);
-						break;
-					case 'jewish':
-					default:
-						$today = new JewishDate($yahrzeit->jd);
-						break;
-					}
-					$td = new Date($today->format('%@ %A %O %E'));
-					$content .= '<td>' . $td->display() . '</td>';
-					$content .= '<td>' . $td->julianDay() . '</td>'; // sortable date
+                    // upcomming yahrzeit dates
+                    switch ($calendar) {
+                    case 'gregorian':
+                        $today = new GregorianDate($yahrzeit->jd);
+                        break;
+                    case 'jewish':
+                    default:
+                        $today = new JewishDate($yahrzeit->jd);
+                        break;
+                    }
+                    $td = new Date($today->format('%@ %A %O %E'));
+                    $content .= '<td>' . $td->display() . '</td>';
+                    $content .= '<td>' . $td->julianDay() . '</td>'; // sortable date
 
-					$content .= '</tr>';
-				}
-			}
-			$content .= '</tbody></table>';
+                    $content .= '</tr>';
+                }
+            }
+            $content .= '</tbody></table>';
 
-			break;
-		}
+            break;
+        }
 
-		if ($template) {
-			if ($block) {
-				$class .= ' small_inner_block';
-			}
+        if ($template) {
+            if ($block) {
+                $class .= ' small_inner_block';
+            }
 
-			return Theme::theme()->formatBlock($id, $title, $class, $content);
-		} else {
-			return $content;
-		}
-	}
+            return Theme::theme()->formatBlock($id, $title, $class, $content);
+        } else {
+            return $content;
+        }
+    }
 
-	/** {@inheritdoc} */
-	public function loadAjax() {
-		return true;
-	}
+    /** {@inheritdoc} */
+    public function loadAjax() {
+        return true;
+    }
 
-	/** {@inheritdoc} */
-	public function isUserBlock() {
-		return true;
-	}
+    /** {@inheritdoc} */
+    public function isUserBlock() {
+        return true;
+    }
 
-	/** {@inheritdoc} */
-	public function isGedcomBlock() {
-		return true;
-	}
+    /** {@inheritdoc} */
+    public function isGedcomBlock() {
+        return true;
+    }
 
-	/**
-	 * An HTML form to edit block settings
-	 *
-	 * @param int $block_id
-	 */
-	public function configureBlock($block_id) {
-		if (Filter::postBool('save') && Filter::checkCsrf()) {
-			$this->setBlockSetting($block_id, 'days', Filter::postInteger('days', 1, 30, 7));
-			$this->setBlockSetting($block_id, 'infoStyle', Filter::post('infoStyle', 'list|table', 'table'));
-			$this->setBlockSetting($block_id, 'calendar', Filter::post('calendar', 'jewish|gregorian', 'jewish'));
-			$this->setBlockSetting($block_id, 'block', Filter::postBool('block'));
-		}
+    /**
+     * An HTML form to edit block settings
+     *
+     * @param int $block_id
+     */
+    public function configureBlock($block_id) {
+        if (Filter::postBool('save') && Filter::checkCsrf()) {
+            $this->setBlockSetting($block_id, 'days', Filter::postInteger('days', 1, 30, 7));
+            $this->setBlockSetting($block_id, 'infoStyle', Filter::post('infoStyle', 'list|table', 'table'));
+            $this->setBlockSetting($block_id, 'calendar', Filter::post('calendar', 'jewish|gregorian', 'jewish'));
+            $this->setBlockSetting($block_id, 'block', Filter::postBool('block'));
+        }
 
-		$days      = $this->getBlockSetting($block_id, 'days', '7');
-		$infoStyle = $this->getBlockSetting($block_id, 'infoStyle', 'table');
-		$calendar  = $this->getBlockSetting($block_id, 'calendar', 'jewish');
-		$block     = $this->getBlockSetting($block_id, 'block', '1');
+        $days      = $this->getBlockSetting($block_id, 'days', '7');
+        $infoStyle = $this->getBlockSetting($block_id, 'infoStyle', 'table');
+        $calendar  = $this->getBlockSetting($block_id, 'calendar', 'jewish');
+        $block     = $this->getBlockSetting($block_id, 'block', '1');
 
-		echo '<tr><td class="descriptionbox wrap width33">';
-		echo I18N::translate('Number of days to show');
-		echo '</td><td class="optionbox">';
-		echo '<input type="text" name="days" size="2" value="' . $days . '">';
-		echo ' <em>', I18N::plural('maximum %s day', 'maximum %s days', 30, I18N::number(30)), '</em>';
-		echo '</td></tr>';
+        echo '<tr><td class="descriptionbox wrap width33">';
+        echo I18N::translate('Number of days to show');
+        echo '</td><td class="optionbox">';
+        echo '<input type="text" name="days" size="2" value="' . $days . '">';
+        echo ' <em>', I18N::plural('maximum %s day', 'maximum %s days', 30, I18N::number(30)), '</em>';
+        echo '</td></tr>';
 
-		echo '<tr><td class="descriptionbox wrap width33">';
-		echo I18N::translate('Presentation style');
-		echo '</td><td class="optionbox">';
-		echo FunctionsEdit::selectEditControl('infoStyle', array('list' => I18N::translate('list'), 'table' => I18N::translate('table')), null, $infoStyle, '');
-		echo '</td></tr>';
+        echo '<tr><td class="descriptionbox wrap width33">';
+        echo I18N::translate('Presentation style');
+        echo '</td><td class="optionbox">';
+        echo FunctionsEdit::selectEditControl('infoStyle', array('list' => I18N::translate('list'), 'table' => I18N::translate('table')), null, $infoStyle, '');
+        echo '</td></tr>';
 
-		echo '<tr><td class="descriptionbox wrap width33">';
-		echo I18N::translate('Calendar');
-		echo '</td><td class="optionbox">';
-		echo FunctionsEdit::selectEditControl('calendar', array(
-			'jewish'    => /* I18N: The Hebrew/Jewish calendar */ I18N::translate('Jewish'),
-			'gregorian' => /* I18N: The gregorian calendar */ I18N::translate('Gregorian'),
-		), null, $calendar, '');
-		echo '</td></tr>';
+        echo '<tr><td class="descriptionbox wrap width33">';
+        echo I18N::translate('Calendar');
+        echo '</td><td class="optionbox">';
+        echo FunctionsEdit::selectEditControl('calendar', array(
+            'jewish'    => /* I18N: The Hebrew/Jewish calendar */ I18N::translate('Jewish'),
+            'gregorian' => /* I18N: The gregorian calendar */ I18N::translate('Gregorian'),
+        ), null, $calendar, '');
+        echo '</td></tr>';
 
-		echo '<tr><td class="descriptionbox wrap width33">';
-		echo /* I18N: label for a yes/no option */ I18N::translate('Add a scrollbar when block contents grow');
-		echo '</td><td class="optionbox">';
-		echo FunctionsEdit::editFieldYesNo('block', $block);
-		echo '</td></tr>';
-	}
+        echo '<tr><td class="descriptionbox wrap width33">';
+        echo /* I18N: label for a yes/no option */ I18N::translate('Add a scrollbar when block contents grow');
+        echo '</td><td class="optionbox">';
+        echo FunctionsEdit::editFieldYesNo('block', $block);
+        echo '</td></tr>';
+    }
 }

Switch Indentation +61 added lines, -61 removed lines

@@ -119,24 +119,24 @@ 
 		}
 
 		switch ($infoStyle) {
-		case 'list':
-			foreach ($yahrzeits as $yahrzeit) {
-				if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
-					$ind = $yahrzeit->getParent();
-					$content .= "<a href=\"" . $ind->getHtmlUrl() . "\" class=\"list_item name2\">" . $ind->getFullName() . "</a>" . $ind->getSexImage();
-					$content .= "<div class=\"indent\">";
-					$content .= $yahrzeit->getDate()->display(true);
-					$content .= ', ' . I18N::translate('%s year anniversary', $yahrzeit->anniv);
-					$content .= "</div>";
-				}
-			}
-			break;
-		case 'table':
-		default:
-			$table_id = Uuid::uuid4(); // table requires a unique ID
-			$controller
-				->addExternalJavascript(WT_JQUERY_DATATABLES_JS_URL)
-				->addInlineJavascript('
+		    case 'list':
+			    foreach ($yahrzeits as $yahrzeit) {
+				    if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
+					    $ind = $yahrzeit->getParent();
+					    $content .= "<a href=\"" . $ind->getHtmlUrl() . "\" class=\"list_item name2\">" . $ind->getFullName() . "</a>" . $ind->getSexImage();
+					    $content .= "<div class=\"indent\">";
+					    $content .= $yahrzeit->getDate()->display(true);
+					    $content .= ', ' . I18N::translate('%s year anniversary', $yahrzeit->anniv);
+					    $content .= "</div>";
+				    }
+			    }
+			    break;
+		    case 'table':
+		    default:
+			    $table_id = Uuid::uuid4(); // table requires a unique ID
+			    $controller
+				    ->addExternalJavascript(WT_JQUERY_DATATABLES_JS_URL)
+				    ->addInlineJavascript('
 					jQuery("#' . $table_id . '").dataTable({
 						dom: \'t\',
 						' . I18N::datatablesI18N() . ',
@@ -160,53 +160,53 @@ 
 					jQuery("#' . $table_id . '").css("visibility", "visible");
 					jQuery(".loading-image").css("display", "none");
 				');
-			$content = '';
-			$content .= '<div class="loading-image">&nbsp;</div>';
-			$content .= '<table id="' . $table_id . '" class="width100" style="visibility:hidden;">';
-			$content .= '<thead><tr>';
-			$content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
-			$content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
-			$content .= '<th>' . GedcomTag::getLabel('DEAT') . '</th>';
-			$content .= '<th>DEAT</th>';
-			$content .= '<th><i class="icon-reminder" title="' . I18N::translate('Anniversary') . '"></i></th>';
-			$content .= '<th>' . GedcomTag::getLabel('_YART') . '</th>';
-			$content .= '<th>_YART</th>';
-			$content .= '</tr></thead><tbody>';
+			    $content = '';
+			    $content .= '<div class="loading-image">&nbsp;</div>';
+			    $content .= '<table id="' . $table_id . '" class="width100" style="visibility:hidden;">';
+			    $content .= '<thead><tr>';
+			    $content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
+			    $content .= '<th>' . GedcomTag::getLabel('NAME') . '</th>';
+			    $content .= '<th>' . GedcomTag::getLabel('DEAT') . '</th>';
+			    $content .= '<th>DEAT</th>';
+			    $content .= '<th><i class="icon-reminder" title="' . I18N::translate('Anniversary') . '"></i></th>';
+			    $content .= '<th>' . GedcomTag::getLabel('_YART') . '</th>';
+			    $content .= '<th>_YART</th>';
+			    $content .= '</tr></thead><tbody>';
 
-			foreach ($yahrzeits as $yahrzeit) {
-				if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
-					$content .= '<tr>';
-					$ind = $yahrzeit->getParent();
-					// Individual name(s)
-					$name = $ind->getFullName();
-					$url  = $ind->getHtmlUrl();
-					$content .= '<td>';
-					$content .= '<a href="' . $url . '">' . $name . '</a>';
-					$content .= $ind->getSexImage();
-					$addname = $ind->getAddName();
-					if ($addname) {
-						$content .= '<br><a href="' . $url . '">' . $addname . '</a>';
-					}
-					$content .= '</td>';
-					$content .= '<td>' . $ind->getSortName() . '</td>';
+			    foreach ($yahrzeits as $yahrzeit) {
+				    if ($yahrzeit->jd >= $startjd && $yahrzeit->jd < $startjd + $days) {
+					    $content .= '<tr>';
+					    $ind = $yahrzeit->getParent();
+					    // Individual name(s)
+					    $name = $ind->getFullName();
+					    $url  = $ind->getHtmlUrl();
+					    $content .= '<td>';
+					    $content .= '<a href="' . $url . '">' . $name . '</a>';
+					    $content .= $ind->getSexImage();
+					    $addname = $ind->getAddName();
+					    if ($addname) {
+						    $content .= '<br><a href="' . $url . '">' . $addname . '</a>';
+					    }
+					    $content .= '</td>';
+					    $content .= '<td>' . $ind->getSortName() . '</td>';
 
-					// death/yahrzeit event date
-					$content .= '<td>' . $yahrzeit->getDate()->display() . '</td>';
-					$content .= '<td>' . $yahrzeit->getDate()->julianDay() . '</td>'; // sortable date
+					    // death/yahrzeit event date
+					    $content .= '<td>' . $yahrzeit->getDate()->display() . '</td>';
+					    $content .= '<td>' . $yahrzeit->getDate()->julianDay() . '</td>'; // sortable date
 
-					// Anniversary
-					$content .= '<td>' . $yahrzeit->anniv . '</td>';
+					    // Anniversary
+					    $content .= '<td>' . $yahrzeit->anniv . '</td>';
 
-					// upcomming yahrzeit dates
-					switch ($calendar) {
-					case 'gregorian':
-						$today = new GregorianDate($yahrzeit->jd);
-						break;
-					case 'jewish':
-					default:
-						$today = new JewishDate($yahrzeit->jd);
-						break;
-					}
+					    // upcomming yahrzeit dates
+					    switch ($calendar) {
+					        case 'gregorian':
+						        $today = new GregorianDate($yahrzeit->jd);
+						        break;
+					        case 'jewish':
+					        default:
+						        $today = new JewishDate($yahrzeit->jd);
+						        break;
+					    }
 					$td = new Date($today->format('%@ %A %O %E'));
 					$content .= '<td>' . $td->display() . '</td>';
 					$content .= '<td>' . $td->julianDay() . '</td>'; // sortable date

Braces +16 added lines, -8 removed lines

@@ -31,14 +31,17 @@ 
 /**
  * Class YahrzeitModule
  */
-class YahrzeitModule extends AbstractModule implements ModuleBlockInterface {
+class YahrzeitModule extends AbstractModule implements ModuleBlockInterface
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module. Yahrzeiten (the plural of Yahrzeit) are special anniversaries of deaths in the Hebrew faith/calendar. */ I18N::translate('Yahrzeiten');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Yahrzeiten” module. A “Hebrew death” is a death where the date is recorded in the Hebrew calendar. */ I18N::translate('A list of the Hebrew death anniversaries that will occur in the near future.');
 	}
 
@@ -51,7 +54,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getBlock($block_id, $template = true, $cfg = array()) {
+	public function getBlock($block_id, $template = true, $cfg = array())
+	{
 		global $ctype, $controller, $WT_TREE;
 
 		$days      = $this->getBlockSetting($block_id, 'days', '7');
@@ -231,17 +235,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function loadAjax() {
+	public function loadAjax()
+	{
 		return true;
 	}
 
 	/** {@inheritdoc} */
-	public function isUserBlock() {
+	public function isUserBlock()
+	{
 		return true;
 	}
 
 	/** {@inheritdoc} */
-	public function isGedcomBlock() {
+	public function isGedcomBlock()
+	{
 		return true;
 	}
 
@@ -250,7 +257,8 @@ 
 	 *
 	 * @param int $block_id
 	 */
-	public function configureBlock($block_id) {
+	public function configureBlock($block_id)
+	{
 		if (Filter::postBool('save') && Filter::checkCsrf()) {
 			$this->setBlockSetting($block_id, 'days', Filter::postInteger('days', 1, 30, 7));
 			$this->setBlockSetting($block_id, 'infoStyle', Filter::post('infoStyle', 'list|table', 'table'));

app/Module/UserFavoritesModule.php

Indentation +63 added lines, -63 removed lines

@@ -29,73 +29,73 @@
  * The "user favorites" module is almost identical to the "family tree favorites" module
  */
 class UserFavoritesModule extends FamilyTreeFavoritesModule {
-	/** {@inheritdoc} */
-	public function getDescription() {
-		return /* I18N: Description of the “Favorites” module */ I18N::translate('Display and manage a user’s favorite pages.');
-	}
+    /** {@inheritdoc} */
+    public function getDescription() {
+        return /* I18N: Description of the “Favorites” module */ I18N::translate('Display and manage a user’s favorite pages.');
+    }
 
-	/**
-	 * Can this block be shown on the user’s home page?
-	 *
-	 * @return bool
-	 */
-	public function isUserBlock() {
-		return true;
-	}
+    /**
+     * Can this block be shown on the user’s home page?
+     *
+     * @return bool
+     */
+    public function isUserBlock() {
+        return true;
+    }
 
-	/**
-	 * Can this block be shown on the tree’s home page?
-	 *
-	 * @return bool
-	 */
-	public function isGedcomBlock() {
-		return false;
-	}
+    /**
+     * Can this block be shown on the tree’s home page?
+     *
+     * @return bool
+     */
+    public function isGedcomBlock() {
+        return false;
+    }
 
-	/**
-	 * Get the favorites for a user (for the current family tree)
-	 *
-	 * @param int $user_id
-	 *
-	 * @return string[][]
-	 */
-	public static function getFavorites($user_id) {
-		global $WT_TREE;
+    /**
+     * Get the favorites for a user (for the current family tree)
+     *
+     * @param int $user_id
+     *
+     * @return string[][]
+     */
+    public static function getFavorites($user_id) {
+        global $WT_TREE;
 
-		return
-			Database::prepare(
-				"SELECT favorite_id AS id, user_id, gedcom_id, xref AS gid, favorite_type AS type, title AS title, note AS note, url AS url" .
-				" FROM `##favorite` WHERE user_id=? AND gedcom_id=?")
-			->execute(array($user_id, $WT_TREE->getTreeId()))
-			->fetchAll(PDO::FETCH_ASSOC);
-	}
+        return
+            Database::prepare(
+                "SELECT favorite_id AS id, user_id, gedcom_id, xref AS gid, favorite_type AS type, title AS title, note AS note, url AS url" .
+                " FROM `##favorite` WHERE user_id=? AND gedcom_id=?")
+            ->execute(array($user_id, $WT_TREE->getTreeId()))
+            ->fetchAll(PDO::FETCH_ASSOC);
+    }
 
-	/**
-	 * This is a general purpose hook, allowing modules to respond to routes
-	 * of the form module.php?mod=FOO&mod_action=BAR
-	 *
-	 * @param string $mod_action
-	 */
-	public function modAction($mod_action) {
-		global $WT_TREE;
+    /**
+     * This is a general purpose hook, allowing modules to respond to routes
+     * of the form module.php?mod=FOO&mod_action=BAR
+     *
+     * @param string $mod_action
+     */
+    public function modAction($mod_action) {
+        global $WT_TREE;
 
-		switch ($mod_action) {
-		case 'menu-add-favorite':
-			// Process the "add to user favorites" menu item on indi/fam/etc. pages
-			$record = GedcomRecord::getInstance(Filter::post('xref', WT_REGEX_XREF), $WT_TREE);
-			if (Auth::check() && $record->canShowName()) {
-				self::addFavorite(array(
-					'user_id'   => Auth::id(),
-					'gedcom_id' => $record->getTree()->getTreeId(),
-					'gid'       => $record->getXref(),
-					'type'      => $record::RECORD_TYPE,
-					'url'       => null,
-					'note'      => null,
-					'title'     => null,
-				));
-				FlashMessages::addMessage(/* I18N: %s is the name of an individual, source or other record */ I18N::translate('“%s” has been added to your favorites.', $record->getFullName()));
-			}
-			break;
-		}
-	}
+        switch ($mod_action) {
+        case 'menu-add-favorite':
+            // Process the "add to user favorites" menu item on indi/fam/etc. pages
+            $record = GedcomRecord::getInstance(Filter::post('xref', WT_REGEX_XREF), $WT_TREE);
+            if (Auth::check() && $record->canShowName()) {
+                self::addFavorite(array(
+                    'user_id'   => Auth::id(),
+                    'gedcom_id' => $record->getTree()->getTreeId(),
+                    'gid'       => $record->getXref(),
+                    'type'      => $record::RECORD_TYPE,
+                    'url'       => null,
+                    'note'      => null,
+                    'title'     => null,
+                ));
+                FlashMessages::addMessage(/* I18N: %s is the name of an individual, source or other record */ I18N::translate('“%s” has been added to your favorites.', $record->getFullName()));
+            }
+            break;
+        }
+    }
 }

Switch Indentation +16 added lines, -16 removed lines

@@ -80,22 +80,22 @@
 		global $WT_TREE;
 
 		switch ($mod_action) {
-		case 'menu-add-favorite':
-			// Process the "add to user favorites" menu item on indi/fam/etc. pages
-			$record = GedcomRecord::getInstance(Filter::post('xref', WT_REGEX_XREF), $WT_TREE);
-			if (Auth::check() && $record->canShowName()) {
-				self::addFavorite(array(
-					'user_id'   => Auth::id(),
-					'gedcom_id' => $record->getTree()->getTreeId(),
-					'gid'       => $record->getXref(),
-					'type'      => $record::RECORD_TYPE,
-					'url'       => null,
-					'note'      => null,
-					'title'     => null,
-				));
-				FlashMessages::addMessage(/* I18N: %s is the name of an individual, source or other record */ I18N::translate('“%s” has been added to your favorites.', $record->getFullName()));
-			}
-			break;
+		    case 'menu-add-favorite':
+			    // Process the "add to user favorites" menu item on indi/fam/etc. pages
+			    $record = GedcomRecord::getInstance(Filter::post('xref', WT_REGEX_XREF), $WT_TREE);
+			    if (Auth::check() && $record->canShowName()) {
+				    self::addFavorite(array(
+					    'user_id'   => Auth::id(),
+					    'gedcom_id' => $record->getTree()->getTreeId(),
+					    'gid'       => $record->getXref(),
+					    'type'      => $record::RECORD_TYPE,
+					    'url'       => null,
+					    'note'      => null,
+					    'title'     => null,
+				    ));
+				    FlashMessages::addMessage(/* I18N: %s is the name of an individual, source or other record */ I18N::translate('“%s” has been added to your favorites.', $record->getFullName()));
+			    }
+			    break;
 		}
 	}
 }

Braces +12 added lines, -6 removed lines

@@ -28,9 +28,11 @@ 
  *
  * The "user favorites" module is almost identical to the "family tree favorites" module
  */
-class UserFavoritesModule extends FamilyTreeFavoritesModule {
+class UserFavoritesModule extends FamilyTreeFavoritesModule
+{
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Favorites” module */ I18N::translate('Display and manage a user’s favorite pages.');
 	}
 
@@ -39,7 +41,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function isUserBlock() {
+	public function isUserBlock()
+	{
 		return true;
 	}
 
@@ -48,7 +51,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function isGedcomBlock() {
+	public function isGedcomBlock()
+	{
 		return false;
 	}
 
@@ -59,7 +63,8 @@ 
 	 *
 	 * @return string[][]
 	 */
-	public static function getFavorites($user_id) {
+	public static function getFavorites($user_id)
+	{
 		global $WT_TREE;
 
 		return
@@ -76,7 +81,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		global $WT_TREE;
 
 		switch ($mod_action) {

app/Module/UserWelcomeModule.php

Indentation +57 added lines, -57 removed lines

@@ -24,71 +24,71 @@
  * Class UserWelcomeModule
  */
 class UserWelcomeModule extends AbstractModule implements ModuleBlockInterface {
-	/** {@inheritdoc} */
-	public function getTitle() {
-		return /* I18N: Name of a module */ I18N::translate('My page');
-	}
+    /** {@inheritdoc} */
+    public function getTitle() {
+        return /* I18N: Name of a module */ I18N::translate('My page');
+    }
 
-	/** {@inheritdoc} */
-	public function getDescription() {
-		return /* I18N: Description of the “My page” module */ I18N::translate('A greeting message and useful links for a user.');
-	}
+    /** {@inheritdoc} */
+    public function getDescription() {
+        return /* I18N: Description of the “My page” module */ I18N::translate('A greeting message and useful links for a user.');
+    }
 
-	/**
-	 * Generate the HTML content of this block.
-	 *
-	 * @param int      $block_id
-	 * @param bool     $template
-	 * @param string[] $cfg
-	 *
-	 * @return string
-	 */
-	public function getBlock($block_id, $template = true, $cfg = array()) {
-		global $WT_TREE;
+    /**
+     * Generate the HTML content of this block.
+     *
+     * @param int      $block_id
+     * @param bool     $template
+     * @param string[] $cfg
+     *
+     * @return string
+     */
+    public function getBlock($block_id, $template = true, $cfg = array()) {
+        global $WT_TREE;
 
-		$id      = $this->getName() . $block_id;
-		$class   = $this->getName() . '_block';
-		$title   = '<span dir="auto">' . /* I18N: A greeting; %s is the user’s name */ I18N::translate('Welcome %s', Auth::user()->getRealNameHtml()) . '</span>';
-		$content = '<table><tr>';
-		$content .= '<td><a href="edituser.php"><i class="icon-mypage"></i><br>' . I18N::translate('My account') . '</a></td>';
+        $id      = $this->getName() . $block_id;
+        $class   = $this->getName() . '_block';
+        $title   = '<span dir="auto">' . /* I18N: A greeting; %s is the user’s name */ I18N::translate('Welcome %s', Auth::user()->getRealNameHtml()) . '</span>';
+        $content = '<table><tr>';
+        $content .= '<td><a href="edituser.php"><i class="icon-mypage"></i><br>' . I18N::translate('My account') . '</a></td>';
 
-		$gedcomid = $WT_TREE->getUserPreference(Auth::user(), 'gedcomid');
+        $gedcomid = $WT_TREE->getUserPreference(Auth::user(), 'gedcomid');
 
-		if ($gedcomid) {
-			if (Module::isActiveChart($WT_TREE, 'pedigree_chart')) {
-				$content .= '<td><a href="pedigree.php?rootid=' . $gedcomid . '&amp;ged=' . $WT_TREE->getNameUrl() . '"><i class="icon-pedigree"></i><br>' . I18N::translate('My pedigree') . '</a></td>';
-			}
-			$content .= '<td><a href="individual.php?pid=' . $gedcomid . '&amp;ged=' . $WT_TREE->getNameUrl() . '"><i class="icon-indis"></i><br>' . I18N::translate('My individual record') . '</a></td>';
-		}
-		$content .= '</tr></table>';
+        if ($gedcomid) {
+            if (Module::isActiveChart($WT_TREE, 'pedigree_chart')) {
+                $content .= '<td><a href="pedigree.php?rootid=' . $gedcomid . '&amp;ged=' . $WT_TREE->getNameUrl() . '"><i class="icon-pedigree"></i><br>' . I18N::translate('My pedigree') . '</a></td>';
+            }
+            $content .= '<td><a href="individual.php?pid=' . $gedcomid . '&amp;ged=' . $WT_TREE->getNameUrl() . '"><i class="icon-indis"></i><br>' . I18N::translate('My individual record') . '</a></td>';
+        }
+        $content .= '</tr></table>';
 
-		if ($template) {
-			return Theme::theme()->formatBlock($id, $title, $class, $content);
-		} else {
-			return $content;
-		}
-	}
+        if ($template) {
+            return Theme::theme()->formatBlock($id, $title, $class, $content);
+        } else {
+            return $content;
+        }
+    }
 
-	/** {@inheritdoc} */
-	public function loadAjax() {
-		return false;
-	}
+    /** {@inheritdoc} */
+    public function loadAjax() {
+        return false;
+    }
 
-	/** {@inheritdoc} */
-	public function isUserBlock() {
-		return true;
-	}
+    /** {@inheritdoc} */
+    public function isUserBlock() {
+        return true;
+    }
 
-	/** {@inheritdoc} */
-	public function isGedcomBlock() {
-		return false;
-	}
+    /** {@inheritdoc} */
+    public function isGedcomBlock() {
+        return false;
+    }
 
-	/**
-	 * An HTML form to edit block settings
-	 *
-	 * @param int $block_id
-	 */
-	public function configureBlock($block_id) {
-	}
+    /**
+     * An HTML form to edit block settings
+     *
+     * @param int $block_id
+     */
+    public function configureBlock($block_id) {
+    }
 }

Braces +16 added lines, -8 removed lines

@@ -23,14 +23,17 @@ 
 /**
  * Class UserWelcomeModule
  */
-class UserWelcomeModule extends AbstractModule implements ModuleBlockInterface {
+class UserWelcomeModule extends AbstractModule implements ModuleBlockInterface
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('My page');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “My page” module */ I18N::translate('A greeting message and useful links for a user.');
 	}
 
@@ -43,7 +46,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getBlock($block_id, $template = true, $cfg = array()) {
+	public function getBlock($block_id, $template = true, $cfg = array())
+	{
 		global $WT_TREE;
 
 		$id      = $this->getName() . $block_id;
@@ -70,17 +74,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function loadAjax() {
+	public function loadAjax()
+	{
 		return false;
 	}
 
 	/** {@inheritdoc} */
-	public function isUserBlock() {
+	public function isUserBlock()
+	{
 		return true;
 	}
 
 	/** {@inheritdoc} */
-	public function isGedcomBlock() {
+	public function isGedcomBlock()
+	{
 		return false;
 	}
 
@@ -89,6 +96,7 @@ 
 	 *
 	 * @param int $block_id
 	 */
-	public function configureBlock($block_id) {
+	public function configureBlock($block_id)
+	{
 	}
 }

app/Module/UserMessagesModule.php

Indentation +158 added lines, -158 removed lines

@@ -28,162 +28,162 @@
  * Class UserMessagesModule
  */
 class UserMessagesModule extends AbstractModule implements ModuleBlockInterface {
-	/** {@inheritdoc} */
-	public function getTitle() {
-		return /* I18N: Name of a module */ I18N::translate('Messages');
-	}
-
-	/** {@inheritdoc} */
-	public function getDescription() {
-		return /* I18N: Description of the “Messages” module */ I18N::translate('Communicate directly with other users, using private messages.');
-	}
-
-	/**
-	 * This is a general purpose hook, allowing modules to respond to routes
-	 * of the form module.php?mod=FOO&mod_action=BAR
-	 *
-	 * @param string $mod_action
-	 */
-	public function modAction($mod_action) {
-		switch ($mod_action) {
-		case 'delete':
-			$stmt = Database::prepare("DELETE FROM `##message` WHERE user_id = :user_id AND message_id = :message_id");
-
-			foreach (Filter::postArray('message_id') as $id) {
-				$stmt->execute(array(
-					'message_id' => $id,
-					'user_id'    => Auth::id(),
-				));
-			}
-		}
-
-		$ged   = Filter::post('ged');
-		$ctype = Filter::post('ctype', 'user|gedcom', 'user');
-
-		header('Location: ' . WT_BASE_URL . 'index.php?ged=' . Filter::escapeUrl($ged) . '&ctype=' . $ctype);
-	}
-
-	/**
-	 * Generate the HTML content of this block.
-	 *
-	 * @param int      $block_id
-	 * @param bool     $template
-	 * @param string[] $cfg
-	 *
-	 * @return string
-	 */
-	public function getBlock($block_id, $template = true, $cfg = array()) {
-		global $ctype, $WT_TREE;
-
-		$block = $this->getBlockSetting($block_id, 'block', '1');
-		foreach (array('block') as $name) {
-			if (array_key_exists($name, $cfg)) {
-				$$name = $cfg[$name];
-			}
-		}
-		$messages = Database::prepare("SELECT message_id, sender, subject, body, UNIX_TIMESTAMP(created) AS created FROM `##message` WHERE user_id=? ORDER BY message_id DESC")
-			->execute(array(Auth::id()))
-			->fetchAll();
-
-		$count = count($messages);
-		$id    = $this->getName() . $block_id;
-		$class = $this->getName() . '_block';
-		$title = I18N::plural('%s message', '%s messages', $count, I18N::number($count));
-		$users = array_filter(User::all(), function (User $user) {
-			return $user->getUserId() !== Auth::id() && $user->getPreference('verified_by_admin') && $user->getPreference('contactmethod') !== 'none';
-		});
-
-		$content = '<form id="messageform" name="messageform" method="post" action="module.php?mod=user_messages&mod_action=delete" onsubmit="return confirm(\'' . I18N::translate('Are you sure you want to delete this message? It cannot be retrieved later.') . '\');">';
-		$content .= '<input type="hidden" name="ged" value="' . $ctype . '">';
-		$content .= '<input type="hidden" name="ctype" value="' . $WT_TREE->getNameHtml() . '">';
-		if ($users) {
-			$content .= '<label for="touser">' . I18N::translate('Send a message') . '</label>';
-			$content .= '<select id="touser" name="touser">';
-			$content .= '<option value="">' . I18N::translate('&lt;select&gt;') . '</option>';
-			foreach ($users as $user) {
-				$content .= sprintf('<option value="%1$s">%2$s - %1$s</option>', Filter::escapeHtml($user->getUserName()), Filter::escapeHtml($user->getRealName()));
-			}
-			$content .= '</select>';
-			$content .= '<input type="button" value="' . I18N::translate('Send') . '" onclick="return message(document.messageform.touser.options[document.messageform.touser.selectedIndex].value, \'messaging2\', \'\');"><br><br>';
-		}
-		if ($messages) {
-			$content .= '<table class="list_table"><tr>';
-			$content .= '<th class="list_label">' . I18N::translate('Delete') . '<br><a href="#" onclick="jQuery(\'#' . $this->getName() . $block_id . ' :checkbox\').prop(\'checked\', true); return false;">' . I18N::translate('All') . '</a></th>';
-			$content .= '<th class="list_label">' . I18N::translate('Subject') . '</th>';
-			$content .= '<th class="list_label">' . I18N::translate('Date sent') . '</th>';
-			$content .= '<th class="list_label">' . I18N::translate('Email address') . '</th>';
-			$content .= '</tr>';
-			foreach ($messages as $message) {
-				$content .= '<tr>';
-				$content .= '<td class="list_value_wrap"><input type="checkbox" name="message_id[]" value="' . $message->message_id . '" id="cb_message' . $message->message_id . '"></td>';
-				$content .= '<td class="list_value_wrap"><a href="#" onclick="return expand_layer(\'message' . $message->message_id . '\');"><i id="message' . $message->message_id . '_img" class="icon-plus"></i> <b dir="auto">' . Filter::escapeHtml($message->subject) . '</b></a></td>';
-				$content .= '<td class="list_value_wrap">' . FunctionsDate::formatTimestamp($message->created + WT_TIMESTAMP_OFFSET) . '</td>';
-				$content .= '<td class="list_value_wrap">';
-				$user = User::findByIdentifier($message->sender);
-				if ($user) {
-					$content .= $user->getRealNameHtml();
-					$content .= '  - <span dir="auto">' . $user->getEmail() . '</span>';
-				} else {
-					$content .= '<a href="mailto:' . Filter::escapeHtml($message->sender) . '">' . Filter::escapeHtml($message->sender) . '</a>';
-				}
-				$content .= '</td>';
-				$content .= '</tr>';
-				$content .= '<tr><td class="list_value_wrap" colspan="4"><div id="message' . $message->message_id . '" style="display:none;">';
-				$content .= '<div dir="auto" style="white-space: pre-wrap;">' . Filter::expandUrls($message->body) . '</div><br>';
-				if (strpos($message->subject, /* I18N: When replying to an email, the subject becomes “RE: <subject>” */ I18N::translate('RE: ')) !== 0) {
-					$message->subject = I18N::translate('RE: ') . $message->subject;
-				}
-				if ($user) {
-					$content .= '<button type="button" onclick="reply(\'' . Filter::escapeJs($message->sender) . '\', \'' . Filter::escapeJs($message->subject) . '\'); return false;">' . I18N::translate('Reply') . '</button> ';
-				}
-				$content .= '<button type="button" onclick="if (confirm(\'' . I18N::translate('Are you sure you want to delete this message? It cannot be retrieved later.') . '\')) {jQuery(\'#messageform :checkbox\').prop(\'checked\', false); jQuery(\'#cb_message' . $message->message_id . '\').prop(\'checked\', true); document.messageform.submit();}">' . I18N::translate('Delete') . '</button></div></td></tr>';
-			}
-			$content .= '</table>';
-			$content .= '<p><button type="submit">' . I18N::translate('Delete selected messages') . '</button></p>';
-		}
-		$content .= '</form>';
-
-		if ($template) {
-			if ($block) {
-				$class .= ' small_inner_block';
-			}
-
-			return Theme::theme()->formatBlock($id, $title, $class, $content);
-		} else {
-			return $content;
-		}
-	}
-
-	/** {@inheritdoc} */
-	public function loadAjax() {
-		return false;
-	}
-
-	/** {@inheritdoc} */
-	public function isUserBlock() {
-		return true;
-	}
-
-	/** {@inheritdoc} */
-	public function isGedcomBlock() {
-		return false;
-	}
-
-	/**
-	 * An HTML form to edit block settings
-	 *
-	 * @param int $block_id
-	 */
-	public function configureBlock($block_id) {
-		if (Filter::postBool('save') && Filter::checkCsrf()) {
-			$this->setBlockSetting($block_id, 'block', Filter::postBool('block'));
-		}
-
-		$block = $this->getBlockSetting($block_id, 'block', '1');
-		echo '<tr><td class="descriptionbox wrap width33">';
-		echo /* I18N: label for a yes/no option */ I18N::translate('Add a scrollbar when block contents grow');
-		echo '</td><td class="optionbox">';
-		echo FunctionsEdit::editFieldYesNo('block', $block);
-		echo '</td></tr>';
-	}
+    /** {@inheritdoc} */
+    public function getTitle() {
+        return /* I18N: Name of a module */ I18N::translate('Messages');
+    }
+
+    /** {@inheritdoc} */
+    public function getDescription() {
+        return /* I18N: Description of the “Messages” module */ I18N::translate('Communicate directly with other users, using private messages.');
+    }
+
+    /**
+     * This is a general purpose hook, allowing modules to respond to routes
+     * of the form module.php?mod=FOO&mod_action=BAR
+     *
+     * @param string $mod_action
+     */
+    public function modAction($mod_action) {
+        switch ($mod_action) {
+        case 'delete':
+            $stmt = Database::prepare("DELETE FROM `##message` WHERE user_id = :user_id AND message_id = :message_id");
+
+            foreach (Filter::postArray('message_id') as $id) {
+                $stmt->execute(array(
+                    'message_id' => $id,
+                    'user_id'    => Auth::id(),
+                ));
+            }
+        }
+
+        $ged   = Filter::post('ged');
+        $ctype = Filter::post('ctype', 'user|gedcom', 'user');
+
+        header('Location: ' . WT_BASE_URL . 'index.php?ged=' . Filter::escapeUrl($ged) . '&ctype=' . $ctype);
+    }
+
+    /**
+     * Generate the HTML content of this block.
+     *
+     * @param int      $block_id
+     * @param bool     $template
+     * @param string[] $cfg
+     *
+     * @return string
+     */
+    public function getBlock($block_id, $template = true, $cfg = array()) {
+        global $ctype, $WT_TREE;
+
+        $block = $this->getBlockSetting($block_id, 'block', '1');
+        foreach (array('block') as $name) {
+            if (array_key_exists($name, $cfg)) {
+                $$name = $cfg[$name];
+            }
+        }
+        $messages = Database::prepare("SELECT message_id, sender, subject, body, UNIX_TIMESTAMP(created) AS created FROM `##message` WHERE user_id=? ORDER BY message_id DESC")
+            ->execute(array(Auth::id()))
+            ->fetchAll();
+
+        $count = count($messages);
+        $id    = $this->getName() . $block_id;
+        $class = $this->getName() . '_block';
+        $title = I18N::plural('%s message', '%s messages', $count, I18N::number($count));
+        $users = array_filter(User::all(), function (User $user) {
+            return $user->getUserId() !== Auth::id() && $user->getPreference('verified_by_admin') && $user->getPreference('contactmethod') !== 'none';
+        });
+
+        $content = '<form id="messageform" name="messageform" method="post" action="module.php?mod=user_messages&mod_action=delete" onsubmit="return confirm(\'' . I18N::translate('Are you sure you want to delete this message? It cannot be retrieved later.') . '\');">';
+        $content .= '<input type="hidden" name="ged" value="' . $ctype . '">';
+        $content .= '<input type="hidden" name="ctype" value="' . $WT_TREE->getNameHtml() . '">';
+        if ($users) {
+            $content .= '<label for="touser">' . I18N::translate('Send a message') . '</label>';
+            $content .= '<select id="touser" name="touser">';
+            $content .= '<option value="">' . I18N::translate('&lt;select&gt;') . '</option>';
+            foreach ($users as $user) {
+                $content .= sprintf('<option value="%1$s">%2$s - %1$s</option>', Filter::escapeHtml($user->getUserName()), Filter::escapeHtml($user->getRealName()));
+            }
+            $content .= '</select>';
+            $content .= '<input type="button" value="' . I18N::translate('Send') . '" onclick="return message(document.messageform.touser.options[document.messageform.touser.selectedIndex].value, \'messaging2\', \'\');"><br><br>';
+        }
+        if ($messages) {
+            $content .= '<table class="list_table"><tr>';
+            $content .= '<th class="list_label">' . I18N::translate('Delete') . '<br><a href="#" onclick="jQuery(\'#' . $this->getName() . $block_id . ' :checkbox\').prop(\'checked\', true); return false;">' . I18N::translate('All') . '</a></th>';
+            $content .= '<th class="list_label">' . I18N::translate('Subject') . '</th>';
+            $content .= '<th class="list_label">' . I18N::translate('Date sent') . '</th>';
+            $content .= '<th class="list_label">' . I18N::translate('Email address') . '</th>';
+            $content .= '</tr>';
+            foreach ($messages as $message) {
+                $content .= '<tr>';
+                $content .= '<td class="list_value_wrap"><input type="checkbox" name="message_id[]" value="' . $message->message_id . '" id="cb_message' . $message->message_id . '"></td>';
+                $content .= '<td class="list_value_wrap"><a href="#" onclick="return expand_layer(\'message' . $message->message_id . '\');"><i id="message' . $message->message_id . '_img" class="icon-plus"></i> <b dir="auto">' . Filter::escapeHtml($message->subject) . '</b></a></td>';
+                $content .= '<td class="list_value_wrap">' . FunctionsDate::formatTimestamp($message->created + WT_TIMESTAMP_OFFSET) . '</td>';
+                $content .= '<td class="list_value_wrap">';
+                $user = User::findByIdentifier($message->sender);
+                if ($user) {
+                    $content .= $user->getRealNameHtml();
+                    $content .= '  - <span dir="auto">' . $user->getEmail() . '</span>';
+                } else {
+                    $content .= '<a href="mailto:' . Filter::escapeHtml($message->sender) . '">' . Filter::escapeHtml($message->sender) . '</a>';
+                }
+                $content .= '</td>';
+                $content .= '</tr>';
+                $content .= '<tr><td class="list_value_wrap" colspan="4"><div id="message' . $message->message_id . '" style="display:none;">';
+                $content .= '<div dir="auto" style="white-space: pre-wrap;">' . Filter::expandUrls($message->body) . '</div><br>';
+                if (strpos($message->subject, /* I18N: When replying to an email, the subject becomes “RE: <subject>” */ I18N::translate('RE: ')) !== 0) {
+                    $message->subject = I18N::translate('RE: ') . $message->subject;
+                }
+                if ($user) {
+                    $content .= '<button type="button" onclick="reply(\'' . Filter::escapeJs($message->sender) . '\', \'' . Filter::escapeJs($message->subject) . '\'); return false;">' . I18N::translate('Reply') . '</button> ';
+                }
+                $content .= '<button type="button" onclick="if (confirm(\'' . I18N::translate('Are you sure you want to delete this message? It cannot be retrieved later.') . '\')) {jQuery(\'#messageform :checkbox\').prop(\'checked\', false); jQuery(\'#cb_message' . $message->message_id . '\').prop(\'checked\', true); document.messageform.submit();}">' . I18N::translate('Delete') . '</button></div></td></tr>';
+            }
+            $content .= '</table>';
+            $content .= '<p><button type="submit">' . I18N::translate('Delete selected messages') . '</button></p>';
+        }
+        $content .= '</form>';
+
+        if ($template) {
+            if ($block) {
+                $class .= ' small_inner_block';
+            }
+
+            return Theme::theme()->formatBlock($id, $title, $class, $content);
+        } else {
+            return $content;
+        }
+    }
+
+    /** {@inheritdoc} */
+    public function loadAjax() {
+        return false;
+    }
+
+    /** {@inheritdoc} */
+    public function isUserBlock() {
+        return true;
+    }
+
+    /** {@inheritdoc} */
+    public function isGedcomBlock() {
+        return false;
+    }
+
+    /**
+     * An HTML form to edit block settings
+     *
+     * @param int $block_id
+     */
+    public function configureBlock($block_id) {
+        if (Filter::postBool('save') && Filter::checkCsrf()) {
+            $this->setBlockSetting($block_id, 'block', Filter::postBool('block'));
+        }
+
+        $block = $this->getBlockSetting($block_id, 'block', '1');
+        echo '<tr><td class="descriptionbox wrap width33">';
+        echo /* I18N: label for a yes/no option */ I18N::translate('Add a scrollbar when block contents grow');
+        echo '</td><td class="optionbox">';
+        echo FunctionsEdit::editFieldYesNo('block', $block);
+        echo '</td></tr>';
+    }
 }

Switch Indentation +9 added lines, -9 removed lines

@@ -46,15 +46,15 @@
 	 */
 	public function modAction($mod_action) {
 		switch ($mod_action) {
-		case 'delete':
-			$stmt = Database::prepare("DELETE FROM `##message` WHERE user_id = :user_id AND message_id = :message_id");
-
-			foreach (Filter::postArray('message_id') as $id) {
-				$stmt->execute(array(
-					'message_id' => $id,
-					'user_id'    => Auth::id(),
-				));
-			}
+		    case 'delete':
+			    $stmt = Database::prepare("DELETE FROM `##message` WHERE user_id = :user_id AND message_id = :message_id");
+
+			    foreach (Filter::postArray('message_id') as $id) {
+				    $stmt->execute(array(
+					    'message_id' => $id,
+					    'user_id'    => Auth::id(),
+				    ));
+			    }
 		}
 
 		$ged   = Filter::post('ged');

Braces +18 added lines, -9 removed lines

@@ -27,14 +27,17 @@ 
 /**
  * Class UserMessagesModule
  */
-class UserMessagesModule extends AbstractModule implements ModuleBlockInterface {
+class UserMessagesModule extends AbstractModule implements ModuleBlockInterface
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('Messages');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Messages” module */ I18N::translate('Communicate directly with other users, using private messages.');
 	}
 
@@ -44,7 +47,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'delete':
 			$stmt = Database::prepare("DELETE FROM `##message` WHERE user_id = :user_id AND message_id = :message_id");
@@ -72,7 +76,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getBlock($block_id, $template = true, $cfg = array()) {
+	public function getBlock($block_id, $template = true, $cfg = array())
+	{
 		global $ctype, $WT_TREE;
 
 		$block = $this->getBlockSetting($block_id, 'block', '1');
@@ -155,17 +160,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function loadAjax() {
+	public function loadAjax()
+	{
 		return false;
 	}
 
 	/** {@inheritdoc} */
-	public function isUserBlock() {
+	public function isUserBlock()
+	{
 		return true;
 	}
 
 	/** {@inheritdoc} */
-	public function isGedcomBlock() {
+	public function isGedcomBlock()
+	{
 		return false;
 	}
 
@@ -174,7 +182,8 @@ 
 	 *
 	 * @param int $block_id
 	 */
-	public function configureBlock($block_id) {
+	public function configureBlock($block_id)
+	{
 		if (Filter::postBool('save') && Filter::checkCsrf()) {
 			$this->setBlockSetting($block_id, 'block', Filter::postBool('block'));
 		}

app/Module/RelatedIndividualsReportModule.php

Indentation +34 added lines, -34 removed lines

@@ -23,42 +23,42 @@
  * Class RelatedIndividualsReportModule
  */
 class RelatedIndividualsReportModule extends AbstractModule implements ModuleReportInterface {
-	/** {@inheritdoc} */
-	public function getTitle() {
-		// This text also appears in the .XML file - update both together
-		return /* I18N: Name of a report */ I18N::translate('Related individuals');
-	}
+    /** {@inheritdoc} */
+    public function getTitle() {
+        // This text also appears in the .XML file - update both together
+        return /* I18N: Name of a report */ I18N::translate('Related individuals');
+    }
 
-	/** {@inheritdoc} */
-	public function getDescription() {
-		// This text also appears in the .XML file - update both together
-		return /* I18N: Description of the “Related individuals” module */ I18N::translate('A report of the individuals that are closely related to an individual.');
-	}
+    /** {@inheritdoc} */
+    public function getDescription() {
+        // This text also appears in the .XML file - update both together
+        return /* I18N: Description of the “Related individuals” module */ I18N::translate('A report of the individuals that are closely related to an individual.');
+    }
 
-	/**
-	 * What is the default access level for this module?
-	 *
-	 * Some modules are aimed at admins or managers, and are not generally shown to users.
-	 *
-	 * @return int
-	 */
-	public function defaultAccessLevel() {
-		return Auth::PRIV_PRIVATE;
-	}
+    /**
+     * What is the default access level for this module?
+     *
+     * Some modules are aimed at admins or managers, and are not generally shown to users.
+     *
+     * @return int
+     */
+    public function defaultAccessLevel() {
+        return Auth::PRIV_PRIVATE;
+    }
 
-	/**
-	 * Return a menu item for this report.
-	 *
-	 * @return Menu
-	 */
-	public function getReportMenu() {
-		global $controller, $WT_TREE;
+    /**
+     * Return a menu item for this report.
+     *
+     * @return Menu
+     */
+    public function getReportMenu() {
+        global $controller, $WT_TREE;
 
-		return new Menu(
-			$this->getTitle(),
-			'reportengine.php?ged=' . $WT_TREE->getNameUrl() . '&action=setup&report=' . WT_MODULES_DIR . $this->getName() . '/report.xml&pid=' . $controller->getSignificantIndividual()->getXref(),
-			'menu-report-' . $this->getName(),
-			array('rel' => 'nofollow')
-		);
-	}
+        return new Menu(
+            $this->getTitle(),
+            'reportengine.php?ged=' . $WT_TREE->getNameUrl() . '&action=setup&report=' . WT_MODULES_DIR . $this->getName() . '/report.xml&pid=' . $controller->getSignificantIndividual()->getXref(),
+            'menu-report-' . $this->getName(),
+            array('rel' => 'nofollow')
+        );
+    }
 }

Braces +10 added lines, -5 removed lines

@@ -22,15 +22,18 @@ 
 /**
  * Class RelatedIndividualsReportModule
  */
-class RelatedIndividualsReportModule extends AbstractModule implements ModuleReportInterface {
+class RelatedIndividualsReportModule extends AbstractModule implements ModuleReportInterface
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		// This text also appears in the .XML file - update both together
 		return /* I18N: Name of a report */ I18N::translate('Related individuals');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		// This text also appears in the .XML file - update both together
 		return /* I18N: Description of the “Related individuals” module */ I18N::translate('A report of the individuals that are closely related to an individual.');
 	}
@@ -42,7 +45,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function defaultAccessLevel() {
+	public function defaultAccessLevel()
+	{
 		return Auth::PRIV_PRIVATE;
 	}
 
@@ -51,7 +55,8 @@ 
 	 *
 	 * @return Menu
 	 */
-	public function getReportMenu() {
+	public function getReportMenu()
+	{
 		global $controller, $WT_TREE;
 
 		return new Menu(

app/Module/CensusAssistantModule.php

Indentation +370 added lines, -370 removed lines

@@ -34,110 +34,110 @@ 
  * Class CensusAssistantModule
  */
 class CensusAssistantModule extends AbstractModule {
-	/** {@inheritdoc} */
-	public function getTitle() {
-		return /* I18N: Name of a module */ I18N::translate('Census assistant');
-	}
-
-	/** {@inheritdoc} */
-	public function getDescription() {
-		return /* I18N: Description of the “Census assistant” module */ I18N::translate('An alternative way to enter census transcripts and link them to individuals.');
-	}
-
-	/**
-	 * This is a general purpose hook, allowing modules to respond to routes
-	 * of the form module.php?mod=FOO&mod_action=BAR
-	 *
-	 * @param string $mod_action
-	 */
-	public function modAction($mod_action) {
-		switch ($mod_action) {
-		case 'census_find':
-			self::censusFind();
-			break;
-		case 'media_find':
-			self::mediaFind();
-			break;
-		case 'media_query_3a':
-			self::mediaQuery();
-			break;
-		default:
-			http_response_code(404);
-		}
-	}
-
-	/**
-	 * Find an individual.
-	 */
-	private static function censusFind() {
-		global $WT_TREE;
-
-		$controller = new SimpleController;
-		$filter     = Filter::get('filter');
-		$action     = Filter::get('action');
-		$census     = Filter::get('census');
-		$census     = new $census;
-
-		$controller
-			->restrictAccess($census instanceof CensusInterface)
-			->setPageTitle(I18N::translate('Find an individual'))
-			->pageHeader();
-
-		echo '<table class="list_table width90" border="0">';
-		echo '<tr><td style="padding: 10px;" class="facts_label03 width90">';
-		echo I18N::translate('Find an individual');
-		echo '</td>';
-		echo '</table>';
-		echo '<br>';
-
-		if ($action == 'filter') {
-			$filter       = trim($filter);
-			$filter_array = explode(' ', preg_replace('/ {2,}/', ' ', $filter));
-
-			// Output Individual for GEDFact Assistant ======================
-			echo '<table class="list_table width90">';
-			$myindilist = FunctionsDb::searchIndividualNames($filter_array, array($WT_TREE));
-			if ($myindilist) {
-				echo '<tr><td class="list_value_wrap"><ul>';
-				usort($myindilist, '\Fisharebest\Webtrees\GedcomRecord::compare');
-				foreach ($myindilist as $indi) {
-					echo '<li>';
-					echo '<a href="#" onclick="window.opener.appendCensusRow(\'' . Filter::escapeJs(self::censusTableRow($census, $indi, null)) . '\'); window.close();">';
-					echo '<b>' . $indi->getFullName() . '</b>';
-					echo '</a>';
-					echo $indi->formatFirstMajorFact(WT_EVENTS_BIRT, 1);
-					echo $indi->formatFirstMajorFact(WT_EVENTS_DEAT, 1);
-					echo '<hr>';
-					echo '</li>';
-				}
-				echo '</ul></td></tr>';
-			} else {
-				echo '<tr><td class="list_value_wrap">';
-				echo I18N::translate('No results found.');
-				echo '</td></tr>';
-			}
-			echo '<tr><td>';
-			echo '<button onclick="window.close();">', I18N::translate('close'), '</button>';
-			echo '</td></tr>';
-			echo '</table>';
-		}
-	}
-
-	/**
-	 * Find a media object.
-	 */
-	private static function mediaFind() {
-		global $WT_TREE;
-
-		$controller = new SimpleController;
-		$filter     = Filter::get('filter');
-		$multiple   = Filter::getBool('multiple');
-
-		$controller
-			->setPageTitle(I18N::translate('Find an individual'))
-			->pageHeader();
-
-		?>
+    /** {@inheritdoc} */
+    public function getTitle() {
+        return /* I18N: Name of a module */ I18N::translate('Census assistant');
+    }
+
+    /** {@inheritdoc} */
+    public function getDescription() {
+        return /* I18N: Description of the “Census assistant” module */ I18N::translate('An alternative way to enter census transcripts and link them to individuals.');
+    }
+
+    /**
+     * This is a general purpose hook, allowing modules to respond to routes
+     * of the form module.php?mod=FOO&mod_action=BAR
+     *
+     * @param string $mod_action
+     */
+    public function modAction($mod_action) {
+        switch ($mod_action) {
+        case 'census_find':
+            self::censusFind();
+            break;
+        case 'media_find':
+            self::mediaFind();
+            break;
+        case 'media_query_3a':
+            self::mediaQuery();
+            break;
+        default:
+            http_response_code(404);
+        }
+    }
+
+    /**
+     * Find an individual.
+     */
+    private static function censusFind() {
+        global $WT_TREE;
+
+        $controller = new SimpleController;
+        $filter     = Filter::get('filter');
+        $action     = Filter::get('action');
+        $census     = Filter::get('census');
+        $census     = new $census;
+
+        $controller
+            ->restrictAccess($census instanceof CensusInterface)
+            ->setPageTitle(I18N::translate('Find an individual'))
+            ->pageHeader();
+
+        echo '<table class="list_table width90" border="0">';
+        echo '<tr><td style="padding: 10px;" class="facts_label03 width90">';
+        echo I18N::translate('Find an individual');
+        echo '</td>';
+        echo '</table>';
+        echo '<br>';
+
+        if ($action == 'filter') {
+            $filter       = trim($filter);
+            $filter_array = explode(' ', preg_replace('/ {2,}/', ' ', $filter));
+
+            // Output Individual for GEDFact Assistant ======================
+            echo '<table class="list_table width90">';
+            $myindilist = FunctionsDb::searchIndividualNames($filter_array, array($WT_TREE));
+            if ($myindilist) {
+                echo '<tr><td class="list_value_wrap"><ul>';
+                usort($myindilist, '\Fisharebest\Webtrees\GedcomRecord::compare');
+                foreach ($myindilist as $indi) {
+                    echo '<li>';
+                    echo '<a href="#" onclick="window.opener.appendCensusRow(\'' . Filter::escapeJs(self::censusTableRow($census, $indi, null)) . '\'); window.close();">';
+                    echo '<b>' . $indi->getFullName() . '</b>';
+                    echo '</a>';
+                    echo $indi->formatFirstMajorFact(WT_EVENTS_BIRT, 1);
+                    echo $indi->formatFirstMajorFact(WT_EVENTS_DEAT, 1);
+                    echo '<hr>';
+                    echo '</li>';
+                }
+                echo '</ul></td></tr>';
+            } else {
+                echo '<tr><td class="list_value_wrap">';
+                echo I18N::translate('No results found.');
+                echo '</td></tr>';
+            }
+            echo '<tr><td>';
+            echo '<button onclick="window.close();">', I18N::translate('close'), '</button>';
+            echo '</td></tr>';
+            echo '</table>';
+        }
+    }
+
+    /**
+     * Find a media object.
+     */
+    private static function mediaFind() {
+        global $WT_TREE;
+
+        $controller = new SimpleController;
+        $filter     = Filter::get('filter');
+        $multiple   = Filter::getBool('multiple');
+
+        $controller
+            ->setPageTitle(I18N::translate('Find an individual'))
+            ->pageHeader();
+
+        ?>
 		<script>
 		function pasterow(id, name, gend, yob, age, bpl) {
 			window.opener.opener.insertRowToTable(id, name, '', gend, '', yob, age, 'Y', '', bpl);
@@ -178,27 +178,27 @@ 
 		</script>
 
 		<?php
-		echo '<div>';
-		echo '<table class="list_table width90" border="0">';
-		echo '<tr><td style="padding: 10px;" class="facts_label03 width90">'; // start column for find text header
-		echo $controller->getPageTitle();
-		echo '</td>';
-		echo '</tr>';
-		echo '</table>';
-		echo '<br>';
-		echo '<button onclick="window.close();">', I18N::translate('close'), '</button>';
-		echo '<br>';
-
-		$filter       = trim($filter);
-		$filter_array = explode(' ', preg_replace('/ {2,}/', ' ', $filter));
-		echo '<table class="tabs_table width90"><tr>';
-		$myindilist = FunctionsDb::searchIndividualNames($filter_array, array($WT_TREE));
-		if ($myindilist) {
-			echo '<td class="list_value_wrap"><ul>';
-			usort($myindilist, '\Fisharebest\Webtrees\GedcomRecord::compare');
-			foreach ($myindilist as $indi) {
-				$nam = Filter::escapeHtml($indi->getFullName());
-				echo "<li><a href=\"#\" onclick=\"pasterow(
+        echo '<div>';
+        echo '<table class="list_table width90" border="0">';
+        echo '<tr><td style="padding: 10px;" class="facts_label03 width90">'; // start column for find text header
+        echo $controller->getPageTitle();
+        echo '</td>';
+        echo '</tr>';
+        echo '</table>';
+        echo '<br>';
+        echo '<button onclick="window.close();">', I18N::translate('close'), '</button>';
+        echo '<br>';
+
+        $filter       = trim($filter);
+        $filter_array = explode(' ', preg_replace('/ {2,}/', ' ', $filter));
+        echo '<table class="tabs_table width90"><tr>';
+        $myindilist = FunctionsDb::searchIndividualNames($filter_array, array($WT_TREE));
+        if ($myindilist) {
+            echo '<td class="list_value_wrap"><ul>';
+            usort($myindilist, '\Fisharebest\Webtrees\GedcomRecord::compare');
+            foreach ($myindilist as $indi) {
+                $nam = Filter::escapeHtml($indi->getFullName());
+                echo "<li><a href=\"#\" onclick=\"pasterow(
 					'" . $indi->getXref() . "' ,
 					'" . $nam . "' ,
 					'" . $indi->getSex() . "' ,
@@ -207,44 +207,44 @@ 
 					'" . $indi->getBirthPlace() . "'); return false;\">
 					<b>" . $indi->getFullName() . "</b>&nbsp;&nbsp;&nbsp;";
 
-				$born = GedcomTag::getLabel('BIRT');
-				echo "</span><br><span class=\"list_item\">", $born, " ", $indi->getBirthYear(), "&nbsp;&nbsp;&nbsp;", $indi->getBirthPlace(), "</span></a></li>";
-				echo "<hr>";
-			}
-			echo '</ul></td></tr><tr><td class="list_label">', I18N::translate('Total individuals: %s', count($myindilist)), '</tr></td>';
-		} else {
-			echo "<td class=\"list_value_wrap\">";
-			echo I18N::translate('No results found.');
-			echo "</td></tr>";
-		}
-		echo "</table>";
-		echo '</div>';
-	}
-
-	/**
-	 * Search for a media object.
-	 */
-	private static function mediaQuery() {
-		global $WT_TREE;
-
-		$iid2 = Filter::get('iid', WT_REGEX_XREF);
-
-		$controller = new SimpleController;
-		$controller
-			->setPageTitle(I18N::translate('Link to an existing media object'))
-			->pageHeader();
-
-		$record = GedcomRecord::getInstance($iid2, $WT_TREE);
-		if ($record) {
-			$headjs = '';
-			if ($record instanceof Family) {
-				if ($record->getHusband()) {
-					$headjs = $record->getHusband()->getXref();
-				} elseif ($record->getWife()) {
-					$headjs = $record->getWife()->getXref();
-				}
-			}
-			?>
+                $born = GedcomTag::getLabel('BIRT');
+                echo "</span><br><span class=\"list_item\">", $born, " ", $indi->getBirthYear(), "&nbsp;&nbsp;&nbsp;", $indi->getBirthPlace(), "</span></a></li>";
+                echo "<hr>";
+            }
+            echo '</ul></td></tr><tr><td class="list_label">', I18N::translate('Total individuals: %s', count($myindilist)), '</tr></td>';
+        } else {
+            echo "<td class=\"list_value_wrap\">";
+            echo I18N::translate('No results found.');
+            echo "</td></tr>";
+        }
+        echo "</table>";
+        echo '</div>';
+    }
+
+    /**
+     * Search for a media object.
+     */
+    private static function mediaQuery() {
+        global $WT_TREE;
+
+        $iid2 = Filter::get('iid', WT_REGEX_XREF);
+
+        $controller = new SimpleController;
+        $controller
+            ->setPageTitle(I18N::translate('Link to an existing media object'))
+            ->pageHeader();
+
+        $record = GedcomRecord::getInstance($iid2, $WT_TREE);
+        if ($record) {
+            $headjs = '';
+            if ($record instanceof Family) {
+                if ($record->getHusband()) {
+                    $headjs = $record->getHusband()->getXref();
+                } elseif ($record->getWife()) {
+                    $headjs = $record->getWife()->getXref();
+                }
+            }
+            ?>
 			<script>
 				function insertId() {
 					if (window.opener.document.getElementById('addlinkQueue')) {
@@ -255,8 +255,8 @@ 
 				}
 			</script>
 			<?php
-		} else {
-			?>
+        } else {
+            ?>
 			<script>
 				function insertId() {
 					window.opener.alert('<?php echo $iid2; ?> - <?php echo I18N::translate('Not a valid individual, family, or source ID'); ?>');
@@ -264,178 +264,178 @@ 
 				}
 			</script>
 			<?php
-		}
-		?>
+        }
+        ?>
 		<script>window.onLoad = insertId();</script>
 		<?php
-	}
-
-	/**
-	 * Convert custom markup into HTML
-	 *
-	 * @param Note $note
-	 *
-	 * @return string
-	 */
-	public static function formatCensusNote(Note $note) {
-		global $WT_TREE;
-
-		if (preg_match('/(.*)((?:\n.*)*)\n\.start_formatted_area\.\n(.+)\n(.+(?:\n.+)*)\n.end_formatted_area\.((?:\n.*)*)/', $note->getNote(), $match)) {
-			// This looks like a census-assistant shared note
-			$title     = Filter::escapeHtml($match[1]);
-			$preamble  = Filter::escapeHtml($match[2]);
-			$header    = Filter::escapeHtml($match[3]);
-			$data      = Filter::escapeHtml($match[4]);
-			$postamble = Filter::escapeHtml($match[5]);
-
-			// Get the column headers for the census to which this note refers
-			// requires the fact place & date to match the specific census
-			// censusPlace() (Soundex match) and censusDate() functions
-			$fmt_headers   = array();
-			$linkedRecords = array_merge($note->linkedIndividuals('NOTE'), $note->linkedFamilies('NOTE'));
-			$firstRecord   = array_shift($linkedRecords);
-			if ($firstRecord) {
-				$countryCode = '';
-				$date        = '';
-				foreach ($firstRecord->getFacts('CENS') as $fact) {
-					if (trim($fact->getAttribute('NOTE'), '@') === $note->getXref()) {
-						$date        = $fact->getAttribute('DATE');
-						$place       = explode(',', strip_tags($fact->getPlace()->getFullName()));
-						$countryCode = Soundex::daitchMokotoff(array_pop($place));
-						break;
-					}
-				}
-
-				foreach (Census::allCensusPlaces() as $censusPlace) {
-					if (Soundex::compare($countryCode, Soundex::daitchMokotoff($censusPlace->censusPlace()))) {
-						foreach ($censusPlace->allCensusDates() as $census) {
-							if ($census->censusDate() == $date) {
-								foreach ($census->columns() as $column) {
-									$abbrev = $column->abbreviation();
-									if ($abbrev) {
-										$description          = $column->title() ? $column->title() : I18N::translate('Description unavailable');
-										$fmt_headers[$abbrev] = '<span title="' . $description . '">' . $abbrev . '</span>';
-									}
-								}
-								break 2;
-							}
-						}
-					}
-				}
-			}
-			// Substitute header labels and format as HTML
-			$thead = '<tr><th>' . strtr(str_replace('|', '</th><th>', $header), $fmt_headers) . '</th></tr>';
-			$thead = str_replace('.b.', '', $thead);
-
-			// Format data as HTML
-			$tbody = '';
-			foreach (explode("\n", $data) as $row) {
-				$tbody .= '<tr>';
-				foreach (explode('|', $row) as $column) {
-					$tbody .= '<td>' . $column . '</td>';
-				}
-				$tbody .= '</tr>';
-			}
-
-			return
-				$title . "\n" . // The newline allows the framework to expand the details and turn the first line into a link
-				'<div class="markdown">' .
-				'<p>' . $preamble . '</p>' .
-				'<table>' .
-				'<thead>' . $thead . '</thead>' .
-				'<tbody>' . $tbody . '</tbody>' .
-				'</table>' .
-				'<p>' . $postamble . '</p>' .
-				'</div>';
-		} else {
-			// Not a census-assistant shared note - apply default formatting
-			return Filter::formatText($note->getNote(), $WT_TREE);
-		}
-	}
-
-	/**
-	 * Generate an HTML row of data for the census header
-	 *
-	 * Add prefix cell (store XREF and drag/drop)
-	 * Add suffix cell (delete button)
-	 *
-	 * @param CensusInterface $census
-	 *
-	 * @return string
-	 */
-	public static function censusTableHeader(CensusInterface $census) {
-		$html = '';
-		foreach ($census->columns() as $column) {
-			$html .= '<th title="' . $column->title() . '">' . $column->abbreviation() . '</th>';
-		}
-
-		return '<tr><th hidden></th>' . $html . '<th></th></tr>';
-	}
-
-	/**
-	 * Generate an HTML row of data for the census
-	 *
-	 * Add prefix cell (store XREF and drag/drop)
-	 * Add suffix cell (delete button)
-	 *
-	 * @param CensusInterface $census
-	 *
-	 * @return string
-	 */
-	public static function censusTableEmptyRow(CensusInterface $census) {
-		return '<tr><td hidden></td>' . str_repeat('<td><input type="text"></td>', count($census->columns())) . '<td><a class="icon-remove" href="#" title="' . I18N::translate('Remove') . '"></a></td></tr>';
-	}
-
-	/**
-	 * Generate an HTML row of data for the census
-	 *
-	 * Add prefix cell (store XREF and drag/drop)
-	 * Add suffix cell (delete button)
-	 *
-	 * @param CensusInterface $census
-	 * @param Individual      $individual
-	 * @param Individual|null $head
-	 *
-	 * @return string
-	 */
-	public static function censusTableRow(CensusInterface $census, Individual $individual, Individual $head = null) {
-		$html = '';
-		foreach ($census->columns() as $column) {
-			$html .= '<td><input type="text" value="' . $column->generate($individual, $head) . '"></td>';
-		}
-
-		return '<tr><td hidden>' . $individual->getXref() . '</td>' . $html . '<td><a class="icon-remove" href="#" title="' . I18N::translate('Remove') . '"></a></td></tr>';
-	}
-
-	/**
-	 * Create a family on the census navigator.
-	 *
-	 * @param CensusInterface $census
-	 * @param Family          $family
-	 * @param Individual      $head
-	 *
-	 * @return string
-	 */
-	public static function censusNavigatorFamily(CensusInterface $census, Family $family, Individual $head) {
-		$headImg2 = '<i class="icon-button_head" title="' . I18N::translate('Head of household') . '"></i>';
-
-		foreach ($family->getSpouses() as $spouse) {
-			$menu = new Menu(Functions::getCloseRelationshipName($head, $spouse));
-			foreach ($spouse->getChildFamilies() as $grandparents) {
-				foreach ($grandparents->getSpouses() as $grandparent) {
-					$submenu = new Menu(
-						Functions::getCloseRelationshipName($head, $grandparent) . ' - ' . $grandparent->getFullName(),
-						'#',
-						'',
-						array('onclick' => 'return appendCensusRow("' . Filter::escapeJs(self::censusTableRow($census, $grandparent, $head)) . '");')
-					);
-					$submenu->addClass('submenuitem', '');
-					$menu->addSubmenu($submenu);
-					$menu->addClass('', 'submenu');
-				}
-			}
-
-			?>
+    }
+
+    /**
+     * Convert custom markup into HTML
+     *
+     * @param Note $note
+     *
+     * @return string
+     */
+    public static function formatCensusNote(Note $note) {
+        global $WT_TREE;
+
+        if (preg_match('/(.*)((?:\n.*)*)\n\.start_formatted_area\.\n(.+)\n(.+(?:\n.+)*)\n.end_formatted_area\.((?:\n.*)*)/', $note->getNote(), $match)) {
+            // This looks like a census-assistant shared note
+            $title     = Filter::escapeHtml($match[1]);
+            $preamble  = Filter::escapeHtml($match[2]);
+            $header    = Filter::escapeHtml($match[3]);
+            $data      = Filter::escapeHtml($match[4]);
+            $postamble = Filter::escapeHtml($match[5]);
+
+            // Get the column headers for the census to which this note refers
+            // requires the fact place & date to match the specific census
+            // censusPlace() (Soundex match) and censusDate() functions
+            $fmt_headers   = array();
+            $linkedRecords = array_merge($note->linkedIndividuals('NOTE'), $note->linkedFamilies('NOTE'));
+            $firstRecord   = array_shift($linkedRecords);
+            if ($firstRecord) {
+                $countryCode = '';
+                $date        = '';
+                foreach ($firstRecord->getFacts('CENS') as $fact) {
+                    if (trim($fact->getAttribute('NOTE'), '@') === $note->getXref()) {
+                        $date        = $fact->getAttribute('DATE');
+                        $place       = explode(',', strip_tags($fact->getPlace()->getFullName()));
+                        $countryCode = Soundex::daitchMokotoff(array_pop($place));
+                        break;
+                    }
+                }
+
+                foreach (Census::allCensusPlaces() as $censusPlace) {
+                    if (Soundex::compare($countryCode, Soundex::daitchMokotoff($censusPlace->censusPlace()))) {
+                        foreach ($censusPlace->allCensusDates() as $census) {
+                            if ($census->censusDate() == $date) {
+                                foreach ($census->columns() as $column) {
+                                    $abbrev = $column->abbreviation();
+                                    if ($abbrev) {
+                                        $description          = $column->title() ? $column->title() : I18N::translate('Description unavailable');
+                                        $fmt_headers[$abbrev] = '<span title="' . $description . '">' . $abbrev . '</span>';
+                                    }
+                                }
+                                break 2;
+                            }
+                        }
+                    }
+                }
+            }
+            // Substitute header labels and format as HTML
+            $thead = '<tr><th>' . strtr(str_replace('|', '</th><th>', $header), $fmt_headers) . '</th></tr>';
+            $thead = str_replace('.b.', '', $thead);
+
+            // Format data as HTML
+            $tbody = '';
+            foreach (explode("\n", $data) as $row) {
+                $tbody .= '<tr>';
+                foreach (explode('|', $row) as $column) {
+                    $tbody .= '<td>' . $column . '</td>';
+                }
+                $tbody .= '</tr>';
+            }
+
+            return
+                $title . "\n" . // The newline allows the framework to expand the details and turn the first line into a link
+                '<div class="markdown">' .
+                '<p>' . $preamble . '</p>' .
+                '<table>' .
+                '<thead>' . $thead . '</thead>' .
+                '<tbody>' . $tbody . '</tbody>' .
+                '</table>' .
+                '<p>' . $postamble . '</p>' .
+                '</div>';
+        } else {
+            // Not a census-assistant shared note - apply default formatting
+            return Filter::formatText($note->getNote(), $WT_TREE);
+        }
+    }
+
+    /**
+     * Generate an HTML row of data for the census header
+     *
+     * Add prefix cell (store XREF and drag/drop)
+     * Add suffix cell (delete button)
+     *
+     * @param CensusInterface $census
+     *
+     * @return string
+     */
+    public static function censusTableHeader(CensusInterface $census) {
+        $html = '';
+        foreach ($census->columns() as $column) {
+            $html .= '<th title="' . $column->title() . '">' . $column->abbreviation() . '</th>';
+        }
+
+        return '<tr><th hidden></th>' . $html . '<th></th></tr>';
+    }
+
+    /**
+     * Generate an HTML row of data for the census
+     *
+     * Add prefix cell (store XREF and drag/drop)
+     * Add suffix cell (delete button)
+     *
+     * @param CensusInterface $census
+     *
+     * @return string
+     */
+    public static function censusTableEmptyRow(CensusInterface $census) {
+        return '<tr><td hidden></td>' . str_repeat('<td><input type="text"></td>', count($census->columns())) . '<td><a class="icon-remove" href="#" title="' . I18N::translate('Remove') . '"></a></td></tr>';
+    }
+
+    /**
+     * Generate an HTML row of data for the census
+     *
+     * Add prefix cell (store XREF and drag/drop)
+     * Add suffix cell (delete button)
+     *
+     * @param CensusInterface $census
+     * @param Individual      $individual
+     * @param Individual|null $head
+     *
+     * @return string
+     */
+    public static function censusTableRow(CensusInterface $census, Individual $individual, Individual $head = null) {
+        $html = '';
+        foreach ($census->columns() as $column) {
+            $html .= '<td><input type="text" value="' . $column->generate($individual, $head) . '"></td>';
+        }
+
+        return '<tr><td hidden>' . $individual->getXref() . '</td>' . $html . '<td><a class="icon-remove" href="#" title="' . I18N::translate('Remove') . '"></a></td></tr>';
+    }
+
+    /**
+     * Create a family on the census navigator.
+     *
+     * @param CensusInterface $census
+     * @param Family          $family
+     * @param Individual      $head
+     *
+     * @return string
+     */
+    public static function censusNavigatorFamily(CensusInterface $census, Family $family, Individual $head) {
+        $headImg2 = '<i class="icon-button_head" title="' . I18N::translate('Head of household') . '"></i>';
+
+        foreach ($family->getSpouses() as $spouse) {
+            $menu = new Menu(Functions::getCloseRelationshipName($head, $spouse));
+            foreach ($spouse->getChildFamilies() as $grandparents) {
+                foreach ($grandparents->getSpouses() as $grandparent) {
+                    $submenu = new Menu(
+                        Functions::getCloseRelationshipName($head, $grandparent) . ' - ' . $grandparent->getFullName(),
+                        '#',
+                        '',
+                        array('onclick' => 'return appendCensusRow("' . Filter::escapeJs(self::censusTableRow($census, $grandparent, $head)) . '");')
+                    );
+                    $submenu->addClass('submenuitem', '');
+                    $menu->addSubmenu($submenu);
+                    $menu->addClass('', 'submenu');
+                }
+            }
+
+            ?>
 			<tr>
 				<td class="optionbox">
 					<?php echo $menu->getMenu(); ?>
@@ -454,38 +454,38 @@ 
 				</td>
 			</tr>
 			<?php
-		}
-
-		foreach ($family->getChildren() as $child) {
-			$menu = new Menu(Functions::getCloseRelationshipName($head, $child));
-			foreach ($child->getSpouseFamilies() as $spouse_family) {
-				foreach ($spouse_family->getSpouses() as $spouse_family_spouse) {
-					if ($spouse_family_spouse != $child) {
-						$submenu = new Menu(
-							Functions::getCloseRelationshipName($head, $spouse_family_spouse) . ' - ' . $spouse_family_spouse->getFullName(),
-							'#',
-							'',
-							array('onclick' => 'return appendCensusRow("' . Filter::escapeJs(self::censusTableRow($census, $spouse_family_spouse, $head)) . '");')
-						);
-						$submenu->addClass('submenuitem', '');
-						$menu->addSubmenu($submenu);
-						$menu->addClass('', 'submenu');
-					}
-				}
-				foreach ($spouse_family->getChildren() as $spouse_family_child) {
-					$submenu = new Menu(
-						Functions::getCloseRelationshipName($head, $spouse_family_child) . ' - ' . $spouse_family_child->getFullName(),
-						'#',
-						'',
-						array('onclick' => 'return appendCensusRow("' . Filter::escapeJs(self::censusTableRow($census, $spouse_family_child, $head)) . '");')
-					);
-					$submenu->addClass('submenuitem', '');
-					$menu->addSubmenu($submenu);
-					$menu->addClass('', 'submenu');
-				}
-			}
-
-			?>
+        }
+
+        foreach ($family->getChildren() as $child) {
+            $menu = new Menu(Functions::getCloseRelationshipName($head, $child));
+            foreach ($child->getSpouseFamilies() as $spouse_family) {
+                foreach ($spouse_family->getSpouses() as $spouse_family_spouse) {
+                    if ($spouse_family_spouse != $child) {
+                        $submenu = new Menu(
+                            Functions::getCloseRelationshipName($head, $spouse_family_spouse) . ' - ' . $spouse_family_spouse->getFullName(),
+                            '#',
+                            '',
+                            array('onclick' => 'return appendCensusRow("' . Filter::escapeJs(self::censusTableRow($census, $spouse_family_spouse, $head)) . '");')
+                        );
+                        $submenu->addClass('submenuitem', '');
+                        $menu->addSubmenu($submenu);
+                        $menu->addClass('', 'submenu');
+                    }
+                }
+                foreach ($spouse_family->getChildren() as $spouse_family_child) {
+                    $submenu = new Menu(
+                        Functions::getCloseRelationshipName($head, $spouse_family_child) . ' - ' . $spouse_family_child->getFullName(),
+                        '#',
+                        '',
+                        array('onclick' => 'return appendCensusRow("' . Filter::escapeJs(self::censusTableRow($census, $spouse_family_child, $head)) . '");')
+                    );
+                    $submenu->addClass('submenuitem', '');
+                    $menu->addSubmenu($submenu);
+                    $menu->addClass('', 'submenu');
+                }
+            }
+
+            ?>
 			<tr>
 				<td class="optionbox">
 					<?php echo $menu->getMenu(); ?>
@@ -504,7 +504,7 @@ 
 				</td>
 			</tr>
 			<?php
-		}
-		echo '<tr><td><br></td></tr>';
-	}
+        }
+        echo '<tr><td><br></td></tr>';
+    }
 }

Switch Indentation +11 added lines, -11 removed lines

@@ -52,17 +52,17 @@
 	 */
 	public function modAction($mod_action) {
 		switch ($mod_action) {
-		case 'census_find':
-			self::censusFind();
-			break;
-		case 'media_find':
-			self::mediaFind();
-			break;
-		case 'media_query_3a':
-			self::mediaQuery();
-			break;
-		default:
-			http_response_code(404);
+		    case 'census_find':
+			    self::censusFind();
+			    break;
+		    case 'media_find':
+			    self::mediaFind();
+			    break;
+		    case 'media_query_3a':
+			    self::mediaQuery();
+			    break;
+		    default:
+			    http_response_code(404);
 		}
 	}
 

Braces +24 added lines, -12 removed lines

@@ -33,14 +33,17 @@ 
 /**
  * Class CensusAssistantModule
  */
-class CensusAssistantModule extends AbstractModule {
+class CensusAssistantModule extends AbstractModule
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('Census assistant');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Census assistant” module */ I18N::translate('An alternative way to enter census transcripts and link them to individuals.');
 	}
 
@@ -50,7 +53,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'census_find':
 			self::censusFind();
@@ -69,7 +73,8 @@ 
 	/**
 	 * Find an individual.
 	 */
-	private static function censusFind() {
+	private static function censusFind()
+	{
 		global $WT_TREE;
 
 		$controller = new SimpleController;
@@ -126,7 +131,8 @@ 
 	/**
 	 * Find a media object.
 	 */
-	private static function mediaFind() {
+	private static function mediaFind()
+	{
 		global $WT_TREE;
 
 		$controller = new SimpleController;
@@ -224,7 +230,8 @@ 
 	/**
 	 * Search for a media object.
 	 */
-	private static function mediaQuery() {
+	private static function mediaQuery()
+	{
 		global $WT_TREE;
 
 		$iid2 = Filter::get('iid', WT_REGEX_XREF);
@@ -277,7 +284,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function formatCensusNote(Note $note) {
+	public static function formatCensusNote(Note $note)
+	{
 		global $WT_TREE;
 
 		if (preg_match('/(.*)((?:\n.*)*)\n\.start_formatted_area\.\n(.+)\n(.+(?:\n.+)*)\n.end_formatted_area\.((?:\n.*)*)/', $note->getNote(), $match)) {
@@ -363,7 +371,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function censusTableHeader(CensusInterface $census) {
+	public static function censusTableHeader(CensusInterface $census)
+	{
 		$html = '';
 		foreach ($census->columns() as $column) {
 			$html .= '<th title="' . $column->title() . '">' . $column->abbreviation() . '</th>';
@@ -382,7 +391,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function censusTableEmptyRow(CensusInterface $census) {
+	public static function censusTableEmptyRow(CensusInterface $census)
+	{
 		return '<tr><td hidden></td>' . str_repeat('<td><input type="text"></td>', count($census->columns())) . '<td><a class="icon-remove" href="#" title="' . I18N::translate('Remove') . '"></a></td></tr>';
 	}
 
@@ -398,7 +408,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function censusTableRow(CensusInterface $census, Individual $individual, Individual $head = null) {
+	public static function censusTableRow(CensusInterface $census, Individual $individual, Individual $head = null)
+	{
 		$html = '';
 		foreach ($census->columns() as $column) {
 			$html .= '<td><input type="text" value="' . $column->generate($individual, $head) . '"></td>';
@@ -416,7 +427,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function censusNavigatorFamily(CensusInterface $census, Family $family, Individual $head) {
+	public static function censusNavigatorFamily(CensusInterface $census, Family $family, Individual $head)
+	{
 		$headImg2 = '<i class="icon-button_head" title="' . I18N::translate('Head of household') . '"></i>';
 
 		foreach ($family->getSpouses() as $spouse) {

app/Module/FamilyTreeNews/Schema/Migration2.php

Indentation +19 added lines, -19 removed lines

@@ -23,24 +23,24 @@
  * Upgrade the database schema from version 2 to version 3.
  */
 class Migration2 implements MigrationInterface {
-	/** {@inheritDoc} */
-	public function upgrade() {
-		// Delete any data that might violate the new constraints
-		Database::exec(
-			"DELETE FROM `##news`" .
-			" WHERE user_id   NOT IN (SELECT user_id   FROM `##user`  )" .
-			" OR    gedcom_id NOT IN (SELECT gedcom_id FROM `##gedcom`)"
-		);
+    /** {@inheritDoc} */
+    public function upgrade() {
+        // Delete any data that might violate the new constraints
+        Database::exec(
+            "DELETE FROM `##news`" .
+            " WHERE user_id   NOT IN (SELECT user_id   FROM `##user`  )" .
+            " OR    gedcom_id NOT IN (SELECT gedcom_id FROM `##gedcom`)"
+        );
 
-		// Add the new constraints
-		try {
-			Database::exec(
-				"ALTER TABLE `##news`" .
-				" ADD FOREIGN KEY `##news_fk1` (user_id  ) REFERENCES `##user`   (user_id)   ON DELETE CASCADE," .
-				" ADD FOREIGN KEY `##news_fk2` (gedcom_id) REFERENCES `##gedcom` (gedcom_id) ON DELETE CASCADE"
-			);
-		} catch (PDOException $ex) {
-			// Already updated?
-		}
-	}
+        // Add the new constraints
+        try {
+            Database::exec(
+                "ALTER TABLE `##news`" .
+                " ADD FOREIGN KEY `##news_fk1` (user_id  ) REFERENCES `##user`   (user_id)   ON DELETE CASCADE," .
+                " ADD FOREIGN KEY `##news_fk2` (gedcom_id) REFERENCES `##gedcom` (gedcom_id) ON DELETE CASCADE"
+            );
+        } catch (PDOException $ex) {
+            // Already updated?
+        }
+    }
 }

Braces +4 added lines, -2 removed lines

@@ -22,9 +22,11 @@
 /**
  * Upgrade the database schema from version 2 to version 3.
  */
-class Migration2 implements MigrationInterface {
+class Migration2 implements MigrationInterface
+{
 	/** {@inheritDoc} */
-	public function upgrade() {
+	public function upgrade()
+	{
 		// Delete any data that might violate the new constraints
 		Database::exec(
 			"DELETE FROM `##news`" .

app/Module/FamilyTreeNews/Schema/Migration1.php

Indentation +44 added lines, -44 removed lines

@@ -23,54 +23,54 @@
  * Upgrade the database schema from version 1 to version 2.
  */
 class Migration1 implements MigrationInterface {
-	/** {@inheritDoc} */
-	public function upgrade() {
-		// Add new columns
-		try {
-			Database::exec(
-				"ALTER TABLE `##news`" .
-				" ADD user_id INTEGER NULL AFTER n_id," .
-				" ADD gedcom_id INTEGER NULL AFTER user_id," .
-				" ADD updated TIMESTAMP ON UPDATE CURRENT_TIMESTAMP DEFAULT CURRENT_TIMESTAMP," .
-				" ADD KEY news_ix1 (user_id, updated)," .
-				" ADD KEY news_ix2 (gedcom_id, updated)"
-			);
-		} catch (PDOException $ex) {
-			// Already updated?
-		}
+    /** {@inheritDoc} */
+    public function upgrade() {
+        // Add new columns
+        try {
+            Database::exec(
+                "ALTER TABLE `##news`" .
+                " ADD user_id INTEGER NULL AFTER n_id," .
+                " ADD gedcom_id INTEGER NULL AFTER user_id," .
+                " ADD updated TIMESTAMP ON UPDATE CURRENT_TIMESTAMP DEFAULT CURRENT_TIMESTAMP," .
+                " ADD KEY news_ix1 (user_id, updated)," .
+                " ADD KEY news_ix2 (gedcom_id, updated)"
+            );
+        } catch (PDOException $ex) {
+            // Already updated?
+        }
 
 // Migrate data from the old columns to the new ones
-		try {
-			Database::exec(
-				"UPDATE `##news` n" .
-				" LEFT JOIN `##gedcom` g ON (n.n_username=g.gedcom_name)" .
-				" LEFT JOIN `##user` u ON (n.n_username=u.user_name)" .
-				" SET n.gedcom_id=g.gedcom_id, n.user_id=u.user_id, updated=FROM_UNIXTIME(n_date)"
-			);
-		} catch (PDOException $ex) {
-			// Already updated?
-		}
+        try {
+            Database::exec(
+                "UPDATE `##news` n" .
+                " LEFT JOIN `##gedcom` g ON (n.n_username=g.gedcom_name)" .
+                " LEFT JOIN `##user` u ON (n.n_username=u.user_name)" .
+                " SET n.gedcom_id=g.gedcom_id, n.user_id=u.user_id, updated=FROM_UNIXTIME(n_date)"
+            );
+        } catch (PDOException $ex) {
+            // Already updated?
+        }
 
 // Delete orphaned rows
-		try {
-			Database::exec(
-				"DELETE FROM `##news` WHERE user_id IS NULL AND gedcom_id IS NULL"
-			);
-		} catch (PDOException $ex) {
-			// Already updated?
-		}
+        try {
+            Database::exec(
+                "DELETE FROM `##news` WHERE user_id IS NULL AND gedcom_id IS NULL"
+            );
+        } catch (PDOException $ex) {
+            // Already updated?
+        }
 
 // Delete/rename old columns
-		try {
-			Database::exec(
-				"ALTER TABLE `##news`" .
-				" DROP n_username, DROP n_date," .
-				" CHANGE n_id news_id INTEGER NOT NULL AUTO_INCREMENT," .
-				" CHANGE n_title subject VARCHAR(255) COLLATE utf8_unicode_ci," .
-				" CHANGE n_text body TEXT COLLATE utf8_unicode_ci"
-			);
-		} catch (PDOException $ex) {
-			// Already updated?
-		}
-	}
+        try {
+            Database::exec(
+                "ALTER TABLE `##news`" .
+                " DROP n_username, DROP n_date," .
+                " CHANGE n_id news_id INTEGER NOT NULL AUTO_INCREMENT," .
+                " CHANGE n_title subject VARCHAR(255) COLLATE utf8_unicode_ci," .
+                " CHANGE n_text body TEXT COLLATE utf8_unicode_ci"
+            );
+        } catch (PDOException $ex) {
+            // Already updated?
+        }
+    }
 }

Braces +4 added lines, -2 removed lines

@@ -22,9 +22,11 @@
 /**
  * Upgrade the database schema from version 1 to version 2.
  */
-class Migration1 implements MigrationInterface {
+class Migration1 implements MigrationInterface
+{
 	/** {@inheritDoc} */
-	public function upgrade() {
+	public function upgrade()
+	{
 		// Add new columns
 		try {
 			Database::exec(

app/Module/FamilyTreeNews/Schema/Migration0.php

Indentation +14 added lines, -14 removed lines

@@ -22,18 +22,18 @@
  * Upgrade the database schema from version 0 (empty database) to version 1.
  */
 class Migration0 implements MigrationInterface {
-	/** {@inheritDoc} */
-	public function upgrade() {
-		Database::exec(
-			"CREATE TABLE IF NOT EXISTS `##news` (" .
-			" n_id       INTEGER AUTO_INCREMENT NOT NULL," .
-			" n_username VARCHAR(100)           NOT NULL," .
-			" n_date     INTEGER                NOT NULL," .
-			" n_title    VARCHAR(255)           NOT NULL," .
-			" n_text     TEXT                   NOT NULL," .
-			" PRIMARY KEY     (n_id)," .
-			"         KEY ix1 (n_username)" .
-			") COLLATE utf8_unicode_ci ENGINE=InnoDB"
-		);
-	}
+    /** {@inheritDoc} */
+    public function upgrade() {
+        Database::exec(
+            "CREATE TABLE IF NOT EXISTS `##news` (" .
+            " n_id       INTEGER AUTO_INCREMENT NOT NULL," .
+            " n_username VARCHAR(100)           NOT NULL," .
+            " n_date     INTEGER                NOT NULL," .
+            " n_title    VARCHAR(255)           NOT NULL," .
+            " n_text     TEXT                   NOT NULL," .
+            " PRIMARY KEY     (n_id)," .
+            "         KEY ix1 (n_username)" .
+            ") COLLATE utf8_unicode_ci ENGINE=InnoDB"
+        );
+    }
 }

Braces +4 added lines, -2 removed lines

@@ -21,9 +21,11 @@
 /**
  * Upgrade the database schema from version 0 (empty database) to version 1.
  */
-class Migration0 implements MigrationInterface {
+class Migration0 implements MigrationInterface
+{
 	/** {@inheritDoc} */
-	public function upgrade() {
+	public function upgrade()
+	{
 		Database::exec(
 			"CREATE TABLE IF NOT EXISTS `##news` (" .
 			" n_id       INTEGER AUTO_INCREMENT NOT NULL," .