From e401b0b8b925787cbe52d0f540b814e40269f228 Mon Sep 17 00:00:00 2001 From: "mkanat%bugzilla.org" <> Date: Sun, 13 Dec 2009 21:59:41 +0000 Subject: The tinderbox was burning from the checkin of the "webservice attachments by ids" patch--it looks like a part of the patch was just missing or something when it was posted to bmo. I've fixed it now. --- Bugzilla/WebService/Bug.pm | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm index 9eb357351..75c6d76b5 100644 --- a/Bugzilla/WebService/Bug.pm +++ b/Bugzilla/WebService/Bug.pm @@ -413,11 +413,8 @@ sub update_see_also { sub attachments { my ($self, $params) = validate(@_, 'ids', 'attachment_ids'); - my $ids = $params->{ids}; - defined $ids || ThrowCodeError('param_required', { param => 'ids' }); - if (!(defined $params->{ids} - || defined $params->{attachment_ids})) + or defined $params->{attachment_ids})) { ThrowCodeError('param_required', { function => 'Bug.attachments', -- cgit v1.2.3-24-g4f1b