Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get $not support working. #6

Closed
ekryski opened this issue Nov 16, 2015 · 3 comments
Closed

Get $not support working. #6

ekryski opened this issue Nov 16, 2015 · 3 comments

Comments

@ekryski
Copy link
Member

ekryski commented Nov 16, 2015

NeDB's $not support is not the same as Mongoose or our Knex implementation. It looks like this:

db.find({ $not: { planet: 'Earth' } }, function (err, docs) {
  // docs contains Mars, Jupiter, Omicron Persei 8
});

but should look like this:

db.find({ planet: { $not: 'Earth' } }, function (err, docs) {
  // docs contains Mars, Jupiter, Omicron Persei 8
});
@daffl
Copy link
Member

daffl commented Nov 16, 2015

$not isn't implemented anymore in the default query filter (it's $ne). This should still work (though not DB independent) right?

@ekryski
Copy link
Member Author

ekryski commented Nov 16, 2015

Yes it's fine, but I'm just putting it down as an issue to track, as I would like to support it at some point.

@daffl
Copy link
Member

daffl commented Jul 20, 2016

This has been open for a while and I don't think we should tackle it individually. Maybe when adding $not support to all databases via the service tests. If you want to use NeDBs not you should just be able to use it directly. Closing.

@daffl daffl closed this as completed Jul 20, 2016
@daffl daffl removed the Backlog label Jul 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants