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!

Showing Posts 1 to 5

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.

tnederlof

tnederlof OP

Thanks for the response. I actually do want one user to have many groups. The groups are specific to each user and will be generated by the user. I really want to build a seed file passing the user struct to the creation of the group.

patrickdm

patrickdm

I see, :sweat_smile: sorry for the misleading answer then!

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.

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!

— All posts loaded —

Where Next? Top

Trending in Questions Top

stjefim
Hello! Suppose you are building workflow (order / task / payment) processing system with the following requirements: Each workflow con...
New
Blokh
Hey guys, I’ve got a huge CSV ( around 10 GB ) that needs to be processed hourly Do you guys have any suggestions what is the best prac...
New
kszambelanczyk
Hello! Could someone please give me a help/sample code, how to delete a file from s3 using waffle/waffle_ecto from Phoenix app. I creat...
New
Onor.io
I have what I’ve heard referred to as a “lookup table” in my database. This is a way of assigning codes to common values. One common lo...
New
jaybe78
Hello, I’m developing a online persistent chat system (what’s app) like using elixir/dynamodb/aws for a mobile app(flutter). The diffic...
New
Trolleger
What approach to take when sending live updates to “random” users Hi! I have a question, I have a little chat app, and when I create a DM...
New
widianto
I think I’ve found a small improvement I could contribute to &lt;%= web_namespace %&gt;.CoreComponents (installer/templates/phx_web/compo...
New

Other Trending Topics Top

garrison
Hobbes is a low-level distributed database for the Elixir programming language. Hobbes provides a simple, safe, and scalable storage lay...
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
Damirados
Hello everyone. After busy few months I am happy to announce v0.1.0 of Emerge &amp; Solve. They are GUI (Emerge) and State management (S...
New
netoum
Corex is an accessible, unstyled UI component library for Phoenix that integrates Zag.js state machines using Vanilla JavaScript and Live...
New
wintermeyer
There are three potential reasons for members of this forum to have a look at https://vutuv.de You are tired or annoyed of LinkedIn. Yo...
New
aseigo
ICal is a library for interacting with iCalendar data. It parses iCalendars into typed Elixir structs via ICal.from_ics, and can prepare ...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews