Gruppenbesitzer kann keine Mitglieder zu einer Gruppe hinzufügen

Beitragen Fehler
1

Ich bin mir ziemlich sicher, dass das früher funktioniert hat.

Erwartetes Verhalten
Gruppenbesitzer können Mitglieder über das Modal „Mitglieder hinzufügen

5 „Gefällt mir“
2

Es funktioniert immer noch mit 2.7.0.beta9 ( 4a2912233c )

3 „Gefällt mir“
4

Gerade den neuesten Code von GitHub gezogen und erneut getestet. Bestätigt, dass es defekt ist. Schritte zur Reproduktion:

  1. Erstellen Sie einen neuen Benutzer; dieser Benutzer sollte weder Moderator noch Administrator sein.
  2. Erstellen Sie eine neue Gruppe mit folgenden Einstellungen: Benutzer können die Gruppe frei verlassen=True, Benutzer können Mitgliedschaftsanfragen an Gruppeninhaber senden=True, Wer kann diese Gruppe sehen?=Alle, Wer kann die Mitglieder dieser Gruppe sehen?=Alle, Wer kann diese Gruppe @erwähnen?=Nur Moderatoren und Administratoren, Wer kann diese Gruppe nachrichten?=Nur Moderatoren und Administratoren.
  3. Setzen Sie den Benutzer als Inhaber der Gruppe.
  4. Versuchen Sie, während Sie als Gruppeninhaber angemeldet sind, einen anderen Benutzer zur Gruppe hinzuzufügen.
5 „Gefällt mir“
5

Ich nehme an, der Benutzer ist TL2 oder höher?

4 „Gefällt mir“
6

Ich habe das Problem endlich gelöst.

Hier ist die Zeile, die den Ärger verursacht hat: discourse/app/controllers/groups_controller.rb at e2e13a70f63605b92b4e521cc9f818061c1ccfb6 · discourse/discourse · GitHub

@Benjamin_D hat es richtig erkannt. Es ist ein Problem mit dem Vertrauenslevel (TL).

Es gibt eine Site-Einstellung min_trust_level_to_allow_invite, die auf TL2 gesetzt war, während der Gruppeninhaber nur TL1 hatte.

Sollte ein Gruppeninhaber erlaubt sein, Benutzer in die Gruppe einzuladen, ohne die Vertrauenslevel-Anforderung zu erfüllen? Das scheint verwirrend.

10 „Gefällt mir“
7

:thinking: Du hast recht, ich schätze, er sollte das dürfen? Oder vielleicht sollte er gar kein Inhaber sein, wenn er die Kriterien für die Vertrauensstufe nicht erfüllt?

6 „Gefällt mir“
8

Das würde auch Sinn ergeben.

Oder man fügt hier einfach eine spezifischere Fehlermeldung hinzu.
Zum Beispiel statt den InvalidAccess-Fehler mit der (en)-Nachricht: You are not permitted to view the requested resource. auszulösen.

Erstelle einen InsufficientTrustLevel-Fehler (vielleicht als Unterklasse von InvalidAccess) mit der (en)-Nachricht: You do not have the required trust level to view the requeted resource. Mindestens dann können Administratoren erkennen, dass es sich um ein TL-Problem und nicht um einen Bug handelt.

Einige Hinweise dazu, wo diese Dinge im Code definiert sind:

Ich kann einen PR dafür erstellen, falls das der richtige Weg zu sein scheint.

7 „Gefällt mir“
10

Obwohl ich zustimme, dass wir hier eine bessere Fehlermeldung benötigen, denke ich eigentlich, dass wir einen Rückschritt gemacht haben.

Die Rolle eines Gruppenbesitzers besteht darin, die Mitgliedschaft in seiner Gruppe zu verwalten. Wenn er keine Benutzer in seine Gruppe einladen/hinzufügen kann, ist die Funktion zur Hälfte defekt…

@dan, kannst du das auf deine Liste setzen?

9 „Gefällt mir“
11

Großartige Detektivarbeit, ihr alle!

Ich mag mich irren, aber ich denke, diese Situation ist im Rahmen unserer Arbeiten zur Verbesserung des Einladungs-Systems entstanden. Wir normalisieren Einladungen, sodass alle dasselbe Einladungs-System verwenden, und trennen die Funktionen für bestehende Nutzer ab. Kürzlich haben wir geändert, dass wir bestehende Nutzer nun über das Teilen-Symbol im Themen-Menü über Themen informieren, getrennt davon, neue Nutzer einzuladen, sich der Seite anzuschließen und zum Thema beizutragen.

Wir werden dasselbe für Gruppen tun wollen – das steht bereits auf unserer Liste, aber jetzt ist es wohl dringender geworden, wenn es bedeutet, dass man niemanden zu einer Gruppe hinzufügen kann, wenn man nicht über genügend TL verfügt, um neue Nutzer auf die Seite einzuladen.

7 „Gefällt mir“
13

Dieses Limit ergibt Sinn, wenn E-Mails zu Gruppen eingeladen werden. Es ergibt jedoch keinen Sinn, wenn bestehende Benutzer eingeladen werden. Ich habe dieses Problem in folgendem Pull Request behoben:

Dies wird das Problem vorübergehend beheben, aber wir planen, diesen Bereich grundlegend zu überarbeiten.

8 „Gefällt mir“
14

Dieses Thema wurde automatisch nach 2 Tagen geschlossen. Neue Antworten sind nicht mehr möglich.