Benutzerrechte für neues Nachrichtenarchiv wird nicht richtig gesetzt

[Xendiadyon]

Ich habe einen Redakteurbenutzer, der neue Newsarchive anlegen darf. Wünschenswert ist, dass er auf die neu erstellten Newsarchive auch automatisch Zugriff erhält.

Wenn er aber versucht, ein neues Newsarchiv anzulegen, hat er für exakt dieses Newsarchiv keine Zugriffsrechte.

Es fehlt die Möglichkeit, bei den Benutzerrechten einzustellen, dass nicht nur aktuell gewählte Newsarchive erlaubt sind, sondern auch alle zukünftigen.
Das Gleiche gilt für andere Erweiterungen, z.B. FAQ, Newsletter, Google Maps, ...

Contao 4.3

[ADoebeling]

Das klingt für mich nach einem Bug und einem Feature-Request:

Wenn er aber versucht, ein neues Newsarchiv anzulegen, hat er für exakt dieses Newsarchiv keine Zugriffsrechte.

Wenn ein User ein Archiv anlegen kann, muss er es natürlich auch bearbeiten können.

Es fehlt die Möglichkeit, bei den Benutzerrechten einzustellen, dass nicht nur aktuell gewählte Newsarchive erlaubt sind, sondern auch alle zukünftigen.

Das könnte ein sinnvolles Feature sein.

Grüße
Andreas

Nachtrag 05.12.2016:
Sorry, Danke @fritzmg für den Hinweis. Copy&Paste korrigiert

[fritzmg]

@ADoebeling was genau ist nun für dich ein Bug und was ein Feature Request? Du hast zwei mal die selbe Stelle gequoted ;).

[ADoebeling]

Ich habe am Wochenende nochmal offline mit @Xendiadyon gesprochen:
Beim Anlegen eines neuen Archives müsste ein Benutzer eigentlich auswählen dürfen, für welche Benutzergruppen er das jeweilige Archiv freischalten möchte. In der Liste dürfen dabei natürlich nur diejenigen Benutzergruppen gelistet sein, denen er selbst zugewiesen ist, bzw. für die er als Administrator berechtigt ist.

[asaage]

😕 der Logik folgend müsste er ja durch das Anlegen eines Archives auch die Rechte bekommen in den Listen-Modulen sein Häkchen für das Archiv zu setzen.
Ich denk das sollte man schon jemandem überlassen, der die nötigen Rechte in der Benutzer- oder Benutzergruppenverwaltung bzw. den Frontendmodulen hat.

[ADoebeling]

@asaage Die Frage ist, wie es besser wäre:

Redakteur legt neues Archiv an und:

• ... hat selbst keine Rechte darauf => IMHO ein Bug
• ... bekommt nur persönlich userbezogene Rechte hierauf => Ziemlich suboptimal im Redakteurteam
• ... das Archiv bekommt automatisch die gleichen Rechtegruppen wie der Benutzer => Suboptimal
• ... kann selbst wählen, welche Benutzergruppen hierauf welche Rechte besitzen => erscheint mir die noch relativ schönste Alternative.

Für eine optimale Lösung müsste mal das bisherige Rechtekonzept überarbeiten und künftig nicht nur den Haken setzen "Darf Archive anlegen", sondern auch "Archive erhalten automatisch folgende Benutzergruppe zugewiesen". Das ist aber IMHO ein BC-Break.

[asaage]

• ... kann selbst wählen, welche Benutzergruppen hierauf welche Rechte besitzen => erscheint mir die noch relativ schönste Alternative.

lässt sich doch realisieren indem du den Redakteuren die Gruppenverwaltung mit den entsprechenden Feldern freischaltest - oder sehe ich das falsch?
Erlaubte Module:
Benutzerverwaltung->Benutzergruppen
Erlaubte Felder:
tl_user_group->Erlaubte Archive

Ich hadere auch hin und wieder mit dem Rechtekonzept :)

[ADoebeling]

Wenn das so funktioniert, ist das zwar vom Workflow für den gemeinen Redakteur etwas "unintuitiv", löst aber tatsächlich das Problem. @Xendiadyon Kannst Du das bitte mal checken?

[leofeyer]

Wünschenswert ist, dass er auf die neu erstellten Newsarchive auch automatisch Zugriff erhält.

