Merge pull request #1 from Matt-Esch/master

Allow checkbox arrays of selected values

index.js

@@ -24,11 +24,15 @@ function valueOfElement(elem) {
 
         return checked ? checked.value : null
     } else if (Array.isArray(elem)) {
-        return elem.map(valueOfElement)
+        return elem.map(valueOfElement).filter(filterNull)
     } else if (elem.tagName === undefined && elem.nodeType === undefined) {
         return FormData(elem)
     } else if (elem.tagName === "INPUT" && isChecked(elem)) {
-        return elem.checked
+        if (elem.hasAttribute("value")) {
+            return elem.checked ? elem.value : null
+        } else {
+            return elem.checked
+        }
     } else if (elem.tagName === "INPUT") {
         return elem.value
     } else if (elem.tagName === "TEXTAREA") {
@@ -65,3 +69,7 @@ function toList(value) {
 function prop(x) {
     return this[x]
 }
+
+function filterNull(val) {
+    return val !== null
+}

test/checkbox.js

@@ -0,0 +1,32 @@
+var test = require("tape")
+var h = require("hyperscript")
+
+var FormData = require("../index")
+
+test("FormData works with <input type='checkbox' value='val' />",
+function (assert) {
+    var elements = {
+        foo: [
+            checkbox("bar", "foo"),
+            checkbox("bar", "bar", true),
+            checkbox("bar", "baz", true)
+        ]
+    }
+
+    var data = FormData(elements)
+
+    assert.deepEqual(data, {
+        foo: ["bar", "baz"]
+    })
+
+    assert.end()
+})
+
+function checkbox(name, value, checked) {
+    return h("input", {
+        type: "checkbox",
+        name: name,
+        value: value,
+        checked: !!checked
+    })
+}

test/index.js

@@ -2,4 +2,5 @@ require("./basic")
 require("./nested-object")
 require("./select")
 require("./radio")
+require("./checkbox")
 require("./arbitrary-functions")