github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-17 01:52:45 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FIX: Allow staff to change group members visibility level for automatic groups (#25281)

Browse Source
This commit is contained in:
Penar Musaraj 2024-01-17 12:54:52 -05:00 committed by GitHub
parent 4cfc0e231a
commit 719956f255
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/controllers/groups_controller.rb
View File

@ -740,9 +740,7 @@ class GroupsController < ApplicationController
flair_upload_id
]
if automatic
attributes.push(:visibility_level)
else
if !automatic
attributes.push(
:title,
:allow_membership_requests,
@ -753,6 +751,8 @@ class GroupsController < ApplicationController
)
end
attributes.push(:visibility_level, :members_visibility_level) if current_user.staff?
if !automatic && current_user.staff?
attributes.push(
:incoming_email,
@ -773,8 +773,6 @@ class GroupsController < ApplicationController
:email_password,
:email_from_alias,
:primary_group,
:visibility_level,
:members_visibility_level,
:name,
:grant_trust_level,
:automatic_membership_email_domains,

3
spec/requests/groups_controller_spec.rb
View File

@ -994,6 +994,7 @@ RSpec.describe GroupsController do
mentionable_level: 2,
messageable_level: 2,
default_notification_level: 2,
members_visibility_level: 2,
)
put "/groups/#{group.id}.json",
@ -1007,6 +1008,7 @@ RSpec.describe GroupsController do
mentionable_level: 1,
messageable_level: 1,
default_notification_level: 1,
members_visibility_level: 1,
tracking_category_ids: [category.id],
tracking_tags: [tag.name],
},
@ -1025,6 +1027,7 @@ RSpec.describe GroupsController do
expect(group.mentionable_level).to eq(1)
expect(group.messageable_level).to eq(1)
expect(group.default_notification_level).to eq(1)
expect(group.members_visibility_level).to eq(1)
expect(group.group_category_notification_defaults.first&.category).to eq(category)
expect(group.group_tag_notification_defaults.first&.tag).to eq(tag)
end