michaelherold

michaelherold

Why does Phoenix.HTML.FormData.Ecto.Changeset.form_for_errors/1 only copy errors into the form when the action is set to :insert, :update, :delete, or :replace? I understand not copying the errors on :ignore, but the nil clause is interesting to me.

A little explanation of what I’m trying to do:

defmodule MyApp.MyDomain.MySchema do
  embedded_schema do
    embeds_one :foo, Foo
    embeds_one :bar, Bar
  end

  @doc """
  This used in my Phoenix controller as shown below.
  """
  def changeset(model, attrs) do
    model
    |> cast(attrs, [])
    |> cast_embed(:foo)
    |> cast_embed(:bar)
  end
end
<%= form_for @changeset, my_path(@conn, :create), fn form -> %>
  <%= inputs_for form, :foo, fn foo_form -> %>
  <% end %>
  <%= inputs_for form, :bar, fn bar_form -> %>
  <% end %>
<% end %>

This schema is basically a compatibility layer that gets transformed into an Ecto.Multi for insertion. Since Ecto.Multi has no implementation for Phoenix.HTML.FormData, this is the way I chose to go about working the problem.

Because the changeset I’m displaying is based on this embedded schema, the :action for the changeset is set to nil (I think … that’s how I understand it anyway). As such, form_for_errors/1 ignores the errors that are on the nested embedded schemas.

Why isn’t form_for_errors/1 written like this?

defp form_for_errors(%{action: :ignore}), do: []
defp form_for_errors(%{errors: errors}), do: errors

This implementation breaks five different tests, so it’s clear that we mean to ignore the errors when %{action: nil}.

Why is this? Is there some other way that I should approach the problem?

Where Next? Top

Trending in Questions Top

RSP87
I’m working on a project that simulates the bumbl example in the programming phoenix book. It acts almost like an email client. We have a...
New
kszambelanczyk
Hello! Could someone please give me a help/sample code, how to delete a file from s3 using waffle/waffle_ecto from Phoenix app. I creat...
New
RemyXRenard
I’m seeing that a list inside a Kino.DataTable will be interpreted as a charlist, even if the Kino.configure() is set to charlists: :as_l...
New
velrest
So my question is quite simple and i have found no conclusive answer on forum, google or AI. Should we use :erlang.float for Integer to ...
New
samoloth
Hi, I’ve just set up an application with ash_authentication. There is only magic link strategy for now, so there is no confirmation add o...
New
FlyingNoodle
If a change or preparation module uses Ash.Changeset.get_argument/2 or Ash.Query.get_argument/2 (or any of the other get_argument functio...
New
ryanwinchester
apply_graft/2 doesn’t rewrite an add_many sub-workflow’s deps on an add step. Grafted jobs cancel with “upstream job was deleted” Version...
New

Other Trending Topics Top

mudasobwa
I am happy to introduce the very α version of the new programming language compiled to BEAM. Welcome Cure. It has literally three kille...
New
garrison
Hobbes is a low-level distributed database for the Elixir programming language. Hobbes provides a simple, safe, and scalable storage lay...
New
marciok
Hi there! We created Gust: A task orchestrator inspired by Airflow. For those who have never heard about Aiflow, it’s a Python-based wor...
New
jimsynz
Beam Bots (or just BB for short) is a framework for building fault-tolerant robotics applications in Elixir using familiar OTP patterns. ...
New
Dmk
Xamal is a deployment tool for Elixir apps that deploys native releases to bare metal servers over SSH. It’s a port of GitHub - basecamp/...
New
Damirados
Hello everyone. After busy few months I am happy to announce v0.1.0 of Emerge &amp; Solve. They are GUI (Emerge) and State management (S...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews