Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 473052: Modify WebService::Bug.add_comment to return the new comment ID

Browse Source 
Patch By Kip Hampton <khampton@totalcinema.com> r=mkanat, a=mkanat


git-svn-id: svn://10.0.0.236/trunk@255868 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
mkanat%bugzilla.org 2009-01-20 20:26:16 +00:00
parent 37bb25c673
commit e3acd9ec21
1 changed files with 19 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
mozilla/webtools/bugzilla/Bugzilla/WebService/Bug.pm
View File

@ -337,11 +337,23 @@ sub add_comment {
# Append comment
$bug->add_comment($comment, { isprivate => $params->{private},
work_time => $params->{work_time} });
# Capture the call to bug->update (which creates the new comment) in
# a transaction so we're sure to get the correct comment_id.
my $dbh = Bugzilla->dbh;
$dbh->bz_start_transaction();
$bug->update();
my $new_comment_id = $dbh->bz_last_key('longdescs', 'comment_id');
$dbh->bz_commit_transaction();
# Send mail.
Bugzilla::BugMail::Send($bug->bug_id, { changer => Bugzilla->user->login });
return undef;
return { id => $self->type('int', $new_comment_id) };
}
1;
@ -977,6 +989,10 @@ be ignored.
=back
=item B<Returns>
A hash with one element, C<id> whose value is the id of the newly-created comment.
=item B<Errors>
=over
@ -1002,6 +1018,8 @@ You did not have the necessary rights to edit the bug.
=item Added in Bugzilla B<3.2>.
=item Modified to return the new comment's id in Bugzilla B<3.4>
=back
=back