fisharebest / webtrees

app/Controller/IndividualController.php

Doc Comments -1 removed lines

@@ -127,7 +127,6 @@
 	/**
 	 * Format a name record
 	 *
-	 * @param int  $primary
 	 * @param Fact $name_fact
 	 *
 	 * @return string

Spacing +3 added lines, -3 removed lines

@@ -95,7 +95,7 @@ 
 	public function getTabs() {
 		$active_tabs = Module::getActiveTabs($this->record->getTree());
 
-		return array_filter($active_tabs, function(ModuleTabInterface $tab) { return $tab->hasTabContent(); });
+		return array_filter($active_tabs, function (ModuleTabInterface $tab) { return $tab->hasTabContent(); });
 	}
 
 	/**
@@ -143,7 +143,7 @@ 
 
 		if ($n === 0) {
 			$content_class = 'collapse show';
-			$aria  = 'true';
+			$aria = 'true';
 
 			// Display gender icon
 			foreach ($individual->getFacts('SEX') as $sex_fact) {
@@ -162,7 +162,7 @@ 
 		echo '<dd class="field">', $dummy->getFullName();
 		if ($this->record->canEdit() && !$name_fact->isPendingDeletion()) {
 			echo "<div class=\"deletelink noprint\"><a class=\"deleteicon\" href=\"#\" onclick=\"return delete_fact('" . I18N::translate('Are you sure you want to delete this fact?') . "', '" . $this->record->getXref() . "', '" . $name_fact->getFactId() . "');\" title=\"" . I18N::translate('Delete this name') . '"><span class="link_text">' . I18N::translate('Delete this name') . '</span></a></div>';
-			echo '<div class="editlink noprint"><a href="edit_interface.php?action=editname&amp;xref=' . $this->record->getXref() . '&amp;fact_id=' . $name_fact->getFactId() . '&amp;ged=' . $this->record->getTree()->getNameHtml() .  '" class="editicon" title="' . I18N::translate('Edit the name') . '"><span class="link_text">' . I18N::translate('Edit the name') . '</span></a></div>';
+			echo '<div class="editlink noprint"><a href="edit_interface.php?action=editname&amp;xref=' . $this->record->getXref() . '&amp;fact_id=' . $name_fact->getFactId() . '&amp;ged=' . $this->record->getTree()->getNameHtml() . '" class="editicon" title="' . I18N::translate('Edit the name') . '"><span class="link_text">' . I18N::translate('Edit the name') . '</span></a></div>';
 		}
 		echo '</dd>';
 		$ct = preg_match_all('/\n2 (\w+) (.*)/', $name_fact->getGedcom(), $nmatch, PREG_SET_ORDER);

individual.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 if (Auth::isAdmin()) {
 	$user = User::findByGenealogyRecord($controller->record);
 	if ($user) {
-		$user_link =  ' —  <a href="admin_users.php?filter=' . Filter::escapeHtml($user->getUserName()) . '">' . Filter::escapeHtml($user->getUserName()) . '</a>';
+		$user_link = ' —  <a href="admin_users.php?filter=' . Filter::escapeHtml($user->getUserName()) . '">' . Filter::escapeHtml($user->getUserName()) . '</a>';
 	};
 }
 

index_edit.php

Spacing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
 			}
 		}
 	}
-	if ($user_id < 0 || $gedcom_id < 0 ) {
+	if ($user_id < 0 || $gedcom_id < 0) {
 		header('Location: ' . WT_BASE_URL . 'admin.php');
 	} elseif ($user_id > 0) {
 		header('Location: ' . WT_BASE_URL . 'index.php?ctype=user&ged=' . $WT_TREE->getNameUrl());

ancestry.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 			<?= I18N::translate('Individual') ?>
 		</label>
 		<div class="col-sm-9">
-			<?= FunctionsEdit::formControlIndividual( $controller->root, ['id' => 'rootid', 'name' => 'rootid']) ?>
+			<?= FunctionsEdit::formControlIndividual($controller->root, ['id' => 'rootid', 'name' => 'rootid']) ?>
 		</div>
 	</div>
 

app/Functions/FunctionsPrint.php

Spacing +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
 			    <div class="modal-content">
 			      <div class="modal-header">
 			        <h3 class="modal-title" id="wt-modal-title">' . $title . '</h3>
-			        <button type="button" class="close" data-dismiss="modal" aria-label="'. I18N::translate('close') .'">
+			        <button type="button" class="close" data-dismiss="modal" aria-label="'. I18N::translate('close') . '">
 			          <span aria-hidden="true">&times;</span>
 			        </button>
 			      </div>

app/Module/LoggedInUsersModule.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 
 use Fisharebest\Webtrees\Auth;
 use Fisharebest\Webtrees\Filter;
-use Fisharebest\Webtrees\Functions\Functions;
 use Fisharebest\Webtrees\I18N;
 use Fisharebest\Webtrees\Individual;
 use Fisharebest\Webtrees\Theme;

app/Module/UserMessagesModule.php

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
 					$message->subject = I18N::translate('RE: ') . $message->subject;
 				}
 				if ($user) {
-					$content .= '<a class="btn btn-secondary" href="message.php?to=' . Filter::escapeUrl($message->sender) . '&amp;subject=' . Filter::escapeUrl($message->subject) . '&amp;ged=' . $WT_TREE->getNameUrl() .'" title="' . I18N::translate('Reply') .'">' . I18N::translate('Reply') . '</a> ';
+					$content .= '<a class="btn btn-secondary" href="message.php?to=' . Filter::escapeUrl($message->sender) . '&amp;subject=' . Filter::escapeUrl($message->subject) . '&amp;ged=' . $WT_TREE->getNameUrl() . '" title="' . I18N::translate('Reply') . '">' . I18N::translate('Reply') . '</a> ';
 				}
 				$content .= '<button type="button" onclick="if (confirm(\'' . I18N::translate('Are you sure you want to delete this message? It cannot be retrieved later.') . '\')) {$(\'#messageform :checkbox\').prop(\'checked\', false); $(\'#cb_message' . $message->message_id . '\').prop(\'checked\', true); document.messageform.submit();}">' . I18N::translate('Delete') . '</button></div></td></tr>';
 			}

message.php

Braces +3 added lines, -1 removed lines

@@ -109,7 +109,8 @@ 
 						</option>
 					</select>
 				</td>
-				<?php else: ?>
+				<?php else {
+	: ?>
 					<td class="descriptionbox wrap"></td>
 					<td class="optionbox wrap"></td>
 				<?php endif ?>
@@ -133,6 +134,7 @@ 
 						<?php
 						foreach (['10', '20', '30', '40', '50', '75', '100', '125', '150', '200'] as $selectEntry) {
 							echo '<option value="', $selectEntry, '" ';
+}
 							if ($selectEntry == $max) {
 								echo 'selected';
 							}