Modify

Opened 10 years ago

Closed 7 years ago

#11932 closed defect (fixed)

ValueError: invalid literal for int() with base 10: '37 163'

Reported by: fossil Owned by: Ryan J Ollos
Priority: normal Component: MasterTicketsPlugin
Severity: normal Keywords:
Cc: Trac Release:

Description

How to Reproduce

While doing a GET operation on /ticket/164, Trac issued an internal error.

I migrated from 0.12 to 1. Tickets 37 163 are dependent on 164. After I've changed the description and saved the ticket 164, I cannot access the ticket 164 anymore. I think the problem is that the dependent ticket numbers in the old version were separated just by spaces and in the line 79 it is assumed, that they are split by comma.

Request parameters:

{'id': u'164'}

User agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/34.0.1847.116 Chrome/34.0.1847.116 Safari/537.36

System Information

Trac 1.0.1
Babel 1.3
Docutils 0.11
Genshi 0.7 (with speedups)
GIT 1.9.1
mod_python 3.3.1
MySQL server: "5.5.38-0ubuntu0.14.04.1", client: "5.5.38", thread-safe: 1
MySQLdb 1.2.3
Pygments 1.6
Python 2.7.6 (default, Mar 22 2014, 23:03:14)
[GCC 4.8.2]
pytz 2012c
RPC 1.1.2
setuptools 3.3
Subversion 1.8.8 (r1568071)
jQuery 1.7.2

Enabled Plugins

TracGit 0.12.0.5dev
TracMasterTickets 3.0.2
TracXMLRPC 1.1.2

Python Traceback

Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/trac/web/main.py", line 497, in _dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.7/dist-packages/trac/web/main.py", line 224, in dispatch
    self._post_process_request(req, *resp)
  File "/usr/lib/python2.7/dist-packages/trac/web/main.py", line 338, in _post_process_request
    resp = f.post_process_request(req, *resp)
  File "/usr/lib/python2.7/dist-packages/mastertickets/web_ui.py", line 79, in post_process_request
    old = set([int(n) for n in field_data['old'].split(',')])
ValueError: invalid literal for int() with base 10: '37 163'

Attachments (0)

Change History (4)

comment:1 in reply to:  description Changed 10 years ago by Ryan J Ollos

Replying to fossil:

I think the problem is that the dependent ticket numbers in the old version were separated just by spaces and in the line 79 it is assumed, that they are split by comma.

Which version is the old version you are referring to?

Duplicate of #8830, where I noted that I could not produce the issue. Please try upgrading to the latest, 3.0.5. If the issue still occurs with that version I will see about putting in a fix.

comment:2 Changed 10 years ago by Ryan J Ollos

Component: ManPageRendererPluginMasterTicketsPlugin
Owner: changed from Piers O'Hanlon to Ryan J Ollos

comment:3 Changed 9 years ago by Ryan J Ollos

#12525 closed as a duplicate.

comment:4 Changed 7 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

In 16038:

4.0.0dev: Use Trac 1.0 database API

Fixes #11932, #12193.

Modify Ticket

Change Properties
Set your email in Preferences
Action
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.