From 3f357a75a12085a6fef37ebf3a043d6b6d29f414 Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Thu, 25 Jun 2020 13:33:41 +0200
Subject: [register] Simplify code

---
 modules/register.inc.php | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/modules/register.inc.php b/modules/register.inc.php
index 22b269d..aa2b94c 100644
--- a/modules/register.inc.php
+++ b/modules/register.inc.php
@@ -24,14 +24,18 @@ class Page_Register extends Page
 		}
 
 		$anonymous = (Request::post('share') !== 'on');
-		if (Request::post('testlogin') !== false) {
+		$testLogin = Request::post('testlogin');
+		if (empty($testLogin)) {
+			$testLogin = false;
+		}
+		if ($testLogin !== false) {
 			// Check if one of firstname, lastname or email matches
 			$user = Database::queryFirst('SELECT firstname, lastname, email, organizationid FROM user WHERE userid = :login LIMIT 1',
-					array('login' => Request::post('testlogin')));
+					array('login' => $testLogin));
 			if ($user === false || User::getOrganizationId() !== $user['organizationid']) {
 				// Invalid Login
 				Message::addError('Test-Account {{0}} unbekannt. '
-					. ' Bitte wenden Sie sich an den {{1}}-Support, wenn dieser Test-Account Ihnen gehört.', Request::post('testlogin'), CONFIG_SUITE);
+					. ' Bitte wenden Sie sich an den {{1}}-Support, wenn dieser Test-Account Ihnen gehört.', $testLogin, CONFIG_SUITE);
 				Util::redirect('?do=Main');
 			}
 			if (User::getLastName() !== $user['lastname']
@@ -39,11 +43,11 @@ class Page_Register extends Page
 				|| User::getMail() !== $user['email']) {
 				// No match by personal information
 				Message::addError('Ihre Metadaten stimmen nicht mit dem Test-Account {{0}} überein. '
-					. ' Bitte wenden Sie sich an den {{1}}-Support, wenn dieser Test-Account Ihnen gehört.', Request::post('testlogin'), CONFIG_SUITE);
+					. ' Bitte wenden Sie sich an den {{1}}-Support, wenn dieser Test-Account Ihnen gehört.', $testLogin, CONFIG_SUITE);
 				Util::redirect('?do=Main');
 			}
 			// Check if anonymous is requested, but user shared VMs with his testacc
-			if ($anonymous && Image::getImageCount(Request::post('testlogin')) > 0) {
+			if ($anonymous && Image::getImageCount($testLogin) > 0) {
 				Message::addError('Sie haben mit Ihrem Test-Account Virtuelle Maschinen auf den Zentral-Server hochgeladen und können"
 					. " sich daher nicht ohne Teilnahme am landesweiten VM-Austausch registrieren.');
 				Util::redirect('?do=Main');
@@ -60,7 +64,7 @@ class Page_Register extends Page
 			} elseif (!User::hasFullName()) {
 				Message::addError('Ihr Identity Provider hat keinen Namen zu Ihrem Account geliefert. Registrierung nicht möglich.');
 				// Put stuff in DB
-			} elseif (User::deploy($anonymous, Request::post('testlogin'))) {
+			} elseif (User::deploy($anonymous, $testLogin)) {
 				Message::addSuccess('Ihr Konto wurde freigeschaltet');
 			} else {
 				Message::addError('Fehler beim Zusammenführen mit Ihrem Test-Account. Bitte wenden Sie sich an den Support.');
-- 
cgit v1.2.3-55-g7522