Updating array column: about moebius HOT 8 CLOSED

This is a Postgres encoding error, as you can see. You can try setting the value to a string, as Postgrex has recommended, and see if that works out.

from moebius.

It's not a postgrex problem: fazibear@e8ea243

from moebius.

This is a diff and doesn't explain anything to me. I can guess as to what you mean or you can simply tell me, which I would prefer. Keep in mind I have 10 other things that are occupying my mind and I'm dropping back into this issue after 30 days of not thinking about it once. I'd like to see if we can minimize the back and forth.

from moebius.

Sorry, I just showed you how I make it work in my fork. Here we have a problem with building params for query. In this example:

Moebius.Query.db(:users)
  |> Moebius.Query.filter(email: "[email protected]") 
  |> Moebius.Query.update(title: "test", roles: ["admin", "other"]) 
  |> App.Db.query

params hash looks like this: ["[email protected]", "test", "admin", "other"], witch is bad. It should look like: ["[email protected]", "test", ["admin", "other"]]. This is what List.flatten do. And it's not good in that place.

from moebius.

I understand that it solves your problem :). We put it there for a reason - what would be interesting is to know how the test suite behaves when running with your changes. I'd also like to know if my original suggestion of passing in a string works for you.

from moebius.

You mean like this ?

Moebius.Query.db(:users)
  |> Moebius.Query.filter(email: "[email protected]") 
  |> Moebius.Query.update(title: "test", roles: "['admin', 'other']") 
  |> App.Db.query

No it's not work, same error.

from moebius.

Yeah - bummer. I know I put the flatten statement in there to deal with the way parameters are passed in some cases. It would help if you ran test suite against your change so I could see what breaks.

from moebius.

Found the problem - thanks for the code :). Will be fixed with v3.

from moebius.