prkhrkat

prkhrkat

I have a table in which I have put unique constraints on the combination of two fields. I had also defined this in changeset.

def changeset(asset, attrs) do
    asset
    |> cast(attrs, @create_attrs)
    |> validate_required(@required_attrs)
    |> unique_constraint([:serial_number, :organization_id])
  end

But I keep getting this error instead of the object {:error, #Ecto.Changeset…} -

Server: localhost:4000 (http)
Request: POST /api/create_asset
** (exit) an exception was raised:
    ** (Ecto.ConstraintError) constraint error when attempting to insert struct:

    * "assets_organization_id_serial_number_index" (unique_constraint)

If you would like to stop this constraint violation from raising an
exception and instead add it as an error to your changeset, please
call `unique_constraint/3` on your changeset with the constraint
`:name` as an option.


Until I found out that order also matters when writing the unique_constraint. It worked when i changed it to -

unique_constraint([:organization_id, :serial_number])

So I just wanted to put it here, so that we can fix this bug in the new release. Would also love to contribute if I’m able to fix it by myself.

Thanks

Showing Posts 1 to 3

sodapopcan

sodapopcan

This isn’t a bug, the “order” matters at the database level too. Ecto is using :org_id and :serial_number to create the name of the index. Otherwise, the order of compound indexes, at least in postgres, does matter.

al2o3cr

al2o3cr

The default behavior of unique_constraint is tripping you up here:

  • Putting organization_id first in the index makes sense since Postgres can use partial indexes from the left-hand side.
  • But putting serial_number first in the unique_constraint call makes sense too, since you likely want the “is already taken” message on the serial number rather than the organization ID.

One or the other of these has to change. Either:

  • tell unique_constraint the correct name of the index to use with the name: "assets_organization_id_serial_number_index" option
  • pass the fields to unique_constraint in the same order as the index (so the name can be automatically generated), then tell it to put the error on serial_number instead with error_key
prkhrkat

prkhrkat OP

Thanks @sodapopcan and @al2o3cr It makes sense.

— All posts loaded —

Where Next? Top

Trending in Discussions Top

juhalehtonen
There has been a thread to discuss the Stack Overflow Developer Survey on this forum every year since 2018, so here’s yet another one for...
New
axelson
Hi there! :wave: @frigidcode and I (but mostly him) have been running an Elixir Book club, we’re almost done with Designing Elixir Syste...
New
budgie
I love Elixir. It’s one of 2 programming languages I’ve ever fallen in love with. But I don’t use it anymore. Serverless was the promis...
New
Null-logic-0
What IDE or editor are you using for Elixir development? Personally, I use Zed, and I really like it, but sometimes I wish there were a ...
New
alexslade
Fly’s CEO posted this recently - Turn And Face The Strange · The Fly Blog It says that Fly is going all-in on sprites, which is a worry ...
New
jam
This was an interesting post by Jose. Curious to hear what others think. Evolving programming languages in the AI era - Dashbit Blog Gen...
New
szsoppa
Hey guys! Curiosum team has just published the State of Elixir results on Elixir Hub: see the results here. 1018 participants, 45 quest...
New

Other Trending Topics Top

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
mudasobwa
I fully migrated to my own harness from Anthropic/Gemini and I think it’s time to share it. Welcome DSH, the DeepSeek Harness, fully writ...
New
jimsynz
Beam Bots (or just BB for short) is a framework for building fault-tolerant robotics applications in Elixir using familiar OTP patterns. ...
New
ausimian
Emily is an Elixir library that runs Nx computations on Apple’s MLX. Install it as the default Nx backend and Nx, defn, Axon, Nx.Serving,...
New
OndrejValenta
(I just needed to vent somewhere and LinkedIn is full of hope, or hype, I’m not sure which exactly) AI dream has many faces, but general...
New
kip
In 2021 I started a new library called Tempo with the objective of modelling time as a set of intervals - not as instants. In 2022 I gave...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews