Merge pull request 'dev' (#43) from dev into main

Reviewed-on: #43

.gitea/workflows/code-analysis.yml

@@ -10,19 +10,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
+        uses: actions/checkout@v4
         with:
           clean: true
-          # Der Analyse-Workflow benötigt keine Historie. Ein flacher Checkout
-          # verhindert, dass der Runner das große Repository-Pack vollständig lädt.
-          fetch-depth: 1
-          # Der Web-Workflow braucht keine Android-Artefakte. Insbesondere liegt
-          # dort ein großer Heap-Dump im aktuellen Commit, der auch bei einem
-          # flachen Checkout übertragen würde und den Runner-Checkout abbricht.
-          sparse-checkout: |
-            /*
-            !/android-app/
-          sparse-checkout-cone-mode: false
+          fetch-depth: 0
 
       - name: Ensure clean workspace
         run: |
@@ -38,7 +29,7 @@ jobs:
           echo "eslint entries in package.json:" && rg '"eslint"' package.json || true
 
       - name: Setup Node.js
-        uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020
+        uses: actions/setup-node@v4
         with:
           node-version: 22
           cache: npm
@@ -125,7 +116,7 @@ jobs:
           chmod +x osv-scanner
           ./osv-scanner --version
           test -f ./package-lock.json
-          ./osv-scanner scan -L ./package-lock.json --config ./.osv-scanner.toml
+          ./osv-scanner --lockfile ./package-lock.json
 
   deploy-production:
     runs-on: ubuntu-latest

.github/workflows/android-ci.yml

@@ -10,14 +10,14 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
+      - uses: actions/checkout@v4
       - name: Set up JDK 17
-        uses: actions/setup-java@c1e323688fd81a25caa38c78aa6df2d33d3e20d9
+        uses: actions/setup-java@v4
         with:
           distribution: 'temurin'
           java-version: '17'
       - name: Cache Gradle
-        uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830
+        uses: actions/cache@v4
         with:
           path: |
             ~/.gradle/caches

.gitignore

@@ -94,8 +94,6 @@ dist
 /android-app/**/build/
 /android-app/local.properties
 /android-app/gradle-local.properties
-# JVM Heap-Dumps sind lokale Diagnoseartefakte und dürfen nie ins Repository.
-*.hprof
 
 # Build output (but keep production data!)
 .output

.osv-scanner.toml

@@ -1,4 +0,0 @@
-[[IgnoredVulns]]
-id = "GHSA-v3m3-f69x-jf25"
-ignoreUntil = 2026-12-31
-reason = "Temporary exception: Quill 2.0.3 is required by the current RichTextEditor implementation, and OSV currently reports no fixed version. Track upstream fix and remove this ignore once a patched release is available."

android-app/app/build.gradle.kts

@@ -78,12 +78,12 @@ val ensureProductionApiBaseUrl = tasks.register("ensureProductionApiBaseUrl") {
 
 android {
     namespace = "de.harheimertc"
-    compileSdk = 36
+    compileSdk = 35
 
     defaultConfig {
         applicationId = "de.harheimertc"
         minSdk = 24
-        targetSdk = 36
+        targetSdk = 35
         versionCode = androidVersionCode
         versionName = androidVersionName
     }

android-app/app/src/main/java/de/harheimertc/data/ApiService.kt

@@ -166,7 +166,6 @@ data class MembershipResponse(
     val success: Boolean = false,
     val message: String? = null,
     val downloadUrl: String? = null,
-    val downloadToken: String? = null,
 )
 data class LoginRequest(
     val email: String,
@@ -653,18 +652,12 @@ interface ApiService {
     @POST("/api/cms/save-csv")
     suspend fun saveCsv(@Body request: SaveCsvRequest): Response<SaveCsvResponse>
 
-    @POST("/api/cms/import-spielplan")
-    suspend fun importSpielplan(): Response<AuthMessageResponse>
-
     @POST("/api/membership/generate-pdf")
     suspend fun generateMembershipPdf(@Body request: MembershipRequest): Response<MembershipResponse>
 
     @Streaming
     @GET
-    suspend fun downloadMembershipPdf(
-        @Url downloadUrl: String,
-        @retrofit2.http.Header("X-Membership-Download-Token") downloadToken: String? = null,
-    ): Response<ResponseBody>
+    suspend fun downloadMembershipPdf(@Url downloadUrl: String): Response<ResponseBody>
 
     @POST("/api/auth/login")
     suspend fun login(@Body request: LoginRequest): Response<LoginResponse>

android-app/app/src/main/java/de/harheimertc/notifications/HarheimerNotifications.kt

@@ -16,16 +16,14 @@ import de.harheimertc.R
 import de.harheimertc.ui.navigation.Destinations
 
 object HarheimerNotifications {
-    // A new ID is intentional: Android does not allow an app update to raise
-    // the importance of an already-created notification channel.
-    const val DEFAULT_CHANNEL_ID = "harheimer_tc_updates_v2"
+    const val DEFAULT_CHANNEL_ID = "harheimer_tc_updates"
 
     fun createChannels(context: Context) {
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) return
         val channel = NotificationChannel(
             DEFAULT_CHANNEL_ID,
             "Harheimer TC",
-            NotificationManager.IMPORTANCE_HIGH,
+            NotificationManager.IMPORTANCE_DEFAULT,
         ).apply {
             description = "Benachrichtigungen des Harheimer TC"
         }
@@ -49,7 +47,7 @@ object HarheimerNotifications {
             .setContentTitle(title)
             .setContentText(message)
             .setStyle(NotificationCompat.BigTextStyle().bigText(message))
-            .setPriority(NotificationCompat.PRIORITY_HIGH)
+            .setPriority(NotificationCompat.PRIORITY_DEFAULT)
             .setContentIntent(createContentIntent(context, notificationId, data))
             .setAutoCancel(true)
             .build()

android-app/app/src/main/java/de/harheimertc/repositories/CmsRepository.kt

@@ -125,7 +125,6 @@ class CmsRepository @Inject constructor(
                 spieler = values[7],
                 informationenLink = values[8],
                 letzteAktualisierung = values[9],
-                spielplanMannschaftId = values.getOrElse(10) { "" },
             )
         }
     }
@@ -147,12 +146,6 @@ class CmsRepository @Inject constructor(
         response.body() ?: SaveCsvResponse(success = false, message = "Leere Antwort")
     }
 
-    suspend fun importSpielplan(): Result<de.harheimertc.data.AuthMessageResponse> = runCatching {
-        val response = api.importSpielplan()
-        if (!response.isSuccessful) error("Spielplan konnte nicht importiert werden.")
-        response.body() ?: de.harheimertc.data.AuthMessageResponse(success = false, message = "Leere Antwort")
-    }
-
     suspend fun spielplan(season: String? = null): Result<SpielplanResponse> = runCatching {
         val response = api.spielplan(season)
         if (!response.isSuccessful) error("Spielplan konnte nicht geladen werden.")
@@ -388,7 +381,6 @@ data class CmsMannschaftRow(
     val spieler: String = "",
     val informationenLink: String = "",
     val letzteAktualisierung: String = "",
-    val spielplanMannschaftId: String = "",
 )
 
 private fun List<CmsMannschaftRow>.toMannschaftenCsv(): String {
@@ -403,7 +395,6 @@ private fun List<CmsMannschaftRow>.toMannschaftenCsv(): String {
         "Spieler",
         "Weitere Informationen Link",
         "Letzte Aktualisierung",
-        "Spielplan Mannschaft ID",
     ).toCsvRow()
     val rows = map { row ->
         listOf(
@@ -417,7 +408,6 @@ private fun List<CmsMannschaftRow>.toMannschaftenCsv(): String {
             row.spieler,
             row.informationenLink,
             row.letzteAktualisierung,
-            row.spielplanMannschaftId,
         ).toCsvRow()
     }
     return listOf(header).plus(rows).joinToString("\n")

android-app/app/src/main/java/de/harheimertc/repositories/MembershipRepository.kt

@@ -9,7 +9,7 @@ import java.io.File
 import javax.inject.Inject
 import javax.inject.Singleton
 
-data class MembershipDocument(val message: String, val uri: String? = null)
+data class MembershipDocument(val message: String, val uri: String)
 
 @Singleton
 class MembershipRepository @Inject constructor(
@@ -18,33 +18,21 @@ class MembershipRepository @Inject constructor(
 ) {
     suspend fun submit(request: MembershipRequest): Result<MembershipDocument> = runCatching {
         val response = api.generateMembershipPdf(request)
-        if (!response.isSuccessful) {
-            val serverMessage = response.errorBody()?.string()
-                ?.let { Regex("\\\"(?:statusMessage|message)\\\"\\s*:\\s*\\\"([^\\\"]+)\\\"").find(it)?.groupValues?.getOrNull(1) ?: it }
-                ?.takeIf { it.isNotBlank() }
-            error(serverMessage ?: "Der Antrag konnte nicht übermittelt werden (HTTP ${response.code()}).")
-        }
+        if (!response.isSuccessful) error("HTTP ${response.code()}")
         val body = response.body() ?: error("Leere Antwort")
         if (!body.success) error(body.message ?: "Antrag konnte nicht erstellt werden.")
-        val uri = body.downloadUrl?.let { downloadUrl ->
-            runCatching {
-                val documentResponse = api.downloadMembershipPdf(downloadUrl, body.downloadToken)
-                if (!documentResponse.isSuccessful) error("PDF konnte nicht heruntergeladen werden.")
-                val directory = File(context.cacheDir, "membership").apply { mkdirs() }
-                val file = File(directory, "beitrittserklaerung.pdf")
-                documentResponse.body()?.byteStream()?.use { input ->
-                    file.outputStream().use { output -> input.copyTo(output) }
-                } ?: error("Leere PDF-Antwort")
-                FileProvider.getUriForFile(context, "${context.packageName}.files", file).toString()
-            }.getOrNull()
-        }
+        val downloadUrl = body.downloadUrl ?: error("PDF-Download fehlt.")
+        val documentResponse = api.downloadMembershipPdf(downloadUrl)
+        if (!documentResponse.isSuccessful) error("PDF konnte nicht heruntergeladen werden.")
+        val directory = File(context.cacheDir, "membership").apply { mkdirs() }
+        val file = File(directory, "beitrittserklaerung.pdf")
+        documentResponse.body()?.byteStream()?.use { input ->
+            file.outputStream().use { output -> input.copyTo(output) }
+        } ?: error("Leere PDF-Antwort")
+        val uri = FileProvider.getUriForFile(context, "${context.packageName}.files", file)
         MembershipDocument(
-            message = if (uri == null) {
-                "${body.message ?: "Mitgliedschaftsantrag erfolgreich übermittelt."} Das PDF konnte nicht auf diesem Gerät gespeichert werden."
-            } else {
-                body.message ?: "Beitrittsformular erfolgreich erstellt."
-            },
-            uri = uri,
+            message = body.message ?: "Beitrittsformular erfolgreich erstellt.",
+            uri = uri.toString(),
         )
     }
 }

android-app/app/src/main/java/de/harheimertc/ui/screens/cms/CmsScreens.kt

@@ -458,9 +458,6 @@ fun CmsSportbetriebScreen(navController: NavController, showBackNavigation: Bool
         "mannschaften" to "Mannschaften",
         "spielplaene" to "Spielpläne",
     )
-    val spielplanTeamOptions = remember(state.sportSpielplanHeaders, state.sportSpielplanRows) {
-        importedSpielplanTeams(state.sportSpielplanHeaders, state.sportSpielplanRows)
-    }
     val terminKategorien = listOf("Training", "Punktspiel", "Turnier", "Veranstaltung", "Sonstiges")
 
     LaunchedEffect(Unit) {
@@ -609,7 +606,6 @@ fun CmsSportbetriebScreen(navController: NavController, showBackNavigation: Bool
                     items(mannschaften.size) { index ->
                         MannschaftEditorCard(
                             row = mannschaften[index],
-                            spielplanTeams = spielplanTeamOptions,
                             onChange = { updated -> mannschaften[index] = updated },
                             onRemove = { mannschaften.removeAt(index) },
                         )
@@ -623,19 +619,9 @@ fun CmsSportbetriebScreen(navController: NavController, showBackNavigation: Bool
                             InfoRow("Datei", fileName)
                             InfoRow("Saison", seasonLabel)
                             InfoRow("Einträge", state.sportSpielplanRows.size.toString())
-                            Row(horizontalArrangement = Arrangement.spacedBy(8.dp), modifier = Modifier.fillMaxWidth()) {
-                                Button(
-                                    onClick = { viewModel.importSportSpielplan() },
-                                    enabled = !state.sportSaving,
-                                    modifier = Modifier.weight(1f),
-                                ) {
-                                    Text(if (state.sportSaving) "Importiert..." else "Von myTischtennis importieren")
-                                }
-                            }
                             Row(horizontalArrangement = Arrangement.spacedBy(8.dp), modifier = Modifier.fillMaxWidth()) {
                                 OutlinedButton(
                                     onClick = { viewModel.loadSportbetrieb() },
-                                    enabled = !state.sportSaving,
                                     modifier = Modifier.weight(1f),
                                 ) {
                                     Text("Neu laden")
@@ -746,32 +732,11 @@ fun CmsSportbetriebScreen(navController: NavController, showBackNavigation: Bool
 @Composable
 private fun MannschaftEditorCard(
     row: CmsMannschaftRow,
-    spielplanTeams: List<ImportedSpielplanTeam>,
     onChange: (CmsMannschaftRow) -> Unit,
     onRemove: () -> Unit,
 ) {
-    var teamPickerOpen by remember(row.mannschaft, spielplanTeams) { mutableStateOf(false) }
     DataCard(row.mannschaft.ifBlank { "Mannschaft" }) {
         OutlinedTextField(value = row.mannschaft, onValueChange = { onChange(row.copy(mannschaft = it)) }, label = { Text("Mannschaft") }, modifier = Modifier.fillMaxWidth())
-        OutlinedTextField(value = row.spielplanMannschaftId, onValueChange = { onChange(row.copy(spielplanMannschaftId = it)) }, label = { Text("Spielplan-Team-ID") }, supportingText = { Text("Eindeutige myTischtennis-Team-ID") }, modifier = Modifier.fillMaxWidth())
-        if (spielplanTeams.isNotEmpty()) {
-            Box(modifier = Modifier.fillMaxWidth()) {
-                OutlinedButton(onClick = { teamPickerOpen = true }, modifier = Modifier.fillMaxWidth()) {
-                    Text("Aus importierten Teams auswählen")
-                }
-                DropdownMenu(expanded = teamPickerOpen, onDismissRequest = { teamPickerOpen = false }) {
-                    spielplanTeams.forEach { team ->
-                        DropdownMenuItem(
-                            text = { Text(team.label) },
-                            onClick = {
-                                onChange(row.copy(spielplanMannschaftId = team.id))
-                                teamPickerOpen = false
-                            },
-                        )
-                    }
-                }
-            }
-        }
         OutlinedTextField(value = row.liga, onValueChange = { onChange(row.copy(liga = it)) }, label = { Text("Liga") }, modifier = Modifier.fillMaxWidth())
         OutlinedTextField(value = row.staffelleiter, onValueChange = { onChange(row.copy(staffelleiter = it)) }, label = { Text("Staffelleiter") }, modifier = Modifier.fillMaxWidth())
         OutlinedTextField(value = row.telefon, onValueChange = { onChange(row.copy(telefon = it)) }, label = { Text("Telefon") }, modifier = Modifier.fillMaxWidth())
@@ -787,35 +752,6 @@ private fun MannschaftEditorCard(
     }
 }
 
-private data class ImportedSpielplanTeam(val id: String, val label: String)
-
-private fun importedSpielplanTeams(headers: List<String>, rows: List<List<String>>): List<ImportedSpielplanTeam> {
-    fun index(name: String) = headers.indexOf(name)
-    fun value(row: List<String>, column: String): String {
-        val columnIndex = index(column)
-        return row.getOrElse(columnIndex) { "" }.trim()
-    }
-
-    val teams = linkedMapOf<String, ImportedSpielplanTeam>()
-    listOf("Heim", "Gast").forEach { side ->
-        rows.forEach { row ->
-            val isHarheimer = value(row, "${side}VereinNr") == "43030" || value(row, "${side}VereinName") == "Harheimer TC"
-            val id = value(row, "${side}MannschaftId")
-            if (!isHarheimer || id.isBlank()) return@forEach
-
-            val teamName = value(row, "${side}Mannschaft")
-            val ageClass = value(row, "${side}MannschaftAltersklasse")
-            val teamNumber = value(row, "${side}MannschaftNr")
-            val label = listOf(teamName, ageClass, teamNumber.takeIf { it.isNotBlank() }?.let { "Nr. $it" })
-                .filterNotNull()
-                .filter { it.isNotBlank() }
-                .joinToString(" · ")
-            teams.putIfAbsent(id, ImportedSpielplanTeam(id, "$label [$id]"))
-        }
-    }
-    return teams.values.sortedBy { it.label }
-}
-
 private fun sportSpielplanCsvText(headers: List<String>, rows: List<List<String>>): String {
     if (headers.isEmpty()) return ""
     return listOf(headers).plus(rows).joinToString("\n") { row -> row.joinToString(";") { it.csvCell(";") } }

android-app/app/src/main/java/de/harheimertc/ui/screens/cms/CmsViewModels.kt

@@ -322,33 +322,6 @@ class CmsViewModel @Inject constructor(
         }
     }
 
-    fun importSportSpielplan() {
-        viewModelScope.launch {
-            _state.value = _state.value.copy(sportSaving = true, error = null, message = null)
-            repository.importSpielplan()
-                .onSuccess { response ->
-                    if (!response.success) {
-                        _state.value = _state.value.copy(
-                            sportSaving = false,
-                            error = response.message ?: "Spielplan konnte nicht importiert werden.",
-                        )
-                        return@onSuccess
-                    }
-                    _state.value = _state.value.copy(
-                        sportSaving = false,
-                        message = response.message ?: "Spielplan aktualisiert.",
-                    )
-                    loadSportbetrieb()
-                }
-                .onFailure { err ->
-                    _state.value = _state.value.copy(
-                        sportSaving = false,
-                        error = ErrorMapper.mapError(err) ?: "Spielplan konnte nicht importiert werden.",
-                    )
-                }
-        }
-    }
-
     fun saveSportSpielplan(headers: List<String>, rows: List<List<String>>) {
         viewModelScope.launch {
             _state.value = _state.value.copy(sportSaving = true, error = null, message = null)