ACHTUNG: Refactoring - Rewrite des Intranet Quellcodes #14

Merged
samuelbias merged 5 commits from dev into main 2026-08-29 21:37:33 +02:00
Owner

Dieser PR führt ein umfassendes Refactoring mehrerer Intranet-Module durch, mit Fokus auf weniger Komplexität, klarere Verantwortlichkeiten, weniger unnötige Endpoints und verbesserte Stabilität/Sicherheit.

Enthalten sind unter anderem:

  • deutliche Vereinfachung von Seelsorge, Kalli, Contentpanel, Dashboard und Brandschutz
  • Entfernung ungenutzter Legacy-/Kompatibilitätsdateien und redundanter AJAX-Endpunkte
  • Auslagerung von gemeinsam genutzter Fachlogik in Helper sowie CSS/JS in zentrale Assets
  • Reduktion von N+1-Datenbankabfragen und unnötigen Requests
  • Umstellung von SMS-Benachrichtigungen in Seelsorge auf das bestehende Mail-System
  • sicherere POST-Flows mit CSRF-Schutz und Post/Redirect/Get
  • Härtung und Vereinfachung der Authorization-Flows
  • Behebung der Login-Regression durch eine Variablenkollision mit database.php
  • zusätzliche Validierung, transaktionale Schreibvorgänge und robustere Datei-/PDF-/XLSX-Verarbeitung
  • kleiner Developerpanel-Fix mit zusätzlichem Wiki-Link
  • UI-Fix in Seelsorge, damit Verfügbarkeitsstatus nicht mehr den gesamten Modal-Hintergrund einfärbt
    Die bestehenden Funktionen bleiben erhalten, die Codebasis ist insgesamt deutlich kompakter und die Architektur konsistenter geworden.

Außerdem noch:
Die CKalender-Synchronisierung wurde robuster umgesetzt und refaktoriert. Der Zeitraum wird jetzt immer explizit vom 1. des aktuellen Monats bis zum letzten Tag des Folgemonats angefordert. Falls CKalender dennoch einen Monat unvollständig liefert, wird dieser gezielt nachgeladen. Zusätzlich wurde die Parser- und PDF-Logik vereinfacht, ohne das bestehende Verhalten zu ändern. Die relevanten Fälle inklusive Monatswechsel und Jahreswechsel wurden getestet.

Nicht ganz passend zum restlichen Ticket, aber hatte Bock drauf, ein Git-RSS-Feed wurde im Developerboard ergänzt und entsprechend des restlichen Projekts implementiert. Die letzten sieben Aktivitäten werden nach Zeiträumen gruppiert dargestellt, weitere Einträge lassen sich bei Bedarf einblenden.

Dieser PR führt ein umfassendes Refactoring mehrerer Intranet-Module durch, mit Fokus auf weniger Komplexität, klarere Verantwortlichkeiten, weniger unnötige Endpoints und verbesserte Stabilität/Sicherheit. Enthalten sind unter anderem: - deutliche Vereinfachung von Seelsorge, Kalli, Contentpanel, Dashboard und Brandschutz - Entfernung ungenutzter Legacy-/Kompatibilitätsdateien und redundanter AJAX-Endpunkte - Auslagerung von gemeinsam genutzter Fachlogik in Helper sowie CSS/JS in zentrale Assets - Reduktion von N+1-Datenbankabfragen und unnötigen Requests - Umstellung von SMS-Benachrichtigungen in Seelsorge auf das bestehende Mail-System - sicherere POST-Flows mit CSRF-Schutz und Post/Redirect/Get - Härtung und Vereinfachung der Authorization-Flows - Behebung der Login-Regression durch eine Variablenkollision mit database.php - zusätzliche Validierung, transaktionale Schreibvorgänge und robustere Datei-/PDF-/XLSX-Verarbeitung - kleiner Developerpanel-Fix mit zusätzlichem Wiki-Link - UI-Fix in Seelsorge, damit Verfügbarkeitsstatus nicht mehr den gesamten Modal-Hintergrund einfärbt Die bestehenden Funktionen bleiben erhalten, die Codebasis ist insgesamt deutlich kompakter und die Architektur konsistenter geworden. Außerdem noch: Die CKalender-Synchronisierung wurde robuster umgesetzt und refaktoriert. Der Zeitraum wird jetzt immer explizit vom 1. des aktuellen Monats bis zum letzten Tag des Folgemonats angefordert. Falls CKalender dennoch einen Monat unvollständig liefert, wird dieser gezielt nachgeladen. Zusätzlich wurde die Parser- und PDF-Logik vereinfacht, ohne das bestehende Verhalten zu ändern. Die relevanten Fälle inklusive Monatswechsel und Jahreswechsel wurden getestet. Nicht ganz passend zum restlichen Ticket, aber hatte Bock drauf, ein Git-RSS-Feed wurde im Developerboard ergänzt und entsprechend des restlichen Projekts implementiert. Die letzten sieben Aktivitäten werden nach Zeiträumen gruppiert dargestellt, weitere Einträge lassen sich bei Bedarf einblenden.
samuelbias added the Refactoring label 2026-08-29 21:32:51 +02:00
samuelbias added 5 commits 2026-08-29 21:32:52 +02:00
samuelbias requested review from Developers 2026-08-29 21:32:52 +02:00
leoniefink approved these changes 2026-08-29 21:37:11 +02:00
leoniefink left a comment
Member

lgtm

lgtm
samuelbias merged commit 494bfa8884 into main 2026-08-29 21:37:33 +02:00
Sign in to join this conversation.