diff options
author | kiko%async.com.br <> | 2004-07-24 07:20:43 +0000 |
---|---|---|
committer | kiko%async.com.br <> | 2004-07-24 07:20:43 +0000 |
commit | ab5b48eafb99cc1c56566424295b4bdc18cb9a0d (patch) | |
tree | 44c8d91c63a7ebe453dfac271bebfbd93f3d0b4e | |
parent | c2719a4b966569473ddd4cc0621ed8af594ceb75 (diff) | |
download | bugs-ab5b48eafb99cc1c56566424295b4bdc18cb9a0d.tar bugs-ab5b48eafb99cc1c56566424295b4bdc18cb9a0d.tar.gz bugs-ab5b48eafb99cc1c56566424295b4bdc18cb9a0d.tar.bz2 bugs-ab5b48eafb99cc1c56566424295b4bdc18cb9a0d.tar.xz bugs-ab5b48eafb99cc1c56566424295b4bdc18cb9a0d.zip |
Fix for bug 252370: Provide edit link for attachment in comment. Adds an
[edit] link next to attachments in comment text, and removes the
&action=view suffix which breaks things when pasting into a shell.
r=bugreport, myk; a=myk.
-rw-r--r-- | globals.pl | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/globals.pl b/globals.pl index 6632833ab..e31c8e717 100644 --- a/globals.pl +++ b/globals.pl @@ -968,10 +968,10 @@ sub GetAttachmentLink { # Now that we know we've got all the information we're gonna get, let's # return the link (which is the whole reason we were called :) my ($title, $className) = @{$::attachlink{$attachid}}; - # $title will be undefined if the bug didn't exist in the database. + # $title will be undefined if the attachment didn't exist in the database. if (defined $title) { - my $linkval = "attachment.cgi?id=$attachid&action=view"; - return qq{<a href="$linkval" class="$className" title="$title">$link_text</a>}; + my $linkval = "attachment.cgi?id=$attachid"; + return qq{<a href="$linkval" class="$className" title="$title">$link_text</a> [<a href="$linkval&action=edit">edit</a>]}; } else { return qq{$link_text}; |