logs activity when bulk-hiding proposals
This commit is contained in:
@@ -15,15 +15,14 @@ class Moderation::ProposalsController < Moderation::BaseController
|
|||||||
end
|
end
|
||||||
|
|
||||||
def hide
|
def hide
|
||||||
@proposal.hide
|
hide_proposal @proposal
|
||||||
Activity.log(current_user, :hide, @proposal)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
def moderate
|
def moderate
|
||||||
@proposals = @proposals.where(id: params[:proposal_ids])
|
@proposals = @proposals.where(id: params[:proposal_ids])
|
||||||
|
|
||||||
if params[:hide_proposals].present?
|
if params[:hide_proposals].present?
|
||||||
@proposals.accessible_by(current_ability, :hide).each(&:hide)
|
@proposals.accessible_by(current_ability, :hide).each{|proposal| hide_proposal proposal}
|
||||||
|
|
||||||
elsif params[:ignore_flags].present?
|
elsif params[:ignore_flags].present?
|
||||||
@proposals.accessible_by(current_ability, :ignore_flag).each(&:ignore_flag)
|
@proposals.accessible_by(current_ability, :ignore_flag).each(&:ignore_flag)
|
||||||
@@ -42,4 +41,9 @@ class Moderation::ProposalsController < Moderation::BaseController
|
|||||||
@proposals = Proposal.accessible_by(current_ability, :moderate)
|
@proposals = Proposal.accessible_by(current_ability, :moderate)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def hide_proposal(proposal)
|
||||||
|
proposal.hide
|
||||||
|
Activity.log(current_user, :hide, proposal)
|
||||||
|
end
|
||||||
|
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -24,6 +24,31 @@ feature 'Admin activity' do
|
|||||||
expect(page).to have_content(@admin.user.username)
|
expect(page).to have_content(@admin.user.username)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
scenario "Shows moderation activity from moderation screen" do
|
||||||
|
proposal1 = create(:proposal)
|
||||||
|
proposal2 = create(:proposal)
|
||||||
|
proposal3 = create(:proposal)
|
||||||
|
|
||||||
|
visit moderation_proposals_path(filter: 'all')
|
||||||
|
|
||||||
|
within("#proposal_#{proposal1.id}") do
|
||||||
|
check "proposal_#{proposal1.id}_check"
|
||||||
|
end
|
||||||
|
|
||||||
|
within("#proposal_#{proposal3.id}") do
|
||||||
|
check "proposal_#{proposal3.id}_check"
|
||||||
|
end
|
||||||
|
|
||||||
|
click_on "Hide proposals"
|
||||||
|
|
||||||
|
visit admin_activity_path
|
||||||
|
|
||||||
|
|
||||||
|
expect(page).to have_content(proposal1.title)
|
||||||
|
expect(page).to_not have_content(proposal2.title)
|
||||||
|
expect(page).to have_content(proposal3.title)
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
end
|
end
|
||||||
Reference in New Issue
Block a user