summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormkanat%bugzilla.org <>2009-12-13 22:59:41 +0100
committermkanat%bugzilla.org <>2009-12-13 22:59:41 +0100
commite401b0b8b925787cbe52d0f540b814e40269f228 (patch)
treeb04e953854e13645160b00a97d6ea4b42d431a6e
parent1f2dc1d01e232afb236f382f85609853aac6868d (diff)
downloadbugzilla-e401b0b8b925787cbe52d0f540b814e40269f228.tar.gz
bugzilla-e401b0b8b925787cbe52d0f540b814e40269f228.tar.xz
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.
-rw-r--r--Bugzilla/WebService/Bug.pm5
1 files changed, 1 insertions, 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',