diff options
author | lpsolit%gmail.com <> | 2006-02-26 08:10:23 +0100 |
---|---|---|
committer | lpsolit%gmail.com <> | 2006-02-26 08:10:23 +0100 |
commit | 645307b7dad849212762afcecf819a268cc810d3 (patch) | |
tree | d4dfdb2ec688efd2ea17f7ff3fc9382175b7c2bc /Bugzilla/Bug.pm | |
parent | 0bd92669e6bd6cde59e85cbf9e827feb94d2536c (diff) | |
download | bugzilla-645307b7dad849212762afcecf819a268cc810d3.tar.gz bugzilla-645307b7dad849212762afcecf819a268cc810d3.tar.xz |
Bug 284875: Move GetBugLink, GetAttachmentLink, and quoteUrls out of globals.pl - Patch by Frédéric Buclin <LpSolit@gmail.com> r=mkanat a=justdave
Diffstat (limited to 'Bugzilla/Bug.pm')
-rwxr-xr-x | Bugzilla/Bug.pm | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 71b6bc7f4..e2222ccb7 100755 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -1221,7 +1221,7 @@ sub ValidateBugAlias { my $vars = {}; $vars->{'alias'} = $alias; if ($id) { - $vars->{'bug_link'} = &::GetBugLink($id, $id); + $vars->{'bug_id'} = $id; ThrowUserError("alias_in_use", $vars); } @@ -1294,19 +1294,12 @@ sub ValidateDependencies { my @deps = @{$deptree{'dependson'}}; my @blocks = @{$deptree{'blocked'}}; - my @union = (); - my @isect = (); my %union = (); my %isect = (); foreach my $b (@deps, @blocks) { $union{$b}++ && $isect{$b}++ } - @union = keys %union; - @isect = keys %isect; + my @isect = keys %isect; if (scalar(@isect) > 0) { - my $both = ""; - foreach my $i (@isect) { - $both .= &::GetBugLink($i, "#" . $i) . " "; - } - ThrowUserError("dependency_loop_multi", { both => $both }); + ThrowUserError("dependency_loop_multi", {'deps' => \@isect}); } return %deps; } |