Support for filtering `null` columns about moebius HOT 5 CLOSED

Probably the easiest way is to do this:

db(:emails)
|> filter("deleted_at IS NULL")
|> all

Although what you wrote above should work. I'm a bit puzzled because we had a filter method in there where you could:

db(:emails)
|> filter(:id, ">", 100)
|> all

... which should allow you to do this. Anyway the query here should work for you (for now) - I''ll leave this open to work on for 1.0.9. Cheers.

from moebius.

I get an unidentified func filter/4 for your 2nd example.

The first example works, but not when I pipe through filter() a second time:

iex(36)> db(:emails) |> filter(user_id: 16) |> count
919

iex(37)> db(:emails) |> filter(user_id: 16) |> filter("deleted_at IS NULL") |> count
36081

## what the queries look like: 

iex(38)> db(:emails) |> filter(user_id: 16)                                         
%Moebius.QueryCommand{columns: nil, error: nil, group_by: nil, join: [""],
 limit: "", offset: "", order: "", params: [16], pid: nil, sql: nil,
 table_name: "emails", type: :select, vals: nil, where: " where user_id = $1",
 where_columns: [:user_id]}

iex(39)> db(:emails) |> filter(user_id: 16) |> filter("deleted_at IS NULL")         
%Moebius.QueryCommand{columns: nil, error: nil, group_by: nil, join: [""],
 limit: "", offset: "", order: "", params: [], pid: nil, sql: nil,
 table_name: "emails", type: :select, vals: nil,
 where: " where deleted_at IS NULL", where_columns: [:user_id]}

So it appears the 2nd pass through filter() is destructive.

from moebius.

You can't pass through to filters now. This is something that I wanted to look at. @taylorbrooks We are doing it but updating :where in the map

—
Thanks,

Johnny

On Thu, Nov 19, 2015 at 9:07 PM, Taylor Brooks [email protected]
wrote:

I get an unidentified func filter/4 for your 2nd example.
The first example works, but not when I pipe through filter() a second time:

iex(36)> db(:emails) |> filter(user_id: 16) |> count
919
iex(37)> db(:emails) |> filter(user_id: 16) |> filter("deleted_at IS NULL") |> count
36081
## what the queries look like: 
iex(38)> db(:emails) |> filter(user_id: 16)                                         
%Moebius.QueryCommand{columns: nil, error: nil, group_by: nil, join: [""],
 limit: "", offset: "", order: "", params: [16], pid: nil, sql: nil,
 table_name: "emails", type: :select, vals: nil, where: " where user_id = $1",
 where_columns: [:user_id]}
iex(39)> db(:emails) |> filter(user_id: 16) |> filter("deleted_at IS NULL")         
%Moebius.QueryCommand{columns: nil, error: nil, group_by: nil, join: [""],
 limit: "", offset: "", order: "", params: [], pid: nil, sql: nil,
 table_name: "emails", type: :select, vals: nil,
 where: " where deleted_at IS NULL", where_columns: [:user_id]}

So it appears the 2nd pass through filter() is destructive.

Reply to this email directly or view it on GitHub:
#35 (comment)

from moebius.

I guess I should look at this issue :)

from moebius.

The filter issue is fixed; null values (from OP) are handled with string pass. If I'm wrong on understanding this please reopen.

from moebius.