Admin.groups.manage.membership.automatic não substituído

martin · Dezembro 9, 2024, 1:41am

Eu acho que a configuração do site que Moin mencionou permite isso, aqui está a verificação relevante do guardião:

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

Mas no cliente, estamos enviando mensagens contraditórias

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))
            );
          }

Aqui está um PR de acompanhamento para mudar para verificar canManageGroup, e também atualizar a versão do modelo de usuário de canManageGroup para ser mais sensata (o único lugar que usa isso é deste controlador de grupo):

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

Tópico		Respostas	Visualizações
Moderators group [en_GB.admin.groups.manage.membership.automatic_tooltip] Site feedback groups	2	38	4 de Dezembro de 2024
Should it be possible to add and remove users from moderators group? UX groups	4	124	31 de Janeiro de 2025
How does visibility of automatic groups work? Support	6	858	4 de Outubro de 2023
Cannot change member_visibility_level of automatic groups Bug groups	10	536	20 de Janeiro de 2024
Respect the visibility settings of all automatic groups Feature groups	14	737	23 de Outubro de 2023

Admin.groups.manage.membership.automatic não substituído

Tópicos relacionados