From 1afb5b3bb0ac579fdd1616f701b27038feb5a375 Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Wed, 4 Apr 2007 20:52:41 +0000 Subject: Bug 376497: validateID() should return an attachment object - Patch by Frédéric Buclin a=LpSolit MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- post_bug.cgi | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'post_bug.cgi') diff --git a/post_bug.cgi b/post_bug.cgi index f755934ae..7d01ab62d 100755 --- a/post_bug.cgi +++ b/post_bug.cgi @@ -187,15 +187,15 @@ if (defined $cgi->param('version')) { # Add an attachment if requested. if (defined($cgi->upload('data')) || $cgi->param('attachurl')) { $cgi->param('isprivate', $cgi->param('commentprivacy')); - my $attach_id = Bugzilla::Attachment->insert_attachment_for_bug(!THROW_ERROR, + my $attachment = Bugzilla::Attachment->insert_attachment_for_bug(!THROW_ERROR, $bug, $user, $timestamp, \$vars); - if ($attach_id) { + if ($attachment) { # Update the comment to include the new attachment ID. # This string is hardcoded here because Template::quoteUrls() # expects to find this exact string. - my $new_comment = "Created an attachment (id=$attach_id)\n" . - $cgi->param('description') . "\n"; + my $new_comment = "Created an attachment (id=" . $attachment->id . ")\n" . + $attachment->description . "\n"; # We can use $bug->longdescs here because we are sure that the bug # description is of type CMT_NORMAL. No need to include it if it's # empty, though. -- cgit v1.2.3-24-g4f1b