Modify

Opened 8 years ago

Closed 5 years ago

Last modified 5 years ago

#936 closed defect (fixed)

Error when no tabs are configured

Reported by: anonymous Owned by: rjollos
Priority: normal Component: TracTabPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description (last modified by coderanger)

Hi,

when logged in, and no tabs added to the ini (as of a fresh install and activation of the plugin via the webinterface as I did) I get the following backtrace:

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 209, in dispatch
    chrome.populate_hdf(req, chosen_handler)
  File "/usr/lib/python2.4/site-packages/trac/web/chrome.py", line 337, in populate_hdf
    req.hdf['chrome.nav.%s.%s' % (category, name)] = text
  File "/usr/lib/python2.4/site-packages/trac/web/clearsilver.py", line 197, in __setitem__
    self.set_value(name, value, True)
  File "/usr/lib/python2.4/site-packages/trac/web/clearsilver.py", line 245, in set_value
    add_value(name, value)
  File "/usr/lib/python2.4/site-packages/trac/web/clearsilver.py", line 223, in add_value
    set_unicode(prefix, unicode(value))
  File "/usr/lib/python2.4/site-packages/trac/web/clearsilver.py", line 213, in set_unicode
    self.hdf.setValue(prefix.encode('utf-8'), value.encode('utf-8'))
Error: Traceback (innermost last):
  File "neo_hdf.c", line 805, in hdf_set_value()
  File "neo_hdf.c", line 795, in _set_value()
AssertError: Unable to set Empty component chrome.nav.mainnav.

However when adding tabs, it works. It would be nice if it does work too without any configured tabs

Attachments (0)

Change History (8)

comment:1 follow-up: Changed 8 years ago by coderanger

  • Description modified (diff)

Fix formatting.

comment:2 in reply to: ↑ 1 ; follow-up: Changed 7 years ago by alex@…

Hi,

Replying to coderanger:

Fix formatting.

What do you mean by this?

comment:3 in reply to: ↑ 2 Changed 7 years ago by coderanger

Replying to alex@frei-welt.de:

Hi,

Replying to coderanger:

Fix formatting.

What do you mean by this?

It means the author didn't actually format the description to be readable. I corrected it.

comment:4 Changed 5 years ago by bobbysmith007

  • Owner changed from bobbysmith007 to rjollos

This plugin will be adopted soon, changing ownership to new owner

comment:5 Changed 5 years ago by rjollos

  • Summary changed from without any tabs inserted it doesnt work when logged in to Error when no tabs are configured

comment:6 Changed 5 years ago by rjollos

  • Severity changed from critical to normal

comment:7 Changed 5 years ago by rjollos

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

(In [7465]) Create no tabs if none are defined. Fixes #936

comment:8 Changed 5 years ago by rjollos

The behavior I saw in Trac 0.11.7stable-r9029 was that if no trac.ini configuration was provided, then a mainnav bar entry with no label would be created. Navigating to this label would result in an error. This is fixed in [7465].

I don't really intend to support 0.10, so I'm only fixing this on 0.11.

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from rjollos. Next status will be '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.