Doesn't act as expected !

Hi, I was just testing the library and I faced that in one exact case it doesn’t return expected result. test1

//given
                        let obj1 = {
                            c: { d: "d" }
                        };
                        let obj2 = {
                            a: "done",
                            b: 34,
                            c: { d: "d", e:"e" }
                        };
 //when
                        let result = sift(obj1, [obj2]); 
// result = [obj2]

test2

//given
//collection contains obj2
//when
                       let result = collection.find(obj1); 
// result = [ ]

Shouldn’t the results of both tests be the same ? Mongodb returns obj2 in that case :

                       let obj1 = {
                            "c.d": "d"
                        };
                       let result = collection.find(obj1); 
// result = [obj2]

[PascalPflaum]

That the two queries collection.find(c: { d: 'd'}); and collection.find('c.d': 'd');

have different results is by purpose in MongoDB and one of the tiny details, that makes this query language so powerful.

• Query 1 looks for a property c that is equal to { d: "d" }
• Query 2 looks for a nested property d in c that is equal to d

I’m in favor of reflecting the mongoDB behaviour as close as possible, something that looks like MongoDB Query Language, but doesn’t behave like MongoDB Query Language is worse than a total different system in my opinion.

[crcn]

If I’m understanding correctly, you’d expect this in sift:

const obj1 = { c: { d: 'd' }};
const obj2 = { a: 'done', c: { d: 'd' }};

const result = sift(obj1, [obj2]); // []

Where result would be an empty array. Currently sift returns obj2 based on the obj1 query – this is intented behavior from when I first added the functionality a few years ago. I’d be open to changing this however to reflect mongodb’s behavior a bit.

Mongodb returns obj2 in that case

Sift supports string queries like this: https://github.com/crcn/sift.js/blob/master/test/operations-test.js#L148