Creepso/NuvioStreaming
1
0
Fork 0
mirror of https://github.com/tapframe/NuvioStreaming.git synced 2026-08-05 11:01:37 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(library): preserve leading articles in title sorting for natural alphabetical order

Browse source
This commit is contained in:
Sobhan 2026-07-27 19:09:05 +05:30
parent 88d3cbdfac
commit cfa49a2cda
2 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
composeApp/src/commonMain/kotlin/com/nuvio/app/features/library/LibraryDisplaySettings.kt
View file

@ -227,12 +227,8 @@ private val LibraryDisplaySettingsJson = Json {
encodeDefaults = true
}
private val LeadingLibraryTitleArticle = Regex("^(the|an|a)\\s+", RegexOption.IGNORE_CASE)
private fun libraryTitleSortKey(item: LibraryItem): String =
libraryTitleTieBreakKey(item)
.trim()
.replace(LeadingLibraryTitleArticle, "")
private fun libraryTitleTieBreakKey(item: LibraryItem): String =
item.name

6
composeApp/src/commonTest/kotlin/com/nuvio/app/features/library/LibraryDisplaySettingsTest.kt
View file

@ -51,7 +51,7 @@ class LibraryDisplaySettingsTest {
}
@Test
fun `title sorting ignores leading English articles`() {
fun `title sorting preserves leading articles for literal alphabetical order`() {
val input = listOf(
item("batman", name = "The Batman"),
item("arrival", name = "Arrival"),
@ -59,11 +59,11 @@ class LibraryDisplaySettingsTest {
)
assertEquals(
listOf("arrival", "batman", "quiet"),
listOf("quiet", "arrival", "batman"),
sortLibraryItems(input, LibrarySortOption.TITLE_ASC, LibrarySourceMode.LOCAL).map { it.id },
)
assertEquals(
listOf("quiet", "batman", "arrival"),
listOf("batman", "arrival", "quiet"),
sortLibraryItems(input, LibrarySortOption.TITLE_DESC, LibrarySourceMode.LOCAL).map { it.id },
)
}