Merge pull request #484 from sj26/fallback-array-form-data

Cope with array form data in fallback

Author: mislav

jquery.pjax.js

@@ -521,6 +521,10 @@ function fallbackPjax(options) {
       var pair = value.split('=')
       form.append($('<input>', {type: 'hidden', name: pair[0], value: pair[1]}))
     })
+  } else if ($.isArray(data)) {
+    $.each(data, function(index, value) {
+      form.append($('<input>', {type: 'hidden', name: value.name, value: value.value}))
+    })
   } else if (typeof data === 'object') {
     for (key in data)
       form.append($('<input>', {type: 'hidden', name: key, value: data[key]}))

test/unit/pjax_fallback.js

@@ -313,6 +313,54 @@ asyncTest("POST with data object"+s, function() {
   })
 })
 
+asyncTest("GET with data array"+s, function() {
+  var frame = this.frame
+
+  this.loaded = function() {
+    equal(frame.location.pathname, "/env.html")
+    equal(frame.location.search, "?foo%5B%5D=bar&foo%5B%5D=baz")
+
+    var env = JSON.parse(frame.$("#env").text())
+    equal(env['REQUEST_METHOD'], "GET")
+    var expected = {'foo': ['bar', 'baz']};
+    if (!disabled) expected._pjax = "#main"
+    deepEqual(env['rack.request.query_hash'], expected)
+
+    start()
+  }
+
+  frame.$.pjax({
+    type: 'GET',
+    url: "env.html",
+    data: [{name: "foo[]", value: "bar"}, {name: "foo[]", value: "baz"}],
+    container: "#main"
+  })
+})
+
+asyncTest("POST with data array"+s, function() {
+  var frame = this.frame
+
+  this.loaded = function() {
+    equal(frame.location.pathname, "/env.html")
+    equal(frame.location.search, "")
+
+    var env = JSON.parse(frame.$("#env").text())
+    equal(env['REQUEST_METHOD'], "POST")
+    var expected = {'foo': ['bar', 'baz']};
+    if (!disabled) expected._pjax = "#main"
+    deepEqual(env['rack.request.form_hash'], expected)
+
+    start()
+  }
+
+  frame.$.pjax({
+    type: 'POST',
+    url: "env.html",
+    data: [{name: "foo[]", value: "bar"}, {name: "foo[]", value: "baz"}],
+    container: "#main"
+  })
+})
+
 asyncTest("GET with data string"+s, function() {
   var frame = this.frame