Modify

Opened 6 years ago

Closed 6 years ago

#2674 closed defect (fixed)

IndexError: list index out of range

Reported by: nulleke76@… Owned by: eblot
Priority: normal Component: RevtreePlugin
Severity: major Keywords:
Cc: nulleke76@… Trac Release: 0.11

Description

Using the revtree plugin on the latest trac svn, it gives the following error (when logged on on the base url, without /wiki):

Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/Trac-0.11dev_r6631-py2.5.egg/trac/web/main.py", line 232, in dispatch
    self._post_process_request(req)
  File "/usr/lib/python2.5/site-packages/Trac-0.11dev_r6631-py2.5.egg/trac/web/main.py", line 297, in _post_process_request
    resp = f.post_process_request(req, *(None,)*arity)
  File "build/bdist.linux-i686/egg/revtree/web_ui.py", line 158, in post_process_request
    if (url_parts > 1) and (url_parts[1] in self.contexts):
IndexError: list index out of range

this is caused in:

    url_parts = req.path_info.split(u'/')
    if (url_parts > 1) and (url_parts[1] in self.contexts):

Attachments (0)

Change History (4)

comment:1 Changed 6 years ago by anonymous

I see the same problem. Only when accessing root URI.

comment:2 Changed 6 years ago by anonymous

I have the same problem. "/trac" gives "list index out of range" but "/trac/wiki" it's ok.

My configuration is:
Trac: 0.11
Python: 2.4.4 (#1, Jan 10 2007, 01:25:01) [C]
setuptools: 0.6c8
SQLite: 3.5.9
pysqlite: 2.4.1
Genshi: 0.5
mod_python: 3.3.1
Subversion: 1.5.0 (r31699)
jQuery: 1.2.3

comment:3 Changed 6 years ago by anonymous

I have solved the problem in my Trac installation:

In the line 158:

  • if (url_parts > 1) and (url_parts[1] in self.contexts): IndexError: list index out of range

"url_parts > 1" is wrong, must be "len(url_parts)"

  • with the next lines works fine:

def post_process_request(self, req, template, data, content_type):

if req.perm.has_permission('REVTREE_VIEW'):

url_parts = req.path_info.split(u'/')
if (len(url_parts) > 1):

if (url_parts[1] in self.contexts):

add_ctxtnav(req, 'Revtree', href=req.href.revtree())

return (template, data, content_type)

comment:4 Changed 6 years ago by eblot

  • Resolution set to fixed
  • Status changed from new to closed

(In [4012]) Closes #2674.

  • Fix up the invalid syntax in url_parts
  • Use the new Trac API for options (so that TracIni now shows up the proper option)


Thanks to anonymous for the original patch.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.