PonyMirrors/philomena
1
0
Fork 0
mirror of https://github.com/philomena-dev/philomena.git synced 2025-02-01 03:46:44 +01:00
Code Issues Projects Releases Packages Wiki Activity

ensure moderators can see duplicate report tab

Browse source
This commit is contained in:
byte[] 2019-12-09 20:36:58 -05:00
parent 2c1725f20a
commit ca3afb26fd
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/philomena/duplicate_reports.ex
View file

@ -161,7 +161,7 @@ defmodule Philomena.DuplicateReports do
end end
def count_duplicate_reports(user) do def count_duplicate_reports(user) do
if Canada.Can.can?(user, :edit, DuplicateReport) do if Canada.Can.can?(user, :index, DuplicateReport) do
DuplicateReport DuplicateReport
|> where(state: "open") |> where(state: "open")
|> Repo.aggregate(:count, :id) |> Repo.aggregate(:count, :id)

6
lib/philomena/users/ability.ex
View file

@ -42,7 +42,10 @@ defimpl Canada.Can, for: [Atom, Philomena.Users.User] do
# View IP addresses and fingerprints # View IP addresses and fingerprints
def can?(%User{role: "moderator"}, :show, :ip_address), do: true def can?(%User{role: "moderator"}, :show, :ip_address), do: true
# View reports # Manage duplicate reports
def can?(%User{role: "moderator"}, :index, DuplicateReport), do: true
# Manage reports
def can?(%User{role: "moderator"}, :index, Report), do: true def can?(%User{role: "moderator"}, :index, Report), do: true
def can?(%User{role: "moderator"}, :show, %Report{}), do: true def can?(%User{role: "moderator"}, :show, %Report{}), do: true
def can?(%User{role: "moderator"}, :edit, %Report{}), do: true def can?(%User{role: "moderator"}, :edit, %Report{}), do: true
@ -64,6 +67,7 @@ defimpl Canada.Can, for: [Atom, Philomena.Users.User] do
def can?(%User{role: "assistant", role_map: %{"Image" => "moderator"}}, :edit, %Image{}), do: true def can?(%User{role: "assistant", role_map: %{"Image" => "moderator"}}, :edit, %Image{}), do: true
# Dupe assistant actions # Dupe assistant actions
def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :index, DuplicateReport), do: true
def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :edit, %DuplicateReport{}), do: true def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :edit, %DuplicateReport{}), do: true
def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :show, %Image{}), do: true def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :show, %Image{}), do: true
def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :edit, %Image{}), do: true def can?(%User{role: "assistant", role_map: %{"DuplicateReport" => "moderator"}}, :edit, %Image{}), do: true