From 59bbb1d7252f2895227ffdfefaca0f3d802a0466 Mon Sep 17 00:00:00 2001 From: Jens-Otto Larsen <46576810+jolarsen@users.noreply.github.com> Date: Fri, 29 Nov 2024 09:03:29 +0100 Subject: [PATCH] =?UTF-8?q?Saksnummer=20som=20prim=C3=A6r=20taskparameter?= =?UTF-8?q?=20fase=201=20(#739)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Saksnummer som primær taskparameter fase 1 * Fikse test --- .../prosesstask/api/ProsessTaskData.java | 64 +++++++++++++------ .../prosesstask/api/ProsessTaskGruppe.java | 14 ++++ .../impl/ProsessTaskRepositoryImplTest.java | 2 +- 3 files changed, 61 insertions(+), 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 index 933524ca..8bc12ff4 100644 --- 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 index 409b11f1..0c377411 100644 --- 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 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 index d05a01dd..1494a7f7 100644 --- 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);