From 6456501c643f506c61a645c2e889c012b3c549bc Mon Sep 17 00:00:00 2001
From: Jan Zickermann <jan.zickermann@dataport.de>
Date: Thu, 5 Sep 2024 14:53:38 +0200
Subject: [PATCH] Revert "OZG-6344 Use required default form id"
This reverts commit a6bf69a6dacbaeb7fa1d00b57a9903f9ef843f73.
---
.../DFoerdermittelEngineBasedSemantikAdapter.java | 8 +++-----
.../DFoerdermittelEngineBasedSemantikAdapterTest.java | 9 +++++----
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/semantik-adapter/src/main/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapter.java b/semantik-adapter/src/main/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapter.java
index a89ba548f..cdba687ee 100644
--- a/semantik-adapter/src/main/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapter.java
+++ b/semantik-adapter/src/main/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapter.java
@@ -42,7 +42,6 @@ public class DFoerdermittelEngineBasedSemantikAdapter implements EngineBasedSema
private static final String KEY_FORM_ID = "ProjectTitle";
static final String DEFAULT_FORM_NAME = "dFördermittelantrag";
- static final String DEFAULT_FORM_ID = "dFördermittelantrag";
@Autowired
private ServiceKontoFactory serviceKontoFactory;
@@ -73,10 +72,9 @@ public class DFoerdermittelEngineBasedSemantikAdapter implements EngineBasedSema
}
FormData addFormId(FormData formData) {
- return mapWithModifiedHeader(formData, headerBuilder -> headerBuilder.formId(
- getNonEmptyFachnachrichtValueByKey(formData, KEY_FORM_ID)
- .orElse(DEFAULT_FORM_ID)
- ));
+ return getNonEmptyFachnachrichtValueByKey(formData, KEY_FORM_ID)
+ .map(formId -> mapWithModifiedHeader(formData, headerBuilder -> headerBuilder.formId(formId)))
+ .orElse(formData);
}
private Optional<String> getNonEmptyFachnachrichtValueByKey(FormData formData, String key) {
diff --git a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java
index 2445fe3b2..110e80b7d 100644
--- a/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java
+++ b/semantik-adapter/src/test/java/de/ozgcloud/eingang/semantik/enginebased/dfoerdermittel/DFoerdermittelEngineBasedSemantikAdapterTest.java
@@ -22,6 +22,7 @@ import de.ozgcloud.common.binaryfile.TempFileUtils;
import de.ozgcloud.common.test.TestUtils;
import de.ozgcloud.eingang.common.formdata.FormData;
import de.ozgcloud.eingang.common.formdata.FormDataTestFactory;
+import de.ozgcloud.eingang.common.formdata.FormHeaderTestFactory;
import de.ozgcloud.eingang.common.formdata.IncomingFile;
import de.ozgcloud.eingang.common.formdata.PostfachAddressTestFactory;
import de.ozgcloud.eingang.common.formdata.ServiceKontoTestFactory;
@@ -86,7 +87,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
@Nested
class HandleFachnachrichtData {
- private final Map<String, Object> extracted = Map.of("name", "Theo");
+ private Map<String, Object> extracted = Map.of("name", "Theo");
@Test
void shouldCallExtractData() {
@@ -185,7 +186,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
void shouldHaveServiceKonto() {
var formData = adapter.addServiceKonto(DFoerdermittelFormDataTestFactory.create());
- assertThat(formData.getHeader().getServiceKonto().getPostfachAddresses().getFirst().getIdentifier())
+ assertThat(formData.getHeader().getServiceKonto().getPostfachAddresses().get(0).getIdentifier())
.asInstanceOf(type(StringBasedIdentifier.class)).extracting(StringBasedIdentifier::getPostfachId)
.isEqualTo(PostfachAddressTestFactory.POSTFACH_ID);
}
@@ -276,7 +277,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
void shouldKeepFormIdIfProjectTitleIsEmpty() {
var formData = adapter.addFormId(createFormDataWithProjectTitle(""));
- assertThat(formData.getHeader().getFormId()).isEqualTo(DEFAULT_FORM_ID);
+ assertThat(formData.getHeader().getFormId()).isEqualTo(FormHeaderTestFactory.FORM_ID);
}
@DisplayName("should keep form id if project title is null")
@@ -284,7 +285,7 @@ class DFoerdermittelEngineBasedSemantikAdapterTest {
void shouldKeepFormIdIfProjectTitleIsNull() {
var formData = adapter.addFormId(createFormDataWithProjectTitle(null));
- assertThat(formData.getHeader().getFormId()).isEqualTo(DEFAULT_FORM_ID);
+ assertThat(formData.getHeader().getFormId()).isEqualTo(FormHeaderTestFactory.FORM_ID);
}
FormData createFormDataWithProjectTitle(String projectTitle) {
--
GitLab