diff options
| -rw-r--r-- | CGI.pl | 3 | ||||
| -rwxr-xr-x | process_bug.cgi | 8 | ||||
| -rw-r--r-- | template/en/default/global/code-error.html.tmpl | 7 |
3 files changed, 6 insertions, 12 deletions
@@ -111,11 +111,10 @@ 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, info => $info }); + ThrowCodeError("undefined_field", { field => $fieldname }); } } diff --git a/process_bug.cgi b/process_bug.cgi index d3155af33..5608c0528 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -842,15 +842,15 @@ if (defined $::FORM{newcc} || defined $::FORM{removecc} || defined $::FORM{massc } if ($cc_add) { - $cc_add =~ s/[\s,]+/ /g; # Change all delimiters to a single space - foreach my $person ( split(" ", $cc_add) ) { + $cc_add =~ s/^[\s,]+//; # Remove leading delimiters. + foreach my $person ( split(/[\s,]+/, $cc_add) ) { my $pid = DBNameToIdAndCheck($person); $cc_add{$pid} = $person; } } if ($cc_remove) { - $cc_remove =~ s/[\s,]+/ /g; # Change all delimiters to a single space - foreach my $person ( split(" ", $cc_remove) ) { + $cc_remove =~ s/^[\s,]+//; # Remove leading delimiters. + foreach my $person ( split(/[\s,]+/, $cc_remove) ) { my $pid = DBNameToIdAndCheck($person); $cc_remove{$pid} = $person; } diff --git a/template/en/default/global/code-error.html.tmpl b/template/en/default/global/code-error.html.tmpl index 458f55c55..73501672b 100644 --- a/template/en/default/global/code-error.html.tmpl +++ b/template/en/default/global/code-error.html.tmpl @@ -208,12 +208,7 @@ I was unable to retrieve your old password from the database. [% ELSIF error == "undefined_field" %] - [% field FILTER html %] was not defined; - [% IF info %] - [% info FILTER html %] - [% ELSE %] - [% Param('browserbugmessage') %] - [% END %] + [% field FILTER html %] was not defined; [% Param('browserbugmessage') %] [% ELSIF error == "unknown_action" %] [% IF action %] |