Merge branch 'is_public-non-list' into 'master'

Fix is_public? for non-lists

See merge request framasoft/mobilizon!647
This commit is contained in:
Thomas Citharel 2020-10-26 08:36:09 +01:00
commit 9e5234059b

View file

@ -20,9 +20,12 @@ defmodule Mobilizon.Federation.ActivityPub.Visibility do
def is_public?(%Activity{data: data}), do: is_public?(data) def is_public?(%Activity{data: data}), do: is_public?(data)
def is_public?(data) when is_map(data) do def is_public?(data) when is_map(data) do
@public in (Map.get(data, "to", []) ++ Map.get(data, "cc", [])) @public in (make_list(Map.get(data, "to", [])) ++ make_list(Map.get(data, "cc", [])))
end end
def is_public?(%Comment{deleted_at: deleted_at}), do: !is_nil(deleted_at) def is_public?(%Comment{deleted_at: deleted_at}), do: !is_nil(deleted_at)
def is_public?(err), do: raise(ArgumentError, message: "Invalid argument #{inspect(err)}") def is_public?(err), do: raise(ArgumentError, message: "Invalid argument #{inspect(err)}")
defp make_list(data) when is_list(data), do: data
defp make_list(data), do: [data]
end end