summaryrefslogtreecommitdiff
path: root/render/form.c
diff options
context:
space:
mode:
Diffstat (limited to 'render/form.c')
-rw-r--r--render/form.c16
1 files changed, 6 insertions, 10 deletions
diff --git a/render/form.c b/render/form.c
index d5704bc6f..bfd9664c3 100644
--- a/render/form.c
+++ b/render/form.c
@@ -131,18 +131,14 @@ bool form_successful_controls(struct form *form,
if (!control->name)
continue;
- /* ignore controls with no value */
- /* this fixes ebay silliness
- * From the spec:
- * "If a control doesn't have a current value when the
- * form is submitted, user agents are not required to
- * treat it as a successful control"
- */
- if (!control->value)
- continue;
-
switch (control->type) {
case GADGET_HIDDEN:
+ /* ignore hidden controls with no value */
+ /* this fixes ebay silliness */
+ if (!control->value)
+ continue;
+
+ /* fall through */
case GADGET_TEXTBOX:
case GADGET_PASSWORD:
value = strdup(control->value);