empty svn:externals break trac browser

[Armin Ronacher]

How to Reproduce

Define svn:externals but keep them empty.

System Information

Trac	0.11dev
Python	2.4.3 (#2, Oct 6 2006, 07:49:22) [GCC 4.0.3 (Ubuntu 4.0.3-1ubuntu5)]
SQLite	3.2.8
pysqlite	2.0.5
Genshi	0.4dev-r493
Pygments	0.7
Subversion	1.3.1

Python Traceback

Traceback (most recent call last):
  File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 401, in dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 213, in dispatch
    resp = chosen_handler.process_request(req)
  File "/usr/lib/python2.4/site-packages/trac/versioncontrol/web_ui/browser.py", line 326, in process_request
    node.get_properties()),
  File "/usr/lib/python2.4/site-packages/trac/versioncontrol/web_ui/browser.py", line 468, in render_properties
    return filter(None, [self.render_property(name, mode, context, props)
  File "/usr/lib/python2.4/site-packages/trac/versioncontrol/web_ui/browser.py", line 480, in render_property
    rendered = renderer.render_property(name, mode, context, props)
  File "/usr/lib/python2.4/site-packages/trac/versioncontrol/svn_fs.py", line 332, in render_property
    return self._render_externals(props[name])
  File "/usr/lib/python2.4/site-packages/trac/versioncontrol/svn_fs.py", line 347, in _render_externals
    localpath, rev, url = elements[0], None, elements[-1]
IndexError: list index out of range

[Christian Boos]

Define svn:externals but keep them empty.

Why on earth would you do that ;)

Ok, I didn't think about that, will fix.

[Christian Boos]

Fixed by r4689.

[Armin Ronacher]

Replying to cboos:

Define svn:externals but keep them empty.

Why on earth would you do that ;)

Obviously because someone was unable to use svn propdel :D