Skip to content

Commit f001a2a

Browse files
Merge pull request #3 from christianbundy/FixSearch
Use querystring to create params.
2 parents 376835d + 8586852 commit f001a2a

File tree

1 file changed

+5
-19
lines changed

1 file changed

+5
-19
lines changed

index.js

Lines changed: 5 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,8 @@
11
'use strict';
22

3+
var querystring = require('querystring');
34
var request = require('request');
45

5-
var objectToParams = function(obj) {
6-
var first = true;
7-
var params = '?';
8-
for (var key in obj) {
9-
if (obj.hasOwnProperty(key)) {
10-
if (first) {
11-
first = false;
12-
} else {
13-
params += '&';
14-
}
15-
params += key + '=' + obj[key];
16-
}
17-
}
18-
};
19-
206
var hn = {
217
// make a request to the specified endpoint
228
call: function (endpoint, cb) {
@@ -145,12 +131,12 @@ var hn = {
145131

146132
// search popular/recent
147133
search: function (obj, cb) {
148-
var params = objectToParams(obj);
149-
hn.call('search' + params, cb);
134+
var params = querystring.stringify(obj);
135+
hn.call('search?' + params, cb);
150136
},
151137
searchLast: function (obj, cb) {
152-
var params = objectToParams(obj);
153-
hn.call('search_by_date' + params, cb);
138+
var params = querystring.stringify(obj);
139+
hn.call('search_by_date?' + params, cb);
154140
}
155141
};
156142

0 commit comments

Comments
 (0)