Edgewall Software
Modify

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#4891 closed defect (wontfix)

Unable to generate a view of an existing ticket due to bug with Master Ticket plugin.

Reported by: dpeterson@… Owned by: Jonas Borgström
Priority: normal Milestone:
Component: general Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I've run a query generating a list of tickets (so they all exist) and then click on one of the ticket links and get the following traceback:

Traceback (most recent call last):
  File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 387, in dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 239, in dispatch
    template, content_type = self._post_process_request(req,
  File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 269, in _post_process_request
    content_type)
  File "build/bdist.linux-x86_64/egg/mastertickets/web_ui.py", line 59, in post_process_request
  File "build/bdist.linux-x86_64/egg/mastertickets/util.py", line 19, in linkify_ids
  File "/usr/lib/python2.4/site-packages/trac/ticket/model.py", line 40, in __init__
    self._fetch_ticket(tkt_id, db)
  File "/usr/lib/python2.4/site-packages/trac/ticket/model.py", line 86, in _fetch_ticket
    raise TracError('Ticket %d does not exist.' % tkt_id,
TypeError: int argument required

Looks like a bug with the master ticket plugin.

Attachments (0)

Change History (3)

comment:1 by dpeterson@…, 17 years ago

Changing util.py's linkify_ids method to the following allows the tickets to display even if the blocking or blocked ticket id's aren't valid. Probably not ideal code, but it works. :-)

def linkify_ids(env, req, ids):
    l = []
    for i in ids:
        try:
            href = req.href.ticket(i)
            t = Ticket(env, i)
            status = t['status']
            l.append(unicode(html.A('#%s'%i, href=href, class_='%s ticket'%status)))
        except:
            l.append(unicode('#%s' % i))
    #l = [unicode(html.A('#%s'%i, href=req.href.ticket(i), class_='%s ticket'%Ticket(env, i)['status'])) for i in ids]

    return Markup(', '.join(l))

comment:2 by Emmanuel Blot, 17 years ago

Resolution: wontfix
Status: newclosed

Please report the bug to the maintainer of the plugin: th:wiki:MasterTicketsPlugin

This Trac is dedicated to the development of Trac itself, not to the plugins.
Thanks in advance.

BTW, do not forget to report the version of Trac you have troubles with when your report an issue.

comment:3 by anonymous, 17 years ago

Okay, created ticket 1280 in the other Trac.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.