Merge pull request #397 from fmasa/fix-career-dupl

fmasa · web-flow · commit aa75a5511ed6 · 2025-01-06T23:42:35.000+01:00
Handle careers with duplicate career level IDs
diff --git a/common/src/commonMain/kotlin/cz/frantisekmasa/wfrp_master/common/core/ui/forms/CareerSelectBox.kt b/common/src/commonMain/kotlin/cz/frantisekmasa/wfrp_master/common/core/ui/forms/CareerSelectBox.kt
@@ -184,7 +184,7 @@ private fun CareerChooserDialog(
                         icon = Resources.Drawable.Career,
                     )
                 },
-                key = { it.first.levelId },
+                key = { it.first.careerId to it.first.levelId },
             ) { (level, label) ->
                 ListItem(
                     modifier =

Original file line number	Diff line number	Diff line change
`@@ -184,7 +184,7 @@ private fun CareerChooserDialog(`
`184`	`184`	`icon = Resources.Drawable.Career,`
`185`	`185`	`)`
`186`	`186`	`},`
`187`		`- key = { it.first.levelId },`
	`187`	`+ key = { it.first.careerId to it.first.levelId },`
`188`	`188`	`) { (level, label) ->`
`189`	`189`	`ListItem(`
`190`	`190`	`modifier =`