Skip to content
This repository has been archived by the owner on Nov 12, 2017. It is now read-only.

random fixes #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

random fixes #6

wants to merge 5 commits into from

Conversation

ghispi
Copy link

@ghispi ghispi commented May 7, 2014

random fixes to errors I came across

ghispi and others added 5 commits May 7, 2014 15:53
- _set_where not needed as it is passed to _count
- in _count its better to use library count then fetching all results and counting them
initially each callback got data from operation and forgot any changes from previous callback function, now changed data is passed to the next callback
 - fix to allow updating subdocument fields
 - fix to allow null values for keys, null-byte injection is about null character in key, not in value
it was updating only one document
@ghispi ghispi closed this Jun 13, 2014
@ghispi ghispi reopened this Jun 13, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant