Commit f53defbc authored by Frédéric Buclin's avatar Frédéric Buclin

Bug 1169767: Upgrading from pre-4.4 to master fails

r=dkl a=glob
parent 1ed6e01b
...@@ -13,6 +13,7 @@ use warnings; ...@@ -13,6 +13,7 @@ use warnings;
use Bugzilla::Config::Common; use Bugzilla::Config::Common;
use Bugzilla::Status; use Bugzilla::Status;
use Bugzilla::Field;
our $sortkey = 500; our $sortkey = 500;
...@@ -31,15 +32,6 @@ sub get_param_list { ...@@ -31,15 +32,6 @@ sub get_param_list {
@closed_bug_statuses = @current_closed_states if scalar(@current_closed_states); @closed_bug_statuses = @current_closed_states if scalar(@current_closed_states);
}; };
my $resolution_field = Bugzilla::Field->new({ name => 'resolution', cache => 1 });
my @resolutions = ();
# The 'fielddefs' table is not yet populated when running checksetup.pl
# for the first time.
if ($resolution_field) {
# The empty resolution is included - it represents "no value"
@resolutions = map {$_->name} @{ $resolution_field->legal_values };
}
my @param_list = ( my @param_list = (
{ {
name => 'duplicate_or_move_bug_status', name => 'duplicate_or_move_bug_status',
...@@ -82,7 +74,7 @@ sub get_param_list { ...@@ -82,7 +74,7 @@ sub get_param_list {
{ {
name => 'resolution_forbidden_with_open_blockers', name => 'resolution_forbidden_with_open_blockers',
type => 's', type => 's',
choices => \@resolutions, choices => \&_get_resolutions,
default => '', default => '',
checker => \&check_resolution, checker => \&check_resolution,
} ); } );
...@@ -90,4 +82,10 @@ sub get_param_list { ...@@ -90,4 +82,10 @@ sub get_param_list {
return @param_list; return @param_list;
} }
sub _get_resolutions {
my $resolution_field = Bugzilla::Field->new({ name => 'resolution', cache => 1 });
# The empty resolution is included - it represents "no value".
return [ map { $_->name } @{ $resolution_field->legal_values } ];
}
1; 1;
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