Merge pull request #6 from jusx/bugfix-request-querystring

Fix get request to send params as querystring and the circleci build.

circle.yml

@@ -1,3 +1,3 @@
-test:
-   override:
-      - mocha
+machine:
+  node:
+    version: 4.5.0

lib/wufoo.js

@@ -127,7 +127,11 @@ Wufoo.prototype.request = function(method, uri, params, fn) {
       }
    }
 
-   if (fn!=undefined && params !=undefined) options["body"] = require('querystring').stringify(params); // params for form post was passed in.
+   if (fn!=undefined && params !=undefined) {
+      var stringified = require("querystring").stringify(params);
+      if (options["method"]=="GET") options["uri"] = options["uri"] +  "?" + stringified;
+      else options["body"] = stringified; // params for form post was passed in.
+   }
    if (fn == undefined && params != undefined) fn = params; // no params was passed in. In which case it is fn callback.
 
    request(options, function(err, res, body) {

package.json

@@ -2,8 +2,13 @@
   "name": "wufoo",
   "description": "Wrapper for the Wufoo API (http://www.wufoo.com/docs/api/v3/)",
   "version": "1.1.0",
-  "author": {"name": "Justin Law", "email": "<hello@justinlaw.org>"},
-  "contributors": ["Ahmader <hello@ahmader.sa>"],
+  "author": {
+    "name": "Justin Law",
+    "email": "<hello@justinlaw.org>"
+  },
+  "contributors": [
+    "Ahmader <hello@ahmader.sa>"
+  ],
   "dependencies": {
     "request": "^2.83.0"
   },
@@ -13,11 +18,23 @@
     "url": "https://github.com/jusx/node-wufoo.git"
   },
   "homepage": "https://github.com/jusx/node-wufoo",
-  "keywords": ["wufoo", "wufoo api", "api"],
-  "devDependencies": { "mocha": "*", "should":"*" },
+  "keywords": [
+    "wufoo",
+    "wufoo api",
+    "api"
+  ],
+  "devDependencies": {
+    "mocha": "*",
+    "should": "*"
+  },
   "license": "MIT",
   "bugs": {
-      "url": "https://github.com/jusx/node-wufoo/issues"
-    },
-  "engines": ["node >= 0.8.0"]
-}
+    "url": "https://github.com/jusx/node-wufoo/issues"
+  },
+  "scripts": {
+    "test": "mocha"
+  },
+  "engines": [
+    "node >= 0.8.0"
+  ]
+}