Bug 529863: The product list when listing all flagtypes in editflagtypes.cgi is empty after editing them - Patch by Frédéric Buclin <LpSolit@gmail.com> r=ghendricks a=LpSolit

1 files changed, 3 insertions, 9 deletions

diff --git a/editflagtypes.cgi b/editflagtypes.cgi
index b730ae2e5..4f85e6c65 100755
--- a/editflagtypes.cgi
+++ b/editflagtypes.cgi
@@ -47,6 +47,9 @@ local our $cgi = Bugzilla->cgi;
 local our $template = Bugzilla->template;
 local our $vars = {};
 
+# We need this everywhere.
+$vars = get_products_and_components($vars);
+
 # Make sure the user is logged in and is an administrator.
 my $user = Bugzilla->login(LOGIN_REQUIRED);
 $user->in_group('editcomponents')
@@ -93,9 +96,6 @@ exit;
 ################################################################################
 
 sub list {
-    # Restrict the list to the given product and component, if given.
-    $vars = get_products_and_components($vars);
-
     my $product = validateProduct(scalar $cgi->param('product'));
     my $component = validateComponent($product, scalar $cgi->param('component'));
     my $product_id = $product ? $product->id : 0;
@@ -178,9 +178,6 @@ sub edit {
         $flag_type = validateID();
     }
 
-    # Fill $vars with products and components data.
-    $vars = get_products_and_components($vars);
-
     $vars->{'last_action'} = $cgi->param('action');
     if ($cgi->param('action') eq 'enter' || $cgi->param('action') eq 'copy') {
         $vars->{'action'} = "insert";
@@ -252,9 +249,6 @@ sub processCategoryChange {
     my %inclusions = clusion_array_to_hash(\@inclusions);
     my %exclusions = clusion_array_to_hash(\@exclusions);
 
-    # Fill $vars with products and components data.
-    $vars = get_products_and_components($vars);
-
     my @groups = Bugzilla::Group->get_all;
     $vars->{'groups'} = \@groups;
     $vars->{'action'} = $cgi->param('action');