Inspection of "Scrutinizer Auto-Fixes (#1524)" - fisharebest/webtrees - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Branch — master (4a3c5b)

by Greg

created 2017-12-03 10:20 UTC

Status

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -13,7 +13,7 @@
 block discarded – undo
  * You should have received a copy of the GNU General Public License
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Fisharebest\Webtrees\Http\Controllers;
 

Please login to merge, or discard this patch.

app/User.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -13,7 +13,7 @@
 block discarded – undo
  * You should have received a copy of the GNU General Public License
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Fisharebest\Webtrees;
 

Please login to merge, or discard this patch.

edit_changes.php 1 patch

Switch Indentation +85 added lines, -85 removed lines patch added patch discarded remove patch

@@ -35,72 +35,72 @@  discard block
 block discarded – undo
 $url       = Filter::get('url', null, 'index.php');
 
 switch ($action) {
-case 'reject':
-	$gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
-	$xref      = Database::prepare("SELECT xref      FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
-	// Reject a change, and subsequent changes to the same record
-	Database::prepare(
-		"UPDATE `##change`" .
-		" SET   status     = 'rejected'" .
-		" WHERE status     = 'pending'" .
-		" AND   gedcom_id  = ?" .
-		" AND   xref       = ?" .
-		" AND   change_id >= ?"
-	)->execute([$gedcom_id, $xref, $change_id]);
-	break;
-case 'accept':
-	$gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
-	$xref      = Database::prepare("SELECT xref      FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
-	// Accept a change, and all previous changes to the same record
-	$all_changes = Database::prepare(
-		"SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
-		" FROM  `##change` c" .
-		" JOIN  `##gedcom` g USING (gedcom_id)" .
-		" WHERE c.status   = 'pending'" .
-		" AND   gedcom_id  = ?" .
-		" AND   xref       = ?" .
-		" AND   change_id <= ?" .
-		" ORDER BY change_id"
-	)->execute([$gedcom_id, $xref, $change_id])->fetchAll();
-	foreach ($all_changes as $change) {
-		if (empty($change->new_gedcom)) {
-			// delete
-			FunctionsImport::updateRecord($change->old_gedcom, $gedcom_id, true);
-		} else {
-			// add/update
-			FunctionsImport::updateRecord($change->new_gedcom, $gedcom_id, false);
+	case 'reject':
+		$gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
+		$xref      = Database::prepare("SELECT xref      FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
+		// Reject a change, and subsequent changes to the same record
+		Database::prepare(
+			"UPDATE `##change`" .
+			" SET   status     = 'rejected'" .
+			" WHERE status     = 'pending'" .
+			" AND   gedcom_id  = ?" .
+			" AND   xref       = ?" .
+			" AND   change_id >= ?"
+		)->execute([$gedcom_id, $xref, $change_id]);
+		break;
+	case 'accept':
+		$gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
+		$xref      = Database::prepare("SELECT xref      FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
+		// Accept a change, and all previous changes to the same record
+		$all_changes = Database::prepare(
+			"SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
+			" FROM  `##change` c" .
+			" JOIN  `##gedcom` g USING (gedcom_id)" .
+			" WHERE c.status   = 'pending'" .
+			" AND   gedcom_id  = ?" .
+			" AND   xref       = ?" .
+			" AND   change_id <= ?" .
+			" ORDER BY change_id"
+		)->execute([$gedcom_id, $xref, $change_id])->fetchAll();
+		foreach ($all_changes as $change) {
+			if (empty($change->new_gedcom)) {
+				// delete
+				FunctionsImport::updateRecord($change->old_gedcom, $gedcom_id, true);
+			} else {
+				// add/update
+				FunctionsImport::updateRecord($change->new_gedcom, $gedcom_id, false);
+			}
+			Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
+			Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
 		}
-		Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
-		Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
-	}
-	break;
-case 'rejectall':
-	Database::prepare(
-		"UPDATE `##change`" .
-		" SET status='rejected'" .
-		" WHERE status='pending' AND gedcom_id=?"
-	)->execute([$WT_TREE->getTreeId()]);
-	break;
-case 'acceptall':
-	$all_changes = Database::prepare(
-		"SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
-		" FROM `##change` c" .
-		" JOIN `##gedcom` g USING (gedcom_id)" .
-		" WHERE c.status='pending' AND gedcom_id=?" .
-		" ORDER BY change_id"
-	)->execute([$WT_TREE->getTreeId()])->fetchAll();
-	foreach ($all_changes as $change) {
-		if (empty($change->new_gedcom)) {
-			// delete
-			FunctionsImport::updateRecord($change->old_gedcom, $change->gedcom_id, true);
-		} else {
-			// add/update
-			FunctionsImport::updateRecord($change->new_gedcom, $change->gedcom_id, false);
+		break;
+	case 'rejectall':
+		Database::prepare(
+			"UPDATE `##change`" .
+			" SET status='rejected'" .
+			" WHERE status='pending' AND gedcom_id=?"
+		)->execute([$WT_TREE->getTreeId()]);
+		break;
+	case 'acceptall':
+		$all_changes = Database::prepare(
+			"SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
+			" FROM `##change` c" .
+			" JOIN `##gedcom` g USING (gedcom_id)" .
+			" WHERE c.status='pending' AND gedcom_id=?" .
+			" ORDER BY change_id"
+		)->execute([$WT_TREE->getTreeId()])->fetchAll();
+		foreach ($all_changes as $change) {
+			if (empty($change->new_gedcom)) {
+				// delete
+				FunctionsImport::updateRecord($change->old_gedcom, $change->gedcom_id, true);
+			} else {
+				// add/update
+				FunctionsImport::updateRecord($change->new_gedcom, $change->gedcom_id, false);
+			}
+			Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
+			Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
 		}
-		Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
-		Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
-	}
-	break;
+		break;
 }
 
 $rows = Database::prepare(
@@ -120,27 +120,27 @@  discard block
 block discarded – undo
 	preg_match('/^0 (?:@' . WT_REGEX_XREF . '@ )?(' . WT_REGEX_TAG . ')/', $row->old_gedcom . $row->new_gedcom, $match);
 
 	switch ($match[1]) {
-	case 'INDI':
-		$row->record = new Individual($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
-	case 'FAM':
-		$row->record = new Family($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
-	case 'SOUR':
-		$row->record = new Source($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
-	case 'REPO':
-		$row->record = new Repository($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
-	case 'OBJE':
-		$row->record = new Media($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
-	case 'NOTE':
-		$row->record = new Note($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
-	default:
-		$row->record = new GedcomRecord($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
-		break;
+		case 'INDI':
+			$row->record = new Individual($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
+		case 'FAM':
+			$row->record = new Family($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
+		case 'SOUR':
+			$row->record = new Source($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
+		case 'REPO':
+			$row->record = new Repository($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
+		case 'OBJE':
+			$row->record = new Media($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
+		case 'NOTE':
+			$row->record = new Note($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
+		default:
+			$row->record = new GedcomRecord($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
+			break;
 	}
 
 	$row->accept_url = Html::url('edit_changes.php', [

Please login to merge, or discard this patch.

expand_view.php 1 patch

Switch Indentation +33 added lines, -33 removed lines patch added patch discarded remove patch

@@ -38,38 +38,38 @@
 block discarded – undo
 
 foreach ($facts as $fact) {
 	switch ($fact->getTag()) {
-	case 'ADDR':
-	case 'ALIA':
-	case 'ASSO':
-	case 'CHAN':
-	case 'CHIL':
-	case 'EMAIL':
-	case 'FAMC':
-	case 'FAMS':
-	case 'HUSB':
-	case 'NAME':
-	case 'NOTE':
-	case 'OBJE':
-	case 'PHON':
-	case 'RESI':
-	case 'RESN':
-	case 'SEX':
-	case 'SOUR':
-	case 'SSN':
-	case 'SUBM':
-	case 'TITL':
-	case 'URL':
-	case 'WIFE':
-	case 'WWW':
-	case '_EMAIL':
-	case '_TODO':
-	case '_UID':
-	case '_WT_OBJE_SORT':
-		// Do not show these
-		break;
-	default:
-		// Simple version of FunctionsPrintFacts::print_fact()
-		echo $fact->summary();
-		break;
+		case 'ADDR':
+		case 'ALIA':
+		case 'ASSO':
+		case 'CHAN':
+		case 'CHIL':
+		case 'EMAIL':
+		case 'FAMC':
+		case 'FAMS':
+		case 'HUSB':
+		case 'NAME':
+		case 'NOTE':
+		case 'OBJE':
+		case 'PHON':
+		case 'RESI':
+		case 'RESN':
+		case 'SEX':
+		case 'SOUR':
+		case 'SSN':
+		case 'SUBM':
+		case 'TITL':
+		case 'URL':
+		case 'WIFE':
+		case 'WWW':
+		case '_EMAIL':
+		case '_TODO':
+		case '_UID':
+		case '_WT_OBJE_SORT':
+			// Do not show these
+			break;
+		default:
+			// Simple version of FunctionsPrintFacts::print_fact()
+			echo $fact->summary();
+			break;
 	}
 }

Please login to merge, or discard this patch.

famlist.php 1 patch

Switch Indentation +26 added lines, -26 removed lines patch added patch discarded remove patch

@@ -37,12 +37,12 @@  discard block
 block discarded – undo
 
 $show_marnm = Filter::get('show_marnm', 'no|yes');
 switch ($show_marnm) {
-case 'no':
-case 'yes':
-	Auth::user()->setPreference('famlist.php_show_marnm', $show_marnm);
-	break;
-default:
-	$show_marnm = Auth::user()->getPreference('famlist.php_show_marnm');
+	case 'no':
+	case 'yes':
+		Auth::user()->setPreference('famlist.php_show_marnm', $show_marnm);
+		break;
+	default:
+		$show_marnm = Auth::user()->getPreference('famlist.php_show_marnm');
 }
 
 // Make sure selections are consistent.
@@ -79,16 +79,16 @@  discard block
 block discarded – undo
 	}
 	$url = '?surname=' . rawurlencode($surname) . '&amp;ged=' . $controller->tree()->getNameUrl();
 	switch ($falpha) {
-	case '':
-		break;
-	case '@':
-		$legend .= ', ' . I18N::translateContext('Unknown given name', '…');
-		$url .= '&amp;falpha=' . rawurlencode($falpha) . '&amp;ged=' . $controller->tree()->getNameUrl();
-		break;
-	default:
-		$legend .= ', ' . Html::escape($falpha) . '…';
-		$url .= '&amp;falpha=' . rawurlencode($falpha) . '&amp;ged=' . $controller->tree()->getNameUrl();
-		break;
+		case '':
+			break;
+		case '@':
+			$legend .= ', ' . I18N::translateContext('Unknown given name', '…');
+			$url .= '&amp;falpha=' . rawurlencode($falpha) . '&amp;ged=' . $controller->tree()->getNameUrl();
+			break;
+		default:
+			$legend .= ', ' . Html::escape($falpha) . '…';
+			$url .= '&amp;falpha=' . rawurlencode($falpha) . '&amp;ged=' . $controller->tree()->getNameUrl();
+			break;
 	}
 	$show = 'indi'; // SURN list makes no sense here
 } elseif ($alpha === '@') {
@@ -173,16 +173,16 @@  discard block
 block discarded – undo
 	if ($show === 'surn') {
 		// Show the surname list
 		switch ($controller->tree()->getPreference('SURNAME_LIST_STYLE')) {
-		case 'style1':
-			echo FunctionsPrintLists::surnameList($surns, 3, true, 'famlist.php', $controller->tree());
-			break;
-		case 'style3':
-			echo FunctionsPrintLists::surnameTagCloud($surns, 'famlist.php', true, $controller->tree());
-			break;
-		case 'style2':
-		default:
-			echo FunctionsPrintLists::surnameTable($surns, 'famlist.php', $controller->tree());
-			break;
+			case 'style1':
+				echo FunctionsPrintLists::surnameList($surns, 3, true, 'famlist.php', $controller->tree());
+				break;
+			case 'style3':
+				echo FunctionsPrintLists::surnameTagCloud($surns, 'famlist.php', true, $controller->tree());
+				break;
+			case 'style2':
+			default:
+				echo FunctionsPrintLists::surnameTable($surns, 'famlist.php', $controller->tree());
+				break;
 		}
 	} else {
 		// Show the list

Please login to merge, or discard this patch.

setup.php 1 patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -53,15 +53,15 @@
 block discarded – undo
 $route   = $request->get('route');
 
 switch ($method . ':' . $route) {
-default:
-	$url      = Html::url('setup.php', ['route' => 'setup']);
-	$response = new RedirectResponse($url);
-	break;
+	default:
+		$url      = Html::url('setup.php', ['route' => 'setup']);
+		$response = new RedirectResponse($url);
+		break;
 
-case 'GET:setup':
-case 'POST:setup':
-	$response = (new SetupController)->setup($request);
-	break;
+	case 'GET:setup':
+	case 'POST:setup':
+		$response = (new SetupController)->setup($request);
+		break;
 }
 
 $response->prepare($request)->send();

Please login to merge, or discard this patch.

descendancy.php 1 patch

Switch Indentation +19 added lines, -19 removed lines patch added patch discarded remove patch

@@ -27,25 +27,25 @@
 block discarded – undo
 // Only generate the content for interactive users (not search robots).
 if (Filter::getBool('ajax') && Session::has('initiated')) {
 	switch ($controller->chart_style) {
-	case 0: // List
-		echo '<ul id="descendancy_chart" class="chart_common">';
-		$controller->printChildDescendancy($controller->root, $controller->generations);
-		echo '</ul>';
-		break;
-	case 1: // Booklet
-		$show_cousins = true;
-		echo '<div id="descendancy_booklet">';
-		$controller->printChildFamily($controller->root, $controller->generations);
-		echo '</div>';
-		break;
-	case 2: // Individual list
-		$descendants = $controller->individualDescendancy($controller->root, $controller->generations, []);
-		echo '<div id="descendancy-list">', FunctionsPrintLists::individualTable($descendants), '</div>';
-		break;
-	case 3: // Family list
-		$descendants = $controller->familyDescendancy($controller->root, $controller->generations, []);
-		echo '<div id="descendancy-list">', FunctionsPrintLists::familyTable($descendants), '</div>';
-		break;
+		case 0: // List
+			echo '<ul id="descendancy_chart" class="chart_common">';
+			$controller->printChildDescendancy($controller->root, $controller->generations);
+			echo '</ul>';
+			break;
+		case 1: // Booklet
+			$show_cousins = true;
+			echo '<div id="descendancy_booklet">';
+			$controller->printChildFamily($controller->root, $controller->generations);
+			echo '</div>';
+			break;
+		case 2: // Individual list
+			$descendants = $controller->individualDescendancy($controller->root, $controller->generations, []);
+			echo '<div id="descendancy-list">', FunctionsPrintLists::individualTable($descendants), '</div>';
+			break;
+		case 3: // Family list
+			$descendants = $controller->familyDescendancy($controller->root, $controller->generations, []);
+			echo '<div id="descendancy-list">', FunctionsPrintLists::familyTable($descendants), '</div>';
+			break;
 	}
 	echo $controller->getJavascript();
 

Please login to merge, or discard this patch.

admin_trees_renumber.php 1 patch

Switch Indentation +140 added lines, -140 removed lines patch added patch discarded remove patch

@@ -82,146 +82,146 @@
 block discarded – undo
 		);
 		$new_xref = $WT_TREE->getNewXref($type);
 		switch ($type) {
-		case 'INDI':
-			Database::prepare(
-				"UPDATE `##individuals` SET i_id = ?, i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_id = ? AND i_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ INDI\n", "0 @$new_xref@ INDI\n", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'HUSB') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " HUSB @$old_xref@", " HUSB @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'WIFE') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " WIFE @$old_xref@", " WIFE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'CHIL') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " CHIL @$old_xref@", " CHIL @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'ASSO') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " ASSO @$old_xref@", " ASSO @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = '_ASSO') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " _ASSO @$old_xref@", " _ASSO @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'ASSO') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " ASSO @$old_xref@", " ASSO @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = '_ASSO') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " _ASSO @$old_xref@", " _ASSO @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##placelinks` SET pl_gid = ? WHERE pl_gid = ? AND pl_file = ?"
-			)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##dates` SET d_gid = ? WHERE d_gid = ? AND d_file = ?"
-			)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##user_gedcom_setting` SET setting_value = ? WHERE setting_value = ? AND gedcom_id = ? AND setting_name IN ('gedcomid', 'rootid')"
-			)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
-			break;
-		case 'FAM':
-			Database::prepare(
-				"UPDATE `##families` SET f_id = ?, f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_id = ? AND f_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ FAM\n", "0 @$new_xref@ FAM\n", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'FAMC') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " FAMC @$old_xref@", " FAMC @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'FAMS') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " FAMS @$old_xref@", " FAMS @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##placelinks` SET pl_gid = ? WHERE pl_gid = ? AND pl_file = ?"
-			)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##dates` SET d_gid = ? WHERE d_gid = ? AND d_file = ?"
-			)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
-			break;
-		case 'SOUR':
-			Database::prepare(
-				"UPDATE `##sources` SET s_id = ?, s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_id = ? AND s_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ SOUR\n", "0 @$new_xref@ SOUR\n", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'SOUR') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'SOUR') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ? AND l_type = 'SOUR') SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
-			)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ? AND l_type = 'SOUR') SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
-			)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
-			break;
-		case 'REPO':
-			Database::prepare(
-				"UPDATE `##other` SET o_id = ?, o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_id = ? AND o_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ REPO\n", "0 @$new_xref@ REPO\n", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ? AND l_type = 'REPO') SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
-			)->execute([$old_xref, " REPO @$old_xref@", " REPO @$new_xref@", $WT_TREE->getTreeId()]);
-			break;
-		case 'NOTE':
-			Database::prepare(
-				"UPDATE `##other` SET o_id = ?, o_gedcom = REPLACE(REPLACE(o_gedcom, ?, ?), ?, ?) WHERE o_id = ? AND o_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ NOTE\n", "0 @$new_xref@ NOTE\n", "0 @$old_xref@ NOTE ", "0 @$new_xref@ NOTE ", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'NOTE') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'NOTE') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ? AND l_type = 'NOTE') SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
-			)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ? AND l_type = 'NOTE') SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
-			)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ? AND l_type = 'NOTE') SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
-			)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
-			break;
-		case 'OBJE':
-			Database::prepare(
-				"UPDATE `##media` SET m_id = ?, m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_id = ? AND m_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ OBJE\n", "0 @$new_xref@ OBJE\n", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##media_file` SET m_id = ? WHERE m_id = ? AND m_file = ?"
-			)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'OBJE') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'OBJE') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ? AND l_type = 'OBJE') SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
-			)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ? AND l_type = 'OBJE') SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
-			)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ? AND l_type = 'OBJE') SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
-			)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
-			break;
-		default:
-			Database::prepare(
-				"UPDATE `##other` SET o_id = ?, o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_id = ? AND o_file = ?"
-			)->execute([$new_xref, "0 @$old_xref@ $type\n", "0 @$new_xref@ $type\n", $old_xref, $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ?) SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
-			)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ?) SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
-			)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ?) SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
-			)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ?) SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
-			)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
-			Database::prepare(
-				"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ?) SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
-			)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
-			break;
+			case 'INDI':
+				Database::prepare(
+					"UPDATE `##individuals` SET i_id = ?, i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_id = ? AND i_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ INDI\n", "0 @$new_xref@ INDI\n", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'HUSB') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " HUSB @$old_xref@", " HUSB @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'WIFE') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " WIFE @$old_xref@", " WIFE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'CHIL') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " CHIL @$old_xref@", " CHIL @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'ASSO') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " ASSO @$old_xref@", " ASSO @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = '_ASSO') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " _ASSO @$old_xref@", " _ASSO @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'ASSO') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " ASSO @$old_xref@", " ASSO @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = '_ASSO') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " _ASSO @$old_xref@", " _ASSO @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##placelinks` SET pl_gid = ? WHERE pl_gid = ? AND pl_file = ?"
+				)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##dates` SET d_gid = ? WHERE d_gid = ? AND d_file = ?"
+				)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##user_gedcom_setting` SET setting_value = ? WHERE setting_value = ? AND gedcom_id = ? AND setting_name IN ('gedcomid', 'rootid')"
+				)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
+				break;
+			case 'FAM':
+				Database::prepare(
+					"UPDATE `##families` SET f_id = ?, f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_id = ? AND f_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ FAM\n", "0 @$new_xref@ FAM\n", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'FAMC') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " FAMC @$old_xref@", " FAMC @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'FAMS') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " FAMS @$old_xref@", " FAMS @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##placelinks` SET pl_gid = ? WHERE pl_gid = ? AND pl_file = ?"
+				)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##dates` SET d_gid = ? WHERE d_gid = ? AND d_file = ?"
+				)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
+				break;
+			case 'SOUR':
+				Database::prepare(
+					"UPDATE `##sources` SET s_id = ?, s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_id = ? AND s_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ SOUR\n", "0 @$new_xref@ SOUR\n", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'SOUR') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'SOUR') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ? AND l_type = 'SOUR') SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
+				)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ? AND l_type = 'SOUR') SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
+				)->execute([$old_xref, " SOUR @$old_xref@", " SOUR @$new_xref@", $WT_TREE->getTreeId()]);
+				break;
+			case 'REPO':
+				Database::prepare(
+					"UPDATE `##other` SET o_id = ?, o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_id = ? AND o_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ REPO\n", "0 @$new_xref@ REPO\n", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ? AND l_type = 'REPO') SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
+				)->execute([$old_xref, " REPO @$old_xref@", " REPO @$new_xref@", $WT_TREE->getTreeId()]);
+				break;
+			case 'NOTE':
+				Database::prepare(
+					"UPDATE `##other` SET o_id = ?, o_gedcom = REPLACE(REPLACE(o_gedcom, ?, ?), ?, ?) WHERE o_id = ? AND o_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ NOTE\n", "0 @$new_xref@ NOTE\n", "0 @$old_xref@ NOTE ", "0 @$new_xref@ NOTE ", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'NOTE') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'NOTE') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ? AND l_type = 'NOTE') SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
+				)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ? AND l_type = 'NOTE') SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
+				)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ? AND l_type = 'NOTE') SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
+				)->execute([$old_xref, " NOTE @$old_xref@", " NOTE @$new_xref@", $WT_TREE->getTreeId()]);
+				break;
+			case 'OBJE':
+				Database::prepare(
+					"UPDATE `##media` SET m_id = ?, m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_id = ? AND m_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ OBJE\n", "0 @$new_xref@ OBJE\n", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##media_file` SET m_id = ? WHERE m_id = ? AND m_file = ?"
+				)->execute([$new_xref, $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ? AND l_type = 'OBJE') SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ? AND l_type = 'OBJE') SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ? AND l_type = 'OBJE') SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
+				)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ? AND l_type = 'OBJE') SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
+				)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ? AND l_type = 'OBJE') SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
+				)->execute([$old_xref, " OBJE @$old_xref@", " OBJE @$new_xref@", $WT_TREE->getTreeId()]);
+				break;
+			default:
+				Database::prepare(
+					"UPDATE `##other` SET o_id = ?, o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_id = ? AND o_file = ?"
+				)->execute([$new_xref, "0 @$old_xref@ $type\n", "0 @$new_xref@ $type\n", $old_xref, $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##individuals` JOIN `##link` ON (l_file = i_file AND l_to = ?) SET i_gedcom = REPLACE(i_gedcom, ?, ?) WHERE i_file = ?"
+				)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##families` JOIN `##link` ON (l_file = f_file AND l_to = ?) SET f_gedcom = REPLACE(f_gedcom, ?, ?) WHERE f_file = ?"
+				)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##media` JOIN `##link` ON (l_file = m_file AND l_to = ?) SET m_gedcom = REPLACE(m_gedcom, ?, ?) WHERE m_file = ?"
+				)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##sources` JOIN `##link` ON (l_file = s_file AND l_to = ?) SET s_gedcom = REPLACE(s_gedcom, ?, ?) WHERE s_file = ?"
+				)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
+				Database::prepare(
+					"UPDATE `##other` JOIN `##link` ON (l_file = o_file AND l_to = ?) SET o_gedcom = REPLACE(o_gedcom, ?, ?) WHERE o_file = ?"
+				)->execute([$old_xref, " @$old_xref@", " @$new_xref@", $WT_TREE->getTreeId()]);
+				break;
 		}
 		Database::prepare(
 			"UPDATE `##name` SET n_id = ? WHERE n_id = ? AND n_file = ?"

Please login to merge, or discard this patch.

edituser.php 1 patch

Switch Indentation +38 added lines, -38 removed lines patch added patch discarded remove patch

@@ -48,46 +48,46 @@
 block discarded – undo
 // Respond to form action
 if ($action !== '' && Filter::checkCsrf()) {
 	switch ($action) {
-	case 'update':
-		if ($username !== Auth::user()->getUserName() && User::findByUserName($username)) {
-			FlashMessages::addMessage(I18N::translate('Duplicate username. A user with that username already exists. Please choose another username.'));
-		} elseif ($email !== Auth::user()->getEmail() && User::findByEmail($email)) {
-			FlashMessages::addMessage(I18N::translate('Duplicate email address. A user with that email already exists.'));
-		} else {
-			// Change username
-			if ($username !== Auth::user()->getUserName()) {
-				Log::addAuthenticationLog('User ' . Auth::user()->getUserName() . ' renamed to ' . $username);
-				Auth::user()->setUserName($username);
+		case 'update':
+			if ($username !== Auth::user()->getUserName() && User::findByUserName($username)) {
+				FlashMessages::addMessage(I18N::translate('Duplicate username. A user with that username already exists. Please choose another username.'));
+			} elseif ($email !== Auth::user()->getEmail() && User::findByEmail($email)) {
+				FlashMessages::addMessage(I18N::translate('Duplicate email address. A user with that email already exists.'));
+			} else {
+				// Change username
+				if ($username !== Auth::user()->getUserName()) {
+					Log::addAuthenticationLog('User ' . Auth::user()->getUserName() . ' renamed to ' . $username);
+					Auth::user()->setUserName($username);
+				}
+
+				// Change password
+				if ($password_1 !== '' && $password_1 === $password_2) {
+					Auth::user()->setPassword($password_1);
+				}
+
+				// Change other settings
+				Auth::user()
+					->setRealName($real_name)
+					->setEmail($email)
+					->setPreference('language', $language)
+					->setPreference('TIMEZONE', $time_zone)
+					->setPreference('contactmethod', $contact_method)
+					->setPreference('visibleonline', $visible_online ? '1' : '0');
+
+				Auth::user()->setPreference('theme', $theme);
+
+				$WT_TREE->setUserPreference(Auth::user(), 'rootid', $root_id);
 			}
-
-			// Change password
-			if ($password_1 !== '' && $password_1 === $password_2) {
-				Auth::user()->setPassword($password_1);
+			break;
+
+		case 'delete':
+			// An administrator can only be deleted by another administrator
+			if (!Auth::user()->getPreference('canadmin')) {
+				$currentUser = Auth::user();
+				Auth::logout();
+				$currentUser->delete();
 			}
-
-			// Change other settings
-			Auth::user()
-				->setRealName($real_name)
-				->setEmail($email)
-				->setPreference('language', $language)
-				->setPreference('TIMEZONE', $time_zone)
-				->setPreference('contactmethod', $contact_method)
-				->setPreference('visibleonline', $visible_online ? '1' : '0');
-
-			Auth::user()->setPreference('theme', $theme);
-
-			$WT_TREE->setUserPreference(Auth::user(), 'rootid', $root_id);
-		}
-		break;
-
-	case 'delete':
-		// An administrator can only be deleted by another administrator
-		if (!Auth::user()->getPreference('canadmin')) {
-			$currentUser = Auth::user();
-			Auth::logout();
-			$currentUser->delete();
-		}
-		break;
+			break;
 	}
 
 	header('Location: edituser.php');

Please login to merge, or discard this patch.

		@@ -53,15 +53,15 @@
		block discarded – undo
53	53	$route = $request->get('route');
54	54
55	55	switch ($method . ':' . $route) {
56		-default:
57		- $url = Html::url('setup.php', ['route' => 'setup']);
58		- $response = new RedirectResponse($url);
59		- break;
	56	+ default:
	57	+ $url = Html::url('setup.php', ['route' => 'setup']);
	58	+ $response = new RedirectResponse($url);
	59	+ break;
60	60
61		-case 'GET:setup':
62		-case 'POST:setup':
63		- $response = (new SetupController)->setup($request);
64		- break;
	61	+ case 'GET:setup':
	62	+ case 'POST:setup':
	63	+ $response = (new SetupController)->setup($request);
	64	+ break;
65	65	}
66	66
67	67	$response->prepare($request)->send();

		@@ -13,7 +13,7 @@
		block discarded – undo
13	13	* You should have received a copy of the GNU General Public License
14	14	* along with this program. If not, see <http://www.gnu.org/licenses/>.
15	15	*/
16		-declare(strict_types=1);
	16	+declare(strict_types = 1);
17	17
18	18	namespace Fisharebest\Webtrees\Http\Controllers;
19	19

		@@ -35,72 +35,72 @@ discard block
		block discarded – undo
35	35	$url = Filter::get('url', null, 'index.php');
36	36
37	37	switch ($action) {
38		-case 'reject':
39		- $gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
40		- $xref = Database::prepare("SELECT xref FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
41		- // Reject a change, and subsequent changes to the same record
42		- Database::prepare(
43		- "UPDATE `##change`" .
44		- " SET status = 'rejected'" .
45		- " WHERE status = 'pending'" .
46		- " AND gedcom_id = ?" .
47		- " AND xref = ?" .
48		- " AND change_id >= ?"
49		- )->execute([$gedcom_id, $xref, $change_id]);
50		- break;
51		-case 'accept':
52		- $gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
53		- $xref = Database::prepare("SELECT xref FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
54		- // Accept a change, and all previous changes to the same record
55		- $all_changes = Database::prepare(
56		- "SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
57		- " FROM `##change` c" .
58		- " JOIN `##gedcom` g USING (gedcom_id)" .
59		- " WHERE c.status = 'pending'" .
60		- " AND gedcom_id = ?" .
61		- " AND xref = ?" .
62		- " AND change_id <= ?" .
63		- " ORDER BY change_id"
64		- )->execute([$gedcom_id, $xref, $change_id])->fetchAll();
65		- foreach ($all_changes as $change) {
66		- if (empty($change->new_gedcom)) {
67		- // delete
68		- FunctionsImport::updateRecord($change->old_gedcom, $gedcom_id, true);
69		- } else {
70		- // add/update
71		- FunctionsImport::updateRecord($change->new_gedcom, $gedcom_id, false);
	38	+ case 'reject':
	39	+ $gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
	40	+ $xref = Database::prepare("SELECT xref FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
	41	+ // Reject a change, and subsequent changes to the same record
	42	+ Database::prepare(
	43	+ "UPDATE `##change`" .
	44	+ " SET status = 'rejected'" .
	45	+ " WHERE status = 'pending'" .
	46	+ " AND gedcom_id = ?" .
	47	+ " AND xref = ?" .
	48	+ " AND change_id >= ?"
	49	+ )->execute([$gedcom_id, $xref, $change_id]);
	50	+ break;
	51	+ case 'accept':
	52	+ $gedcom_id = Database::prepare("SELECT gedcom_id FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
	53	+ $xref = Database::prepare("SELECT xref FROM `##change` WHERE change_id=?")->execute([$change_id])->fetchOne();
	54	+ // Accept a change, and all previous changes to the same record
	55	+ $all_changes = Database::prepare(
	56	+ "SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
	57	+ " FROM `##change` c" .
	58	+ " JOIN `##gedcom` g USING (gedcom_id)" .
	59	+ " WHERE c.status = 'pending'" .
	60	+ " AND gedcom_id = ?" .
	61	+ " AND xref = ?" .
	62	+ " AND change_id <= ?" .
	63	+ " ORDER BY change_id"
	64	+ )->execute([$gedcom_id, $xref, $change_id])->fetchAll();
	65	+ foreach ($all_changes as $change) {
	66	+ if (empty($change->new_gedcom)) {
	67	+ // delete
	68	+ FunctionsImport::updateRecord($change->old_gedcom, $gedcom_id, true);
	69	+ } else {
	70	+ // add/update
	71	+ FunctionsImport::updateRecord($change->new_gedcom, $gedcom_id, false);
	72	+ }
	73	+ Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
	74	+ Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
72	75	}
73		- Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
74		- Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
75		- }
76		- break;
77		-case 'rejectall':
78		- Database::prepare(
79		- "UPDATE `##change`" .
80		- " SET status='rejected'" .
81		- " WHERE status='pending' AND gedcom_id=?"
82		- )->execute([$WT_TREE->getTreeId()]);
83		- break;
84		-case 'acceptall':
85		- $all_changes = Database::prepare(
86		- "SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
87		- " FROM `##change` c" .
88		- " JOIN `##gedcom` g USING (gedcom_id)" .
89		- " WHERE c.status='pending' AND gedcom_id=?" .
90		- " ORDER BY change_id"
91		- )->execute([$WT_TREE->getTreeId()])->fetchAll();
92		- foreach ($all_changes as $change) {
93		- if (empty($change->new_gedcom)) {
94		- // delete
95		- FunctionsImport::updateRecord($change->old_gedcom, $change->gedcom_id, true);
96		- } else {
97		- // add/update
98		- FunctionsImport::updateRecord($change->new_gedcom, $change->gedcom_id, false);
	76	+ break;
	77	+ case 'rejectall':
	78	+ Database::prepare(
	79	+ "UPDATE `##change`" .
	80	+ " SET status='rejected'" .
	81	+ " WHERE status='pending' AND gedcom_id=?"
	82	+ )->execute([$WT_TREE->getTreeId()]);
	83	+ break;
	84	+ case 'acceptall':
	85	+ $all_changes = Database::prepare(
	86	+ "SELECT change_id, gedcom_id, gedcom_name, xref, old_gedcom, new_gedcom" .
	87	+ " FROM `##change` c" .
	88	+ " JOIN `##gedcom` g USING (gedcom_id)" .
	89	+ " WHERE c.status='pending' AND gedcom_id=?" .
	90	+ " ORDER BY change_id"
	91	+ )->execute([$WT_TREE->getTreeId()])->fetchAll();
	92	+ foreach ($all_changes as $change) {
	93	+ if (empty($change->new_gedcom)) {
	94	+ // delete
	95	+ FunctionsImport::updateRecord($change->old_gedcom, $change->gedcom_id, true);
	96	+ } else {
	97	+ // add/update
	98	+ FunctionsImport::updateRecord($change->new_gedcom, $change->gedcom_id, false);
	99	+ }
	100	+ Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
	101	+ Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
99	102	}
100		- Database::prepare("UPDATE `##change` SET status='accepted' WHERE change_id=?")->execute([$change->change_id]);
101		- Log::addEditLog("Accepted change {$change->change_id} for {$change->xref} / {$change->gedcom_name} into database");
102		- }
103		- break;
	103	+ break;
104	104	}
105	105
106	106	$rows = Database::prepare(
		@@ -120,27 +120,27 @@ discard block
		block discarded – undo
120	120	preg_match('/^0 (?:@' . WT_REGEX_XREF . '@ )?(' . WT_REGEX_TAG . ')/', $row->old_gedcom . $row->new_gedcom, $match);
121	121
122	122	switch ($match[1]) {
123		- case 'INDI':
124		- $row->record = new Individual($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
125		- break;
126		- case 'FAM':
127		- $row->record = new Family($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
128		- break;
129		- case 'SOUR':
130		- $row->record = new Source($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
131		- break;
132		- case 'REPO':
133		- $row->record = new Repository($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
134		- break;
135		- case 'OBJE':
136		- $row->record = new Media($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
137		- break;
138		- case 'NOTE':
139		- $row->record = new Note($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
140		- break;
141		- default:
142		- $row->record = new GedcomRecord($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
143		- break;
	123	+ case 'INDI':
	124	+ $row->record = new Individual($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	125	+ break;
	126	+ case 'FAM':
	127	+ $row->record = new Family($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	128	+ break;
	129	+ case 'SOUR':
	130	+ $row->record = new Source($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	131	+ break;
	132	+ case 'REPO':
	133	+ $row->record = new Repository($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	134	+ break;
	135	+ case 'OBJE':
	136	+ $row->record = new Media($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	137	+ break;
	138	+ case 'NOTE':
	139	+ $row->record = new Note($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	140	+ break;
	141	+ default:
	142	+ $row->record = new GedcomRecord($row->xref, $row->old_gedcom, $row->new_gedcom, $tree);
	143	+ break;
144	144	}
145	145
146	146	$row->accept_url = Html::url('edit_changes.php', [

		@@ -38,38 +38,38 @@
		block discarded – undo
38	38
39	39	foreach ($facts as $fact) {
40	40	switch ($fact->getTag()) {
41		- case 'ADDR':
42		- case 'ALIA':
43		- case 'ASSO':
44		- case 'CHAN':
45		- case 'CHIL':
46		- case 'EMAIL':
47		- case 'FAMC':
48		- case 'FAMS':
49		- case 'HUSB':
50		- case 'NAME':
51		- case 'NOTE':
52		- case 'OBJE':
53		- case 'PHON':
54		- case 'RESI':
55		- case 'RESN':
56		- case 'SEX':
57		- case 'SOUR':
58		- case 'SSN':
59		- case 'SUBM':
60		- case 'TITL':
61		- case 'URL':
62		- case 'WIFE':
63		- case 'WWW':
64		- case '_EMAIL':
65		- case '_TODO':
66		- case '_UID':
67		- case '_WT_OBJE_SORT':
68		- // Do not show these
69		- break;
70		- default:
71		- // Simple version of FunctionsPrintFacts::print_fact()
72		- echo $fact->summary();
73		- break;
	41	+ case 'ADDR':
	42	+ case 'ALIA':
	43	+ case 'ASSO':
	44	+ case 'CHAN':
	45	+ case 'CHIL':
	46	+ case 'EMAIL':
	47	+ case 'FAMC':
	48	+ case 'FAMS':
	49	+ case 'HUSB':
	50	+ case 'NAME':
	51	+ case 'NOTE':
	52	+ case 'OBJE':
	53	+ case 'PHON':
	54	+ case 'RESI':
	55	+ case 'RESN':
	56	+ case 'SEX':
	57	+ case 'SOUR':
	58	+ case 'SSN':
	59	+ case 'SUBM':
	60	+ case 'TITL':
	61	+ case 'URL':
	62	+ case 'WIFE':
	63	+ case 'WWW':
	64	+ case '_EMAIL':
	65	+ case '_TODO':
	66	+ case '_UID':
	67	+ case '_WT_OBJE_SORT':
	68	+ // Do not show these
	69	+ break;
	70	+ default:
	71	+ // Simple version of FunctionsPrintFacts::print_fact()
	72	+ echo $fact->summary();
	73	+ break;
74	74	}
75	75	}

		@@ -37,12 +37,12 @@ discard block
		block discarded – undo
37	37
38	38	$show_marnm = Filter::get('show_marnm', 'no\|yes');
39	39	switch ($show_marnm) {
40		-case 'no':
41		-case 'yes':
42		- Auth::user()->setPreference('famlist.php_show_marnm', $show_marnm);
43		- break;
44		-default:
45		- $show_marnm = Auth::user()->getPreference('famlist.php_show_marnm');
	40	+ case 'no':
	41	+ case 'yes':
	42	+ Auth::user()->setPreference('famlist.php_show_marnm', $show_marnm);
	43	+ break;
	44	+ default:
	45	+ $show_marnm = Auth::user()->getPreference('famlist.php_show_marnm');
46	46	}
47	47
48	48	// Make sure selections are consistent.
		@@ -79,16 +79,16 @@ discard block
		block discarded – undo
79	79	}
80	80	$url = '?surname=' . rawurlencode($surname) . '&ged=' . $controller->tree()->getNameUrl();
81	81	switch ($falpha) {
82		- case '':
83		- break;
84		- case '@':
85		- $legend .= ', ' . I18N::translateContext('Unknown given name', '…');
86		- $url .= '&falpha=' . rawurlencode($falpha) . '&ged=' . $controller->tree()->getNameUrl();
87		- break;
88		- default:
89		- $legend .= ', ' . Html::escape($falpha) . '…';
90		- $url .= '&falpha=' . rawurlencode($falpha) . '&ged=' . $controller->tree()->getNameUrl();
91		- break;
	82	+ case '':
	83	+ break;
	84	+ case '@':
	85	+ $legend .= ', ' . I18N::translateContext('Unknown given name', '…');
	86	+ $url .= '&falpha=' . rawurlencode($falpha) . '&ged=' . $controller->tree()->getNameUrl();
	87	+ break;
	88	+ default:
	89	+ $legend .= ', ' . Html::escape($falpha) . '…';
	90	+ $url .= '&falpha=' . rawurlencode($falpha) . '&ged=' . $controller->tree()->getNameUrl();
	91	+ break;
92	92	}
93	93	$show = 'indi'; // SURN list makes no sense here
94	94	} elseif ($alpha === '@') {
		@@ -173,16 +173,16 @@ discard block
		block discarded – undo
173	173	if ($show === 'surn') {
174	174	// Show the surname list
175	175	switch ($controller->tree()->getPreference('SURNAME_LIST_STYLE')) {
176		- case 'style1':
177		- echo FunctionsPrintLists::surnameList($surns, 3, true, 'famlist.php', $controller->tree());
178		- break;
179		- case 'style3':
180		- echo FunctionsPrintLists::surnameTagCloud($surns, 'famlist.php', true, $controller->tree());
181		- break;
182		- case 'style2':
183		- default:
184		- echo FunctionsPrintLists::surnameTable($surns, 'famlist.php', $controller->tree());
185		- break;
	176	+ case 'style1':
	177	+ echo FunctionsPrintLists::surnameList($surns, 3, true, 'famlist.php', $controller->tree());
	178	+ break;
	179	+ case 'style3':
	180	+ echo FunctionsPrintLists::surnameTagCloud($surns, 'famlist.php', true, $controller->tree());
	181	+ break;
	182	+ case 'style2':
	183	+ default:
	184	+ echo FunctionsPrintLists::surnameTable($surns, 'famlist.php', $controller->tree());
	185	+ break;
186	186	}
187	187	} else {
188	188	// Show the list

		@@ -27,25 +27,25 @@
		block discarded – undo
27	27	// Only generate the content for interactive users (not search robots).
28	28	if (Filter::getBool('ajax') && Session::has('initiated')) {
29	29	switch ($controller->chart_style) {
30		- case 0: // List
31		- echo '<ul id="descendancy_chart" class="chart_common">';
32		- $controller->printChildDescendancy($controller->root, $controller->generations);
33		- echo '</ul>';
34		- break;
35		- case 1: // Booklet
36		- $show_cousins = true;
37		- echo '<div id="descendancy_booklet">';
38		- $controller->printChildFamily($controller->root, $controller->generations);
39		- echo '</div>';
40		- break;
41		- case 2: // Individual list
42		- $descendants = $controller->individualDescendancy($controller->root, $controller->generations, []);
43		- echo '<div id="descendancy-list">', FunctionsPrintLists::individualTable($descendants), '</div>';
44		- break;
45		- case 3: // Family list
46		- $descendants = $controller->familyDescendancy($controller->root, $controller->generations, []);
47		- echo '<div id="descendancy-list">', FunctionsPrintLists::familyTable($descendants), '</div>';
48		- break;
	30	+ case 0: // List
	31	+ echo '<ul id="descendancy_chart" class="chart_common">';
	32	+ $controller->printChildDescendancy($controller->root, $controller->generations);
	33	+ echo '</ul>';
	34	+ break;
	35	+ case 1: // Booklet
	36	+ $show_cousins = true;
	37	+ echo '<div id="descendancy_booklet">';
	38	+ $controller->printChildFamily($controller->root, $controller->generations);
	39	+ echo '</div>';
	40	+ break;
	41	+ case 2: // Individual list
	42	+ $descendants = $controller->individualDescendancy($controller->root, $controller->generations, []);
	43	+ echo '<div id="descendancy-list">', FunctionsPrintLists::individualTable($descendants), '</div>';
	44	+ break;
	45	+ case 3: // Family list
	46	+ $descendants = $controller->familyDescendancy($controller->root, $controller->generations, []);
	47	+ echo '<div id="descendancy-list">', FunctionsPrintLists::familyTable($descendants), '</div>';
	48	+ break;
49	49	}
50	50	echo $controller->getJavascript();
51	51

		@@ -48,46 +48,46 @@
		block discarded – undo
48	48	// Respond to form action
49	49	if ($action !== '' && Filter::checkCsrf()) {
50	50	switch ($action) {
51		- case 'update':
52		- if ($username !== Auth::user()->getUserName() && User::findByUserName($username)) {
53		- FlashMessages::addMessage(I18N::translate('Duplicate username. A user with that username already exists. Please choose another username.'));
54		- } elseif ($email !== Auth::user()->getEmail() && User::findByEmail($email)) {
55		- FlashMessages::addMessage(I18N::translate('Duplicate email address. A user with that email already exists.'));
56		- } else {
57		- // Change username
58		- if ($username !== Auth::user()->getUserName()) {
59		- Log::addAuthenticationLog('User ' . Auth::user()->getUserName() . ' renamed to ' . $username);
60		- Auth::user()->setUserName($username);
	51	+ case 'update':
	52	+ if ($username !== Auth::user()->getUserName() && User::findByUserName($username)) {
	53	+ FlashMessages::addMessage(I18N::translate('Duplicate username. A user with that username already exists. Please choose another username.'));
	54	+ } elseif ($email !== Auth::user()->getEmail() && User::findByEmail($email)) {
	55	+ FlashMessages::addMessage(I18N::translate('Duplicate email address. A user with that email already exists.'));
	56	+ } else {
	57	+ // Change username
	58	+ if ($username !== Auth::user()->getUserName()) {
	59	+ Log::addAuthenticationLog('User ' . Auth::user()->getUserName() . ' renamed to ' . $username);
	60	+ Auth::user()->setUserName($username);
	61	+ }
	62	+
	63	+ // Change password
	64	+ if ($password_1 !== '' && $password_1 === $password_2) {
	65	+ Auth::user()->setPassword($password_1);
	66	+ }
	67	+
	68	+ // Change other settings
	69	+ Auth::user()
	70	+ ->setRealName($real_name)
	71	+ ->setEmail($email)
	72	+ ->setPreference('language', $language)
	73	+ ->setPreference('TIMEZONE', $time_zone)
	74	+ ->setPreference('contactmethod', $contact_method)
	75	+ ->setPreference('visibleonline', $visible_online ? '1' : '0');
	76	+
	77	+ Auth::user()->setPreference('theme', $theme);
	78	+
	79	+ $WT_TREE->setUserPreference(Auth::user(), 'rootid', $root_id);
61	80	}
62		-
63		- // Change password
64		- if ($password_1 !== '' && $password_1 === $password_2) {
65		- Auth::user()->setPassword($password_1);
	81	+ break;
	82	+
	83	+ case 'delete':
	84	+ // An administrator can only be deleted by another administrator
	85	+ if (!Auth::user()->getPreference('canadmin')) {
	86	+ $currentUser = Auth::user();
	87	+ Auth::logout();
	88	+ $currentUser->delete();
66	89	}
67		-
68		- // Change other settings
69		- Auth::user()
70		- ->setRealName($real_name)
71		- ->setEmail($email)
72		- ->setPreference('language', $language)
73		- ->setPreference('TIMEZONE', $time_zone)
74		- ->setPreference('contactmethod', $contact_method)
75		- ->setPreference('visibleonline', $visible_online ? '1' : '0');
76		-
77		- Auth::user()->setPreference('theme', $theme);
78		-
79		- $WT_TREE->setUserPreference(Auth::user(), 'rootid', $root_id);
80		- }
81		- break;
82		-
83		- case 'delete':
84		- // An administrator can only be deleted by another administrator
85		- if (!Auth::user()->getPreference('canadmin')) {
86		- $currentUser = Auth::user();
87		- Auth::logout();
88		- $currentUser->delete();
89		- }
90		- break;
	90	+ break;
91	91	}
92	92
93	93	header('Location: edituser.php');

fisharebest / webtrees

Branch — master (4a3c5b)

Status

Category

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Switch Indentation +85 added lines, -85 removed lines patch added patch discarded remove patch

Switch Indentation +33 added lines, -33 removed lines patch added patch discarded remove patch

Switch Indentation +26 added lines, -26 removed lines patch added patch discarded remove patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Switch Indentation +19 added lines, -19 removed lines patch added patch discarded remove patch

Switch Indentation +140 added lines, -140 removed lines patch added patch discarded remove patch

Switch Indentation +38 added lines, -38 removed lines patch added patch discarded remove patch