BugzillaLPPlugin.fetchComments() passes incorrect parameters to the remote API

Bug #269386 reported by Graham Binns
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

At the moment, BugzillaLPPlguin.fetchComments() passes dict(bug_ids=[...], ids=[...]) to the Launchpad.comments() API method on the remote Bugzilla. Instead, it should be passing dict(bug_ids=[...], comment_ids=[...]); `ids` is out of date; `comment_ids` is the correct parameter name.

Note that this bug masked bug 269382 and bug 269384; it'll probably only be fixable once those bugs are fixed since to fix it now - at least for Bugzilla 3.2 instances (bug 269384) - would actually make things worse.

Revision history for this message
Graham Binns (gmb) wrote :

Marking this medium priority since it's a performance issue (we get all comments for a bug instead of just one) but not a major one.

description: updated
Changed in malone:
importance: Undecided → Medium
status: New → Triaged
Curtis Hovey (sinzui)
Changed in launchpad:
importance: Medium → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.