Unique_constraint issue

benonymus · August 16, 2018, 5:39pm

Hey,
I am trying to add unique_contrain to my projects, and it is working beautifully with one of my tables that has only one field, but i with the other one it is not working.
I want to show to the user that a name is already taken on creation or at edit, but it keeps giving me an error that assign @teams not available in eex template.

but I think that the unique_constarint should kick in before this, that it should not get to the point that it redirects, so why is this happening?

the code: https://github.com/benonymus/userteamroleapi

OvermindDL1 · August 16, 2018, 5:55pm

This means that the render call for that template did not include a teams: ... argument. What is the exact url to that render call?

OvermindDL1 · August 16, 2018, 5:57pm

I’m guessing the error is happening at?

github.com

benonymus/userteamroleapi/blob/master/lib/userteam1_web/templates/user/form.html.eex#L22


</div>


<div class="form-group">
  <%= label f, "Wachtwoord", class: "control-label" %>
  <%= text_input f, :password, class: "form-control" %>
  <%= error_tag f, :password %>
</div>


<div class="form-group">
  <%= label f, :team_id, class: "control-label" %>
  <%= select f, :team_id, @teams, class: "form-control" %>
  <%= error_tag f, :team_id %>
</div>


<div class="form-group">
  <%= label f, "GebruikersType", class: "control-label" %>
  <%= select f, :role_id, @roles, class: "form-control" %>
  <%= error_tag f, :role_id %>
</div>


<div class="form-group">

If so then two render calls are missing teams: ..., one at:

github.com

benonymus/userteamroleapi/blob/master/lib/userteam1_web/controllers/user_controller.ex#L43




  roles =
    Repo.all(Userteam1.Web.Role)
    |> Enum.map(&{&1.name, &1.id})


  case Web.create_user(user_params) do
    {:ok, user} ->
      conn
      |> put_flash(:info, "User created successfully.")
      |> redirect(to: user_path(conn, :show, user))


    {:error, %Ecto.Changeset{} = changeset} ->
      render(conn, "new.html", changeset: changeset, teams: teams, roles: roles)
  end
end


def show(conn, %{"id" => id}) do
  user = Web.get_user!(id)
  render(conn, "show.html", user: user)
end

And the other at:

github.com

benonymus/userteamroleapi/blob/master/lib/userteam1_web/controllers/user_controller.ex#L80




def update(conn, %{"id" => id, "user" => user_params}) do
  user = Web.get_user!(id)


  teams =
    Repo.all(Userteam1.Web.Team)
    |> Enum.map(&{&1.name, &1.id})


  roles =
    Repo.all(Userteam1.Web.Role)
    |> Enum.map(&{&1.name, &1.id})


  # to have emtpy option for the user
  teams = [{"", ""} | teams]


  case Web.update_user(user, user_params) do
    {:ok, user} ->
      conn
      |> put_flash(:info, "User updated successfully.")
      |> redirect(to: user_path(conn, :show, user))

pedromvieira · August 16, 2018, 6:40pm

In addition to those, he should also reference @teams on both new and edit templates (that calls form).

github.com

benonymus/userteamroleapi/blob/master/lib/userteam1_web/templates/user/edit.html.eex

<h2>Edit User</h2>

<%= render "form.html", Map.put(assigns, :action, user_path(@conn, :update, @user)) %>

<span><%= link "Terug", to: user_path(@conn, :index) %></span>

github.com

benonymus/userteamroleapi/blob/master/lib/userteam1_web/templates/user/new.html.eex

<h2>New User</h2>

<%= render "form.html", Map.put(assigns, :action, user_path(@conn, :create)) %>

<span><%= link "Terug", to: user_path(@conn, :index) %></span>

Something like this.

   <%=
    render "form.html",
    ...
    teams: @teams,
    ...
  %>