Oops the action should be :update here instead of :edit (which is just a GET request to show the form).
Kurisu
0
sd4code
Post method is best in certain cases where you don’t want a url query string on the location bar, because it hides the internal parameters. This is an application. Not a website. So those XML web developers need to know that xml is a transport language only in applications. So there is no destinations to remote query. I imagine you could get the jsnode to generate something, but it could lead to security exploits.
0
ashok
In case of add new card I am able to insert the card new details also able to show validation message like for currency if some one tries to enter text input form willl not get submit and shows invalid data error. But in my form all values filled by user got clear instead of showing old values.
Please suggest how can I get form values filled by user instead of showing all fields blank this will help to reduce user rework from filling all values again and again
0
Kurisu
Well you have already the path set here:
Example of User controller:
def new(conn, _params) do
changeset = Accounts.change_user(%User{})
render(conn, "new.html", changeset: changeset)
end
def create(conn, %{"user" => user_params}) do
case Accounts.create_user(user_params) do
{:ok, user} ->
conn
|> put_flash(:info, "User created successfully.")
|> redirect(to: Routes.user_path(conn, :show, user))
{:error, %Ecto.Changeset{} = changeset} ->
render(conn, "new.html", changeset: changeset)
end
end
def edit(conn, %{"id" => id}) do
user = Accounts.get_user!(id)
changeset = Accounts.change_user(user)
render(conn, "edit.html", user: user, changeset: changeset)
end
def update(conn, %{"id" => id, "user" => user_params}) do
user = Accounts.get_user!(id)
case Accounts.update_user(user, user_params) do
{:ok, user} ->
conn
|> put_flash(:info, "User updated successfully.")
|> redirect(to: Routes.user_path(conn, :show, user))
{:error, %Ecto.Changeset{} = changeset} ->
render(conn, "edit.html", user: user, changeset: changeset)
end
end
Then the Accounts Context
def create_user(attrs \\ %{}) do
%User{}
|> User.changeset(attrs)
|> Repo.insert()
end
def update_user(%User{} = user, attrs) do
user
|> User.changeset(attrs)
|> Repo.update()
end
def change_user(%User{} = user) do
User.changeset(user, %{})
end
For the forms only the action and the page title should be different.
You set Routes.user_path(@conn, :create) for new form
and You set Routes.user_path(@conn, :update, @user) for edit form.
If you’re facing the same issue even though you follow this logic maybe it is because you’re nesting complex level of parents and childrens associations in your form ? In this case maybe the below thread I created myself some times ago could help you? Good luck. ^^
0