Skip to content

Commit 8691766

Browse files
di0nys1usdi0nys1us
committed
fjerner time()-funksjon, tror dette kan løses mer sentralt med JDBC-monitorering
fjerner interface for alle stores
1 parent 9c4b76f commit 8691766

File tree

20 files changed

+687
-849
lines changed

20 files changed

+687
-849
lines changed

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/AzureADRoutes.kt‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ import no.nav.hjelpemidler.soknad.db.domain.BehovsmeldingType
1515
import no.nav.hjelpemidler.soknad.db.domain.HotsakTilknytningData
1616
import no.nav.hjelpemidler.soknad.db.domain.OrdrelinjeData
1717
import no.nav.hjelpemidler.soknad.db.domain.PapirSøknadData
18-
import no.nav.hjelpemidler.soknad.db.domain.SoknadData
1918
import no.nav.hjelpemidler.soknad.db.domain.Status
2019
import no.nav.hjelpemidler.soknad.db.domain.StatusMedÅrsak
20+
import no.nav.hjelpemidler.soknad.db.domain.SøknadData
2121
import no.nav.hjelpemidler.soknad.db.domain.VedtaksresultatData
2222
import no.nav.hjelpemidler.soknad.db.exception.feilmelding
2323
import no.nav.hjelpemidler.soknad.db.ktor.søknadId
@@ -38,7 +38,7 @@ fun Route.azureADRoutes(
3838
}
3939

