Add support to consume uploads in resource actions

[marot]

At the moment, after the resource action was handled, handle_uploads will consume the uploaded entries and call the consume_upload callback.

In use cases where the content of the file is necessary for the resource action (e.g. a csv import), the file should be consumed in the handle callback. If this is done, handle_uploads will fail due to attempting to consume the already consumed upload.

A possible solution is to check in handle_uploads if the upload is already consumed, and not call the consume_upload hook in that case.

I can help to provide a PR if the change is accepted :)

[marot]

The easiest solution is probably to allow skipping the consume_upload callback. If none is provided, the assumption is that the handle callback will consume the upload.

diff --git a/lib/backpex/live_components/form_component.ex b/lib/backpex/live_components/form_component.ex
index 3fe4551..9192f09 100644
--- a/lib/backpex/live_components/form_component.ex
+++ b/lib/backpex/live_components/form_component.ex
@@ -473,11 +473,15 @@ defmodule Backpex.FormComponent do
   defp handle_uploads(%{assigns: %{uploads: _uploads}} = socket, item) do
     for {_name, %{upload_key: upload_key} = field_options} = _field <- socket.assigns.fields do
       if Map.has_key?(socket.assigns.uploads, upload_key) do
-        %{consume_upload: consume_upload, remove_uploads: remove_uploads} = field_options
+        %{remove_uploads: remove_uploads} = field_options

-        consume_uploaded_entries(socket, upload_key, fn meta, entry ->
-          consume_upload.(socket, item, meta, entry)
-        end)
+        if Map.has_key?(field_options, :consume_upload) do
+          consume_upload = Map.get(field_options, :consume_upload)
+
+          consume_uploaded_entries(socket, upload_key, fn meta, entry ->
+            consume_upload.(socket, item, meta, entry)
+          end)
+        end