diff --git a/webtools/bugzilla/enter_bug.cgi b/webtools/bugzilla/enter_bug.cgi index 068e4fb0e27..4b39cb7504c 100755 --- a/webtools/bugzilla/enter_bug.cgi +++ b/webtools/bugzilla/enter_bug.cgi @@ -54,9 +54,7 @@ use vars qw( @legal_severity ); -# If we're using bug groups to restrict bug entry, we need to know who the -# user is right from the start. -Bugzilla->login(LOGIN_REQUIRED) if AnyEntryGroups(); +my $user = Bugzilla->login(LOGIN_REQUIRED); my $cloned_bug; my $cloned_bug_id; @@ -68,8 +66,6 @@ my $vars = {}; my $product = trim($cgi->param('product') || ''); if ($product eq '') { - my $user = Bugzilla->login(); - # If the user cannot enter bugs in any product, stop here. my @enterable_products = @{$user->get_enterable_products}; ThrowUserError('no_products') unless scalar(@enterable_products); @@ -293,8 +289,6 @@ sub pickos { # End of subroutines ############################################################################## -Bugzilla->login(LOGIN_REQUIRED) if (!(AnyEntryGroups())); - # If a user is trying to clone a bug # Check that the user has authorization to view the parent bug # Create an instance of Bug that holds the info from the parent @@ -302,7 +296,7 @@ $cloned_bug_id = $cgi->param('cloned_bug_id'); if ($cloned_bug_id) { ValidateBugID($cloned_bug_id); - $cloned_bug = new Bugzilla::Bug($cloned_bug_id, Bugzilla->user->id); + $cloned_bug = new Bugzilla::Bug($cloned_bug_id, $user->id); } # We need to check and make sure @@ -310,7 +304,7 @@ if ($cloned_bug_id) { my $prod_obj = new Bugzilla::Product({name => $product}); # Update the product name to get the correct case. $product = $prod_obj->name if defined $prod_obj; -Bugzilla->user->can_enter_product($product, 1); +$user->can_enter_product($product, 1); GetVersionTable(); diff --git a/webtools/bugzilla/globals.pl b/webtools/bugzilla/globals.pl index 5d7c9e25034..38791a93bbc 100644 --- a/webtools/bugzilla/globals.pl +++ b/webtools/bugzilla/globals.pl @@ -185,33 +185,6 @@ sub GetVersionTable { $::VersionTableLoaded = 1; } -# -# This function checks if there are any entry groups defined. -# If called with no arguments, it identifies -# entry groups for all products. If called with a product -# id argument, it checks for entry groups associated with -# one particular product. -sub AnyEntryGroups { - my $product_id = shift; - $product_id = 0 unless ($product_id); - return $::CachedAnyEntryGroups{$product_id} - if defined($::CachedAnyEntryGroups{$product_id}); - my $dbh = Bugzilla->dbh; - PushGlobalSQLState(); - my $query = "SELECT 1 FROM group_control_map WHERE entry != 0"; - $query .= " AND product_id = $product_id" if ($product_id); - $query .= " " . $dbh->sql_limit(1); - SendSQL($query); - if (MoreSQLData()) { - $::CachedAnyEntryGroups{$product_id} = MoreSQLData(); - FetchSQLData(); - PopGlobalSQLState(); - return $::CachedAnyEntryGroups{$product_id}; - } else { - return undef; - } -} -# # This function checks if there are any default groups defined. # If so, then groups may have to be changed when bugs move from # one bug to another.