Admin.groups.manage.membership.automatic が置き換えられていません

martin · 2024 年 12 月 9 日午前 1:41

サイト設定のMoinが言及していたものはこれを許可していると思いますが、関連するガーディアンチェックは以下の通りです。

lib/guardian/group_guardian.rb

4ddf7e61f


      
          def can_admin_group?(group)
            is_admin? ||
              (
                SiteSetting.moderators_manage_categories_and_groups && is_moderator? && can_see?(group) &&
                  group.id != Group::AUTO_GROUPS[:admins]
              )
          end

しかし、クライアント側では矛盾したメッセージを送っています

github.com/discourse/discourse

app/assets/javascripts/discourse/app/models/user.js

4ddf7e61f


      
          canManageGroup(group) {
            return group.get("automatic")
              ? false
              : group.get("can_admin_group") || group.get("is_group_owner");
          }

github.com/discourse/discourse

app/assets/javascripts/discourse/app/controllers/group.js

4ddf7e61f


      
          @discourseComputed("model", "model.automatic")
          canManageGroup(model, automatic) {
            return (
              this.currentUser &&
              (this.currentUser.canManageGroup(model) ||
                (model.can_admin_group && automatic))
            );
          }

canManageGroup のチェックを変更し、ユーザーモデルの canManageGroup のバージョンもより合理的なものにする（このグループコントローラーから使用されている唯一の場所）ためのフォローアップPRを以下に示します。

github.com/discourse/discourse

FIX: Allow anyone who can manage groups to see Automatic tooltip

main ← issue/manage-groups-followup

opened 01:40AM - 09 Dec 24 UTC

martin-brennan

+56 -37

Followup 35ecd0335c70e4f008ae86ec5804dc60dbf41101 Since we have the moderator…s_manage_categories_and_groups setting, more than admins can manage groups, so we need to allow others to see this Automatic tooltip as well. Also fixes an inconsistency with canManageGroup between the User model and Group controller, the latter is correct, allowing management of automatic groups if can_admin_group permission is true

トピック		返信	表示
Moderators group [en_GB.admin.groups.manage.membership.automatic_tooltip] Site feedback groups	2	38	2024 年 12 月 4 日
Should it be possible to add and remove users from moderators group? UX groups	4	124	2025 年 1 月 31 日
How does visibility of automatic groups work? Support	6	858	2023 年 10 月 4 日
Cannot change member_visibility_level of automatic groups Bug groups	10	536	2024 年 1 月 20 日
Respect the visibility settings of all automatic groups Feature groups	14	737	2023 年 10 月 23 日

Admin.groups.manage.membership.automatic が置き換えられていません

関連トピック