Closed
Conversation
- Emojis durch Font Awesome 6 Icons ersetzt - YRewrite Domain-Verwaltung hinzugefügt mit Migration - Settings in übersichtliche Unterseiten reorganisiert (Wartung, Einstellungen, Ankündigung, Domains) - Mehrsprachige Wartungstexte (Deutsch/Englisch) mit Sprachswitcher - Modernes upkeep-inspiriertes Design übernommen - Card-basiertes Layout mit Animationen - Dark Mode Support - Responsive Design - Dropdown-Sprachauswahl mit Globe-Icon - IP-Adressen per Klick zur Whitelist hinzufügen - Login-Formular nur bei gesetztem Passwort anzeigen - Intelligente Bypass-URL-Anzeige (nur für gesperrte Domains) - Sidebar-Komponente für alle Frontend-Seiten - Editor-Konfiguration in Einstellungen verschoben - Begriffe vereinheitlicht (Wartungsmodus -> Wartung) - Weiterleitungs-URL zu allgemeinen Einstellungen verschoben
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Check rex_addon::exists('yrewrite') before calling methods
- Prevents exceptions when yrewrite addon is not installed
- Affected files: frontend.php, frontend.sidebar.php (2x), Maintenance.php (2x)
- Fix: Internationalize hardcoded German text 'hinzugefügt' Added maintenance_ip_added translation key - Fix: Correct spelling 'authentification' to 'authentication' Updated in package.yml, lang files, PHP code Added migration for existing installations - Already fixed: Ternary operator logic in frontend.php - Already fixed: YRewrite existence checks
- Fix login.php: Use authentication_mode instead of authentification_mode - Improve update.php migration: Always remove old config key - All files now consistently use 'authentication' (correct spelling)
README changes: - Set Thomas Skerbis as lead developer - Remove sponsor references - Update project team structure Console command enhancements: - Add 'status' command to show current maintenance state - Add 'frontend on/off' for frontend maintenance - Add 'backend on/off' for backend maintenance - Add 'all on/off' to toggle all modes at once - Add 'domain <name> --lock/--unlock' for YRewrite domains - Maintain backward compatibility with legacy 'on/off' commands - Add comprehensive documentation for all commands
Problem: When only German text (maintenance_text_de) was filled and English text was empty, no text was displayed because the DE block didn't receive the 'active' class. Solution: Add fallback logic for DE block: - DE is active if multilanguage is disabled OR only DE text exists - EN is active if multilanguage is disabled OR only EN text exists This ensures at least one text block is always visible when text is provided.
Removed overly restrictive condition that required both password mode AND a set secret. The login form should be displayed whenever password mode is selected, regardless of whether a secret is actually configured. This allows users to see the login form immediately after selecting password authentication mode, even before setting a password.
- Add id attribute to password input field for better accessibility - Add autocomplete='off' for better security - Improve code formatting and comments - Add debug comment placeholder for troubleshooting
Implements feature request for permanent site blocking without maintenance messaging, ideal for staging environments in deployment processes. Features: - New 'silent_mode' config option - Sends only HTTP status code (503/403) without HTML content - Prevents information disclosure about CMS - Perfect for staging systems that should be permanently offline - Accessible only via backend login, IP whitelist, or secret URL Changes: - Added silent_mode to package.yml default_config - Added silent_mode checkbox in advanced settings - Implemented logic in Maintenance::checkFrontend() - Added German translations - Updated README with Silent Mode documentation Use case: Production preview systems that need permanent protection without revealing 'maintenance mode' or CMS information.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
- Behebt Ternary-Operator Fehler: EN-Text war immer als 'active' markiert - DE-Text ist jetzt standardmäßig 'active', EN-Text nur wenn kein DE-Text vorhanden
- DE-Text ist jetzt standardmäßig aktiv - EN-Text nur aktiv wenn kein DE-Text vorhanden - Konsistente Logik zwischen frontend.php und frontend_old.php
- Behebt kritischen Bug: Early-Return umging block_frontend-Schalter - checkFrontend() prüft jetzt sowohl block_frontend als auch domainbasierte Wartung - Frontend wird nur dann erlaubt, wenn BEIDE Flags inaktiv sind Fixes: #156 (comment)
- YRewrite Existenz-Check in domains.php hinzugefügt (verhindert Exception) - Redundanten Domain-Check in boot.php entfernt (isDomainInMaintenance in checkFrontend) - Silent Mode Check früher durchgeführt (direkt nach Maintenance-Check) * Spart unnötige IP/Host/Secret-Checks bei Silent Mode * Verbessert Performance für Staging-Umgebungen
Implementiert automatische Aktivierung/Deaktivierung des Wartungsmodus: Features: - Request-basierte Prüfung: Automatische Aktivierung bei jedem Request - Optionaler Cronjob für präzisere Steuerung (jede Minute) - Konfiguration über Start- und Endzeitpunkt (YYYY-MM-DD HH:MM:SS) - Automatische Bereinigung nach Deaktivierung - Status-Anzeige im Backend Neue Config-Felder: - scheduled_start: Automatischer Aktivierungszeitpunkt - scheduled_end: Automatischer Deaktivierungszeitpunkt Neue Klassen: - Maintenance::checkScheduledMaintenance(): Request-basierte Prüfung - rex_cronjob_scheduled_maintenance: Optionaler Cronjob UI-Erweiterungen: - Zeitgesteuerte Wartung Sektion in Frontend-Einstellungen - Echtzeit-Status der geplanten Wartung - Mehrsprachige Labels (DE) Dokumentation: - README erweitert mit Verwendungsbeispielen - Inline-Kommentare für alle neuen Funktionen
Probleme behoben: - Button zum Hinzufügen der eigenen IP fehlte in Sidebar - JavaScript nutzt jetzt Event-Delegation (document.on) - Timeout für Tokenfield-Initialisierung hinzugefügt - Feedback für bereits vorhandene IPs Verbesserungen: - Zeigt eigene IP-Adresse in Sidebar (nur auf Advanced-Seite) - Button wechselt zu Erfolg (grün) oder Warnung (gelb) - CSP-Nonce für inline JavaScript - Neue Sprachschlüssel: maintenance_my_ip_title, maintenance_add_my_ip
- rex_i18n::msg() funktioniert nicht im Frontend (AddOn-Sprachdateien nicht geladen) - Hardcode 'Language' statt translate:maintenance_language - Behebt [translate:maintenance_language] Anzeige im Frontend
Fixes:
- $currentPage wird jetzt am Anfang der Sidebar definiert (Zeile 8 statt 94)
- Verhindert 'Undefined variable' Warning
IP-Button Verbesserungen:
- waitForTokenfield() Funktion für robuste Tokenfield-Initialisierung
- Verwendet tokenfield('getTokens') statt String-Splitting
- Besseres Timing-Handling mit Retry-Mechanismus
- Entfernt btn-xs Klasse bei Feedback für bessere Sichtbarkeit
- Console-Fehlerbehandlung
- Tokenfield-Abhängigkeit entfernt, einfaches Textfeld verwendet - IP-Sektion aus Sidebar entfernt - Vereinfachte JavaScript-Logik nach Upkeep-Vorbild - Verbesserte UI mit formatierten IP-Zeilen - Sprachschlüssel ergänzt (maintenance_your_ip, maintenance_server_ip, etc.)
- package.yml: Cronjob-Definition vereinfacht (war falsch formatiert) - boot.php: Tokenfield JavaScript/CSS entfernt (nicht mehr benötigt) - Cronjob sollte nun im Cronjob-Addon sichtbar sein
- lib/cronjob/scheduled_maintenance.php → lib/cronjob.php (REDAXO-Standard) - package.yml: Cronjob-Deklaration entfernt (automatische Erkennung) - Inline JavaScript nach assets/js/frontend-advanced.js ausgelagert - Inline JavaScript nach assets/js/domains.js ausgelagert - Inline CSS nach assets/css/ip-addresses.css ausgelagert - IP-Adressen als data-Attribute statt PHP-Interpolation - Fragments behalten inline CSS/JS (Sperrseiten)
- Überschrift zeigt 'Maintenance' (EN) / 'Wartung' (DE) je nach Sprachwahl - Domain wird unter der Überschrift angezeigt (YRewrite oder ServerName) - CSS angepasst für inline maintenance-text Spans in Überschrift - Funktioniert nur bei aktivierter Mehrsprachigkeit
- max-width: 450px, width: 100% für Container - padding-top: 4rem (Desktop) / 3.5rem (Mobile) für Platz über Icon - Language-Switcher bleibt in oberer rechter Ecke (auch mobil) - Kompaktere Buttons auf Mobile (120px statt 140px) - Verhindert Überlappung mit Info-Icon
- Überschrift 'Wartung' (DE) wird initial angezeigt - Entspricht dem Verhalten der Texte (DE ist Standard) - Sprachwechsler funktioniert wie bisher
- Language-Option DE initial als 'active' markiert - Überschrift 'Wartung' (DE) mit 'active' Klasse - Konsistent mit den Nachrichtentexten (DE ist Standard) - Sprachwechsler zeigt nun korrekt DE als gewählte Sprache
- Initial 'Deutsch' statt 'Language' - Button-Text wechselt bei Sprachwahl (English/Deutsch) - Extrahiert Sprachname aus Option-Text (ohne EN/DE Prefix) - Konsistent mit aktivem Dropdown-Item
- execute(): bool → execute() - getTypeName(): string → getTypeName() - getParamFields(): array → getParamFields() - Entspricht REDAXO-Standard (wie PHPMailer, Backup) - Cronjob sollte nun in Auswahl erscheinen
- Feldauswahl mit name-Attribut statt ID (rex_config_form Standard) - Event-Delegation mit $(document).on() für dynamische Elemente - Null-Check für data-ip Attribut - Error-Logging wenn Feld nicht gefunden - Entspricht genau der Upkeep-Implementierung
- Exakte Struktur wie phpmailer cronjob - Eigenständige Logik ohne Aufruf von Maintenance::checkScheduledMaintenance() - Direkte Config-Manipulation für aktivieren/deaktivieren - Besseres Logging mit Status-Updates
- rex_cronjob_manager::registerType() im boot.php - Prüfung ob Cronjob-Addon verfügbar ist - Folgt dem Standard von Backup/PHPMailer AddOns
- Schritt-für-Schritt-Anleitung zur Cronjob-Konfiguration - Erklärung aller relevanten Einstellungen - Hinweis auf optionale Nutzung (Request-basiert funktioniert auch)
- checkScheduledMaintenance() aus checkFrontend() entfernt - Cronjob-Status-Warnung in Frontend-Einstellungen - Direkt-Link zum Cronjob erstellen wenn nicht vorhanden - README und Sprachdateien aktualisiert - Klarstellung: Funktion nur über Cronjob, nicht request-basiert
…niert - Neue Unterseite 'Planung' (frontend.scheduled.php) unter Frontend - Zeitgesteuerte Wartung von frontend.index.php entfernt - Wartungsankündigung von frontend.announcement.php integriert - Beide Features jetzt unter Maintenance > Frontend > Planung - package.yml: scheduled als Unterseite von frontend - Icon: fa-calendar-alt für Planungs-Seite - boot.php: OUTPUT_FILTER für frontend/scheduled aktiviert - Editor-Einstellung wird wieder berücksichtigt - Sidebar mit Schnellzugriffen und Code-Beispiel für Ankündigung
Contributor
|
Caution Review failedThe pull request is closed. Moin. WalkthroughDas Maintenance AddOn wird auf Version 4.0.0-beta1 erweitert mit zeitgesteuerten Wartungsfenstern (cron-basiert), Silent-Mode für HTTP-Only-Responses, mehrsprachiger Frontend-Wartungsseite, domänenspezifischen Wartungssperren via YRewrite und erweiterten CLI-Befehlen. Die Konfiguration wird migriert, das UI reorganisiert und Legacy-Kompatibilität gewährleistet. Changes
Sequence Diagram(s)sequenceDiagram
participant Cron as Cronjob<br/>(periodisch)
participant Check as checkScheduledMaintenance()
participant Config as Config
participant Frontend as Frontend<br/>Blocking
Cron->>Check: Führe aus
Check->>Config: Lese scheduled_start,<br/>scheduled_end, block_frontend
alt Zeitfenster aktiv & blocking inaktiv
Check->>Config: Setze block_frontend=true
Check->>Frontend: Wartung aktiviert
else Zeitfenster abgelaufen & blocking aktiv
Check->>Config: Setze block_frontend=false<br/>Lösche Zeiten
Check->>Frontend: Wartung deaktiviert
else Kein Zustandswechsel
Check->>Frontend: Status unverändert
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes
Possibly related PRs
Suggested reviewers
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (22)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces version 4.0.0-beta1 of the Maintenance addon with significant feature enhancements and a major refactoring of the UI and architecture. The update focuses on improving domain-specific maintenance management, adding scheduled maintenance capabilities, and modernizing the user interface.
Key Changes:
- Adds scheduled/timed maintenance mode with cronjob support
- Introduces domain-specific maintenance management for YRewrite
- Implements Silent Mode for staging environments
- Refactors frontend settings pages into modular subpages (index, advanced, scheduled)
- Corrects misspelling:
authentification_mode→authentication_mode
Reviewed Changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
update.php |
Migration logic for config renaming and domain status conversion |
package.yml |
Version bump, new subpages, updated default configs |
lib/Maintenance.php |
New methods for domain-based and scheduled maintenance |
lib/cronjob.php |
New cronjob class for scheduled maintenance |
lib/command/maintenance_mode.php |
Enhanced CLI with domain management and status commands |
pages/frontend.index.php |
New main frontend settings page (split from old frontend.php) |
pages/frontend.advanced.php |
New admin-only advanced settings page |
pages/frontend.scheduled.php |
New scheduled maintenance configuration page |
pages/frontend.sidebar.php |
Shared sidebar for frontend pages with bypass URLs |
pages/domains.php |
New domain management interface for YRewrite |
fragments/maintenance/frontend.php |
Complete redesign with multilingual support and modern UI |
fragments/maintenance/login.php |
Updated authentication mode reference |
boot.php |
Cronjob registration and domain-based maintenance check |
lang/*.lang |
Updated translations with new features |
assets/js/*.js |
New JS for IP management and domain toggles |
assets/css/*.css |
New styling for icons and IP addresses |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Authentifizierung prüfen - für URL-Parameter und auch bei leerem Modus | ||
| $authentification_mode = (string) self::getConfig('authentification_mode', ''); | ||
| if (('' === $authentification_mode || 'URL' === $authentification_mode || 'password' === $authentification_mode) | ||
| $authentication_mode = (string) self::getConfig('authentication_mode', ''); |
There was a problem hiding this comment.
Duplicate assignment: $authentication_mode is already assigned on line 169. Remove the redundant assignment on line 172.
Suggested change
| $authentication_mode = (string) self::getConfig('authentication_mode', ''); |
Comment on lines
+4
to
+5
| // Debug output - remove this later | ||
| // echo '<!-- DEBUG: authentication_mode = ' . htmlspecialchars($authentication_mode) . ' -->'; |
There was a problem hiding this comment.
Commented debug code should be removed before release. This debug output is no longer needed in production code.
Suggested change
| // Debug output - remove this later | |
| // echo '<!-- DEBUG: authentication_mode = ' . htmlspecialchars($authentication_mode) . ' -->'; |
| } | ||
|
|
||
| if (rex_addon::get('cronjob')->isAvailable()) { | ||
| $content .= '<li class="list-group-item"><a href="' . rex_url::backendPage('cronjob/cronjobs') . '"><i class="rex-icon fa-clock-o"></i> Cronjob-Verwaltung</a></li>'; |
There was a problem hiding this comment.
Hardcoded German text 'Cronjob-Verwaltung' should use i18n translation for consistency with the rest of the addon. Use $addon->i18n('maintenance_cronjob_management') or similar.
Suggested change
| $content .= '<li class="list-group-item"><a href="' . rex_url::backendPage('cronjob/cronjobs') . '"><i class="rex-icon fa-clock-o"></i> Cronjob-Verwaltung</a></li>'; | |
| $content .= '<li class="list-group-item"><a href="' . rex_url::backendPage('cronjob/cronjobs') . '"><i class="rex-icon fa-clock-o"></i> ' . $addon->i18n('maintenance_cronjob_management') . '</a></li>'; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
Release Notes – Maintenance AddOn 4.0.0-beta1
Neue Funktionen
Breaking Changes
Verbesserungen