Skip to content

Commit b828def

Browse files
committed
Remove nesting via with
1 parent efa8cd0 commit b828def

File tree

1 file changed

+10
-18
lines changed

1 file changed

+10
-18
lines changed

lib/philomena/application.ex

Lines changed: 10 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -85,24 +85,16 @@ defmodule Philomena.Application do
8585

8686
if not Enum.empty?(filters) do
8787
allow_log_event? = fn event ->
88-
case(Map.get(event.meta, :mfa)) do
89-
nil ->
90-
false
91-
92-
{module, function, _arity} ->
93-
scope = "#{inspect(module)}.#{function}"
94-
95-
filters
96-
|> Enum.find(fn {selector, _level} ->
97-
is_nil(selector) or String.starts_with?(scope, selector)
98-
end)
99-
|> case do
100-
nil ->
101-
false
102-
103-
{_selector, level} ->
104-
Logger.compare_levels(event.level, level) != :lt
105-
end
88+
with {module, function, _arity} <- Map.get(event.meta, :mfa),
89+
scope <- "#{inspect(module)}.#{function}",
90+
{_selector, level} when not is_nil(filters) <-
91+
filters
92+
|> Enum.find(fn {selector, _level} ->
93+
is_nil(selector) or String.starts_with?(scope, selector)
94+
end) do
95+
Logger.compare_levels(event.level, level) != :lt
96+
else
97+
_ -> false
10698
end
10799
end
108100

0 commit comments

Comments
 (0)