Merge pull request #89 from ChrisNLott/Swift-and-ObjC-cannot-handle-JSON-Obj-with-null-value

Swift and ObjC cannot handle JSON Obj with null value

Author: ahmadnassri

src/targets/objc/helpers.js

@@ -64,6 +64,9 @@ module.exports = {
       case '[object Boolean]':
         return value ? '@YES' : '@NO'
       default:
+        if (value === null || value === undefined) {
+          return ''
+        }
         return '@"' + value.toString().replace(/"/g, '\\"') + '"'
     }
   }

src/targets/swift/helpers.js

@@ -74,6 +74,9 @@ module.exports = {
       case '[object Boolean]':
         return value.toString()
       default:
+        if (value === null || value === undefined) {
+          return ''
+        }
         return '"' + value.toString().replace(/"/g, '\\"') + '"'
     }
   }

test/fixtures/output/c/libcurl/jsonObj-null-value.c

@@ -0,0 +1,12 @@
+CURL *hnd = curl_easy_init();
+
+curl_easy_setopt(hnd, CURLOPT_CUSTOMREQUEST, "POST");
+curl_easy_setopt(hnd, CURLOPT_URL, "http://mockbin.com/har");
+
+struct curl_slist *headers = NULL;
+headers = curl_slist_append(headers, "content-type: application/json");
+curl_easy_setopt(hnd, CURLOPT_HTTPHEADER, headers);
+
+curl_easy_setopt(hnd, CURLOPT_POSTFIELDS, "{\"foo\":null}");
+
+CURLcode ret = curl_easy_perform(hnd);

test/fixtures/output/csharp/restsharp/jsonObj-null-value.cs

@@ -0,0 +1,5 @@
+var client = new RestClient("http://mockbin.com/har");
+var request = new RestRequest(Method.POST);
+request.AddHeader("content-type", "application/json");
+request.AddParameter("application/json", "{\"foo\":null}", ParameterType.RequestBody);
+IRestResponse response = client.Execute(request);

test/fixtures/output/go/native/jsonObj-null-value.go

@@ -0,0 +1,28 @@
+package main
+
+import (
+	"fmt"
+	"strings"
+	"net/http"
+	"io/ioutil"
+)
+
+func main() {
+
+	url := "http://mockbin.com/har"
+
+	payload := strings.NewReader("{\"foo\":null}")
+
+	req, _ := http.NewRequest("POST", url, payload)
+
+	req.Header.Add("content-type", "application/json")
+
+	res, _ := http.DefaultClient.Do(req)
+
+	defer res.Body.Close()
+	body, _ := ioutil.ReadAll(res.Body)
+
+	fmt.Println(res)
+	fmt.Println(string(body))
+
+}

test/fixtures/output/java/okhttp/jsonObj-null-value.java

@@ -0,0 +1,11 @@
+OkHttpClient client = new OkHttpClient();
+
+MediaType mediaType = MediaType.parse("application/json");
+RequestBody body = RequestBody.create(mediaType, "{\"foo\":null}");
+Request request = new Request.Builder()
+  .url("http://mockbin.com/har")
+  .post(body)
+  .addHeader("content-type", "application/json")
+  .build();
+
+Response response = client.newCall(request).execute();

test/fixtures/output/java/unirest/jsonObj-null-value.java

@@ -0,0 +1,4 @@
+HttpResponse<String> response = Unirest.post("http://mockbin.com/har")
+  .header("content-type", "application/json")
+  .body("{\"foo\":null}")
+  .asString();

test/fixtures/output/javascript/jquery/jsonObj-null-value.js

@@ -0,0 +1,15 @@
+var settings = {
+  "async": true,
+  "crossDomain": true,
+  "url": "http://mockbin.com/har",
+  "method": "POST",
+  "headers": {
+    "content-type": "application/json"
+  },
+  "processData": false,
+  "data": "{\"foo\":null}"
+}
+
+$.ajax(settings).done(function (response) {
+  console.log(response);
+});

test/fixtures/output/javascript/xhr/jsonObj-null-value.js

@@ -0,0 +1,17 @@
+var data = JSON.stringify({
+  "foo": null
+});
+
+var xhr = new XMLHttpRequest();
+xhr.withCredentials = true;
+
+xhr.addEventListener("readystatechange", function () {
+  if (this.readyState === this.DONE) {
+    console.log(this.responseText);
+  }
+});
+
+xhr.open("POST", "http://mockbin.com/har");
+xhr.setRequestHeader("content-type", "application/json");
+
+xhr.send(data);

test/fixtures/output/node/native/jsonObj-null-value.js

@@ -0,0 +1,27 @@
+var http = require("http");
+
+var options = {
+  "method": "POST",
+  "hostname": "mockbin.com",
+  "port": null,
+  "path": "/har",
+  "headers": {
+    "content-type": "application/json"
+  }
+};
+
+var req = http.request(options, function (res) {
+  var chunks = [];
+
+  res.on("data", function (chunk) {
+    chunks.push(chunk);
+  });
+
+  res.on("end", function () {
+    var body = Buffer.concat(chunks);
+    console.log(body.toString());
+  });
+});
+
+req.write(JSON.stringify({ foo: null }));
+req.end();