Das ist eigentlich auch der Fall:

https://github.com/contao/core/blob/master/system/modules/news/dca/tl_news_archive.php#L313-L362

Klappt es in der Onlinedemo nicht?

[Xendiadyon]

Wenn man die Benutzerrechte feiner aufspaltet (z.B. darf eine Benutzergruppe Seiten/Artikel anlegen, eine andere darf News eintragen oder erstellen, aber KEINE Seiten/Artikel anlegen) und einem Benutzer die Mitgliedschaft in beide Gruppen zuweist, dann gibt es den Fehler:

Und zwar genau dann, wenn die News-Redakteurgruppe NICHT die Seitenbearbeitungsrechte hat und in der Reihenfolge der User-Groups unter der Seiten-Redakteurgruppe steht.
Dann kommt nämlich beim Auswählen der Weiterleitungsseite ein Rechtefehler und anschließend besitzt man nicht die Rechte, das (trotzdem angelegte) Newsarchiv zu bearbeiten. (Meine Theorie: Weil intern durch die Auswahl der Weiterleitungsseite auf die Seitenredakteurgruppe geschaltet wird und diese nun mal keine Rechte für irgendwelche Newsarchive hat.

Das erklärt aber nicht, warum es bei der Google Maps Erweiterung nicht klappt, wo man idR keine Weiterleitungsseite anlegen muss.

Eventuell wird auch nur versucht, der ersten Benutzergruppe die Rechte zu verpassen, denn wenn die News-Redakteurgruppe VOR der Seitenredakteurgruppe stehen hat, klappt es in der Online-Demo.

Ich werde versuchen, den News-Redakteuren (genau wie den Google-Maps-Redakteuren) zusätzlich als Gruppenrechte Seitenbearbeitungsrechte zu geben. Dann ist zwar die Berechtigung nicht so dediziert gestaffelt wie ich es mir wünsche, aber es ist an sich auch völlig okay, wenn ein Benutzer, der Newsarchive anlegen kann, auch Seitenbearbeitungsrechte besitzt.

[leofeyer]

Das ist eigentlich auch der Fall:

https://github.com/contao/core/blob/master/system/modules/news/dca/tl_news_archive.php#L313-L362

Klappt es in der Onlinedemo nicht?

[fritzmg]

@Xendiadyon ging es dir nicht ursprünglich um die Nachrichten Archive in Contao 4?

[Xendiadyon]

Ja, es geht mir um Contao4;
ich habe dieses Problem wie am 5.12. geschrieben auch genauso in der OnlineDemo reproduzieren können:

1. neuen Benutzer "News-Redakteur" anlegen und ihm die Rechte für Newsarchive löschen/bearbeiten geben, aber NICHT für Seiten editieren. Nicht vergessen, ihn mit Rechten für tl_news und tl_news_archive auszustatten.

2. Editors bearbeiten, die Rechte für das Editieren von News-Archiven entfernen (der Editor hat sowieso keine Rechte, um ein neues News-Archiv anzulegen). Den Pagemount für Editor setzen.

3. Helen Lewis bearbeiten und ihr zusätzlich der Gruppe News-Redakteur zuweisen.

4. Als Helen einloggen und ein neues News-Archiv anlegen.

5. Gegentest: Als Admin einloggen, die Reihenfolge der Gruppen ändern (erst News-Redakteur, dann Editor). Als Helen einloggen, Newsarchiv anlegen: Klappt.

6. Weil es so unfassbar ist, nochmal Triple-Test: Als Admin einloggen, Reihenfolge der Gruppen bei Helen wieder tauschen (erst Editor, dann News-Redakteur). Als Helen einloggen, Newsarchiv anlegen: Fehler.

Dieser Fehler existiert sowohl in Contao 3 (Demo) als auch in Contao 4.

Ich hoffe, ich konnte helfen?

[leofeyer]

Das Problem war, dass Contao versucht hat, die ID des neuen Archivs der Benutzergruppe zuzuordnen; die Rechte waren aber auf dem Benutzer selbst gesetzt. Hättest Du die Rechte zum Anlegen neuer Archive der Gruppe gegeben, wäre uns der Fehler nie aufgefallen – insofern gut. 😄

Behoben in 95c9fd7.