Skip to content

Cleanup und Refactoring

  • Jobbörse, Stundenplan aus Build entfernt
  • Modul reward komplett entfernt
  • Durchgängig Verwendung von Domain-Objects anstelle von Long-IDs
  • Service-Abhängigkeiten untereinander auf ein Minimum reduziert, um zyklische Abhängigkeiten zu vermeiden. Services werden jetzt überwiegend im Controller aufgerufen.
  • Tests überarbeitet
  • Reihenfolge der Parameter auf allen Ebenen einheitlich (In Entitys wurde Reihenfolge der Felder auch leicht angepasst):
    1. Domain-Object (z.B. TimelineEntry)
    2. Ggf. User
    3. Verschiedenes wie Pages etc.
  • Fehlermeldungen in Exceptions einheitlich geschrieben: "" gefolgt von "ID:" (groß, Doppelpunkt)
  • Timeline in eigenes Paket verschoben (de.thm.kim.tc.app.timeline)
  • Homescreen, Profil, Campus, Department in eigenes Paket verschoben (de.thm.kim.tc.app.account)
  • Alle Methoden aus CampusService entfernt. Ganz abschaffen?
  • Alle Methoden aus DepartmentService entfernt. Ganz abschaffen?
  • Viele Methoden aus SourceService entfernt (Querys nach Campus und Fachbereich etc.)
  • Eigenes Paket für gemeinsame Exceptions und ein globaler Handler (de.thm.kim.tc.app.common)
  • Keine explizite Angabe der zu scannenden Pakete für JPA-Entitys und -Repos mehr in der Main-Klasse
  • Einige Hibernate-Repository-Methoden sinnvoll benannt (findByX anstelle von findDomainObjectByX)
  • DTOs und Embedded-IDs package-private gesetzt
  • Gradle-Wrapper auf v5.6.2 aktualisiert
  • Unnötige Kommentare entfernt
  • Whitespace

Closes #26 (closed)

Edited by Robert Palm

Merge request reports