OZG-2566 OZG-2684 tiny cleanup on kommentar; adjust FIELD_PATH_PREFIX in...

OZG-2566 OZG-2684 tiny cleanup on kommentar; adjust FIELD_PATH_PREFIX in client for validation messages

goofy-client/libs/kommentar/src/lib/kommentar-list-in-vorgang-container/kommentar-form/kommentar.formservice.ts

@@ -8,7 +8,8 @@ import { Observable } from "rxjs";
 export class KommentarFormService extends AbstractFormService {
 
 	static readonly TEXT = 'text';
-	static readonly FIELD_PATH_PREFIX = 'command.body';
+
+	static readonly FIELD_PATH_PREFIX = 'kommentar';
 
 	constructor(formBuilder: FormBuilder, private kommentarService: KommentarService) {
 		super(formBuilder);

goofy-client/libs/wiedervorlage/src/lib/wiedervorlage-page-container/wiedervorlage-page/wiedervorlage-form/wiedervorlage.formservice.ts

@@ -12,7 +12,7 @@ export class WiedervorlageFormService extends AbstractFormService {
 	static readonly FIELD_FRIST = 'frist';
 	static readonly FIELD_ATTACHMENTS = 'attachments';
 
-	static readonly FIELD_PATH_PREFIX = 'command.body';
+	static readonly FIELD_PATH_PREFIX = 'wiedervorlage';
 
 	constructor(
 		formBuilder: FormBuilder,

goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarCommandController.java

@@ -25,7 +25,7 @@ public class KommentarCommandController {
 	@PostMapping
 	public ResponseEntity<Void> editKommentar(@RequestBody KommentarCommand kommentarCommand, @PathVariable String kommentarId,
 			@PathVariable long kommentarVersion) {
-		var createdCommand = service.editKommentar(kommentarCommand, kommentarId, kommentarVersion);
+		var createdCommand = service.editKommentar(kommentarCommand.getKommentar(), kommentarId, kommentarVersion);
 
 		return buildResponseLink(createdCommand);
 	}
@@ -45,7 +45,7 @@ public class KommentarCommandController {
 
 		@PostMapping
 		public ResponseEntity<Void> createKommentar(@RequestBody KommentarCommand command, @PathVariable String vorgangId) {
-			var createdCommand = service.createKommentar(command, vorgangId);
+			var createdCommand = service.createKommentar(command.getKommentar(), vorgangId);
 
 			return ResponseEntity.created(linkTo(CommandController.class).slash(createdCommand.getId()).toUri()).build();
 		}

goofy-server/src/main/java/de/itvsh/goofy/kommentar/KommentarService.java

@@ -24,8 +24,8 @@ class KommentarService {
 	@Autowired
 	private VorgangAttachedItemService vorgangAttachedItemService;
 
-	public Command createKommentar(@Valid KommentarCommand kommentarCommand, String vorgangId) {
-		return vorgangAttachedItemService.createNewKommentar(addCreated(kommentarCommand.getKommentar()), vorgangId);
+	public Command createKommentar(@Valid Kommentar kommentar, String vorgangId) {
+		return vorgangAttachedItemService.createNewKommentar(addCreated(kommentar), vorgangId);
 	}
 
 	Kommentar addCreated(Kommentar kommentar) {
@@ -35,9 +35,9 @@ class KommentarService {
 				.build();
 	}
 
-	public Command editKommentar(@Valid KommentarCommand kommentarCommand, String kommentarId, long kommentarVersion) {
+	public Command editKommentar(@Valid Kommentar kommentar, String kommentarId, long kommentarVersion) {
 		var loadedKommentar = getById(kommentarId);
-		var preparedKommentar = loadedKommentar.toBuilder().text(kommentarCommand.getKommentar().getText()).build();
+		var preparedKommentar = loadedKommentar.toBuilder().text(kommentar.getText()).build();
 
 		return vorgangAttachedItemService.editKommentar(preparedKommentar, kommentarId, kommentarVersion);
 	}

goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandControllerTest.java

@@ -53,7 +53,7 @@ class KommentarCommandControllerTest {
 		void shouldCallService() throws Exception {
 			doRequest();
 
-			verify(service).createKommentar(any(KommentarCommand.class), eq(VorgangHeaderTestFactory.ID));
+			verify(service).createKommentar(any(Kommentar.class), eq(VorgangHeaderTestFactory.ID));
 		}
 
 		@Test
@@ -93,7 +93,7 @@ class KommentarCommandControllerTest {
 		void shouldCallService() throws Exception {
 			doRequest();
 
-			verify(service).editKommentar(any(KommentarCommand.class), eq(KommentarTestFactory.ID), eq(KommentarTestFactory.VERSION));
+			verify(service).editKommentar(any(Kommentar.class), eq(KommentarTestFactory.ID), eq(KommentarTestFactory.VERSION));
 		}
 
 		@Test

goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarCommandITCase.java

@@ -61,8 +61,6 @@ class KommentarCommandITCase {
 		@Nested
 		class TestValidation {
 
-			private static final String FIELD_PREFIX = "kommentarCommand.kommentar";
-
 			@DisplayName("for null Text")
 			@Test
 			void createCommandWithInvalidText() throws Exception {
@@ -70,7 +68,7 @@ class KommentarCommandITCase {
 
 				doRequestByKommentarId(content).andExpect(status().isUnprocessableEntity())
 						.andExpect(jsonPath("$.issues.length()").value(1))
-						.andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"))
+						.andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"))
 						.andExpect(jsonPath("$.issues.[0].messageCode").value(ValidationMessageCodes.FIELD_IS_EMPTY));
 			}
 
@@ -80,7 +78,7 @@ class KommentarCommandITCase {
 				String content = buildContentWithText(StringUtils.EMPTY);
 
 				doRequestByKommentarId(content).andExpect(status().isUnprocessableEntity())
-						.andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"));
+						.andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"));
 
 			}
 
@@ -132,8 +130,6 @@ class KommentarCommandITCase {
 		@Nested
 		class TestValidation {
 
-			private static final String FIELD_PREFIX = "kommentarCommand.kommentar";
-
 			@DisplayName("for null Text")
 			@Test
 			void createCommandWithInvalidText() throws Exception {
@@ -141,7 +137,7 @@ class KommentarCommandITCase {
 
 				doRequestByVorgangId(content).andExpect(status().isUnprocessableEntity())
 						.andExpect(jsonPath("$.issues.length()").value(1))
-						.andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"))
+						.andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"))
 						.andExpect(jsonPath("$.issues.[0].messageCode").value(ValidationMessageCodes.FIELD_IS_EMPTY));
 			}
 
@@ -151,7 +147,7 @@ class KommentarCommandITCase {
 				String content = buildContentWithText(StringUtils.EMPTY);
 
 				doRequestByVorgangId(content).andExpect(status().isUnprocessableEntity())
-						.andExpect(jsonPath("$.issues.[0].field").value(FIELD_PREFIX + ".text"));
+						.andExpect(jsonPath("$.issues.[0].field").value("kommentar.text"));
 
 			}
 

goofy-server/src/test/java/de/itvsh/goofy/kommentar/KommentarServiceTest.java

@@ -97,7 +97,7 @@ class KommentarServiceTest {
 		}
 
 		private Command callCreateKommentar() {
-			return service.createKommentar(KommentarCommandTestFactory.create(), VorgangHeaderTestFactory.ID);
+			return service.createKommentar(KommentarTestFactory.create(), VorgangHeaderTestFactory.ID);
 		}
 	}
 
@@ -137,7 +137,7 @@ class KommentarServiceTest {
 		}
 
 		private Command callEditKommentar() {
-			return service.editKommentar(KommentarCommandTestFactory.create(), KommentarTestFactory.ID, KommentarTestFactory.VERSION);
+			return service.editKommentar(KommentarTestFactory.create(), KommentarTestFactory.ID, KommentarTestFactory.VERSION);
 		}
 	}