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.

The tinderbox was burning from the checkin of the
e401b0b8 · mkanat%bugzilla.org · 1f2dc1d0 · e401b0b8
Commit e401b0b8 authored Dec 13, 2009 by mkanat%bugzilla.org
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 4 deletions

Bug.pm Bugzilla/WebService/Bug.pm +1 -4

No files found.
--- 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',