Commit 60ac8cee authored by kiko%async.com.br's avatar kiko%async.com.br

Landing fix for bug 153811: default severity should be parametrized.

r,a=myk.
parent da7ecb07
......@@ -66,6 +66,16 @@ sub check_priority {
return "";
}
sub check_severity {
my ($value) = (@_);
&::GetVersionTable();
if (lsearch(\@::legal_severity, $value) < 0) {
return "Must be a legal severity value: one of " .
join(", ", @::legal_severity);
}
return "";
}
sub check_platform {
my ($value) = (@_);
&::GetVersionTable();
......@@ -761,6 +771,14 @@ You will get this message once a day until you\'ve dealt with these bugs!
},
{
name => 'defaultseverity',
desc => 'This is the severity that newly entered bugs are set to.',
type => 't',
default => 'normal',
checker => \&check_severity
},
{
name => 'defaultplatform',
desc => 'This is the platform that is preselected on the bug '.
'entry form.<br>'.
......
......@@ -327,7 +327,7 @@ $vars->{'priority'} = \@legal_priority;
$default{'priority'} = formvalue('priority', Param('defaultpriority'));
$vars->{'bug_severity'} = \@legal_severity;
$default{'bug_severity'} = formvalue('bug_severity', 'normal');
$default{'bug_severity'} = formvalue('bug_severity', Param('defaultseverity'));
$vars->{'rep_platform'} = \@legal_platform;
$default{'rep_platform'} = pickplatform();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment