Skip to content

TODOs aus PR #418 - Code-Verbesserungen #419

@skerbis

Description

@skerbis

🔍 Code-Qualität & Architektur

Fragmente verbessern

  • In Fragmenten im Header die erwarteten Variablen beschreiben
  • Prüfen, was in Fragmente ausgelagert werden kann für bessere Trennung von Code und Ausgabe
  • Insbesondere Code-Stellen, die JS erzeugen, in Fragmente auslagern

SQL-Queries modernisieren

  • $sql->setQuery durch einzelne Methoden (setTable/setWhere/select) ersetzen

Datenstrukturen

  • Komplexere Arrays (Domain, Cookie) durch Klassen ersetzen
    • Vereinfacht Folgecode (keine ständigen Existenz- und Datentyp-Prüfungen mehr nötig)

🧹 Code-Cleanup

Ungenutzte Parameter

  • Methoden mit ungenutzten Parametern überprüfen
    • Können die Parameter entfernt werden?
    • Oder werden sie noch gebraucht?

Weitere RexStan-Analysen

  • Vollständige RexStan-Analyse durchführen
  • Weitere Type-Hints ergänzen wo sinnvoll

Kommentierung

  • Header-Kommentar uneinheitlich; oft fehlend / tw. mit @package

Sonstiges

  • copilot-instructions.md an die Realität anpassen

📋 Vorgehensweise

Jeder Punkt sollte:

  1. Fachkenntnis zum Addon erfordern (daher nicht in PR Code-Überprüfung / Änderungsvorschlage (TODO: etc) / kleine Verbesserungen #418)
  2. Als separater, thematisch abgeschlossener PR bearbeitet werden
  3. Mit Tests/Review abgesichert werden

Ursprung: PR #418 von @christophboecker
Diskussion: #418

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions