Return to create form after validation error for Ash authentication

Yes, you’d put the redirect in your failure/3 handler. I think the rendering of failure.html should eventually be removed from the guide and the default behavior should be to redirect.

What is the best way to solve this? The following code doesn’t display the old values and doesn’t give the validation errors:

  def failure(conn, _activity, _reason) do
    conn =
      conn
      |> assign(:form_id, "sign-up-form")
      |> assign(:cta, "Create account")
      |> assign(:action, ~p"/auth/user/password/register")
      |> assign(:is_register?, true)
      |> assign(
        :form,
        Form.for_create(User, :register_with_password, api: Accounts, as: "user")
      )

    render(conn, :register, layout: false)
  end

The tooling clearly could use some improvement for the non-live view cases here.

@jimsynz might be able to help more than me. Does the reason contain a failed form in this case? Or just the error from the form? Or perhaps the conn contains the failed form in the assigns?

What you’d want to do ideally is reassigned the form that failed to submit. That would render errors and old values.

How can I use this reason to assign the needed values to the new form?

iex(8)> reason
%Ash.Error.Invalid{
  errors: [
    %Ash.Error.Changes.InvalidArgument{
      field: :password,
      message: "length must be greater than or equal to %{min}",
      value: "1234",
      changeset: nil,
      query: nil,
      error_context: [],
      vars: [min: 8],
      path: [],
      stacktrace: #Stacktrace<>,
      class: :invalid
    }
  ],
  stacktraces?: true,
  changeset: #Ash.Changeset<
    api: Animina.Accounts,
    action_type: :create,
    action: :register_with_password,
    attributes: %{
      id: "1a2be27e-d2eb-41a8-955a-013a30035fe4",
      username: "example",
      email: #Ash.CiString<"example@example.com">
    },
    relationships: %{},
    arguments: %{},
    errors: [
      %Ash.Error.Changes.InvalidArgument{
        field: :password,
        message: "length must be greater than or equal to %{min}",
        value: "1234",
        changeset: nil,
        query: nil,
        error_context: [],
        vars: [min: 8],
        path: [],
        stacktrace: #Stacktrace<>,
        class: :invalid
      }
    ],
    data: #Animina.Accounts.User<
      __meta__: #Ecto.Schema.Metadata<:built, "users">,
      id: nil,
      email: nil,
      username: nil,
      aggregates: %{},
      calculations: %{},
      ...
    >,
    context: %{authorize?: false, actor: nil},
    valid?: false
  >,
  query: nil,
  error_context: [nil],
  vars: [],
  path: [],
  stacktrace: #Stacktrace<>,
  class: :invalid
}

I’m adding a new utility to AshPhoenix to support this. Specifically, it will be AshPhoenix.Form.add_error(form, error).

Is there a way to add reason.changeset to the form so that I at have the old values?

Oh, interesting. You can replace the form source, i.e %{form | source: reason.changeset}

Oh, interesting. You can replace the form source, i.e %{form | source: reason.changeset}. I’ve added AshPhoenix.Form.add_error/3 to ash_phoenix main that you can try as well.

I don’t understand that because this doesn’t work (because form is not there):

    conn =
      conn
      |> assign(:form_id, "sign-up-form")
      |> assign(:cta, "Account anlegen")
      |> assign(:action, ~p"/auth/user/password/register")
      |> assign(:is_register?, true)
      |> assign(
        :form,
        %{form | source: reason.changeset}
      )

The error:

    error: undefined variable "form"
    │
 30 │         %{form | source: reason.changeset}
    │           ^^^^
    │
    └─ lib/animina_web/controllers/auth_controller.ex:30:11: AniminaWeb.AuthController.failure/3

Right, I was referring the form that you were creating manually Form.for_create(User, :register_with_password, api: Accounts, as: "user")

I think in your case, the best thing to do would be something like this:

# not all failures will have the changeset. this signals a design issue to me, specifically we should always provide the params that failed. This will need to be added in a future release. Until then this should work well enough.

params = 
  if reason.changeset do
    reason.changeset.params
  else
    %{}
  end

      ....
      |> assign(
        :form,
        Form.for_create(User, :register_with_password, api: Accounts, as: "user", params: params)
      )