Vor und Nachname separat speichern sowie Profilbearbeitung #18

Merged
niklas.ortmann merged 5 commits from Profilbearbeitung into dev 2026-06-03 20:13:03 +02:00
Collaborator
No description provided.
caroline.slt added 4 commits 2026-06-02 22:45:34 +02:00
niklas.ortmann requested changes 2026-06-03 06:45:31 +02:00
niklas.ortmann left a comment
Owner

Einige Dinge fehlen noch (s. Anmerkungen).

Einige Dinge fehlen noch (s. Anmerkungen).
@@ -9,23 +9,62 @@ if (!isset($_SESSION["user"])) {
exit();
}
$dao = new LocalUserDAO();
Owner

Das sollte zwingend im try-Block bleiben. Wenn beim Instanziieren eines LocalUserDAO ein fehler auftritt, dann wird dieser nicht abgefangen!

Das sollte zwingend im try-Block bleiben. Wenn beim Instanziieren eines LocalUserDAO ein fehler auftritt, dann wird dieser nicht abgefangen!
caroline.slt marked this conversation as resolved
@@ -15,0 +35,4 @@
if ($existingUser && $newEmail !== $oldEmail) {
$error = "Diese E-Mail-Adresse wird bereits verwendet.";
} else {
$updated = $dao->updateUser(
Owner

Eingabevalidierung fehlt noch.
Was darf ein Nutzer überhaupt alles eingeben? Z.B. Buchstaben a-z, A-Z, Sonderzeichen?
(Wenn du das zu dieser Abgabe nicht schaffst, dann bitte ein TODO setzen, dass das später noch erfolgt (wichtig) und einen Hinweis in der ReadMe hinterlassen)!

Eingabevalidierung fehlt noch. Was darf ein Nutzer überhaupt alles eingeben? Z.B. Buchstaben a-z, A-Z, Sonderzeichen? (Wenn du das zu dieser Abgabe nicht schaffst, dann bitte ein TODO setzen, dass das später noch erfolgt (wichtig) und einen Hinweis in der ReadMe hinterlassen)!
caroline.slt marked this conversation as resolved
@@ -39,4 +47,4 @@
$users = $this->loadUsers();
$users[] = [
"email" => $email,
Owner

Werden die Daten überprüft?
Können Fehler auftreten?
Werden Exceptions geworfen, abgefangen und an die Instanz, die diese Methode aufruft, weitergegeben?

Werden die Daten überprüft? Können Fehler auftreten? Werden Exceptions geworfen, abgefangen und an die Instanz, die diese Methode aufruft, weitergegeben?
caroline.slt marked this conversation as resolved
@@ -28,0 +35,4 @@
* @param string $vorname Neuer Vorname
* @param string $nachname Neuer Nachname
* @param string|null $password Neues Passwort oder null
* @return bool true, wenn der Benutzer aktualisiert wurde, sonst false
Owner

Die möglichen Exceptions müssen angeben werden.
(Können wir auch noch nachreichen, aber dann würde ich hier zumindest ein TODO schreiben, dass das noch gemacht werden muss).

Die möglichen Exceptions müssen angeben werden. (Können wir auch noch nachreichen, aber dann würde ich hier zumindest ein TODO schreiben, dass das noch gemacht werden muss).
caroline.slt marked this conversation as resolved
caroline.slt added 1 commit 2026-06-03 20:08:02 +02:00
niklas.ortmann merged commit 1343496a12 into dev 2026-06-03 20:13:03 +02:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: niklas.ortmann/webprogrammierung#18