From fa2c116330bd2023dc14a09c0e8a986d62dc6be0 Mon Sep 17 00:00:00 2001
From: "timeless%mozdev.org" <>
Date: Thu, 16 Oct 2003 05:48:09 +0000
Subject: Bug 108528 - knob is not defined doesn't explain to 2001110503 users
 what to do patch by caduvall@glue.umd.edu r=timeless a=justdave

---
 CGI.pl                                          | 3 ++-
 process_bug.cgi                                 | 2 +-
 template/en/default/global/code-error.html.tmpl | 7 ++++++-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/CGI.pl b/CGI.pl
index bcd9a7f90..06db6fe55 100644
--- a/CGI.pl
+++ b/CGI.pl
@@ -111,10 +111,11 @@ sub CheckFormField (\%$;\@) {
 sub CheckFormFieldDefined (\%$) {
     my ($formRef,                # a reference to the form to check (a hash)
         $fieldname,              # the fieldname to check
+        $info,                   # (optional) error message to give
        ) = @_;
 
     if (!defined $formRef->{$fieldname}) {
-        ThrowCodeError("undefined_field", { field => $fieldname });
+        ThrowCodeError("undefined_field", { field => $fieldname, info => $info });
     }
 }
 
diff --git a/process_bug.cgi b/process_bug.cgi
index e504fd7ea..d3155af33 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -858,7 +858,7 @@ if (defined $::FORM{newcc} || defined $::FORM{removecc} || defined $::FORM{massc
 }
 
 
-CheckFormFieldDefined(\%::FORM, 'knob');
+CheckFormFieldDefined(\%::FORM, 'knob', "check that \"Leave as...\" was selected.");
 SWITCH: for ($::FORM{'knob'}) {
     /^none$/ && do {
         last SWITCH;
diff --git a/template/en/default/global/code-error.html.tmpl b/template/en/default/global/code-error.html.tmpl
index 73501672b..458f55c55 100644
--- a/template/en/default/global/code-error.html.tmpl
+++ b/template/en/default/global/code-error.html.tmpl
@@ -208,7 +208,12 @@
      I was unable to retrieve your old password from the database.
 
   [% ELSIF error == "undefined_field" %]
-    [% field FILTER html %] was not defined; [% Param('browserbugmessage') %]
+    [% field FILTER html %] was not defined; 
+    [% IF info %]
+      [% info FILTER html %]
+    [% ELSE %]
+      [% Param('browserbugmessage') %]
+    [% END %]
 
   [% ELSIF error == "unknown_action" %]
     [% IF action %]
-- 
cgit v1.2.3-24-g4f1b