tnederlof

tnederlof

I have a Phoenix app which is using ecto/postgres. I have setup two schemas (User has many Groups) and a seed file (contents below). I am running into an error when I add the group (which is supposed to have a user). I feel like the user association, where the constraint goes (requiring user_id vs assoc_constraint(:user)) and then how the seeding flows I am confused on.

Error I am getting

** (Ecto.InvalidChangesetError) could not perform insert because changeset is invalid.

Errors

    %{user_id: [{"can't be blank", [validation: :required]}]}

Applied changes

    %{description: "Random text", name: "Family"}

Params

    %{
      "description" => "Random text",
      "name" => "Family",
      "user" => %XXXX.Accounts.User{
        __meta__: #Ecto.Schema.Metadata<:loaded, "users">,
        email: "example@gmail.com",
        groups: #Ecto.Association.NotLoaded<association :groups is not loaded>,
        id: 1,
        inserted_at: ~N[2020-05-19 22:43:11],
        name: "Example User",
        password: "secret",
        password_hash: "XXXXXXXX",
        phonenumber: "1 203-247-7113",
        updated_at: ~N[2020-05-19 22:43:11],
        username: "exampleuser"
      }
    }

Changeset

    #Ecto.Changeset<
      action: :insert,
      changes: %{description: "Random text", name: "Family"},
      errors: [user_id: {"can't be blank", [validation: :required]}],
      data: #XXXX.People.Group<>,
      valid?: false

User Schema

defmodule XXXX.Accounts.User do
  use Ecto.Schema
  import Ecto.Changeset

  schema "users" do
    field :email, :string
    field :name, :string
    field :password_hash, :string
    field :password, :string, virtual: true
    field :phonenumber, :string
    field :username, :string

    has_many :groups, XXXX.People.Group

    timestamps()
  end

  def changeset(user, attrs) do
    user
    |> cast(attrs, [:name, :username, :email, :password, :phonenumber])
    |> validate_required([:name, :username, :email, :password, :phonenumber])
    |> validate_length(:username, min: 4)
    |> validate_length(:password, min: 4)
    |> unique_constraint(:username)
    |> unique_constraint(:email)
    |> unique_constraint(:phonenumber)
    |> hash_password()
  end

  defp hash_password(changeset) do
    case changeset do
      %Ecto.Changeset{valid?: true, changes: %{password: password}} ->
        put_change(changeset, :password_hash, Pbkdf2.hash_pwd_salt(password))
      _ ->
        changeset
    end
  end
end

User Migration

defmodule XXXX.Repo.Migrations.CreateUsers do
  use Ecto.Migration

  def change do
    create table(:users) do
      add :name, :string, null: false
      add :username, :string, null: false
      add :email, :string, null: false
      add :password_hash, :string, null: false
      add :phonenumber, :string, null: false

      timestamps()
    end

    create unique_index(:users, [:username, :email, :phonenumber])
  end
end

Group Schema

defmodule XXXX.People.Group do
  use Ecto.Schema
  import Ecto.Changeset

  schema "groups" do
    field :description, :string
    field :name, :string

    belongs_to :user, XXXX.Accounts.User

    timestamps()
  end

  def changeset(group, attrs) do
    required_fields = [:name, :user_id]
    optional_fields = [:description]

    group
    |> cast(attrs, required_fields ++ optional_fields)
    |> validate_required(required_fields)
    |> assoc_constraint(:user)
  end
end

Group Migration

defmodule XXXX.Repo.Migrations.CreateGroups do
  use Ecto.Migration

  def change do
    create table(:groups) do
      add :name, :string, null: false
      add :description, :text
      add :user_id, references(:users, on_delete: :delete_all), null: false

      timestamps()
    end

  end
end

seeds.exs

alias XXXX.Repo
alias XXXX.People.Group
alias XXXX.Accounts.User


user_example_1 = %User{} |> User.changeset(%{
  name: "Example User",
  username: "exampleuser",
  phonenumber: "1 834-000-0000",
  email: "example@gmail.com",
  password: "secret"
}) |> Repo.insert!

example_group1 = %Group{} |> Group.changeset(%{
  name: "Family",
  description: "Random text",
  user: user_example_1
}) |> Repo.insert!

First Post!

patrickdm

patrickdm

Hello, I am not sure I understand you use case: with this schema each groups record is related to only one users record (groups record belong to a single user, via the user_id foreign key): so you can have many groups records related to the same user, but you can’t have multiple users in the same group. Then I’m not sure this is what you want to model with your schema. In my opinion it would make more sense to relate users and groups with a many-to-many relationship, so you can have a user belonging to multiple groups and a group containing many different users. If that is the case, you can find more details in the ecto docs https://hexdocs.pm/ecto/Ecto.Association.ManyToMany.html#content, else I apologize for misunderstanding your question.

Most Liked

kokolegorille

kokolegorille

It is usually not a good idea to expose id field in the changeset.

You can do it in different ways, but if You have already a user, You could use put_assoc.

Another way to do it is to use build_assoc.

Last Post!

tnederlof

tnederlof OP

Thank you! Reading through put_assoc I learned a lot more about associations and changesets. I can see why exposing the id field to the changeset is a bad idea, I will stick with using put and build for now. I appreciate your help again and patience, I am learning a lot!

Where Next? Top

Trending in Questions Top

katta
I having some trouble figuring out if I have set myself too strict of standards for my production server. Currently I can handle 75% of r...
New
achenet
Hello, I’m trying to build a basic Phoenix web-app, and I’d like to use Tailwind. However, when I launch mix phx.server, I get an error...
New
kpanic
Hi everyone, I am toying with the idea of building a “match maker” for giving personal help to people that wants to start coding. I sta...
New
Cxx-mlr
I’m working on a small exercise involving update_in/3, and I came up with this solution: data = %{ name: "Periodic Table", category:...
New
ChrisAmelia
I’ve got trouble wrapping my head around the order in which functions are called in this snippet (from Phoenix’s authentication): toke...
New
dillonoconnor
Is there any way to avoid the Hologram compiler running when using iex? It seems like the front-end code could potentially be disregarded...
New
thiagogsr
** (ArgumentError) expected :max_attempts to be a positive integer, got: {:@, [line: 10, column: 19], [{:max_attempts, [line: 10, column:...
New

Other Trending Topics Top

GenericJam
Edit: 2026 May 15 - This post is archived. Mob is alive!! Main docs: mob v0.7.11 — Documentation A bit of explanation for the slightly c...
New
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
budgie
A little off-topic, but I feel like people here have a good head on their shoulders. I used to be quite good at making software. Was luc...
New
KristerV
Hey. Is there anyone here who creates agents in their apps? Not talking about using agents, but creating them. I’m finding it pretty diff...
New
mcass19
ExRatatui lets you cook up rich terminal UIs in Elixir, powered by Rust’s ratatui via Rustler NIFs. Build interactive terminal applicatio...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews