OZG-7872 OZG-8039 add related resource link on postfach

alfa-service/src/main/java/de/ozgcloud/alfa/common/command/CommandModelAssembler.java

@@ -25,6 +25,7 @@ package de.ozgcloud.alfa.common.command;
 
 import static org.springframework.hateoas.server.mvc.WebMvcLinkBuilder.*;
 
+import java.util.List;
 import java.util.function.Predicate;
 import java.util.stream.Stream;
 
@@ -33,7 +34,6 @@ import org.springframework.hateoas.EntityModel;
 import org.springframework.hateoas.Link;
 import org.springframework.hateoas.LinkRelation;
 import org.springframework.hateoas.server.RepresentationModelAssembler;
-import org.springframework.hateoas.server.mvc.WebMvcLinkBuilder;
 import org.springframework.stereotype.Component;
 
 import de.ozgcloud.alfa.bescheid.BescheidController;
@@ -41,10 +41,11 @@ import de.ozgcloud.alfa.bescheid.DocumentController;
 import de.ozgcloud.alfa.collaboration.CollaborationController.CollaborationByVorgangController;
 import de.ozgcloud.alfa.forwarding.ForwardingController;
 import de.ozgcloud.alfa.kommentar.KommentarController;
-import de.ozgcloud.alfa.postfach.PostfachMailController;
 import de.ozgcloud.alfa.vorgang.VorgangController;
 import de.ozgcloud.alfa.wiedervorlage.WiedervorlageController;
+import lombok.RequiredArgsConstructor;
 
