Bugfix: Access/edit checks

[hermann-p]

• 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

Summary

• AC#1013 - bearbeiten von key und displayKey Spalten in Settings-Tabellen verbieten
• Bei "final" gesetzten Zeilen die "Link löschen"-Aktion deaktivieren
• AC#1051 - Rowcreator nur anzeigen, wenn in der Zieltabelle auch Zeilen erstellt werden könnnen

[McHunkyTrunk]

Code passt von meiner Seite.
@Zwergal Ich habe das Frontend zum Testen auf Winora Testcenter deployed und einen neuen User angelegt:

• user: test-link-edit
• pass: test0000
  Der User besitzt die Rollen dev und deny-view-variants (die Rolle existiert nur aufm Testserver), womit man den Rowcreator testen kann (man sollte keine neuen Varianten für Modelle anlegen können).
  Beispiel: Batavus Modelle
  https://pdc-test.winora.grud.de/de/tables/191

[Zwergal]

@McHunkyTrunk @hermann-p Ich weiß jetzt nicht wie die Rechte sind, aber ich sehe nun gar keinen Button mehr um neue Varianten im LinkOverlay anzulegen:

Auch hier bei Modelljahr nicht, obwohl ich das beim Modelljahr machen darf.

Es ist egal bei welchem LinkOverlay, der Button erscheint nie.

Bzgl. 1013 sieht es gut aus.

[Zwergal]

@hermann-p Der Part mit dem "Minus"-Button im Entity-View wenn man kein Recht auf die Link-Spalte hat besteht weiterhin:

Ist aus dem Ticket https://app.activecollab.com/116706/projects/2?modal=Task-46156-2 (1004). Ein Ticket 1051 gibt es in AC gar nicht. Ist falsch angegeben in der Beschreibung, oder?

[Zwergal]

Siehe Kommentare direkt am PR

[McHunkyTrunk]

@Zwergal @hermann-p Die Rolle deny-view-variants, die ich zum Testen angelegt habe, sieht wie folgt aus:

"deny-view-variants": [
    {
      "type": "deny",
      "action": [
        "viewTable",
        "viewColumn",
        "viewCellValue"
      ],
      "condition": {
         "table": {
             "name": ".*_variant"
         }
      }
    }
  ]

@Zwergal Du hast Recht. beim "Campudus" User sind jetzt auch keine Buttons im LinkOverlay zu sehen.