GRUD_DEV-1247/refactor group display value

[smnhgn]

Submit a pull request

Related Ticket: GRUD_DEV-1247

Please make sure the following is true

• I gave the PR a meaningful name
• I checked that the correct target branch is selected
• I rebased the branch on the target branch and it can be merged
• I ran the linter and it did pass
• I checked for unused code / dead code / debug code
• I checked that variables/functions have meaningful names
• I checked that the behaviour is as the documentation/task describes and I tested it
• I updated the docs / specifications if possible
• I could explain all that code when someone wakes me up at 3am
• I checked that the code considers failures and not just the happy path
• There are no new dependencies OR I listed them and explained them below
• PR introduces no breaking changes OR I listed them and described them below
• I added/updated tests for new/modified unit-testable functions/helpers
• I ran the tests and they did pass

Other information/comments (e.g. reasons why points are not checked from above)

Reason for this PR

Setzt auf dem PR #572 Link-Attribute auf.

Erweitert die GroupCell (Tabelle und Preview) und GroupView (EntityView) um eine alternative Anzeige von Boolean-Werten.

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

_{Tip: disable this comment in your organization's Code Review settings.}

[hermann-p]

Ich bin mir nicht sicher, ob das mit deinem PR reingekommen ist. Wenn du eine Gruppenzelle mit mehreren Booleans auswählst (z.B. "Boolean eins / Boolean zwei" in dem Dump den du mir geschickt hat), diese Zelle kopiert und in eine andere Gruppenzelle pastest, dann werden nicht alle Updates korrekt durchgezogen (manchmal sind nicht beide Boolean-Checkboxen getoggelt, manchmal nicht alle Gruppenwerte aktualisiert).

Das könnte am Ende ziemlich fieselig werden herauszufinden bzw. zu synchronisieren (vermutlich weil das ein Paste in mehreren Zellen ist?) und evtl. den Rahmen hier sprengen, vor Allem wenn es vor diesem PR auch schon kaputt war…

Ansonsten funktioniert das alles gut, auch wenn die Zelldarstellung jetzt natürlich noch komplexer wird ;)

[smnhgn]

Ich bin mir nicht sicher, ob das mit deinem PR reingekommen ist. Wenn du eine Gruppenzelle mit mehreren Booleans auswählst (z.B. "Boolean eins / Boolean zwei" in dem Dump den du mir geschickt hat), diese Zelle kopiert und in eine andere Gruppenzelle pastest, dann werden nicht alle Updates korrekt durchgezogen (manchmal sind nicht beide Boolean-Checkboxen getoggelt, manchmal nicht alle Gruppenwerte aktualisiert).

Es gibt auf jeden Fall noch ein Problem bei der Cache-Invalidierung von abhängigen Werten speziell in der aktuell angezeigten Tabelle.
Im Dump ist auch eine etwas speziellere Konstellation von (Boolean)-Spalten die in mehreren Gruppen-Spalten und zugleich Identifier sind (-> auch in Concat-Spalte enthalten).
Evtl. hängt das Problem mit dem Multi-Copy-Paste auch damit zusammen.

[hermann-p]

Für mich wirkt jetzt auch der Bug gelöst.

[hermann-p]

Hier ist es ein Bisschen schwierig, den eigentlichen Change rauszuziehen, da auch noch Checkbox-Änderungen mit drin sind; es fehlt evtl. noch ein Rebase unter Berücksichtigung der Boolean-Darstellung?

Prinzipiell ist das Ganze sauber. Mir fällt nur eine Verhaltensänderung auf:
Wir hatten für das UX definiert, dass sich das Gruppenoverlay immer öffnen lassen sollte, auch wenn keiner der Members editierbar ist. Es ist nicht garantiert, dass die Memberspalten für den User direkt sichtbar/angezeigt sind; evtl. will der User die Einzelwerte im Detail nachschlagen ohne ggf. den Spaltenfilter zu verändern o.ä.. Durch den canEdit-Check sollte das innerhalb der Gruppen-Entityview keine Probleme machen.
Oder wurde mittlerweile eine Entscheidung getroffen, dass dieses Verhalten geändert werden soll?

The merge-base changed after approval.

[smnhgn]

Wir hatten für das UX definiert, dass sich das Gruppenoverlay immer öffnen lassen sollte, auch wenn keiner der Members editierbar ist. Es ist nicht garantiert, dass die Memberspalten für den User direkt sichtbar/angezeigt sind; evtl. will der User die Einzelwerte im Detail nachschlagen ohne ggf. den Spaltenfilter zu verändern o.ä..

Ok, das ist ein guter Punkt.
Ich passe das in dem anderen PR zum Permission-Handling (#579) an, dass sich das Group-Overlay unabhängig von den Edit-Permissions öffnen lässt.

[hermann-p]

Passt, unter der Annahme, dass die Öffnen-Permissions im nächsten Branch mitgefixt werden.