PonyMirrors/philomena
1
0
Fork 0
mirror of https://github.com/philomena-dev/philomena.git synced 2024-11-27 13:47:58 +01:00
Code Issues Projects Releases Packages Wiki Activity

more comprehensive blankness check for personal titles and secondary roles

Browse source
This commit is contained in:
byte[] 2019-12-07 21:30:47 -05:00
parent 6ce30ee27c
commit c79dfd5b21
4 changed files with 14 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
lib/philomena_web/controllers/filter/hide_controller.ex
View file

@ -3,7 +3,6 @@ defmodule PhilomenaWeb.Filter.HideController do
alias Philomena.Filters
alias Philomena.Tags.Tag
alias Philomena.Repo
plug PhilomenaWeb.FilterBannedUsersPlug
plug :authorize_filter

1
lib/philomena_web/controllers/filter/spoiler_controller.ex
View file

@ -3,7 +3,6 @@ defmodule PhilomenaWeb.Filter.SpoilerController do
alias Philomena.Filters
alias Philomena.Tags.Tag
alias Philomena.Repo
plug PhilomenaWeb.FilterBannedUsersPlug
plug :authorize_filter

1
lib/philomena_web/controllers/tag/watch_controller.ex
View file

@ -3,7 +3,6 @@ defmodule PhilomenaWeb.Tag.WatchController do
alias Philomena.Tags.Tag
alias Philomena.Users
alias Philomena.Repo
plug :load_resource, model: Tag, id_field: "slug", id_name: "tag_id", persisted: true

22
lib/philomena_web/views/user_attribution_view.ex
View file

@ -49,15 +49,21 @@ defmodule PhilomenaWeb.UserAttributionView do
|> staff_role(user)
end
defp personal_title(labels, %{personal_title: t}) when t not in [nil, ""],
do: [{"label--primary", t} | labels]
defp personal_title(labels, _user),
do: labels
defp personal_title(labels, %{personal_title: t}) do
case blank?(t) do
true -> labels
false -> [{"label--primary", t} | labels]
end
end
defp personal_title(labels, _user), do: labels
defp secondary_role(labels, %{secondary_role: t}) when t not in [nil, ""],
do: [{"label--warning", t} | labels]
defp secondary_role(labels, _user),
do: labels
defp secondary_role(labels, %{secondary_role: t}) do
case blank?(t) do
true -> labels
false -> [{"label--warning", t} | labels]
end
end
defp secondary_role(labels, _user), do: labels
defp staff_role(labels, %{hide_default_role: false, role: "admin"}),
do: [{"label--danger", "Site Administrator"} | labels]