comment_ids parameter is ignored by Launchpad.comments() in bugzilla-launchpad 0.9-3.0

Bug #269382 reported by Graham Binns
2
Affects Status Importance Assigned to Milestone
Bugzilla Launchpad Plugin
Invalid
Undecided
Unassigned
Bugzilla-3.0
Invalid
Undecided
Unassigned

Bug Description

Launchpad.comments() can take a comment_ids parameter. Specifying this means that Launchpad.comments() should only return the comments whose IDs are referenced in that parameter (which is a list / array of comment IDs).

However, passing comment_ids to a Bugzilla Launchpad Plugin instance version 0.9-3.0 seems to have no effect. If a bug_ids parameter is also passed to Launchpad.comments(), all the comments for that bug are returned. If no bug_ids parameter is specified then no comments at all are returned. This seems to suggest that the comment_ids parameter is being ignored altogether by Launchpad.comments().

Revision history for this message
Max Kanat-Alexander (mkanat) wrote :

Is it only the 3.0 plugin that has this problem?

Revision history for this message
Graham Binns (gmb) wrote : Re: [Bug 269382] Re: comment_ids parameter is ignored by Launchpad.comments() in bugzilla-launchpad 0.9-3.0

> Is it only the 3.0 plugin that has this problem?

Bug 269384 affects the 3.2 plugin, so I guess so (i.e. the fact that
in the 3.2 plugin supplying comment_ids raises a DB error suggests
it's not being ignored there.

Revision history for this message
Max Kanat-Alexander (mkanat) wrote :

Now that I've fixed bug 269384, I can't reproduce this locally.

Revision history for this message
Max Kanat-Alexander (mkanat) wrote :

Invalid on all affected branches as far as I can tell.

Changed in bugzilla-launchpad:
status: New → Invalid
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.