From: mkanat%bugzilla.org <>
Date: Thu, 15 Jun 2006 04:17:21 +0000 (+0000)
Subject: Bug 341453: Bugzilla::Series looks for functions in global scope that are not global... 
X-Git-Tag: bugzilla-2.23.2~108
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a59456eb3675e8d716f9516a4f9f0c140da29c9f;p=thirdparty%2Fbugzilla.git

Bug 341453: Bugzilla::Series looks for functions in global scope that are not global functions
Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=LpSolit, a=myk
---

diff --git a/Bugzilla/Series.pm b/Bugzilla/Series.pm
index 521357488d..4693a7131b 100644
--- a/Bugzilla/Series.pm
+++ b/Bugzilla/Series.pm
@@ -32,6 +32,7 @@ use lib ".";
 
 package Bugzilla::Series;
 
+use Bugzilla::Error;
 use Bugzilla::Util;
 use Bugzilla::User;
 
@@ -85,8 +86,8 @@ sub initFromDatabase {
     my $self = shift;
     my $series_id = shift;
     
-    &::detaint_natural($series_id) 
-      || &::ThrowCodeError("invalid_series_id", { 'series_id' => $series_id });
+    detaint_natural($series_id) 
+      || ThrowCodeError("invalid_series_id", { 'series_id' => $series_id });
     
     my $dbh = Bugzilla->dbh;
     my @series = $dbh->selectrow_array("SELECT series.series_id, cc1.name, " .
@@ -135,26 +136,26 @@ sub initFromCGI {
     $self->{'series_id'} = $cgi->param('series_id') || undef;
     if (defined($self->{'series_id'})) {
         detaint_natural($self->{'series_id'})
-          || &::ThrowCodeError("invalid_series_id", 
+          || ThrowCodeError("invalid_series_id", 
                                { 'series_id' => $self->{'series_id'} });
     }
     
     $self->{'category'} = $cgi->param('category')
       || $cgi->param('newcategory')
-      || &::ThrowUserError("missing_category");
+      || ThrowUserError("missing_category");
 
     $self->{'subcategory'} = $cgi->param('subcategory')
       || $cgi->param('newsubcategory')
-      || &::ThrowUserError("missing_subcategory");
+      || ThrowUserError("missing_subcategory");
 
     $self->{'name'} = $cgi->param('name')
-      || &::ThrowUserError("missing_name");
+      || ThrowUserError("missing_name");
 
     $self->{'creator'} = Bugzilla->user->id;
 
     $self->{'frequency'} = $cgi->param('frequency');
     detaint_natural($self->{'frequency'})
-      || &::ThrowUserError("missing_frequency");
+      || ThrowUserError("missing_frequency");
 
     $self->{'query'} = $cgi->canonicalise_query("format", "ctype", "action",
                                         "category", "subcategory", "name",
@@ -209,7 +210,7 @@ sub writeToDatabase {
         $self->{'series_id'} = $dbh->selectrow_array("SELECT MAX(series_id) " .
                                                      "FROM series");
         $self->{'series_id'}
-          || &::ThrowCodeError("missing_series_id", { 'series' => $self });
+          || ThrowCodeError("missing_series_id", { 'series' => $self });
     }
     
     $dbh->bz_unlock_tables();