summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Rettberg2020-06-25 13:33:41 +0200
committerSimon Rettberg2020-06-25 13:33:41 +0200
commit3f357a75a12085a6fef37ebf3a043d6b6d29f414 (patch)
treec21e0addfbd868b5c729d82ac8155b7b3ecf85dc
parentUpdate exmaple config (diff)
downloadbwlp-webadmin-3f357a75a12085a6fef37ebf3a043d6b6d29f414.tar.gz
bwlp-webadmin-3f357a75a12085a6fef37ebf3a043d6b6d29f414.tar.xz
bwlp-webadmin-3f357a75a12085a6fef37ebf3a043d6b6d29f414.zip
[register] Simplify code
-rw-r--r--modules/register.inc.php16
1 files 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.');