diff options
author | mkanat%bugzilla.org <> | 2006-06-15 06:17:21 +0200 |
---|---|---|
committer | mkanat%bugzilla.org <> | 2006-06-15 06:17:21 +0200 |
commit | a59456eb3675e8d716f9516a4f9f0c140da29c9f (patch) | |
tree | 0aee329a8e549e4a91fe074b5ba8d18a4664f39a /Bugzilla | |
parent | 9d8e3ef8873724dff896687a783dbd1ff3295297 (diff) | |
download | bugzilla-a59456eb3675e8d716f9516a4f9f0c140da29c9f.tar.gz bugzilla-a59456eb3675e8d716f9516a4f9f0c140da29c9f.tar.xz |
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
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/Series.pm | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/Bugzilla/Series.pm b/Bugzilla/Series.pm index 521357488..4693a7131 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(); |