Modify

Opened 10 years ago

Closed 7 years ago

Last modified 7 years ago

#936 closed defect (fixed)

Error when no tabs are configured

Reported by: anonymous Owned by: Ryan J Ollos
Priority: normal Component: TracTabPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description (last modified by Noah Kantrowitz)

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 Changed 10 years ago by Noah Kantrowitz

Description: modified (diff)

Fix formatting.

comment:2 in reply to:  1 ; Changed 9 years ago by alex@…

Hi,

Replying to coderanger:

Fix formatting.

What do you mean by this?

comment:3 in reply to:  2 Changed 9 years ago by Noah Kantrowitz

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 8 years ago by Russ Tyndall

Owner: changed from Russ Tyndall to Ryan J Ollos

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

comment:5 Changed 7 years ago by Ryan J Ollos

Summary: without any tabs inserted it doesnt work when logged inError when no tabs are configured

comment:6 Changed 7 years ago by Ryan J Ollos

Severity: criticalnormal

comment:7 Changed 7 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

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

comment:8 Changed 7 years ago by Ryan J Ollos

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.

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.