Kontofunktionen vor dem Go-Live und Landingpage-Kuerzung
Vier Luecken geschlossen, die im Alltag sofort aufgefallen waeren: - Passwort aendern war im eingeloggten Zustand gar nicht moeglich; es gab nur den Reset per Mail-Link. saas_change_password() prueft das aktuelle Passwort, verlangt ein tatsaechlich anderes und erneuert danach die Session-ID. Formular in konto.php. - mandant-auswahl.php war nur direkt nach dem Login erreichbar. Wer bei mehreren Mandanten Mitglied ist, musste sich zum Wechseln abmelden. Die Seite bedient jetzt beide Wege, die Mandantenpruefung bleibt unveraendert ueber saas_identity_for_user_tenant(). Menuepunkt ab zwei Mitgliedschaften. - email_verified_at wurde nirgends geprueft, nur angezeigt - bei offener Selbstregistrierung konnte sich jemand mit fremder Adresse anmelden und alles nutzen. Erzwungen wird jetzt gezielt dort, wo eine Aktion nach aussen wirkt: Einladung, Info-Mail, Jahresabschluss. Der Login selbst bleibt bewusst frei, sonst waeren alle migrierten Bestandsnutzer mit NULL-Verifikation ausgesperrt. Zusaetzlich setzt der Passwort-Reset die Verifikation mit, weil der Mail-Link den Postfachzugriff nachweist - sonst blieben eingeladene Mitglieder dauerhaft unbestaetigt. - Login landete auf konto.php statt auf dem Dashboard. Landingpage: die gruene Vertrauenszeile auf "DSGVO-konform" gekuerzt, die Eintraege zu Paragraf 19 UStG und "Bestehende Ablaeufe bleiben" entfernt. Geprueft: neues scripts/check-konto-und-mandantenwechsel.php mit 18 Assertions gruen, Passwortformular zusaetzlich manuell inkl. CSRF (419). Bestehende Suiten unveraendert gruen: HTTP-Smoke 34 Seiten, Rollenmatrix 55, Mandanten-Isolation 12, M3-Auth 9, M3-Settings 15. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
+22
-8
@@ -3,26 +3,35 @@
|
||||
require_once __DIR__ . '/functions.php';
|
||||
|
||||
$pdo = app_db_pdo();
|
||||
|
||||
// Zwei Wege auf diese Seite: direkt nach dem Login (noch keine Session,
|
||||
// nur eine vorgemerkte User-ID) oder als Wechsel aus einer bestehenden
|
||||
// Sitzung heraus. Ohne den zweiten Weg muesste sich jemand, der bei
|
||||
// mehreren Mandanten Mitglied ist, zum Wechseln erst abmelden.
|
||||
$pendingUserId = saas_pending_tenant_user_id();
|
||||
if ($pendingUserId === null) {
|
||||
$currentUser = $pendingUserId === null ? saas_current_user($pdo) : null;
|
||||
$isSwitching = $currentUser !== null;
|
||||
$userId = $pendingUserId ?? ($currentUser !== null ? (int)$currentUser['user_id'] : null);
|
||||
|
||||
if ($userId === null) {
|
||||
header('Location: login.php');
|
||||
exit;
|
||||
}
|
||||
|
||||
$errors = [];
|
||||
$memberships = saas_list_user_memberships($pdo, $pendingUserId);
|
||||
$memberships = saas_list_user_memberships($pdo, $userId);
|
||||
|
||||
if ($_SERVER['REQUEST_METHOD'] === 'POST') {
|
||||
app_require_csrf();
|
||||
|
||||
$tenantId = isset($_POST['tenant_id']) ? (int)$_POST['tenant_id'] : 0;
|
||||
$identity = $tenantId > 0 ? saas_identity_for_user_tenant($pdo, $pendingUserId, $tenantId) : null;
|
||||
$identity = $tenantId > 0 ? saas_identity_for_user_tenant($pdo, $userId, $tenantId) : null;
|
||||
|
||||
if ($identity === null) {
|
||||
$errors[] = 'Dieser Mandant ist für dein Konto nicht verfügbar.';
|
||||
} else {
|
||||
saas_session_login($identity);
|
||||
header('Location: konto.php');
|
||||
header('Location: index.php');
|
||||
exit;
|
||||
}
|
||||
}
|
||||
@@ -34,7 +43,7 @@ include 'nav.php';
|
||||
|
||||
<section id="banner">
|
||||
<div class="content">
|
||||
<h2>Mandant auswählen</h2>
|
||||
<h2><?php echo $isSwitching ? 'Mandant wechseln' : 'Mandant auswählen'; ?></h2>
|
||||
|
||||
<?php if ($errors !== []): ?>
|
||||
<div class="hint-box error">
|
||||
@@ -54,7 +63,8 @@ include 'nav.php';
|
||||
<label for="tenant_id">Mandant</label>
|
||||
<select name="tenant_id" id="tenant_id" required>
|
||||
<?php foreach ($memberships as $membership): ?>
|
||||
<option value="<?php echo saas_html($membership['tenant_id']); ?>">
|
||||
<option value="<?php echo saas_html($membership['tenant_id']); ?>"
|
||||
<?php echo $isSwitching && (int)$membership['tenant_id'] === (int)$currentUser['tenant_id'] ? 'selected' : ''; ?>>
|
||||
<?php echo saas_html($membership['tenant_name']); ?>
|
||||
(<?php echo saas_html($membership['role']); ?>)
|
||||
</option>
|
||||
@@ -63,8 +73,12 @@ include 'nav.php';
|
||||
</div>
|
||||
</div>
|
||||
<ul class="actions">
|
||||
<li><button type="submit">Weiter</button></li>
|
||||
<li><a href="logout.php" class="button alt">Abbrechen</a></li>
|
||||
<li><button type="submit"><?php echo $isSwitching ? 'Wechseln' : 'Weiter'; ?></button></li>
|
||||
<?php if ($isSwitching): ?>
|
||||
<li><a href="index.php" class="button alt">Abbrechen</a></li>
|
||||
<?php else: ?>
|
||||
<li><a href="logout.php" class="button alt">Abbrechen</a></li>
|
||||
<?php endif; ?>
|
||||
</ul>
|
||||
</form>
|
||||
<?php endif; ?>
|
||||
|
||||
Reference in New Issue
Block a user