fisharebest / webtrees

app/Module/SiteMapModule.php

Braces +18 added lines, -9 removed lines

@@ -30,17 +30,20 @@ 
 /**
  * Class SiteMapModule
  */
-class SiteMapModule extends AbstractModule implements ModuleConfigInterface {
+class SiteMapModule extends AbstractModule implements ModuleConfigInterface
+{
 	const RECORDS_PER_VOLUME = 500; // Keep sitemap files small, for memory, CPU and max_allowed_packet limits.
 	const CACHE_LIFE         = 1209600; // Two weeks
 
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module - see http://en.wikipedia.org/wiki/Sitemaps */ I18N::translate('Sitemaps');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Sitemaps” module */ I18N::translate('Generate sitemap files for search engines.');
 	}
 
@@ -50,7 +53,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'admin':
 			$this->admin();
@@ -68,7 +72,8 @@ 
 	 *
 	 * @param string $file
 	 */
-	private function generate($file) {
+	private function generate($file)
+	{
 		if ($file == 'sitemap.xml') {
 			$this->generateIndex();
 		} elseif (preg_match('/^sitemap-(\d+)-([isrmn])-(\d+).xml$/', $file, $match)) {
@@ -82,7 +87,8 @@ 
 	 * The index file contains references to all the other files.
 	 * These files are the same for visitors/users/admins.
 	 */
-	private function generateIndex() {
+	private function generateIndex()
+	{
 		// Check the cache
 		$timestamp = $this->getSetting('sitemap.timestamp');
 		if ($timestamp > WT_TIMESTAMP - self::CACHE_LIFE) {
@@ -150,7 +156,8 @@ 
 	 * @param string $rec_type
 	 * @param string $volume
 	 */
-	private function generateFile($ged_id, $rec_type, $volume) {
+	private function generateFile($ged_id, $rec_type, $volume)
+	{
 		$tree = Tree::findById($ged_id);
 		// Check the cache
 		$timestamp = $this->getSetting('sitemap-' . $ged_id . '-' . $rec_type . '-' . $volume . '.timestamp');
@@ -271,7 +278,8 @@ 
 	/**
 	 * Edit the configuration
 	 */
-	private function admin() {
+	private function admin()
+	{
 		$controller = new PageController;
 		$controller
 			->restrictAccess(Auth::isAdmin())
@@ -339,7 +347,8 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function getConfigLink() {
+	public function getConfigLink()
+	{
 		return 'module.php?mod=' . $this->getName() . '&mod_action=admin';
 	}
 }

app/Module/FamilyTreeNewsModule.php

Braces +18 added lines, -9 removed lines

@@ -25,7 +25,8 @@ 
 /**
  * Class FamilyTreeNewsModule
  */
-class FamilyTreeNewsModule extends AbstractModule implements ModuleBlockInterface {
+class FamilyTreeNewsModule extends AbstractModule implements ModuleBlockInterface
+{
 	// How to update the database schema for this module
 	const SCHEMA_TARGET_VERSION   = 3;
 	const SCHEMA_SETTING_NAME     = 'NB_SCHEMA_VERSION';
@@ -36,7 +37,8 @@ 
 	 *
 	 * @param string $directory Where is this module installed
 	 */
-	public function __construct($directory) {
+	public function __construct($directory)
+	{
 		parent::__construct($directory);
 
 		// Create/update the database tables.
@@ -49,7 +51,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('News');
 	}
 
@@ -58,7 +61,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “News” module */ I18N::translate('Family news and site announcements.');
 	}
 
@@ -71,7 +75,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;
 
 		switch (Filter::get('action')) {
@@ -145,17 +150,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function loadAjax() {
+	public function loadAjax()
+	{
 		return false;
 	}
 
 	/** {@inheritdoc} */
-	public function isUserBlock() {
+	public function isUserBlock()
+	{
 		return false;
 	}
 
 	/** {@inheritdoc} */
-	public function isGedcomBlock() {
+	public function isGedcomBlock()
+	{
 		return true;
 	}
 
@@ -164,6 +172,7 @@ 
 	 *
 	 * @param int $block_id
 	 */
-	public function configureBlock($block_id) {
+	public function configureBlock($block_id)
+	{
 	}
 }

app/Module/SourcesTabModule.php

Braces +20 added lines, -10 removed lines

@@ -25,39 +25,46 @@ 
 /**
  * Class SourcesTabModule
  */
-class SourcesTabModule extends AbstractModule implements ModuleTabInterface {
+class SourcesTabModule extends AbstractModule implements ModuleTabInterface
+{
 	/** @var Fact[] All facts belonging to this source. */
 	private $facts;
 
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('Sources');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Sources” module */ I18N::translate('A tab showing the sources linked to an individual.');
 	}
 
 	/** {@inheritdoc} */
-	public function defaultTabOrder() {
+	public function defaultTabOrder()
+	{
 		return 30;
 	}
 
 	/** {@inheritdoc} */
-	public function hasTabContent() {
+	public function hasTabContent()
+	{
 		global $WT_TREE;
 
 		return Auth::isEditor($WT_TREE) || $this->getFactsWithSources();
 	}
 
 	/** {@inheritdoc} */
-	public function isGrayedOut() {
+	public function isGrayedOut()
+	{
 		return !$this->getFactsWithSources();
 	}
 
 	/** {@inheritdoc} */
-	public function getTabContent() {
+	public function getTabContent()
+	{
 		global $controller;
 
 		ob_start();
@@ -113,7 +120,8 @@ 
 	 *
 	 * @return Fact[]
 	 */
-	private function getFactsWithSources() {
+	private function getFactsWithSources()
+	{
 		global $controller;
 
 		if ($this->facts === null) {
@@ -138,12 +146,14 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function canLoadAjax() {
+	public function canLoadAjax()
+	{
 		return !Auth::isSearchEngine(); // Search engines cannot use AJAX
 	}
 
 	/** {@inheritdoc} */
-	public function getPreLoadContent() {
+	public function getPreLoadContent()
+	{
 		return '';
 	}
 }

app/Module/FamiliesSidebarModule.php

Braces +22 added lines, -11 removed lines

@@ -26,14 +26,17 @@ 
 /**
  * Class FamiliesSidebarModule
  */
-class FamiliesSidebarModule extends AbstractModule implements ModuleSidebarInterface {
+class FamiliesSidebarModule extends AbstractModule implements ModuleSidebarInterface
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module/sidebar */ I18N::translate('Family list');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Families” module */ I18N::translate('A sidebar showing an alphabetic list of all the families in the family tree.');
 	}
 
@@ -43,7 +46,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'ajax':
 			header('Content-Type: text/html; charset=UTF-8');
@@ -56,17 +60,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function defaultSidebarOrder() {
+	public function defaultSidebarOrder()
+	{
 		return 50;
 	}
 
 	/** {@inheritdoc} */
-	public function hasSidebarContent() {
+	public function hasSidebarContent()
+	{
 		return true;
 	}
 
 	/** {@inheritdoc} */
-	public function getSidebarAjaxContent() {
+	public function getSidebarAjaxContent()
+	{
 		global $WT_TREE;
 
 		$alpha   = Filter::get('alpha'); // All surnames beginning with this letter where "@"=unknown and ","=none
@@ -89,7 +96,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getSidebarContent() {
+	public function getSidebarContent()
+	{
 		global $controller, $WT_TREE;
 
 		// Fetch a list of the initial letters of all surnames in the database
@@ -181,7 +189,8 @@ 
 	 *
 	 * @return string
 	 */
-	private function getAlphaSurnames(Tree $tree, $alpha) {
+	private function getAlphaSurnames(Tree $tree, $alpha)
+	{
 		$surnames = QueryName::surnames($tree, '', $alpha, true, true);
 		$out      = '<ul>';
 		foreach (array_keys($surnames) as $surname) {
@@ -203,7 +212,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getSurnameFams(Tree $tree, $alpha, $surname) {
+	public function getSurnameFams(Tree $tree, $alpha, $surname)
+	{
 		$families = QueryName::families($tree, $surname, $alpha, '', true);
 		$out      = '<ul>';
 		foreach ($families as $family) {
@@ -231,7 +241,8 @@ 
 	 *
 	 * @return string
 	 */
-	private function search(Tree $tree, $query) {
+	private function search(Tree $tree, $query)
+	{
 		if (strlen($query) < 2) {
 			return '';
 		}

app/Module/StoriesModule.php

Braces +39 added lines, -19 removed lines

@@ -32,12 +32,14 @@ 
  */
 class StoriesModule extends AbstractModule implements ModuleTabInterface, ModuleConfigInterface, ModuleMenuInterface {
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('Stories');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “Stories” module */ I18N::translate('Add narrative stories to individuals in the family tree.');
 	}
 
@@ -47,7 +49,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'admin_edit':
 			$this->edit();
@@ -68,17 +71,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function getConfigLink() {
+	public function getConfigLink()
+	{
 		return 'module.php?mod=' . $this->getName() . '&mod_action=admin_config';
 	}
 
 	/** {@inheritdoc} */
-	public function defaultTabOrder() {
+	public function defaultTabOrder()
+	{
 		return 55;
 	}
 
 	/** {@inheritdoc} */
-	public function getTabContent() {
+	public function getTabContent()
+	{
 		global $controller, $WT_TREE;
 
 		$block_ids =
@@ -117,12 +123,14 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function hasTabContent() {
+	public function hasTabContent()
+	{
 		return $this->getTabContent() != '';
 	}
 
 	/** {@inheritdoc} */
-	public function isGrayedOut() {
+	public function isGrayedOut()
+	{
 		global $controller;
 
 		$count_of_stories =
@@ -142,19 +150,22 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function canLoadAjax() {
+	public function canLoadAjax()
+	{
 		return false;
 	}
 
 	/** {@inheritdoc} */
-	public function getPreLoadContent() {
+	public function getPreLoadContent()
+	{
 		return '';
 	}
 
 	/**
 	 * Show and process a form to edit a story.
 	 */
-	private function edit() {
+	private function edit()
+	{
 		global $WT_TREE;
 
 		if (Auth::isEditor($WT_TREE)) {
@@ -283,7 +294,8 @@ 
 	/**
 	 * Respond to a request to delete a story.
 	 */
-	private function delete() {
+	private function delete()
+	{
 		global $WT_TREE;
 
 		if (Auth::isEditor($WT_TREE)) {
@@ -305,7 +317,8 @@ 
 	/**
 	 * The admin view - list, create, edit, delete stories.
 	 */
-	private function config() {
+	private function config()
+	{
 		global $WT_TREE;
 
 		$controller = new PageController;
@@ -389,8 +402,11 @@ 
 						<a href="<?php echo $individual->getHtmlUrl(); ?>#stories">
 							<?php echo $individual->getFullName(); ?>
 						</a>
-						<?php else: ?>
-							<?php echo $story->xref; ?>
+						<?php else {
+    : ?>
+							<?php echo $story->xref;
+}
+?>
 						<?php endif; ?>
 						</td>
 						<td>
@@ -416,7 +432,8 @@ 
 	/**
 	 * Show the list of stories
 	 */
-	private function showList() {
+	private function showList()
+	{
 		global $controller, $WT_TREE;
 
 		$controller = new PageController;
@@ -482,7 +499,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function defaultMenuOrder() {
+	public function defaultMenuOrder()
+	{
 		return 30;
 	}
 
@@ -493,7 +511,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function defaultAccessLevel() {
+	public function defaultAccessLevel()
+	{
 		return Auth::PRIV_HIDE;
 	}
 
@@ -502,7 +521,8 @@ 
 	 *
 	 * @return Menu|null
 	 */
-	public function getMenu() {
+	public function getMenu()
+	{
 		$menu = new Menu($this->getTitle(), 'module.php?mod=' . $this->getName() . '&amp;mod_action=show_list', 'menu-story');
 
 		return $menu;

app/Module/CompactTreeChartModule.php

Braces +12 added lines, -6 removed lines

@@ -23,13 +23,15 @@ 
 /**
  * Class CompactTreeChartModule
  */
-class CompactTreeChartModule extends AbstractModule implements ModuleChartInterface {
+class CompactTreeChartModule extends AbstractModule implements ModuleChartInterface
+{
 	/**
 	 * How should this module be labelled on tabs, menus, etc.?
 	 *
 	 * @return string
 	 */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module/chart */ I18N::translate('Compact tree');
 	}
 
@@ -38,7 +40,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “CompactTreeChart” module */ I18N::translate('A chart of an individual’s ancestors, as a compact tree.');
 	}
 
@@ -49,7 +52,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function defaultAccessLevel() {
+	public function defaultAccessLevel()
+	{
 		return Auth::PRIV_PRIVATE;
 	}
 
@@ -58,7 +62,8 @@ 
 	 *
 	 * @return Menu|null
 	 */
-	public function getChartMenu(Individual $individual) {
+	public function getChartMenu(Individual $individual)
+	{
 		return new Menu(
 			$this->getTitle(),
 			'compact.php?rootid=' . $individual->getXref() . '&ged=' . $individual->getTree()->getNameUrl(),
@@ -72,7 +77,8 @@ 
 	 *
 	 * @return Menu|null
 	 */
-	public function getBoxChartMenu(Individual $individual) {
+	public function getBoxChartMenu(Individual $individual)
+	{
 		return $this->getChartMenu($individual);
 	}
 }

app/Module/IndividualSidebarModule.php

Braces +22 added lines, -11 removed lines

@@ -27,14 +27,17 @@ 
 /**
  * Class IndividualSidebarModule
  */
-class IndividualSidebarModule extends AbstractModule implements ModuleSidebarInterface {
+class IndividualSidebarModule extends AbstractModule implements ModuleSidebarInterface
+{
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module */ I18N::translate('Individual list');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of “Individuals” module */ I18N::translate('A sidebar showing an alphabetic list of all the individuals in the family tree.');
 	}
 
@@ -44,7 +47,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'ajax':
 			header('Content-Type: text/html; charset=UTF-8');
@@ -57,17 +61,20 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function defaultSidebarOrder() {
+	public function defaultSidebarOrder()
+	{
 		return 40;
 	}
 
 	/** {@inheritdoc} */
-	public function hasSidebarContent() {
+	public function hasSidebarContent()
+	{
 		return !Auth::isSearchEngine();
 	}
 
 	/** {@inheritdoc} */
-	public function getSidebarAjaxContent() {
+	public function getSidebarAjaxContent()
+	{
 		global $WT_TREE;
 
 		$alpha   = Filter::get('alpha'); // All surnames beginning with this letter where "@"=unknown and ","=none
@@ -90,7 +97,8 @@ 
 	 *
 	 * @return string
 	 */
-	public function getSidebarContent() {
+	public function getSidebarContent()
+	{
 		global $controller, $WT_TREE;
 
 		// Fetch a list of the initial letters of all surnames in the database
@@ -182,7 +190,8 @@ 
 	 *
 	 * @return string
 	 */
-	private function getAlphaSurnames(Tree $tree, $alpha) {
+	private function getAlphaSurnames(Tree $tree, $alpha)
+	{
 		$surnames = QueryName::surnames($tree, '', $alpha, true, false);
 		$out      = '<ul>';
 		foreach (array_keys($surnames) as $surname) {
@@ -204,7 +213,8 @@ 
 	 *
 	 * @return string
 	 */
-	private function getSurnameIndis(Tree $tree, $alpha, $surname) {
+	private function getSurnameIndis(Tree $tree, $alpha, $surname)
+	{
 		$indis = QueryName::individuals($tree, $surname, $alpha, '', true, false);
 		$out   = '<ul>';
 		foreach ($indis as $person) {
@@ -232,7 +242,8 @@ 
 	 *
 	 * @return string
 	 */
-	private function search(Tree $tree, $query) {
+	private function search(Tree $tree, $query)
+	{
 		if (strlen($query) < 2) {
 			return '';
 		}

app/Module/PedigreeReportModule.php

Braces +10 added lines, -5 removed lines

@@ -22,15 +22,18 @@ 
 /**
  * Class PedigreeReportModule
  */
-class PedigreeReportModule extends AbstractModule implements ModuleReportInterface {
+class PedigreeReportModule 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('Pedigree');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		// This text also appears in the .XML file - update both together
 		return /* I18N: Description of the “Pedigree” module */ I18N::translate('A report of an individual’s ancestors, formatted as a tree.');
 	}
@@ -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/FrequentlyAskedQuestionsModule.php

Braces +26 added lines, -13 removed lines

@@ -30,12 +30,14 @@ 
  */
 class FrequentlyAskedQuestionsModule extends AbstractModule implements ModuleMenuInterface, ModuleConfigInterface {
 	/** {@inheritdoc} */
-	public function getTitle() {
+	public function getTitle()
+	{
 		return /* I18N: Name of a module. Abbreviation for “Frequently Asked Questions” */ I18N::translate('FAQ');
 	}
 
 	/** {@inheritdoc} */
-	public function getDescription() {
+	public function getDescription()
+	{
 		return /* I18N: Description of the “FAQ” module */ I18N::translate('A list of frequently asked questions and answers.');
 	}
 
@@ -45,7 +47,8 @@ 
 	 *
 	 * @param string $mod_action
 	 */
-	public function modAction($mod_action) {
+	public function modAction($mod_action)
+	{
 		switch ($mod_action) {
 		case 'admin_config':
 			$this->config();
@@ -86,14 +89,16 @@ 
 	}
 
 	/** {@inheritdoc} */
-	public function getConfigLink() {
+	public function getConfigLink()
+	{
 		return 'module.php?mod=' . $this->getName() . '&mod_action=admin_config';
 	}
 
 	/**
 	 * Action from the configuration page
 	 */
-	private function editSave() {
+	private function editSave()
+	{
 		if (Filter::checkCsrf()) {
 			$block_id = Filter::postInteger('block_id');
 			if ($block_id) {
@@ -125,7 +130,8 @@ 
 	/**
 	 * Action from the configuration page
 	 */
-	private function edit() {
+	private function edit()
+	{
 		global $WT_TREE;
 
 		$controller = new PageController;
@@ -241,7 +247,8 @@ 
 	/**
 	 * Delete an FAQ.
 	 */
-	private function delete() {
+	private function delete()
+	{
 		$block_id = Filter::getInteger('block_id');
 
 		Database::prepare(
@@ -256,7 +263,8 @@ 
 	/**
 	 * Move an FAQ up the list.
 	 */
-	private function moveup() {
+	private function moveup()
+	{
 		$block_id = Filter::getInteger('block_id');
 
 		$block_order = Database::prepare(
@@ -294,7 +302,8 @@ 
 	/**
 	 * Move an FAQ down the list.
 	 */
-	private function movedown() {
+	private function movedown()
+	{
 		$block_id = Filter::get('block_id');
 
 		$block_order = Database::prepare(
@@ -334,7 +343,8 @@ 
 	/**
 	 * Show a list of FAQs
 	 */
-	private function show() {
+	private function show()
+	{
 		global $controller, $WT_TREE;
 
 		$controller = new PageController;
@@ -396,7 +406,8 @@ 
 	/**
 	 * Provide a form to manage the FAQs.
 	 */
-	private function config() {
+	private function config()
+	{
 		global $WT_TREE;
 
 		$controller = new PageController;
@@ -509,7 +520,8 @@ 
 	 *
 	 * @return int
 	 */
-	public function defaultMenuOrder() {
+	public function defaultMenuOrder()
+	{
 		return 40;
 	}
 
@@ -518,7 +530,8 @@ 
 	 *
 	 * @return Menu|null
 	 */
-	public function getMenu() {
+	public function getMenu()
+	{
 		global $WT_TREE;
 
 		$faqs = Database::prepare(