From f53defbc217f63e9504da972aa05664ab0bd9336 Mon Sep 17 00:00:00 2001
From: Frédéric Buclin <LpSolit@gmail.com>
Date: Mon, 15 Jun 2015 20:10:50 +0200
Subject: Bug 1169767: Upgrading from pre-4.4 to master fails r=dkl a=glob

---
 Bugzilla/Config/BugChange.pm | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

(limited to 'Bugzilla')

diff --git a/Bugzilla/Config/BugChange.pm b/Bugzilla/Config/BugChange.pm
index 986a74485..212f4c171 100644
--- a/Bugzilla/Config/BugChange.pm
+++ b/Bugzilla/Config/BugChange.pm
@@ -13,6 +13,7 @@ use warnings;
 
 use Bugzilla::Config::Common;
 use Bugzilla::Status;
+use Bugzilla::Field;
 
 our $sortkey = 500;
 
@@ -31,15 +32,6 @@ sub get_param_list {
       @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 = (
   {
    name => 'duplicate_or_move_bug_status',
@@ -82,7 +74,7 @@ sub get_param_list {
   {
    name => 'resolution_forbidden_with_open_blockers',
    type => 's',
-   choices => \@resolutions,
+   choices => \&_get_resolutions,
    default => '',
    checker => \&check_resolution,
   } );
@@ -90,4 +82,10 @@ sub get_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;
-- 
cgit v1.2.3-24-g4f1b