Fix merge of nested criterias like #23

Merged
merged 1 commit into from Oct 10, 2012

Conversation

Projects
None yet
3 participants
@divoxx

divoxx commented May 23, 2012

When merging criterias wich has a deeper object structure, like $elemMatch, the current implementation would transform the object into arrays.

Improve the CriteriaHash#merge method, to allow recursively merge nested structures which also fix this problem.

@luis-ca

This comment has been minimized.

Show comment Hide comment
@luis-ca

luis-ca Sep 24, 2012

I have come across this as well, specifically when merging the $in operator. Any reason why this pull request hasn't been accepted?

luis-ca commented Sep 24, 2012

I have come across this as well, specifically when merging the $in operator. Any reason why this pull request hasn't been accepted?

jnunemaker added a commit that referenced this pull request Oct 10, 2012

@jnunemaker jnunemaker merged commit 76dc51b into mongomapper:master Oct 10, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment