[CODINGSTANDARD] Moodle-PHPCS-Verstöße und lose Typvergleiche systematisch bereinigen

[ralferlebach]

Problem

Der Code enthält zahlreiche else if, lose Vergleiche, nicht-strikte in_array-Aufrufe und uneinheitliche Typisierung. Dies erschwert sichere Vergleiche von IDs, Statuswerten und Konfigurationen.

Ursache

Moodle PHPCS ist offenbar nicht als verpflichtendes Merge-Gate etabliert.

Lösung

• Moodle Codechecker auf alle drei Plugins anwenden.
• elseif, strikte Vergleiche und typisierte Eingaben verwenden.
• in_array(..., true) einsetzen, sofern keine Konversion beabsichtigt ist.
• Parameter- und Rückgabetypen ergänzen.
• Begründete Ausnahmen lokal dokumentieren.

Manuelles Testverfahren

Testschritte

1. PHPCS lokal und in CI ausführen.
2. Kritische ID- und Statuspfade nach Typänderung manuell testen.

Aktuelles Ist-Verhalten

Der Code enthält zahlreiche else if, lose Vergleiche, nicht-strikte in_array-Aufrufe und uneinheitliche Typisierung. Dies erschwert sichere Vergleiche von IDs, Statuswerten und Konfigurationen.

Erwartetes Soll-Verhalten

Moodle Coding Standards sind grün oder Ausnahmen explizit begründet.

Automatisierte Tests

• PHPCS als verpflichtender CI-Job.
• Unit-Tests für String-/Integer-ID-Grenzfälle.

Akzeptanzkriterien

• Moodle Coding Standards sind grün oder Ausnahmen explizit begründet.
• Vergleiche sind typstabil.
• Keine unbeabsichtigten Verhaltensänderungen.