Saksnummer som primær taskparameter fase 1 (#739)

* Saksnummer som primær taskparameter fase 1 * Fikse test
navikt · Nov 29, 2024 · 59bbb1d · 59bbb1d
1 parent 1bf97bf
commit 59bbb1d
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 19 deletions.
diff --git a/task/src/main/java/no/nav/vedtak/felles/prosesstask/api/ProsessTaskData.java b/task/src/main/java/no/nav/vedtak/felles/prosesstask/api/ProsessTaskData.java
@@ -22,11 +22,6 @@ public class ProsessTaskData implements ProsessTaskInfo {
     public static final String MANGLER_PROPS = "PT-492717";
     public static final Pattern VALID_KEY_PATTERN = Pattern.compile("[a-zA-Z0-9_\\.]+$");
 
-    private static final String SAKSNUMMER = "saksnummer";
-    private static final String FAGSAK_ID = "fagsakId";
-    private static final String BEHANDLING_ID = "behandlingId";
-    private static final String AKTØR_ID = "aktørId";
-
     private final Properties props = new Properties();
     private final TaskType taskType;
     private int antallFeiledeForsøk;
@@ -317,30 +312,52 @@ public String getSaksnummer() {
     }
 
     public void setSaksnummer(String saksnummer) {
-        setProperty(CommonTaskProperties.SAKSNUMMER, Objects.requireNonNull(saksnummer, SAKSNUMMER));
+        setProperty(CommonTaskProperties.SAKSNUMMER, Objects.requireNonNull(saksnummer, CommonTaskProperties.SAKSNUMMER));
     }
 
+    /**
+     * Convenience API - setBehandling/setFagsak med flere parametere
+     *
+     * saksnummer   offisielt saksnummer
+     * fagsakId     intern tabell-Id for sak - trengs en stund pga taskrammeverk for saksrekkefølge
+     * behandlingId angitt behandlingId definert av fagsystem (kan være Long, UUID, etc)
+     */
+
+    @Deprecated(forRemoval = true) // Fagsakprosesstask må fikses - PTv2
     public void setBehandling(Long fagsakId, Long behandlingId) {
-        Objects.requireNonNull(fagsakId, FAGSAK_ID);
-        Objects.requireNonNull(behandlingId, BEHANDLING_ID);
+        Objects.requireNonNull(fagsakId, CommonTaskProperties.FAGSAK_ID);
+        Objects.requireNonNull(behandlingId, CommonTaskProperties.BEHANDLING_ID);
+
+        setFagsakId(fagsakId);
+        setBehandlingId(behandlingId.toString());
+    }
+
+    @Deprecated(forRemoval = true) // Fagsakprosesstask må fikses - PTv2
+    public void setBehandling(String saksnummer, Long fagsakId, Long behandlingId) {
+        Objects.requireNonNull(saksnummer, CommonTaskProperties.SAKSNUMMER);
+        Objects.requireNonNull(fagsakId, CommonTaskProperties.FAGSAK_ID);
+        Objects.requireNonNull(behandlingId, CommonTaskProperties.BEHANDLING_ID);
 
+        setSaksnummer(saksnummer);
         setFagsakId(fagsakId);
         setBehandlingId(behandlingId.toString());
     }
 
+    @Deprecated(forRemoval = true) // Impending removal next release
     public void setBehandling(Long fagsakId, Long behandlingId, String aktørId) {
-        Objects.requireNonNull(fagsakId, FAGSAK_ID);
-        Objects.requireNonNull(behandlingId, BEHANDLING_ID);
-        Objects.requireNonNull(aktørId, AKTØR_ID);
+        Objects.requireNonNull(fagsakId, CommonTaskProperties.FAGSAK_ID);
+        Objects.requireNonNull(behandlingId, CommonTaskProperties.BEHANDLING_ID);
+        Objects.requireNonNull(aktørId, CommonTaskProperties.AKTØR_ID);
 
         setFagsakId(fagsakId);
         setBehandlingId(behandlingId.toString());
         setAktørId(aktørId);
     }
 
+    @Deprecated(forRemoval = true) // Impending removal next release
     public void setBehandling(String saksnummer, String behandlingId) {
-        Objects.requireNonNull(saksnummer, SAKSNUMMER);
-        Objects.requireNonNull(behandlingId, BEHANDLING_ID);
+        Objects.requireNonNull(saksnummer, CommonTaskProperties.SAKSNUMMER);
+        Objects.requireNonNull(behandlingId, CommonTaskProperties.BEHANDLING_ID);
 
         setSaksnummer(saksnummer);
         setBehandlingId(behandlingId);
@@ -353,24 +370,35 @@ public void setBehandling(String saksnummer, String behandlingId) {
      * @param behandlingId angitt behandlingId definert av fagsystem (kan være Long, UUID, etc)
      * @param aktørId angitt AktørId gyldig i AktørRegisteret.
      */
+    @Deprecated(forRemoval = true) // Impending removal next release
     public void setBehandling(String saksnummer, String behandlingId, String aktørId) {
-        Objects.requireNonNull(saksnummer, SAKSNUMMER);
-        Objects.requireNonNull(behandlingId, BEHANDLING_ID);
-        Objects.requireNonNull(aktørId, AKTØR_ID);
+        Objects.requireNonNull(saksnummer, CommonTaskProperties.SAKSNUMMER);
+        Objects.requireNonNull(behandlingId, CommonTaskProperties.BEHANDLING_ID);
+        Objects.requireNonNull(aktørId, CommonTaskProperties.AKTØR_ID);
 
         setSaksnummer(saksnummer);
         setBehandlingId(behandlingId);
         setAktørId(aktørId);
     }
 
+    @Deprecated(forRemoval = true) // Impending removal next release
     public void setFagsak(Long fagsakId, String aktørId) {
-        Objects.requireNonNull(fagsakId, FAGSAK_ID);
-        Objects.requireNonNull(aktørId, AKTØR_ID);
+        Objects.requireNonNull(fagsakId, CommonTaskProperties.FAGSAK_ID);
+        Objects.requireNonNull(aktørId, CommonTaskProperties.AKTØR_ID);
 
         setFagsakId(fagsakId);
         setAktørId(aktørId);
     }
 
+    @Deprecated(forRemoval = true) // Fagsakprosesstask må fikses - PTv2
+    public void setFagsak(String saksnummer, Long fagsakId) {
+        Objects.requireNonNull(saksnummer, CommonTaskProperties.SAKSNUMMER);
+        Objects.requireNonNull(fagsakId, CommonTaskProperties.FAGSAK_ID);
+
+        setSaksnummer(saksnummer);
+        setFagsakId(fagsakId);
+    }
+
     public void setOpprettetTid(LocalDateTime opprettetTid) {
         this.opprettetTid = opprettetTid;
     }

diff --git a/task/src/main/java/no/nav/vedtak/felles/prosesstask/api/ProsessTaskGruppe.java b/task/src/main/java/no/nav/vedtak/felles/prosesstask/api/ProsessTaskGruppe.java
@@ -69,10 +69,24 @@ public ProsessTaskGruppe addNesteParallell(Collection<ProsessTaskData> prosessTa
 
     public static record Entry(String sekvens, ProsessTaskData task) {}
 
+    @Deprecated(forRemoval = true) // Immediate next patch
     public void setBehandling(Long fagsakId, Long behandlingId, String aktørId) {
         this.getTasks().forEach(e -> e.task().setBehandling(fagsakId, behandlingId, aktørId));
     }
 
+    @Deprecated(forRemoval = true) // Immediate next patch
+    public void setBehandling(Long fagsakId, Long behandlingId) {
+        this.getTasks().forEach(e -> e.task().setBehandling(fagsakId, behandlingId));
+    }
+
+    public void setBehandling(String saksnummer, Long fagsakId, Long behandlingId) {
+        this.getTasks().forEach(e -> e.task().setBehandling(saksnummer, fagsakId, behandlingId));
+    }
+
+    public void setFagsak(String saksnummer, Long fagsakId) {
+        this.getTasks().forEach(e -> e.task().setFagsak(saksnummer, fagsakId));
+    }
+
     public void setCallId(String callId) {
         setProperty(CallId.CALL_ID, callId);
     }

diff --git a/task/src/test/java/no/nav/vedtak/felles/prosesstask/impl/ProsessTaskRepositoryImplTest.java b/task/src/test/java/no/nav/vedtak/felles/prosesstask/impl/ProsessTaskRepositoryImplTest.java
@@ -118,7 +118,7 @@ private ProsessTaskEntitet lagTestEntitet(ProsessTaskStatus status, LocalDateTim
         data.setSisteFeilKode("feilkode-123");
         data.setSisteFeil("siste-feil");
         data.setAntallFeiledeForsøk(2);
-        data.setBehandling(1L, 2L, "3");
+        data.setBehandling("345", 1L, 2L);
         data.setGruppe("gruppe");
         data.setNesteKjøringEtter(nesteKjøringEtter);
         data.setPrioritet(2);