Comment 9 for bug 341997

Revision history for this message
Tim Hatch (timhatch) wrote :

I replied last night over email, but I think Launchpad ate my reply...

I can't reproduce any better with the mysql backend. Seems to not be the issue. Various validations are happening in the xml-rpc system itself, such as long ints being passed in so that's not it. When bugid (that is, the Trac bug) is a string it does produce the issue.

The current behavior is slightly incorrect (talking about tracext/launchpad/rpc.py:252-254 here), and I see that although it's linking to the correct bug in the linkified custom field, the one displayed in the ticket comment is incorrect. I've pushed a fix for this, which will fix the issue only if Malone is actually passing the Trac bug id as a string. I'm holding out on making a new release until we know whether there's some way that's possible (Graham's previous comment suggests it's not).

Ronny, can you check the log to see if an exception like this one is showing up? Specifically is it line 252 in the last stack frame?

2009-03-13 21:50:21,894 Trac[web_ui] ERROR: Traceback (most recent call last):
  File "/Users/tim/code/trac/xmlrpcplugin/tracrpc/web_ui.py", line 60, in process_request
  File "/Users/tim/code/trac/xmlrpcplugin/tracrpc/api.py", line 85, in __call__
    return default
  File "/Users/tim/code/trac/trac-launchpad/tracext/launchpad/rpc.py", line 252, in set_launchpad_bug
TypeError: int argument required