Backup_Repos/komga
1
0
Fork 0
mirror of https://github.com/gotson/komga.git synced 2026-05-08 12:35:30 +02:00
Code Issues Projects Releases Wiki Activity

perf: don't sort books and series by default

Browse source
This commit is contained in:
Gauthier Roebroeck 2022-07-15 22:23:44 +08:00
parent 43a1fc7a0a
commit 31c89fc298
4 changed files with 12 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
komga/src/main/kotlin/org/gotson/komga/interfaces/api/rest/BookController.kt
View file

@ -122,7 +122,7 @@ class BookController(
when { when {
page.sort.isSorted -> page.sort page.sort.isSorted -> page.sort
!searchTerm.isNullOrBlank() -> Sort.by("relevance") !searchTerm.isNullOrBlank() -> Sort.by("relevance")
else -> Sort.by(Sort.Order.asc("metadata.title")) else -> Sort.unsorted()
} }
val pageRequest = val pageRequest =

2
komga/src/main/kotlin/org/gotson/komga/interfaces/api/rest/SeriesController.kt
View file

@ -144,7 +144,7 @@ class SeriesController(
when { when {
page.sort.isSorted -> page.sort page.sort.isSorted -> page.sort
!searchTerm.isNullOrBlank() -> Sort.by("relevance") !searchTerm.isNullOrBlank() -> Sort.by("relevance")
else -> Sort.by(Sort.Order.asc("metadata.titleSort")) else -> Sort.unsorted()
} }
val pageRequest = val pageRequest =

10
komga/src/test/kotlin/org/gotson/komga/interfaces/api/rest/BookControllerTest.kt
View file

@ -177,7 +177,7 @@ class BookControllerTest(
mockMvc.get("/api/v1/books/${book15.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${book15.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books") mockMvc.get("/api/v1/books?sort=metadata.title")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -235,7 +235,7 @@ class BookControllerTest(
mockMvc.get("/api/v1/books/${book10.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${book10.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books/${book15.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${book15.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books") mockMvc.get("/api/v1/books?sort=metadata.title")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -293,7 +293,7 @@ class BookControllerTest(
mockMvc.get("/api/v1/books/${bookAdult.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${bookAdult.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books") mockMvc.get("/api/v1/books?sort=metadata.title")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -351,7 +351,7 @@ class BookControllerTest(
mockMvc.get("/api/v1/books/${bookAdult.id}").andExpect { status { isOk() } } mockMvc.get("/api/v1/books/${bookAdult.id}").andExpect { status { isOk() } }
mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isOk() } } mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isOk() } }
mockMvc.get("/api/v1/books") mockMvc.get("/api/v1/books?sort=metadata.title")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -409,7 +409,7 @@ class BookControllerTest(
mockMvc.get("/api/v1/books/${bookAdult.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${bookAdult.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/books/${book.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/books") mockMvc.get("/api/v1/books?sort=metadata.title")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }

10
komga/src/test/kotlin/org/gotson/komga/interfaces/api/rest/SeriesControllerTest.kt
View file

@ -290,7 +290,7 @@ class SeriesControllerTest(
mockMvc.get("/api/v1/series/${series15.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${series15.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series") mockMvc.get("/api/v1/series?sort=metadata.titleSort")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -344,7 +344,7 @@ class SeriesControllerTest(
mockMvc.get("/api/v1/series/${series16.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${series16.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series/${series18.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${series18.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series") mockMvc.get("/api/v1/series?sort=metadata.titleSort")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -398,7 +398,7 @@ class SeriesControllerTest(
mockMvc.get("/api/v1/series/${seriesAdult.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${seriesAdult.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series") mockMvc.get("/api/v1/series?sort=metadata.titleSort")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -452,7 +452,7 @@ class SeriesControllerTest(
mockMvc.get("/api/v1/series/${seriesAdult.id}").andExpect { status { isOk() } } mockMvc.get("/api/v1/series/${seriesAdult.id}").andExpect { status { isOk() } }
mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isOk() } } mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isOk() } }
mockMvc.get("/api/v1/series") mockMvc.get("/api/v1/series?sort=metadata.titleSort")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }
@ -506,7 +506,7 @@ class SeriesControllerTest(
mockMvc.get("/api/v1/series/${seriesAdult.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${seriesAdult.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isForbidden() } } mockMvc.get("/api/v1/series/${series.id}").andExpect { status { isForbidden() } }
mockMvc.get("/api/v1/series") mockMvc.get("/api/v1/series?sort=metadata.titleSort")
.andExpect { .andExpect {
status { isOk() } status { isOk() }
jsonPath("$.content.length()") { value(2) } jsonPath("$.content.length()") { value(2) }