Opened 2 years ago

Closed 2 years ago

#13361 closed defect (fixed)

Change lstrip parameter in

Reported by: anonymous Owned by: Ryan J Ollos
Priority: normal Component: ChildTicketsPlugin
Severity: normal Keywords:
Cc: qbi@… Trac Release: 1.2


The Tor Project runs an trac instance. A user recently reported an error:

The line

pid = int(ticket.values.get('parent').lstrip('#'))

led to a traceback and the user suggested to change the line to:

pid = int(ticket.values.get('parent').lstrip('$'))

Could you consider this change and maybe patch the plugin accordingly?


Attachments (0)

Change History (2)

comment:1 Changed 2 years ago by Ryan J Ollos

Owner: changed from Mark Ryan to Ryan J Ollos
Status: newaccepted

I don't follow. Why is the user entering $24207 for a ticket id? Isn't that an invalid input for a parent ticket id?

I think we could at least raise an error that says "$24207 is an invalid ticket id".

comment:2 Changed 2 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

In 17042:

Tracchildtickets 2.5.5dev: Fix TypeError on invalid input

Fixes #13361.

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.