Skip to content

Commit 6d0a2be

Browse files
committed
Refaktor av sammenligne feltverdier, fjerner privat
1 parent 71828d9 commit 6d0a2be

File tree

1 file changed

+13
-15
lines changed

1 file changed

+13
-15
lines changed

src/main/kotlin/no/nav/k9/los/nyoppgavestyring/mottak/oppgave/OppgaveV3PartisjonertRepository.kt

+13-15
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,19 @@ class OppgaveV3PartisjonertRepository(val oppgavetypeRepository: OppgavetypeRepo
3939
eksisterendeFelter: List<OppgaveFeltverdi>,
4040
nyeFelter: List<OppgaveFeltverdi>
4141
): Boolean {
42-
return eksisterendeFelter.size != nyeFelter.size ||
43-
!eksisterendeFelter.map { FeltverdiNøkkel(it) }.containsAll(nyeFelter.map { FeltverdiNøkkel(it) }) ||
44-
!nyeFelter.map { FeltverdiNøkkel(it) }.containsAll(eksisterendeFelter.map { FeltverdiNøkkel(it) })
42+
val mapSammenlignbareVerdier = { verdi: OppgaveFeltverdi ->
43+
Triple(
44+
verdi.oppgavefelt.feltDefinisjon.eksternId,
45+
verdi.verdi,
46+
verdi.verdiBigInt
47+
)
48+
}
49+
50+
val eksisterende = eksisterendeFelter.map(mapSammenlignbareVerdier)
51+
val nye = nyeFelter.map(mapSammenlignbareVerdier)
52+
return eksisterendeFelter.size != nyeFelter.size ||
53+
!eksisterende.containsAll(nye) ||
54+
!nye.containsAll(eksisterende)
4555
}
4656

4757
private fun hentOppgave(
@@ -180,16 +190,4 @@ class OppgaveV3PartisjonertRepository(val oppgavetypeRepository: OppgavetypeRepo
180190
).asUpdate,
181191
)
182192
}
183-
184-
private data class FeltverdiNøkkel(
185-
val feltdefinisjonEksternId: String,
186-
val verdi: String,
187-
val verdiBigInt: Long?
188-
) {
189-
constructor(fv: OppgaveFeltverdi) : this(
190-
feltdefinisjonEksternId = fv.oppgavefelt.feltDefinisjon.eksternId,
191-
verdi = fv.verdi,
192-
verdiBigInt = fv.verdiBigInt
193-
)
194-
}
195193
}

0 commit comments

Comments
 (0)