Skip to content

Fjern tynne wrapper-services (pt. 2)#851

Open
FredrikMeyer wants to merge 2 commits into
mainfrom
simplify/fjern-tynne-wrapper-services-ny
Open

Fjern tynne wrapper-services (pt. 2)#851
FredrikMeyer wants to merge 2 commits into
mainfrom
simplify/fjern-tynne-wrapper-services-ny

Conversation

@FredrikMeyer

Copy link
Copy Markdown
Contributor

Beskrivelse

Implementerer punkt 2 fra ARCHITECTURE.md: fjerner PersonService, SakService og PostmottakBehandlingService som kun videresender til et repository.

Endringer

Logikken er flyttet direkte inn i repository-klassene som default interface-metoder:

  • IPersonRepository.hentEllerLagre(ident) — fra PersonService.hentEllerLagrePerson
  • SakRepository.hentEllerSettInn(person, saksnummer, sakStatus) — fra SakService.hentEllerSettInnSak
  • PostmottakBehandlingRepository.oppdaterEllerOpprett(behandling) — fra PostmottakBehandlingService.oppdaterEllerOpprettBehandling

Gevinst

  • 3 klasser slettet (PersonService.kt, SakService.kt, PostmottakBehandlingService.kt)
  • Færre lag i call-stacken

⚠️ Denne PR-en ble gjenskapt etter en force push. Kan ha konflikter som trenger manuell gjennomgang.

FredrikMeyer and others added 2 commits May 27, 2026 12:42
Sletter seksjonene for #3, #6 og #7 (alle ferdig).
Renummererer gjenværende tiltak til 1–4.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…handlingService

Flytt logikken inn i repository-klassene:
- IPersonRepository.hentEllerLagre() (fra PersonService.hentEllerLagrePerson)
- SakRepository.hentEllerSettInn() (fra SakService.hentEllerSettInnSak)
- PostmottakBehandlingRepository.oppdaterEllerOpprett() (fra PostmottakBehandlingService.oppdaterEllerOpprettBehandling)

Oppdater alle kallsteder til å bruke repository direkte:
HendelsesService, ResendHendelseService, OppgaveHistorikkLagrer,
LagrePostmottakHendelseJobbUtfører og tilhørende jobb-konstruktører.

Fjerner 3 klasser og 2 sekundærkonstruktører.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@FredrikMeyer FredrikMeyer requested a review from a team as a code owner May 27, 2026 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant