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