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.

diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm
index 9eb35735160e98b4b0f6391a2717ad8eec90d368..75c6d76b53aaa064d3c6344bbb15ac91a82ee96b 100644 (file)
--- 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',