Fix Edge form support (#178)

* Fix looping through form elements in Edge

* Update lockfile

lib/proto/attach-form.js

@@ -45,13 +45,10 @@ var formAction = function(el, event) {
 
 function parseFormElements(el) {
   var requestParams = []
+  var formElements = el.elements
 
-  for (var elementKey in el.elements) {
+  for (var i = 0; i < formElements.length; i++) {
-    if (Number.isNaN(Number(elementKey))) {
+    var element = formElements[i]
-      continue;
-    }
-
-    var element = el.elements[elementKey]
     var tagName = element.tagName.toLowerCase()
     // jscs:disable disallowImplicitTypeConversion
     if (!!element.name && element.attributes !== undefined && tagName !== "button") {
@@ -65,8 +62,8 @@ function parseFormElements(el) {
         if (tagName === "select") {
           var opt
 
-          for (var i = 0; i < element.options.length; i++) {
+          for (var j = 0; j < element.options.length; j++) {
-            opt = element.options[i]
+            opt = element.options[j]
             if (opt.selected && !opt.disabled) {
               values.push(opt.hasAttribute("value") ? opt.value : opt.text)
             }
@@ -76,10 +73,10 @@ function parseFormElements(el) {
           values.push(element.value)
         }
 
-        for (var j = 0; j < values.length; j++) {
+        for (var k = 0; k < values.length; k++) {
           requestParams.push({
             name: encodeURIComponent(element.name),
-            value: encodeURIComponent(values[j])
+            value: encodeURIComponent(values[k])
           })
         }
       }

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "pjax",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {