Skip to content
Snippets Groups Projects
Commit 6456501c authored by Jan Zickermann's avatar Jan Zickermann
Browse files

Revert "OZG-6344 Use required default form id"

This reverts commit a6bf69a6.
parent 62164426
No related branches found
No related tags found
No related merge requests found
......@@ -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) {
......
......@@ -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) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment