From d00ba360f291f82ec3413d069b0ae143c699abf1 Mon Sep 17 00:00:00 2001 From: Zach Daniel Date: Thu, 9 Jul 2020 03:25:40 -0400 Subject: [PATCH] chore: fix lint --- lib/ash/engine/request.ex | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/lib/ash/engine/request.ex b/lib/ash/engine/request.ex index 545dfcf2..7d763fa1 100644 --- a/lib/ash/engine/request.ex +++ b/lib/ash/engine/request.ex @@ -421,13 +421,11 @@ defmodule Ash.Engine.Request do {:filter, filter} -> request |> Map.update!(:query, &Ash.Query.filter(&1, filter)) - |> Map.update(:authorization_filter, filter, fn existing_authorization_filter -> - if existing_authorization_filter do - Ash.Filter.add_to_filter(existing_authorization_filter, filter) - else - Ash.Filter.parse!(request.resource, filter) - end - end) + |> Map.update( + :authorization_filter, + filter, + &add_to_or_parse(&1, filter, request.resource) + ) |> set_authorizer_state(authorizer, :authorized) |> try_resolve([request.path ++ [:query]], false, false) @@ -472,6 +470,14 @@ defmodule Ash.Engine.Request do end end + defp add_to_or_parse(existing_authorization_filter, filter, resource) do + if existing_authorization_filter do + Ash.Filter.add_to_filter(existing_authorization_filter, filter) + else + Ash.Filter.parse!(resource, filter) + end + end + defp check(authorizers, request) do authorizers |> Enum.reject(&(authorizer_state(request, &1) == :authorized))