#108 events vereinheitlichen

[peter-mueller]

Beschreibung:

Die Verwendung des EventBus wurde stark vereinfacht, dazu wurde ein Model eingeführt, das die Momentanen Daten speichert:

public class BuergerModel implements BuergerReadOnlyModel {
    /** All the kinders of the selectedBuerger. */
    private final BeanItemContainer<Buerger> selectedBuergerKinder = new BeanItemContainer<>(Buerger.class);
    /** All the partners of the selectedBuerger. */
    private final BeanItemContainer<Buerger> selectedBuergerPartner = new BeanItemContainer<>(Buerger.class);
    /** A List of all the Buergers, possible reduced by the query. */
    private final BeanItemContainer<Buerger> buergers = new BeanItemContainer<>(Buerger.class);
    /** The current (single or none) selected Buerger in the GUI. */
    private Optional<Buerger> selectedBuerger = Optional.empty();
    /** The query to filter the buergers. */
    private Optional<String> query = Optional.empty();

}

Im EventBus wurden die einfachen Keys (=Selector) und die Keys für Entitäten (Request und Response) modelliert:

.
├── EventBus.java
├── events
│   ├── Association.java
│   └── Eventable.java
└── selector
    ├── entity
    │   ├── BaseEntityKey.java
    │   ├── RequestEntityKey.java
    │   ├── RequestEvent.java
    │   └── ResponseEntityKey.java
    ├── Keyable.java
    └── Key.java

Auditing geht noch nicht, und wird erst nach dem PR gefixt, ein Ticket dazu wird dann erstellt. thanks @FabianHoltkoetter

Branch-Checklist:

• [x] Doku im Code vollständig
• [ ] Im Wiki dokumentiert
• [x] MicroService getestet
• [x] GUI getestet
• [x] Barakuda Issue erstellt
• [x] Auditing angepasst

  Bestätigungen:

• [x] @xdoo Model :ok:
• [x] @FabianHoltkoetter
• [x] @darenegade EventBus :ok:
• [x] @Arne2 BuergerGrid :ok:

  Referenz:

closes #108

[Arne2]

GenericGrid kann so nicht bleiben, entweder entfernst du es ganz, oder du führst es wieder zurück so wie es gedacht war. Du kannst einfach eine Klasse übergeben dafür ist die entityClass da. Jetzt jetzt hardgecoded mit der entityClass Bürger drin. Denke daran die ViewFactory zu überarbeiten und wenn die GenericGrid woanders noch verwendet wird bitte auch.

[FabianWilms]

Momentan wird die Fehlermeldung, wenn ein Bürger nicht gelöscht werden konnte, da er in einem anderen Bürger referenziert wird, nicht angezeigt. (Erro 409 abfangen)

[peter-mueller]

@FabianHoltkoetter Das GenericGrid fliegt raus, oder? Fehlermeldungen sollen über #95 gelöst werden.

[FabianWilms]

So war der Plan. Es wird ja sowieso generiert, und bevor wir uns jetzt mit einer "generischeren" Lösung aufhalten würde ich es rauswerfen.

[peter-mueller]

@FabianHoltkoetter Momentan geh ich davon aus dass die Relation immer vom gleichen Typ ist:

        @SuppressWarnings("unchecked") final Association<Buerger> association = (Association<Buerger>) event.getData();
        final Buerger buerger;
        if (association.getAssociation().getId() == null) {
            buerger = service.create(association.getAssociation());
        } else {
            buerger = association.getAssociation();
        }

        final Buerger.Rel rel = Buerger.Rel.valueOf(association.getRel());
        if (Buerger.Rel.kinder == rel)
            addBuergerKind(buerger);
        if (Buerger.Rel.partner == rel)
            setBuergerPartner(buerger);
...
}

Ich würde das fürs generieren so ändern, dass man erst die Relation prüft und dann casted ?!

[FabianWilms]

@peter-mueller auf jedenfall. Ein Beispiel wäre ja schon die Wohnung eines Bürgers wo das der Fall sein könnte.

[darenegade]

• Nach einem Create oder Update sollte nicht nochmal die komplette Liste der Entitäten vom Service geholt werden (Optimierung des ResponseKey) :white_check_mark: neu geladen wird jetzt nur noch beim view change
• Der ViewDatastore (bei Peter: Modell) sollte den gleichen Container haben wie die Komponenten, sodass man sich ein paar Events spart und doppelte Listenführung. Damit darf die Liste natürlich im Datastore nicht neu gesetzt werden dürfen (Singleton) :white_check_mark:
• Die GenericGrid fliegt nicht raus, sondern muss nur richtig implementiert werden (#140) Wir könnten zwar alles generieren, aber die Templates sollten einfach gehalten werden, sodass wir nicht immer den gleichen fixen Code in alle Dateinen rein generieren. Sonst könnten wir uns ja alle Libs sparen und alles in eine Main reinknallen.

[FabianWilms]

@peter-mueller Bitte die Nutzung des Eventbus mit neuem Key/Model-Ansatz im Wiki dokumentieren