feat: yki_suoritus: sort columns #831
Conversation
saku-koodari
force-pushed
the
feat/frontend-sort
branch
7 times, most recently
from
5a55afc to
99af61f
Compare
| <th>Yleisarvosana</th> | ||
| {{#header}} | ||
| <th> | ||
| <a href="?{{search}}&{{includeVersionHistory}}&{{sortColumn}}&{{sortDirection}}"> |
There was a problem hiding this comment.
Pitäisikö sivutuslinkeillä olla myös nämä uudet parametrit?
There was a problem hiding this comment.
Tehtävänannossa ei ollut periaatteessa määritystä tälle :D
server/src/main/kotlin/fi/oph/kitu/yki/suoritukset/YkiSuoritusRepository.kt
Show resolved
Hide resolved
| data class Column( | ||
| val databaseColumn: String, | ||
| val uiValue: String, | ||
| ) |
There was a problem hiding this comment.
Tämän voisi ehkä toteuttaa aavistuksen siistimmin Enum classeilla. Yhtenä etuna ainakin, että String -> EnumClass muunnos onnistuu EnumClass.valueOf(..):lla hieman kivemmin (vrt. nykyisen toteutuksen ykiSuoritusColumns.first { .. }).
Hyvällä tuurilla myös YkiViewController:n /suoritukset GET-mapping pystyisi käyttämään enumia parametrina merkkijonon sijasta, jolloin saataisiin parametrin validointi melko ilmaisena lisänä. En ole varma miten enum classit käyttäytyvät tuollaisessa tilanteessa, joten en osaa kokeilematta sanoa saako Spring mankeloitua asiat oikeaan muotoon.
There was a problem hiding this comment.
Se SortDirection on jo nyt enum class, joten sen pitäisi onnistua
saku-koodari
force-pushed
the
feat/frontend-sort
branch
from
a0285fd to
aea2867
Compare
fix wrong column
saku-koodari
force-pushed
the
feat/frontend-sort
branch
from
a52a529 to
90f0e34
Compare
| distinct: Boolean, | ||
| limit: Int?, | ||
| offset: Int?, | ||
| ): Iterable<YkiSuoritusEntity> { | ||
| val searchStr = "%$searchBy%" | ||
|
|
||
| val columnName = ykiSuoritusColumns.first { it.databaseColumn == orderBy }.databaseColumn |
There was a problem hiding this comment.
Vai olisko siistimpi, jos first olis find?
No description provided.