Ecto model is not yet saved, when Arc is saving the file, resulting in empty scope about arc_ecto HOT 10 CLOSED

Thank you @stavro, i think that is a reasonable solution.

Maybe we should add this example to the documentation, i can imagine other could run into the same wall, specially if you are comming from carrierwave in Rails 😃

from arc_ecto.

What if Article.avatar_changeset(article, article_params) |> Repo.update fails? There's no transaction and no rollback.

from arc_ecto.

This information should definitely be in the documentation. I just spent a couple of hours scratching my head and looking for a solution. I don't mind that it doesn't work like Paperclip, but this use case is the most common, specially if you want to reuse a generic Upload schema for e.g. User, Photo, etc. In that case, the scope will not exist before the file is uploaded.

I just realized that you could scope the file to an existing field of the struct, but in any case, this should be documented.

from arc_ecto.

@Bounga maybe try something like this

Repo.transaction(fn ->
  with {:ok, product} <- Product.changeset(%Product{}, params) |> Repo.insert,
       {:ok, product_with_photo} <- Product.photo_changeset(product, params) |> Repo.update
  do
    {:ok, product_with_photo}
  else
    {:error, changeset} ->
      Repo.rollback(changeset)
      {:error, changeset}
  end
end)

That said, it would be so nice if it works like Paperclip allowing you to add the attachment at the same time as creating new object.

from arc_ecto.

Using option (2) above (waiting until after the model is saved), you could:

#
# Model
#
def creation_changeset(params) do
  %Article{} |> cast(params, ~w(name), ~w())
end

def avatar_changeset(article, params) do
  article |> cast_attachments(params, ~w(avatar), ~w())
end

#
# Controller
#
def create(conn, %{"article" => article_params}) do
  # Add author.
  article_params = Dict.put_new(article_params, "admin_id", Plug.Conn.get_session(conn, :current_user))
  changeset = Article.creation_changeset(%Article{}, article_params)

  case Repo.insert(changeset) do
    {:ok, article} ->
      article = Article.avatar_changeset(article, article_params) |> Repo.update

      conn
      |> put_flash(:info, "Article created successfully.")
      |> redirect(to: admin_article_path(conn, :index))
    {:error, changeset} ->
      render(conn, "new.html", changeset: changeset)
  end
end

If you need it all saved with one call to the database then cannot use the id in the storage directory.

from arc_ecto.

I had the same problem. Thanks for addressing this issue.

from arc_ecto.

If you need to upload a file before the model is saved, then you cannot use the id of that model.

You should either generate a UUID for the file name or wait until after the model is saved.

Can you explain a little more about the workflow and model? There's probably a better structure to the files that we can come up with.

from arc_ecto.

Hallo @stavro,

It's a new/create action in Phoenix, almost standard code generated by Phoenix. Basically i enter my form, and select images to my form.

How would i wait for the model to be saved, isn't the image saved as a part of cast_attachments? So i would have to manually call store?

from arc_ecto.

Thanks for the solution. This works!

from arc_ecto.

Thanks!!!!

from arc_ecto.