DatabaseUserManager #25

Merged
niklas.ortmann merged 12 commits from DatabaseUserManager into dev 2026-06-10 16:15:38 +02:00
Collaborator
No description provided.
caroline.slt added 24 commits 2026-06-05 23:47:48 +02:00
niklas.ortmann changed title from DataBaseArticleManager + Methoden to DatabaseUserManager 2026-06-06 09:35:42 +02:00
niklas.ortmann requested changes 2026-06-06 09:46:40 +02:00
niklas.ortmann left a comment
Owner

Der Code sieht gut aus!
Ein paar Kleinigkeiten sind angemerkt.

Der Code sieht gut aus! Ein paar Kleinigkeiten sind angemerkt.
@@ -4,1 +3,3 @@
$dao = new LocalUserManager();
require_once "php/model/UserManager.php";
$dao = UserManager::getInstance();
Owner

Das muss in einen try-catch-Block

Das muss in einen try-catch-Block
caroline.slt marked this conversation as resolved
@@ -0,0 +26,4 @@
throw new RuntimeException("Benutzerdatenbank konnte nicht erstellt werden.");
}
}
Owner

Kommentare ergänzen

Kommentare ergänzen
caroline.slt marked this conversation as resolved
@@ -0,0 +41,4 @@
throw new RuntimeException("Verbindung zur Benutzerdatenbank fehlgeschlagen.");
}
}
Owner

Kommentar ergänzen

Kommentar ergänzen
caroline.slt marked this conversation as resolved
@@ -7,2 +7,4 @@
private string $file = "data/users.json";
public static function getInstance(){
// TODO: implement the getIsntance method.
Owner

Eigentlich brauchen wir die Klasse ja sowieso nicht mehr, aber hier könntest du die getInstance() aus dem LocalArticleManager übernehmen.

Eigentlich brauchen wir die Klasse ja sowieso nicht mehr, aber hier könntest du die getInstance() aus dem LocalArticleManager übernehmen.
caroline.slt marked this conversation as resolved
@@ -1,2 +1,2 @@
<?php
Owner

Kommentar ergänzen

Kommentar ergänzen
caroline.slt marked this conversation as resolved
@@ -7,3 +6,4 @@
$namePattern = '/^[a-zA-ZäöüÄÖÜß\s-]{2,20}$/u';
return preg_match($namePattern, $name) === 1;
}
Owner

Kommentar ergänzen

Kommentar ergänzen
caroline.slt marked this conversation as resolved
@@ -14,5 +14,5 @@
{
$name = trim($name);
$namePattern = '/^[a-zA-ZäöüÄÖÜß\s-]{2,50}$/u';
if (preg_match($namePattern, $name)) {
Owner

Kommentar ergänzen

Kommentar ergänzen
caroline.slt marked this conversation as resolved
@@ -23,3 +23,3 @@
return $zeichenAnzahl >= 8 && $zeichenAnzahl <= 72;
return $zeichenAnzahl >= 5 && $zeichenAnzahl <= 12;
}
Owner

Kommentar ergänzen

Kommentar ergänzen
caroline.slt marked this conversation as resolved
caroline.slt added 1 commit 2026-06-08 20:10:20 +02:00
caroline.slt added 1 commit 2026-06-08 20:18:11 +02:00
caroline.slt added 1 commit 2026-06-08 20:20:11 +02:00
caroline.slt requested review from niklas.ortmann 2026-06-08 20:20:32 +02:00
niklas.ortmann approved these changes 2026-06-09 15:26:18 +02:00
niklas.ortmann left a comment
Owner

Pfadanpassungen noch machen, sonst passt es.

Pfadanpassungen noch machen, sonst passt es.
@@ -18,3 +18,3 @@
if ($pfad === "logout") {
include_once "content/logout.php";
include_once "content/logout-controller.php";
Owner

Pfad anpassen

Pfad anpassen
niklas.ortmann marked this conversation as resolved
@@ -23,3 +23,3 @@
if ($pfad === "deleteAccount") {
include_once "content/deleteAccount.php";
include_once "content/deleteAccount-controller.php";
Owner

Pfad anpassen

Pfad anpassen
niklas.ortmann marked this conversation as resolved
niklas.ortmann merged commit d00f2d9d53 into dev 2026-06-10 16:15:38 +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#25