mirror of
https://github.com/ash-project/ash.git
synced 2024-09-20 13:33:20 +12:00
improvement: small improvements for policy breakdown formatting
This commit is contained in:
parent
60cc9a6f3b
commit
c171bfae5e
2 changed files with 5 additions and 19 deletions
|
@ -319,7 +319,7 @@ defmodule Ash.Error.Forbidden.Policy do
|
||||||
end
|
end
|
||||||
|
|
||||||
title(
|
title(
|
||||||
[Enum.map(condition_description, &[" ", &1]), Enum.map(description, &[" ", &1])],
|
["\n", Enum.map(condition_description, &[" ", &1]), "\n", Enum.map(description, &[" ", &1]), "\n"],
|
||||||
[
|
[
|
||||||
" ",
|
" ",
|
||||||
bypass,
|
bypass,
|
||||||
|
@ -372,25 +372,10 @@ defmodule Ash.Error.Forbidden.Policy do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
{[["condition: ", describe(mod, opts, actor, subject)] | conditions], new_status}
|
{[["condition: ", describe(mod, opts, actor, subject) <> "\n"] | conditions], new_status}
|
||||||
end)
|
end)
|
||||||
|> then(fn {conditions, status} ->
|
|> then(fn {conditions, status} ->
|
||||||
conditions =
|
{Enum.reverse(conditions), status}
|
||||||
conditions
|
|
||||||
|> Enum.reverse()
|
|
||||||
|> case do
|
|
||||||
[] ->
|
|
||||||
[]
|
|
||||||
|
|
||||||
conditions ->
|
|
||||||
[
|
|
||||||
conditions
|
|
||||||
|> Enum.intersperse("\n"),
|
|
||||||
"\n"
|
|
||||||
]
|
|
||||||
end
|
|
||||||
|
|
||||||
{conditions, status}
|
|
||||||
end)
|
end)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
@ -1384,7 +1384,8 @@ defmodule Ash.Policy.Authorizer do
|
||||||
authorizer.facts,
|
authorizer.facts,
|
||||||
filter,
|
filter,
|
||||||
authorizer.policies,
|
authorizer.policies,
|
||||||
success?: true
|
success?: true,
|
||||||
|
help_text?: false
|
||||||
)
|
)
|
||||||
]
|
]
|
||||||
)
|
)
|
||||||
|
|
Loading…
Reference in a new issue