fix: accept ids in code interfaces again

ash-project · May 15, 2024 · 5b18e71 · 5b18e71
1 parent 835187d
commit 5b18e71
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/lib/ash/code_interface.ex b/lib/ash/code_interface.ex
@@ -798,11 +798,7 @@ defmodule Ash.CodeInterface do
                           {:atomic, :stream, list}
 
                         other ->
-                          if Keyword.keyword?(other) || is_map(other) do
-                            {:atomic, :id, other}
-                          else
-                            {:atomic, :stream, other}
-                          end
+                          {:atomic, :id, other}
                       end
                   end
                 else
@@ -1005,11 +1001,7 @@ defmodule Ash.CodeInterface do
                           {:atomic, :stream, list}
 
                         other ->
-                          if Keyword.keyword?(other) || is_map(other) do
-                            {:atomic, :id, other}
-                          else
-                            {:atomic, :stream, other}
-                          end
+                          {:atomic, :id, other}
                       end
                   end
                 else

diff --git a/test/code_interface_test.exs b/test/code_interface_test.exs
@@ -303,6 +303,11 @@ defmodule Ash.Test.CodeInterfaceTest do
       assert %Ash.BulkResult{status: :success} = User.destroy!([bob3, bob4], context: @context)
     end
 
+    test "bulk destroy can take an id" do
+      bob1 = User.create!("bob", context: @context)
+      User.destroy!(bob1.id)
+    end
+
     test "bulk destroy can take an empty list" do
       assert %Ash.BulkResult{status: :success} = User.destroy([])
     end