diff options
| author | mkanat%bugzilla.org <> | 2009-12-13 22:59:41 +0100 |
|---|---|---|
| committer | mkanat%bugzilla.org <> | 2009-12-13 22:59:41 +0100 |
| commit | e401b0b8b925787cbe52d0f540b814e40269f228 (patch) | |
| tree | b04e953854e13645160b00a97d6ea4b42d431a6e /Bugzilla | |
| parent | 1f2dc1d01e232afb236f382f85609853aac6868d (diff) | |
| download | bugzilla-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.
Diffstat (limited to 'Bugzilla')
| -rw-r--r-- | Bugzilla/WebService/Bug.pm | 5 |
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', |