codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-30 10:58:31 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

FIX: Admin logging should not log permissions if none has been set.

Browse source
This commit is contained in:
Guo Xiang Tan 2015-12-06 15:29:13 +08:00
parent ad98f270c9
commit 8b6b3cf858
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/services/staff_action_logger.rb
View file

@ -215,7 +215,7 @@ class StaffActionLogger
changed_attributes = category.previous_changes.slice(*category_params.keys) changed_attributes = category.previous_changes.slice(*category_params.keys)
if old_permissions != category_params[:permissions] if !old_permissions.empty? && (old_permissions != category_params[:permissions])
changed_attributes.merge!({ permissions: [old_permissions.to_json, category_params[:permissions].to_json] }) changed_attributes.merge!({ permissions: [old_permissions.to_json, category_params[:permissions].to_json] })
end end

11
spec/services/staff_action_logger_spec.rb
View file

@ -306,6 +306,17 @@ describe StaffActionLogger do
expect(name_user_history.previous_value).to eq('haha') expect(name_user_history.previous_value).to eq('haha')
expect(name_user_history.new_value).to eq('new_name') expect(name_user_history.new_value).to eq('new_name')
end end
it "does not log permissions changes for category visible to everyone" do
attributes = { name: 'new_name' }
old_permission = category.permissions_params
category.update!(attributes)
logger.log_category_settings_change(category, attributes.merge({ permissions: { "everyone" => 1 } }), old_permission)
expect(UserHistory.count).to eq(1)
expect(UserHistory.find_by_subject('name').category).to eq(category)
end
end end
describe 'log_category_deletion' do describe 'log_category_deletion' do