Merge pull request #94 from telime/master

mreinstein · web-flow · commit 4dc4f71fa978 · 2022-04-26T11:45:29.000-07:00
Allowing empty string
diff --git a/lib/parse.js b/lib/parse.js
@@ -129,8 +129,9 @@ function parsePlistXML (node) {
       counter += 1;
     }
     if (counter % 2 === 1) {
-      throw new Error('Missing value for "' + key + '" while parsing <dict/>');
+      new_obj[key] = '';
     }
+    
     return new_obj;
 
   } else if (node.nodeName === 'array') {
diff --git a/test/parse.js b/test/parse.js
@@ -164,11 +164,10 @@ U=</data>
         parseFixture('<dict><key>a</key><key>b</key></dict>');
       });
     });
-
-    it('should throw if value is missing', function () {
-      assert.throws(function () {
-        parseFixture('<dict><key>a</key></dict>');
-      });
+    
+    it('should parse to empry string if value is missing', function () {
+      var parsed = parseFixture('<dict><key>a</key></dict>');
+      assert.deepEqual(parsed, { 'a': '' });
     });
 
     it('should parse an empty key', function () {

Original file line number	Diff line number	Diff line change
`@@ -129,8 +129,9 @@ function parsePlistXML (node) {`
`129`	`129`	`counter += 1;`
`130`	`130`	`}`
`131`	`131`	`if (counter % 2 === 1) {`
`132`		`- throw new Error('Missing value for "' + key + '" while parsing <dict/>');`
	`132`	`+ new_obj[key] = '';`
`133`	`133`	`}`
	`134`	`+`
`134`	`135`	`return new_obj;`
`135`	`136`
`136`	`137`	`} else if (node.nodeName === 'array') {`