4040
post("/soknad/bruker") {
41-
val søknad = call.receive<SoknadData>()
41+
val søknad = call.receive<SøknadData>()
4242
logg.info { "Digital behovsmelding mottatt for lagring, søknadId: ${søknad.soknadId}" }
4343
val rowsUpdated = transaction { søknadStore.save(søknad) }
4444
call.respond(HttpStatusCode.Created, rowsUpdated)
@@ -183,7 +183,7 @@ fun Route.azureADRoutes(
183183

184184
get("/soknadsdata/bruker/{soknadId}") {
185185
val søknadId = call.søknadId
186-
val søknad = transaction { søknadStore.hentSoknadData(søknadId) }
186+
val søknad = transaction { søknadStore.hentSøknadData(søknadId) }
187187
when (søknad) {
188188
null -> call.feilmelding(HttpStatusCode.NotFound)
189189
else -> call.respond(søknad)
@@ -217,7 +217,7 @@ fun Route.azureADRoutes(
217217

218218
get("/soknad/opprettet-dato/{soknadId}") {
219219
val søknadId = call.søknadId
220-
val opprettetDato = transaction { søknadStore.hentSoknadOpprettetDato(søknadId) }
220+
val opprettetDato = transaction { søknadStore.hentSøknadOpprettetDato(søknadId) }
221221
when (opprettetDato) {
222222
null -> call.feilmelding(HttpStatusCode.NotFound)
223223
else -> call.respond(opprettetDato)

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/Brukerpassbyttedata.kt‎ renamed to ‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/BrukerpassbytteData.kt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package no.nav.hjelpemidler.soknad.db.domain
22

33
import java.util.UUID
44

5-
data class Brukerpassbyttedata(
5+
data class BrukerpassbytteData(
66
val id: UUID,
77
val hjelpemiddel: BytteHjelpemiddel,
88
val bytteårsak: BrukerpassBytteårsak,

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/SoknadData.kt‎ renamed to ‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/SøknadData.kt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import java.util.UUID
66
/**
77
* @see [no.nav.hjelpemidler.behovsmeldingsmodell.BehovsmeldingResponse]
88
*/
9-
data class SoknadData(
9+
data class SøknadData(
1010
val fnrBruker: String,
1111
val navnBruker: String,
1212
val fnrInnsender: String?,

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/SøknadForBruker.kt‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ class SøknadForBruker private constructor(
2222
val fullmakt: Boolean,
2323
val fnrBruker: String,
2424
val søknadsdata: Søknadsdata?,
25-
val brukerpassbyttedata: Brukerpassbyttedata?,
25+
val brukerpassbyttedata: BrukerpassbytteData?,
2626
val er_digital: Boolean,
2727
val soknadGjelder: String?,
2828
var ordrelinjer: List<SøknadForBrukerOrdrelinje>,
@@ -68,7 +68,7 @@ class SøknadForBruker private constructor(
6868
},
6969
brukerpassbyttedata = when (behovsmeldingType) {
7070
BehovsmeldingType.SØKNAD, BehovsmeldingType.BESTILLING, BehovsmeldingType.BYTTE -> null
71-
BehovsmeldingType.BRUKERPASSBYTTE -> jsonMapper.treeToValue<Brukerpassbyttedata>(søknad["brukerpassbytte"])
71+
BehovsmeldingType.BRUKERPASSBYTTE -> jsonMapper.treeToValue<BrukerpassbytteData>(søknad["brukerpassbytte"])
7272
},
7373
er_digital = er_digital,
7474
soknadGjelder = soknadGjelder,

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/SoknadMedStatus.kt‎ renamed to ‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/domain/SøknadMedStatus.kt‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import com.fasterxml.jackson.databind.JsonNode
44
import java.util.Date
55
import java.util.UUID
66

7-
class SoknadMedStatus private constructor(
7+
class SøknadMedStatus private constructor(
88
val soknadId: UUID,
99
val behovsmeldingType: BehovsmeldingType,
1010
val journalpostId: String?,
@@ -30,7 +30,7 @@ class SoknadMedStatus private constructor(
3030
soknadGjelder: String?,
3131
valgteÅrsaker: List<String>,
3232
) =
33-
SoknadMedStatus(
33+
SøknadMedStatus(
3434
soknadId,
3535
behovsmeldingType,
3636
journalpostId,
@@ -57,7 +57,7 @@ class SoknadMedStatus private constructor(
5757
soknadGjelder: String?,
5858
valgteÅrsaker: List<String>,
5959
) =
60-
SoknadMedStatus(
60+
SøknadMedStatus(
6161
soknadId,
6262
behovsmeldingType,
6363
journalpostId,

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/ordre/OrdreService.kt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class OrdreService(
1111
private val grunndataClient: GrunndataClient,
1212
) {
1313
suspend fun finnOrdreForSøknad(søknadId: UUID): List<SøknadForBrukerOrdrelinje> {
14-
val ordrelinjer = transaction { ordreStore.ordreForSoknad(søknadId) }
14+
val ordrelinjer = transaction { ordreStore.finnOrdreForSøknad(søknadId) }
1515
val produkter = grunndataClient
1616
.hentProdukterMedHmsnrs(ordrelinjer.map(SøknadForBrukerOrdrelinje::artikkelNr).toSet())
1717
.groupBy(Product::hmsArtNr)

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/store/Database.kt‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,10 @@ class Database(private val dataSource: DataSource) : Transaction, Closeable {
2828
}
2929

3030
class StoreProvider(tx: JdbcOperations) {
31-
val hotsakStore = HotsakStorePostgres(tx)
32-
val infotrygdStore = InfotrygdStorePostgres(tx)
33-
val ordreStore = OrdreStorePostgres(tx)
34-
val søknadStore = SøknadStorePostgres(tx)
35-
val søknadStoreInnsender = SøknadStoreInnsenderPostgres(tx)
31+
val hotsakStore = HotsakStore(tx)
32+
val infotrygdStore = InfotrygdStore(tx)
33+
val ordreStore = OrdreStore(tx)
34+
val søknadStore = SøknadStore(tx)
35+
val søknadStoreInnsender = SøknadStoreInnsender(tx)
3636
}
3737
}

‎app/src/main/kotlin/no/nav/hjelpemidler/soknad/db/store/HotsakStore.kt‎

Lines changed: 45 additions & 66 deletions
Original file line numberDiff line numberDiff line change
@@ -6,90 +6,69 @@ import no.nav.hjelpemidler.soknad.db.domain.VedtaksresultatHotsakData
66
import java.time.LocalDate
77
import java.util.UUID
88

9-
interface HotsakStore {
10-
fun lagKnytningMellomSakOgSøknad(hotsakTilknytningData: HotsakTilknytningData): Int
11-
fun lagreVedtaksresultat(
12-
søknadId: UUID,
13-
vedtaksresultat: String,
14-
vedtaksdato: LocalDate,
15-
): Int
16-
17-
fun hentVedtaksresultatForSøknad(søknadId: UUID): VedtaksresultatHotsakData?
18-
fun hentSøknadsIdForHotsakNummer(saksnummer: String): UUID?
19-
fun harVedtakForSøknadId(søknadId: UUID): Boolean
20-
fun hentFagsakIdForSøknad(søknadId: UUID): String?
21-
}
9+
class HotsakStore(private val tx: JdbcOperations) {
10+
fun lagKnytningMellomSakOgSøknad(hotsakTilknytningData: HotsakTilknytningData): Int =
11+
tx.update(
12+
"""
13+
INSERT INTO v1_hotsak_data (soknads_id, saksnummer)
14+
VALUES (:soknadId, :saksnummer)
15+
ON CONFLICT DO NOTHING
16+
""".trimIndent(),
17+
mapOf(
18+
"soknadId" to hotsakTilknytningData.søknadId,
19+
"saksnummer" to hotsakTilknytningData.saksnr,
20+
),
21+
).actualRowCount
2222

23-
class HotsakStorePostgres(private val tx: JdbcOperations) : HotsakStore {
24-
override fun lagKnytningMellomSakOgSøknad(hotsakTilknytningData: HotsakTilknytningData): Int =
25-
time("insert_knytning_mellom_søknad_og_hotsak") {
26-
tx.update(
27-
"""
28-
INSERT INTO v1_hotsak_data (soknads_id, saksnummer)
29-
VALUES (:soknadId, :saksnummer)
30-
ON CONFLICT DO NOTHING
31-
""".trimIndent(),
32-
mapOf(
33-
"soknadId" to hotsakTilknytningData.søknadId,
34-
"saksnummer" to hotsakTilknytningData.saksnr,
35-
),
36-
).actualRowCount
37-
}
38-
39-
override fun lagreVedtaksresultat(
23+
fun lagreVedtaksresultat(
4024
søknadId: UUID,
4125
vedtaksresultat: String,
4226
vedtaksdato: LocalDate,
43-
): Int =
44-
time("oppdater_vedtaksresultat_fra_hotsak") {
45-
tx.update(
46-
"""
47-
UPDATE v1_hotsak_data
48-
SET vedtaksresultat = :vedtaksresultat,
49-
vedtaksdato = :vedtaksdato
50-
WHERE soknads_id = :soknadId
51-
""".trimIndent(),
52-
mapOf(
53-
"vedtaksresultat" to vedtaksresultat,
54-
"vedtaksdato" to vedtaksdato,
55-
"soknadId" to søknadId,
56-
),
57-
).actualRowCount
58-
}
27+
): Int = tx.update(
28+
"""
29+
UPDATE v1_hotsak_data
30+
SET vedtaksresultat = :vedtaksresultat,
31+
vedtaksdato = :vedtaksdato
32+
WHERE soknads_id = :soknadId
33+
""".trimIndent(),
34+
mapOf(
35+
"vedtaksresultat" to vedtaksresultat,
36+
"vedtaksdato" to vedtaksdato,
37+
"soknadId" to søknadId,
38+
),
39+
).actualRowCount
5940

60-
override fun hentVedtaksresultatForSøknad(søknadId: UUID): VedtaksresultatHotsakData? {
61-
return time("hent_søknadid_fra_resultat") {
62-
tx.singleOrNull(
63-
"""
64-
SELECT soknads_id, saksnummer, vedtaksresultat, vedtaksdato
65-
FROM v1_hotsak_data
66-
WHERE soknads_id = :soknadId
67-
""".trimIndent(),
68-
mapOf("soknadId" to søknadId),
69-
) {
70-
VedtaksresultatHotsakData(
71-
søknadId = it.uuid("soknads_id"),
72-
saksnr = it.string("saksnummer"),
73-
vedtaksresultat = it.stringOrNull("vedtaksresultat"),
74-
vedtaksdato = it.localDateOrNull("vedtaksdato"),
75-
)
76-
}
41+
fun hentVedtaksresultatForSøknad(søknadId: UUID): VedtaksresultatHotsakData? {
42+
return tx.singleOrNull(
43+
"""
44+
SELECT soknads_id, saksnummer, vedtaksresultat, vedtaksdato
45+
FROM v1_hotsak_data
46+
WHERE soknads_id = :soknadId
47+
""".trimIndent(),
48+
mapOf("soknadId" to søknadId),
49+
) {
50+
VedtaksresultatHotsakData(
51+
søknadId = it.uuid("soknads_id"),
52+
saksnr = it.string("saksnummer"),
53+
vedtaksresultat = it.stringOrNull("vedtaksresultat"),
54+
vedtaksdato = it.localDateOrNull("vedtaksdato"),
55+
)
7756
}
7857
}
7958

80-
override fun hentSøknadsIdForHotsakNummer(saksnummer: String): UUID? =
59+
fun hentSøknadsIdForHotsakNummer(saksnummer: String): UUID? =
8160
tx.singleOrNull(
8261
"SELECT soknads_id FROM v1_hotsak_data WHERE saksnummer = :saksnummer",
8362
mapOf("saksnummer" to saksnummer),
8463
) { it.uuid("soknads_id") }
8564

86-
override fun harVedtakForSøknadId(søknadId: UUID): Boolean =
65+
fun harVedtakForSøknadId(søknadId: UUID): Boolean =
8766
tx.singleOrNull(
8867
"SELECT 1 FROM v1_hotsak_data WHERE soknads_id = :soknadId AND vedtaksresultat IS NOT NULL",
8968
mapOf("soknadId" to søknadId),
9069
) { true } ?: false
9170

92-
override fun hentFagsakIdForSøknad(søknadId: UUID): String? =
71+
fun hentFagsakIdForSøknad(søknadId: UUID): String? =
9372
tx.singleOrNull(
9473
"SELECT saksnummer FROM v1_hotsak_data WHERE soknads_id = :soknadId",
9574
mapOf("soknadId" to søknadId),

0 commit comments

Comments
 (0)