summaryrefslogtreecommitdiffstats
path: root/Bugzilla
diff options
context:
space:
mode:
authorlpsolit%gmail.com <>2006-07-03 18:21:42 +0200
committerlpsolit%gmail.com <>2006-07-03 18:21:42 +0200
commit70711889debb8b059c20c94435414d8c25f57e76 (patch)
tree949784efbdda8de7b155a23a6d2f792ee5586eac /Bugzilla
parent085a4fb5da54ab0031ee44a98f50ccde0a10852d (diff)
downloadbugzilla-70711889debb8b059c20c94435414d8c25f57e76.tar.gz
bugzilla-70711889debb8b059c20c94435414d8c25f57e76.tar.xz
Bug 343364: Changing Multiple Bugs is broken - Patch by Olav Vitters <bugzilla-mozilla@bkor.dhs.org> r=LpSolit a=justdave
Diffstat (limited to 'Bugzilla')
-rwxr-xr-xBugzilla/Bug.pm14
1 files changed, 7 insertions, 7 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 91a92cbe2..857557def 100755
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -107,13 +107,6 @@ sub initBug {
# If the bug ID isn't numeric, it might be an alias, so try to convert it.
$bug_id = bug_alias_to_id($bug_id) if $bug_id !~ /^0*[1-9][0-9]*$/;
- if ((! defined $bug_id) || (!$bug_id) || (!detaint_natural($bug_id))) {
- # no bug number given or the alias didn't match a bug
- $self->{'bug_id'} = $old_bug_id;
- $self->{'error'} = "InvalidBugId";
- return $self;
- }
-
# If the user is not logged in, sets $user_id to 0.
# Else gets $user_id from the user login name if this
# argument is not numeric.
@@ -126,6 +119,13 @@ sub initBug {
$self->{'who'} = new Bugzilla::User($user_id);
+ if ((! defined $bug_id) || (!$bug_id) || (!detaint_natural($bug_id))) {
+ # no bug number given or the alias didn't match a bug
+ $self->{'bug_id'} = $old_bug_id;
+ $self->{'error'} = "InvalidBugId";
+ return $self;
+ }
+
my $custom_fields = "";
if (scalar(Bugzilla->custom_field_names) > 0) {
$custom_fields = ", " . join(", ", Bugzilla->custom_field_names);