Emit empty string for nulls

Fixes TooTallNate#79
OutSystems · Jan 12, 2017 · 3faaedf · 3faaedf
1 parent 290cb60
commit 3faaedf
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/lib/build.js b/lib/build.js
@@ -133,5 +133,8 @@ function walk_obj(next, next_child) {
     // a typed array
     next_child.ele('data').raw(base64.fromByteArray(new Uint8Array(next.buffer), next_child));
 
+  } else if (next === null) {
+    next_child.ele('string');
+
   }
 }
diff --git a/test/build.js b/test/build.js
@@ -128,6 +128,20 @@ describe('plist', function () {
 */}));
     });
 
+    it('should emit an empty string for nulls', function () {
+      var xml = build({ a: null });
+      assert.strictEqual(xml, multiline(function () {/*
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+  <dict>
+    <key>a</key>
+    <string/>
+  </dict>
+</plist>
+*/}));
+    });
+
     it('should omit undefined values', function () {
       var xml = build({ a: undefined });
       assert.strictEqual(xml, multiline(function () {/*