Fikse limit-bug i Oppgaver (#1256)

navikt · Mar 6, 2023 · 135a143 · 135a143
1 parent 00cc14f
commit 135a143
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 24 deletions.
diff --git a/...ient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/AbstractOppgaveKlient.java b/...ient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/AbstractOppgaveKlient.java
@@ -40,39 +40,28 @@ public Oppgave opprettetOppgave(OpprettOppgave oppgave) {
         return restKlient.send(request, Oppgave.class);
     }
 
-    @Override
-    public List<Oppgave> finnÅpneOppgaver(String aktørId, String tema, List<String> oppgaveTyper) {
-        return hentOppgaverFor(aktørId, tema, oppgaveTyper, null, null);
-    }
-
     @Override
     public List<Oppgave> finnÅpneOppgaverAvType(Oppgavetype oppgaveType, String aktørId, String enhetsNr, String limit) {
         Objects.requireNonNull(oppgaveType, "Oppgvetype er påkrevd");
-        return hentOppgaverFor(aktørId, TEMA_FORELDREPENGER, List.of(oppgaveType.getKode()), null, null);
-    }
-
-    @Override
-    public List<Oppgave> finnÅpneOppgaverForEnhet(String tema, List<String> oppgaveTyper, String tildeltEnhetsnr, String limit) {
-        return hentOppgaverFor(null, tema, oppgaveTyper, tildeltEnhetsnr, limit);
+        return hentOppgaverFor(aktørId, List.of(oppgaveType.getKode()), enhetsNr, limit);
     }
 
     @Override
     public List<Oppgave> finnÅpneOppgaver(List<String> oppgaveTyper, String aktørId, String enhetsNr, String limit) {
-        return hentOppgaverFor(aktørId, TEMA_FORELDREPENGER, oppgaveTyper, enhetsNr, limit);
+        return hentOppgaverFor(aktørId, oppgaveTyper, enhetsNr, limit);
     }
 
-    private List<Oppgave> hentOppgaverFor(String aktørId, String tema, List<String> oppgaveTyper, String tildeltEnhetsnr, String limit) {
-        var builder = UriBuilder.fromUri(restConfig.endpoint());
+    private List<Oppgave> hentOppgaverFor(String aktørId, List<String> oppgaveTyper, String tildeltEnhetsnr, String limit) {
+        var builder = UriBuilder.fromUri(restConfig.endpoint())
+            .queryParam("tema", AbstractOppgaveKlient.TEMA_FORELDREPENGER)
+            .queryParam("statuskategori", STATUSKATEGORI_AAPEN);
+
         if (aktørId != null) {
             builder.queryParam("aktoerId", aktørId);
         }
-        if (tema != null) {
-            builder.queryParam("tema", tema);
-        }
         if (tildeltEnhetsnr != null) {
             builder.queryParam("tildeltEnhetsnr", tildeltEnhetsnr);
         }
-        builder.queryParam("statuskategori", STATUSKATEGORI_AAPEN);
         //settes dersom man ønsker flere enn default limit på 10 fra api
         if (limit != null) {
             builder.queryParam("limit", limit);

diff --git a/...pgave-rest-klient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/Oppgaver.java b/...pgave-rest-klient/src/main/java/no/nav/vedtak/felles/integrasjon/oppgave/v1/Oppgaver.java
@@ -12,16 +12,10 @@ public interface Oppgaver {
 
     void ferdigstillOppgave(String oppgaveId);
 
-    @Deprecated(forRemoval = true)
-    List<Oppgave> finnÅpneOppgaver(String aktørId, String tema, List<String> oppgaveTyper);
-
     // Henter åpne oppgaver med Tema-kode Foreldrepenger for gitt Oppgavetyper.
     // Øvrige argumenter kan være null,  men hvis det ventes mange oppgaver så sett aktør, enhet eller limit
     List<Oppgave> finnÅpneOppgaverAvType(Oppgavetype oppgaveType, String aktørId, String enhetsNr, String limit);
 
-    @Deprecated(forRemoval = true)
-    List<Oppgave> finnÅpneOppgaverForEnhet(String tema, List<String> oppgaveTyper, String enhetsNr, String limit);
-
     // Henter åpne oppgaver med Tema-kode Foreldrepenger for gitte Opgavetyper-kodeverk (kan være tom).
     // Øvrige argumenter kan være null, men hvis det ventes mange oppgaver så sett aktør, enhet eller limit
     List<Oppgave> finnÅpneOppgaver(List<String> oppgaveTyper, String aktørId, String enhetsNr, String limit);