Skip to content

Add support for matching array values à la mongo. #710

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

Merged
merged 2 commits into from
Aug 31, 2015
Merged

Add support for matching array values à la mongo. #710

merged 2 commits into from
Aug 31, 2015

Conversation

mazesoul
Copy link
Contributor

Support mongo style matcher in where clause.

embedsMany, referencesMany all have type array properties that can be filtered with when using loopback-mongo-connector. These don't have counterparts in relational dbs but precluding the use of the filters in the memory connectors complicates tests for mongo based apps.

Fixes the memory connector support mentionned in strongloop/loopback#953, strongloop/loopback#517.

Provides a baseline for the memory connector support of #625. At least for unit tests of a mongo based app.

Other issues related to this: #147

@mazesoul
Copy link
Contributor Author

Trying to retrigger build by close and open

@mazesoul
Copy link
Contributor Author

@raymondfeng Could you have a look at this, it's the "simple array of values" counter part of the change made in #697. This is useful when using 'referencesMany'. I know other connector don't support this but I doesn't prevent anything and is massively useful when building unit test for a mongo base app.

Thanks.

raymondfeng added a commit that referenced this pull request Aug 31, 2015
…matching

Add support for matching array values à la mongo.
@raymondfeng raymondfeng merged commit e91b5cf into loopbackio:master Aug 31, 2015
@mazesoul mazesoul deleted the feature/support-implicit-array-matching branch September 2, 2015 18:30
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

Successfully merging this pull request may close these issues.

3 participants