From 5885d76f37d622759a0b85cdc245feb561083a07 Mon Sep 17 00:00:00 2001
From: Behind The Math <info@behindthemath.io>
Date: Tue, 29 May 2018 22:21:32 -0400
Subject: [PATCH 1/2] Fix a bug where the value of <option> would not get sent
 if falsy

According to the spec, the value attribute should be sent if it
exists, even if it's falsy.
---
 lib/proto/attach-form.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/proto/attach-form.js b/lib/proto/attach-form.js
index 44c4d48..0bfec63 100644
--- a/lib/proto/attach-form.js
+++ b/lib/proto/attach-form.js
@@ -68,7 +68,7 @@ function parseFormElements(el) {
           for (var i = 0; i < element.options.length; i++) {
             opt = element.options[i]
             if (opt.selected) {
-              values.push(opt.value || opt.text)
+              values.push(opt.hasAttribute("value") ? opt.value : opt.text)
             }
           }
         }
-- 
2.49.1


From 29d7b4c664f42259f3324276a8877fa0b1851c44 Mon Sep 17 00:00:00 2001
From: Behind The Math <info@behindthemath.io>
Date: Tue, 29 May 2018 22:22:09 -0400
Subject: [PATCH 2/2] Don't send an <option> tag if it's disabled, even if it's
 selected

---
 lib/proto/attach-form.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/proto/attach-form.js b/lib/proto/attach-form.js
index 0bfec63..39333a2 100644
--- a/lib/proto/attach-form.js
+++ b/lib/proto/attach-form.js
@@ -67,7 +67,7 @@ function parseFormElements(el) {
 
           for (var i = 0; i < element.options.length; i++) {
             opt = element.options[i]
-            if (opt.selected) {
+            if (opt.selected && !opt.disabled) {
               values.push(opt.hasAttribute("value") ? opt.value : opt.text)
             }
           }
-- 
2.49.1