summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Mark Bell <jmb@netsurf-browser.org>2004-08-18 01:45:29 +0000
committerJohn Mark Bell <jmb@netsurf-browser.org>2004-08-18 01:45:29 +0000
commitd1eb18f62ef6162a61fe4531faaff5fafc458604 (patch)
tree92ff6fce97e0909aa6f4296b6ac3e0742dee3dea
parent4ff7d738bc0fc1cbea37eb8a1ac7db73432bf493 (diff)
downloadnetsurf-d1eb18f62ef6162a61fe4531faaff5fafc458604.tar.gz
netsurf-d1eb18f62ef6162a61fe4531faaff5fafc458604.tar.bz2
[project @ 2004-08-18 01:45:29 by jmb]
Restrict last change to hidden inputs only - breaks sf.net bug tracker otherwise svn path=/import/netsurf/; revision=1243
-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);