comment_ids parameter causes a DB error in Launchpad.comments() in bugzilla-launchpad 0.9-3.2

Bug #269384 reported by Graham Binns
4
Affects Status Importance Assigned to Milestone
Bugzilla Launchpad Plugin
Status tracked in Bugzilla-3.2
Bugzilla-3.0
Fix Released
High
Max Kanat-Alexander
Bugzilla-3.2
Fix Released
High
Max Kanat-Alexander

Bug Description

Passing a comment_ids parameter to Launchpad.comments() on a Bugzilla-Launchpad 0.9-3.2 instance results in a database error on the remote system (traceback attached).

Revision history for this message
Graham Binns (gmb) wrote :
Revision history for this message
Björn Tillenius (bjornt) wrote :

Marking this as High, since it doesn't affect any users that much. The work around is to not supply comment_ids, which will be a bit less efficient, but it works.

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

Okay, I think I've fixed this (the fix was pretty obvious, when I looked at it), and committed the fix:

Committing to: /var/www/html/everythingsolved.com/bzr/canonical/3.2/
modified extensions/launchpad/lib/WSLaunchpad.pm
Committed revision 6057.

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

It looks like this was actually also affecting 3.0, when I checked. So I fixed it there:

Committing to: /var/www/html/everythingsolved.com/bzr/canonical/3.0/
modified extensions/launchpad/lib/WSLaunchpad.pm
Committed revision 4996.

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.