-
Notifications
You must be signed in to change notification settings - Fork 70
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
Count method on index - argument are ignored #53
Comments
Solved it for me by adding the needed arguments at line 501: "count": function(range) { and line 278: "count": function(range) { This is a great plugin, thanks for all the hard work ! |
Can you submit a pull request for this please ? |
Im new here, dont know what a pull request is |
This bug nearly made me throw my machine out of the window (especially: this is not fucking documented ANYWHERE)! Let me write up a PR. |
@axemclion see #57 . @whodies next time, just fork the repository, clone your forked repository, commit and push your fix. Then create a pull request by going on your forked repo and clicking "create pull request". |
All this stuff is chinese to me. I should read up on what forking and cloning means. Appreciate your help though ! |
Seems that the index count method automatically submit an empty array of argument rather than the arguments I give it - Line 501 :
"count": function() {
return indexOp("count", indexName , []);
},
The text was updated successfully, but these errors were encountered: