Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verbesserungen beim Wartungsmodus und Clean Code #119

Merged
merged 9 commits into from
Nov 13, 2024

Conversation

skerbis
Copy link
Member

@skerbis skerbis commented Nov 12, 2024

Verbesserungen beim Wartungsmodus und Clean Code

Problem

Redakteure wurden nicht korrekt gesperrt und der Code enthielt viele PHPStan-Warnungen.

Lösung

  • Korrektur der Redakteurs-Sperrung durch Verwendung des korrekten Config-Keys
  • Verbesserte statische Code-Analyse Kompatibilität (PHPStan)
  • Robustere Typisierung und Fehlerbehandlung

Wichtigste Änderungen:

  1. Redakteure-Sperrung korrigiert

    • Korrekter Config-Key block_frontend_rex_user wird nun verwendet
    • Klare Logik: Admins haben immer Zugriff, Redakteure nur wenn sie nicht explizit gesperrt sind
  2. Bessere Code-Qualität

    • Statische Typisierung durch korrekte PHPDoc-Blöcke
    • Zentrale Config-Zugriffsmethoden
    • Entfernung aller @phpstan-ignore-line Direktiven
    • Verbesserte Fehlerbehandlung

Geprüft

  • Wartungsmodus funktioniert wie erwartet
  • Admins haben immer Zugriff
  • Redakteure werden korrekt gesperrt wenn aktiviert
  • PHPStan läuft ohne Warnungen durch
  • Bestehende Tests laufen erfolgreich durch
  • Code ist abwärtskompatibel

Copy link
Member

@alxndr-w alxndr-w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Danke für die Anpassungen, insb das Lösen der PHPstan-Meldungen, die ich nicht lösen konnte.

Kleine Anmerkungen, scheint aber alles soweit richtig zu sein.

lib/Maintenance.php Outdated Show resolved Hide resolved
lib/Maintenance.php Show resolved Hide resolved
lib/Maintenance.php Outdated Show resolved Hide resolved
lib/Maintenance.php Outdated Show resolved Hide resolved
@alxndr-w alxndr-w merged commit f0005c0 into main Nov 13, 2024
1 check passed
@alxndr-w alxndr-w deleted the fix-backenduser-blocked branch November 13, 2024 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants