Fjern FRISINN fra køer (for k9sak) #2463
Conversation
| @@ -69,6 +69,10 @@ class OppgaveV3( | |||
| } | |||
| return oppgavefelter | |||
| } | |||
|
|
|||
| fun gjelderFRISINN( oppgave : OppgaveV3) : Boolean { | |||
There was a problem hiding this comment.
Ikke helt ideélt å ha frisinn (eller ytelsespesifikke ting) på den generiske OppgaveV3, men alternativet blir fort en del mer kode.
F.eks at vi har et felt på oppgaven som settes av adapterne, som avgjør om oppgaven skal skjules i sin helhet i los. Et flagg inn i sjekkDuplikatOgProsesser kunne også vært en mulighet, men tror feltdefinisjoner er det som er mest riktig med tanke på hvordan resten av los virker.
There was a problem hiding this comment.
Jeg har forsøkt et kompromiss nå ved å flytte koden fra OppgaveV3 til FrisinnUtil
Jeg tror dette er den beste løsningen inntil alle hendelsene for FRISINN slettes fra LOS, siden vi da er helt sikre på at de ikke dukker opp i køene, ikke påvirker ytelse for køene, og er enkelt å forstå (tror jeg)
There was a problem hiding this comment.
Ja, enig, det er et greit kompromiss. Det hadde nok blitt unødvendig mye kode med den løsningen jeg foreslo
| @@ -37,7 +37,12 @@ class OppgaveV3Repository( | |||
| val oppgaveId = nyOppgaveversjon(oppgave, nyVersjon, tx) | |||
| lagreFeltverdier(oppgaveId, oppgave, tx) | |||
|
|
|||
| AktivOppgaveRepository.ajourholdAktivOppgave(oppgave, nyVersjon, tx) | |||
| val ignorerForKøer = OppgaveV3.gjelderFRISINN(oppgave) && oppgave.oppgavetype.eksternId == "k9sak" | |||
There was a problem hiding this comment.
Frisinn er ikke i los for noen av systemene. Det blir behandlet av en annen enhet som ikke bruker los
There was a problem hiding this comment.
Nice, har fjernet k9sak-sjekken nå
|
Jeg vil anta at dette vil fungere fint, men hadde det vært mulig å slette eventene med FRISINN i stedet? Så slipper vi å ha referanser til dette i koden til evig tid. |
No description provided.