+@RequiredArgsConstructor
 @Component
 class CommandModelAssembler implements RepresentationModelAssembler<Command, EntityModel<Command>> {
 
@@ -57,33 +58,61 @@ class CommandModelAssembler implements RepresentationModelAssembler<Command, Ent
 
 	static final Predicate<Command> HAS_KNOWN_COMMAND_ORDER = command -> command.getCommandOrder() != CommandOrder.UNBEKANNT;
 
+	private static final List<String> NOT_PROCESSABLE_ORDER = List.of(
+			CommandOrder.RECEIVE_POSTFACH_NACHRICHT.name(),
+			CommandOrder.RESEND_POSTFACH_MAIL.name(),
+			CommandOrder.SEND_POSTFACH_MAIL.name(),
+			CommandOrder.SEND_POSTFACH_NACHRICHT.name(),
+			CommandOrder.RESEND_POSTFACH_MAIL.name(),
+			CommandOrder.RECEIVE_POSTFACH_NACHRICHT.name());
+
+	static final Predicate<Command> IS_PROCESSABLE_ORDER = command -> !NOT_PROCESSABLE_ORDER.contains(command.getOrder());
+
+	private final List<CommandProcessor> processors;
+
 	@Override
 	public EntityModel<Command> toModel(Command command) {
-		return EntityModel.of(command)
+		var entityModel = EntityModel.of(command)
 				.add(linkTo(CommandController.class).slash(command.getId()).withSelfRel())
-				.addIf(CommandHelper.IS_DONE.and(HAS_KNOWN_COMMAND_ORDER).test(command), () -> effectedResourceLinkByOrderType(command))
+				.addIf(CommandHelper.IS_DONE.and(HAS_KNOWN_COMMAND_ORDER).and(IS_PROCESSABLE_ORDER).test(command),
+						() -> effectedResourceLinkByOrderType(command))
 				.addIf(CommandHelper.IS_PENDING.test(command), () -> linkTo(CommandController.class).slash(command.getId()).withRel(REL_UPDATE))
 				.addIf(IS_NOT_LOESCH_ANFORDERUNG_AND_REVOKEABLE.test(command),
 						() -> linkTo(methodOn(CommandController.class).revoke(command.getId(), null)).withRel(REL_REVOKE));
+
+		processors.forEach(processor -> processor.process(entityModel));
+
+		return entityModel;
 	}
 
 	Link effectedResourceLinkByOrderType(Command entity) {
 		var type = entity.getCommandOrder().getType();
 
-		WebMvcLinkBuilder linkBuilder = switch (type) {
-			case FORWARDING -> linkTo(methodOn(ForwardingController.class).findByVorgangId(entity.getVorgangId()));
-			case KOMMENTAR -> linkTo(KommentarController.class).slash(entity.getRelationId());
-			case POSTFACH -> linkTo(methodOn(PostfachMailController.class).getAll(entity.getVorgangId()));
-			case VORGANG -> linkTo(VorgangController.class).slash(entity.getRelationId());
-			case VORGANG_LIST -> linkTo(VorgangController.class);
-			case WIEDERVORLAGE -> linkTo(WiedervorlageController.class).slash(entity.getRelationId());
-			case BESCHEID -> linkTo(methodOn(BescheidController.class).getDraft(entity.getVorgangId()));
-			case DOCUMENT -> linkTo(DocumentController.class).slash(entity.getCreatedResource());
-			case COLLABORATION -> linkTo(methodOn(CollaborationByVorgangController.class).getAllByVorgangId(entity.getVorgangId()));
-			case NONE -> throw new IllegalArgumentException("Unknown CommandOrder: " + entity.getOrder());
-		};
-
-		return linkBuilder.withRel(REL_EFFECTED_RESOURCE);
+		if (type == CommandOrder.Type.FORWARDING) {
+			return linkTo(methodOn(ForwardingController.class).findByVorgangId(entity.getVorgangId())).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.KOMMENTAR) {
+			return linkTo(KommentarController.class).slash(entity.getRelationId()).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.VORGANG) {
+			return linkTo(VorgangController.class).slash(entity.getRelationId()).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.VORGANG_LIST) {
+			return linkTo(VorgangController.class).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.WIEDERVORLAGE) {
+			return linkTo(WiedervorlageController.class).slash(entity.getRelationId()).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.BESCHEID) {
+			return linkTo(methodOn(BescheidController.class).getDraft(entity.getVorgangId())).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.DOCUMENT) {
+			return linkTo(DocumentController.class).slash(entity.getCreatedResource()).withRel(REL_EFFECTED_RESOURCE);
+		}
+		if (type == CommandOrder.Type.COLLABORATION) {
+			return linkTo(methodOn(CollaborationByVorgangController.class).getAllByVorgangId(entity.getVorgangId())).withRel(REL_EFFECTED_RESOURCE);
+		}
+		throw new IllegalArgumentException("Unknown CommandOrder: " + entity.getOrder());
 	}
 
 	public CollectionModel<EntityModel<Command>> toCollectionModel(Stream<Command> entities) {

[Felix Reichenbach]

Gibt es einen Grund warum das hier manuell aufgerufen wird und nicht von Spring gemacht wird? Wenn die jeweiligen Processoren RepresentationModelProcessor<EntityModel<Command>> implementieren, kümmert sich Spring darum.

[Martin Küster]

Ja, das hat mehrere Gründe. Dadurch das es explizit aufgerufen wird kann man es einfacher verstehen und nachvollziehen und zum anderen brauchen wir dann nicht immer die null Abfrage auf den Content der ja aktuell auch immer wieder dupliziert wird in den Processoren.

[Felix Reichenbach]

In CommandProcessor gibt es doch trotzdem die Nullabfrage. Ich finde es überraschend in einer Spring Boot Anwendung nicht die Spring Boot Features zu benutzen die es gibt. Ich bin aber auch nicht dagegen es explizit zu machen, weil es hier nicht viel Arbeit ist

[Martin Küster]

Die gibt es jetzt aber nur einmal zentral im CommandProcessor, nicht in jedem fachlichen Prozessor. Es kann in die ein oder andere Richtung überraschen, das ist denke ich Geschmackssache. (hab da auch schon etwas länger mit Tobias drüber philosophiert)

[Felix Reichenbach]

Nur mal so aus Neugier, warum wurde das hier umgestellt? Die alte Variante mit dem switch case hatte ja den Vorteil, dass man nciht immer wieder withRel(...) schreiben muss.

[Martin Küster]

Beim switch case musst du jeden Fall behandeln, deshalb habe ich das auf ifs umgestellt. Jetzt kann man die jeweiligen fachlichen Prozessoren nach und nach aufbauen und hier einfach den jeweiligen if part rausnehmen.

[Felix Reichenbach]

Man muss nicht jeden Fall behandeln wenn man einen default case hat

[Martin Küster]

Ja, das stimmt, ich fand:

case NONE, POSTFACH -> throw new IllegalArgumentException("Unknown CommandOrder: " + entity.getOrder());

Nur etwas irritierend :/