Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fjerne ubrukt kode, parameter, metode, variabler og retter opp på noe bruk av deprecated metoder #1390

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,4 @@ enum class Saksbehandlingstype {
ORDINÆR,
AUTOMATISK_IKKE_INNKREVING_LAVT_BELØP,
AUTOMATISK_IKKE_INNKREVING_UNDER_4X_RETTSGEBYR,
;

fun erAutomatisk() = this == AUTOMATISK_IKKE_INNKREVING_LAVT_BELØP || this == AUTOMATISK_IKKE_INNKREVING_UNDER_4X_RETTSGEBYR
}
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ import no.nav.familie.tilbake.behandlingskontroll.domain.Behandlingsstegstilstan
import no.nav.familie.tilbake.behandlingskontroll.domain.Venteårsak
import no.nav.familie.tilbake.common.exceptionhandler.Feil
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.FeatureToggleService
import no.nav.familie.tilbake.datavarehus.saksstatistikk.BehandlingTilstandService
import no.nav.familie.tilbake.dokumentbestilling.manuell.brevmottaker.ManuellBrevmottakerRepository
import no.nav.familie.tilbake.historikkinnslag.HistorikkTaskService
Expand All @@ -35,7 +34,6 @@ class BehandlingskontrollService(
private val behandlingTilstandService: BehandlingTilstandService,
private val kravgrunnlagRepository: KravgrunnlagRepository,
private val historikkTaskService: HistorikkTaskService,
private val featureToggleService: FeatureToggleService,
private val brevmottakerRepository: ManuellBrevmottakerRepository,
) {
private val log = LoggerFactory.getLogger(this::class.java)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ enum class Behandlingssteg(
sekvens: Int,
brevmottakerErstatterVerge: Boolean = false,
): Behandlingssteg {
for (behandlingssteg in values()) {
for (behandlingssteg in entries) {
if (sekvens == behandlingssteg.sekvens) {
return when (behandlingssteg) {
BREVMOTTAKER, VERGE -> if (brevmottakerErstatterVerge) BREVMOTTAKER else VERGE
Expand All @@ -82,7 +82,7 @@ enum class Behandlingssteg(
}

fun fraNavn(navn: String): Behandlingssteg {
return values().firstOrNull { it.name == navn }
return entries.firstOrNull { it.name == navn }
?: throw IllegalArgumentException("Ukjent Behandlingssteg $navn")
}
}
Expand Down
50 changes: 0 additions & 50 deletions src/main/kotlin/no/nav/familie/tilbake/common/RessursUtils.kt

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,12 @@ import no.nav.familie.tilbake.behandling.domain.Fagsak
import no.nav.familie.tilbake.behandlingskontroll.domain.Behandlingssteg
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.Constants
import no.nav.familie.tilbake.config.FeatureToggleService
import no.nav.familie.tilbake.dokumentbestilling.DistribusjonshåndteringService
import no.nav.familie.tilbake.dokumentbestilling.manuell.brevmottaker.ManuellBrevmottakerRepository
import no.nav.familie.tilbake.dokumentbestilling.manuellAdresse
import no.nav.familie.tilbake.dokumentbestilling.somBrevmottager
import no.nav.familie.tilbake.dokumentbestilling.vedtak.Vedtaksbrevgrunnlag
import no.nav.familie.tilbake.organisasjon.OrganisasjonService
import org.slf4j.LoggerFactory
import org.springframework.stereotype.Service
import java.util.UUID

Expand All @@ -26,10 +24,7 @@ class BrevmetadataUtil(
private val manuelleBrevmottakerRepository: ManuellBrevmottakerRepository,
private val eksterneDataForBrevService: EksterneDataForBrevService,
private val organisasjonService: OrganisasjonService,
private val featureToggleService: FeatureToggleService,
) {
private val logger = LoggerFactory.getLogger(this::class.java)

fun genererMetadataForBrev(
behandlingId: UUID,
vedtaksbrevgrunnlag: Vedtaksbrevgrunnlag? = null,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
package no.nav.familie.tilbake.dokumentbestilling.felles

import no.nav.familie.kontrakter.felles.tilbakekreving.Vergetype
import no.nav.familie.tilbake.behandling.domain.Behandling
import no.nav.familie.tilbake.behandling.domain.Fagsak
import no.nav.familie.kontrakter.felles.tilbakekreving.Verge as VergeDto
import no.nav.familie.tilbake.behandling.domain.Verge as DomainVerge

Expand Down Expand Up @@ -44,17 +42,4 @@ object BrevmottagerUtil {
verge?.navn ?: ""
}
}

fun utledBrevmottager(
behandling: Behandling,
fagsak: Fagsak,
): Brevmottager {
return if (behandling.harVerge) {
Brevmottager.VERGE
} else if (fagsak.institusjon != null) {
Brevmottager.INSTITUSJON
} else {
Brevmottager.BRUKER
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,7 @@ import no.nav.familie.prosessering.AsyncTaskStep
import no.nav.familie.prosessering.TaskStepBeskrivelse
import no.nav.familie.prosessering.domene.Task
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.FagsakRepository
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.FeatureToggleService
import no.nav.familie.tilbake.config.PropertyName
import org.springframework.stereotype.Component
import java.time.LocalDateTime
Expand All @@ -26,8 +24,6 @@ import java.util.UUID
class SendHenleggelsesbrevTask(
private val henleggelsesbrevService: HenleggelsesbrevService,
private val behandlingRepository: BehandlingRepository,
private val fagsakRepository: FagsakRepository,
private val featureToggleService: FeatureToggleService,
) : AsyncTaskStep {
override fun doTask(task: Task) {
val taskdata: SendBrevTaskdata = objectMapper.readValue(task.payload)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,10 @@ import no.nav.familie.prosessering.AsyncTaskStep
import no.nav.familie.prosessering.TaskStepBeskrivelse
import no.nav.familie.prosessering.domene.Task
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.FagsakRepository
import no.nav.familie.tilbake.behandlingskontroll.BehandlingskontrollService
import no.nav.familie.tilbake.behandlingskontroll.domain.Venteårsak
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.Constants
import no.nav.familie.tilbake.config.FeatureToggleService
import no.nav.familie.tilbake.config.PropertyName
import no.nav.familie.tilbake.oppgave.OppgaveTaskService
import org.springframework.stereotype.Service
Expand All @@ -33,8 +31,6 @@ class InnhentDokumentasjonbrevTask(
private val innhentDokumentasjonBrevService: InnhentDokumentasjonbrevService,
private val behandlingskontrollService: BehandlingskontrollService,
private val oppgaveTaskService: OppgaveTaskService,
private val fagsakRepository: FagsakRepository,
private val featureToggleService: FeatureToggleService,
) : AsyncTaskStep {
override fun doTask(task: Task) {
val taskdata: InnhentDokumentasjonbrevTaskdata = objectMapper.readValue(task.payload)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,6 @@
package no.nav.familie.tilbake.dokumentbestilling.manuell.brevmottaker

import no.nav.familie.kontrakter.felles.dokdist.AdresseType
import no.nav.familie.kontrakter.felles.dokdist.AdresseType.norskPostadresse
import no.nav.familie.kontrakter.felles.dokdist.AdresseType.utenlandskPostadresse
import no.nav.familie.kontrakter.felles.dokdist.ManuellAdresse
import no.nav.familie.kontrakter.felles.historikkinnslag.Aktør
import no.nav.familie.kontrakter.felles.tilbakekreving.MottakerType.BRUKER_MED_UTENLANDSK_ADRESSE
import no.nav.familie.tilbake.api.dto.ManuellBrevmottakerRequestDto
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.FagsakService
Expand Down Expand Up @@ -210,27 +205,3 @@ class ManuellBrevmottakerService(
}
}
}

private fun findAdresseType(brevmottaker: ManuellBrevmottaker): AdresseType {
return when {
brevmottaker.landkode == "NO" && brevmottaker.type != BRUKER_MED_UTENLANDSK_ADRESSE -> norskPostadresse
brevmottaker.landkode != "NO" && brevmottaker.type == BRUKER_MED_UTENLANDSK_ADRESSE -> utenlandskPostadresse
else -> throw Feil("landkode stemmer ikke overens med type for brevmottaker ${brevmottaker.id}")
}
}

fun List<ManuellBrevmottaker>.toManuelleAdresser(): List<ManuellAdresse> =
this.mapNotNull { manuellBrevmottaker ->
if (manuellBrevmottaker.hasManuellAdresse()) {
ManuellAdresse(
adresseType = findAdresseType(manuellBrevmottaker),
adresselinje1 = manuellBrevmottaker.adresselinje1,
adresselinje2 = manuellBrevmottaker.adresselinje2,
postnummer = manuellBrevmottaker.postnummer,
poststed = manuellBrevmottaker.poststed,
land = manuellBrevmottaker.landkode!!,
)
} else {
null
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@ import no.nav.familie.prosessering.AsyncTaskStep
import no.nav.familie.prosessering.TaskStepBeskrivelse
import no.nav.familie.prosessering.domene.Task
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.FagsakRepository
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.FeatureToggleService
import org.springframework.stereotype.Service
import java.util.UUID

Expand All @@ -20,8 +18,6 @@ import java.util.UUID
class SendVarselbrevTask(
private val varselbrevService: VarselbrevService,
private val behandlingRepository: BehandlingRepository,
private val fagsakRepository: FagsakRepository,
private val featureToggleService: FeatureToggleService,
) : AsyncTaskStep {
override fun doTask(task: Task) {
val behandlingId = UUID.fromString(task.payload)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,10 @@ import no.nav.familie.prosessering.AsyncTaskStep
import no.nav.familie.prosessering.TaskStepBeskrivelse
import no.nav.familie.prosessering.domene.Task
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.FagsakRepository
import no.nav.familie.tilbake.behandlingskontroll.BehandlingskontrollService
import no.nav.familie.tilbake.behandlingskontroll.domain.Venteårsak
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.Constants
import no.nav.familie.tilbake.config.FeatureToggleService
import no.nav.familie.tilbake.config.PropertyName
import no.nav.familie.tilbake.dokumentbestilling.brevmaler.Dokumentmalstype
import no.nav.familie.tilbake.oppgave.OppgaveTaskService
Expand All @@ -33,8 +31,6 @@ class SendManueltVarselbrevTask(
private val manueltVarselBrevService: ManueltVarselbrevService,
private val behandlingskontrollService: BehandlingskontrollService,
private val oppgaveTaskService: OppgaveTaskService,
private val fagsakRepository: FagsakRepository,
private val featureToggleService: FeatureToggleService,
) : AsyncTaskStep {
override fun doTask(task: Task) {
val taskdata: SendManueltVarselbrevTaskdata = objectMapper.readValue(task.payload)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,7 @@ internal object AvsnittUtil {

private fun parseUnderavsnittstype(tekst: String): Underavsnittstype? {
val rest = Vedtaksbrevsfritekst.fjernFritekstmarkering(tekst)
return Underavsnittstype.values().firstOrNull { it.name == rest }
return Underavsnittstype.entries.firstOrNull { it.name == rest }
}

private fun erOverskrift(tekst: String): Boolean {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import no.nav.familie.tilbake.behandling.domain.Behandlingstype
import no.nav.familie.tilbake.behandling.domain.Behandlingsårsakstype
import no.nav.familie.tilbake.behandling.domain.Saksbehandlingstype
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.FeatureToggleService
import no.nav.familie.tilbake.config.PropertyName
import no.nav.familie.tilbake.datavarehus.saksstatistikk.SendVedtaksoppsummeringTilDvhTask
import no.nav.familie.tilbake.iverksettvedtak.task.AvsluttBehandlingTask
Expand All @@ -31,7 +30,6 @@ class SendVedtaksbrevTask(
private val fagsakRepository: FagsakRepository,
private val vedtaksbrevService: VedtaksbrevService,
private val taskService: TaskService,
private val featureToggleService: FeatureToggleService,
) : AsyncTaskStep {
private val log = LoggerFactory.getLogger(this::class.java)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ package no.nav.familie.tilbake.dokumentbestilling.vedtak
import no.nav.familie.tilbake.api.dto.FritekstavsnittDto
import no.nav.familie.tilbake.api.dto.HentForhåndvisningVedtaksbrevPdfDto
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.FagsakRepository
import no.nav.familie.tilbake.behandling.domain.Behandling
import no.nav.familie.tilbake.common.repository.findByIdOrThrow
import no.nav.familie.tilbake.config.FeatureToggleConfig
Expand All @@ -24,7 +23,6 @@ class VedtaksbrevService(
private val vedtaksbrevgrunnlagService: VedtaksbrevgunnlagService,
private val faktaRepository: FaktaFeilutbetalingRepository,
private val vilkårsvurderingRepository: VilkårsvurderingRepository,
private val fagsakRepository: FagsakRepository,
private val vedtaksbrevsoppsummeringRepository: VedtaksbrevsoppsummeringRepository,
private val vedtaksbrevsperiodeRepository: VedtaksbrevsperiodeRepository,
private val pdfBrevService: PdfBrevService,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import no.nav.familie.prosessering.internal.TaskService
import no.nav.familie.tilbake.api.forvaltning.Forvaltningsinfo
import no.nav.familie.tilbake.behandling.BehandlingRepository
import no.nav.familie.tilbake.behandling.BehandlingsvedtakService
import no.nav.familie.tilbake.behandling.HentFagsystemsbehandlingService
import no.nav.familie.tilbake.behandling.domain.Behandling
import no.nav.familie.tilbake.behandling.domain.Behandlingsresultat
import no.nav.familie.tilbake.behandling.domain.Behandlingsresultatstype
Expand Down Expand Up @@ -60,7 +59,6 @@ class ForvaltningService(
private val oppgaveTaskService: OppgaveTaskService,
private val tellerService: TellerService,
private val taskService: TaskService,
private val hentFagsystemsbehandlingService: HentFagsystemsbehandlingService,
private val endretKravgrunnlagEventPublisher: EndretKravgrunnlagEventPublisher,
) {
private val logger: Logger = LoggerFactory.getLogger(this.javaClass)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ object KravgrunnlagUtil {
}

fun tilYtelsestype(fagområdekode: String): Ytelsestype {
return Ytelsestype.values().firstOrNull { it.kode == fagområdekode }
return Ytelsestype.entries.firstOrNull { it.kode == fagområdekode }
?: throw IllegalArgumentException("Ukjent Ytelsestype for $fagområdekode")
}

Expand Down
Loading