Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#6598 closed defect (fixed)

Bad interaction with GanttCalendarPlugin

Reported by: alephnull@… Owned by: Robert Corsaro
Priority: low Component: AnnouncerPlugin
Severity: minor Keywords: gantt
Cc: Trac Release: 0.11


The GanttCalendarPlugin adds two top level navigation items "Gantt ticket", URL /ticketcalendar and "Calendar", URL /ticketgantt. I run the installation off the / location and not the /trac location.

When accessing either of these URLs, the traceback is:

2010-02-09 18:19:35,295 Trac[main] ERROR: need more than 1 value to unpack
Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/trac/web/", line 233, in dispatch
  File "/usr/lib/python2.5/site-packages/trac/web/", line 301, in _post_process_request
    f.post_process_request(req, *(None,)*extra_arg_count)
  File "build/bdist.linux-x86_64/egg/announcerplugin/subscribers/", line 138, in post_process_request
    realm, _ = path.split('/', 1)
ValueError: need more than 1 value to unpack

It appears that the watcher module does not identify these two URLs as unwatchable and the fact that there is no /trac prefix to all URLs is causing the problem. Is there a way to specify URL paths to ignore ?

Attachments (0)

Change History (4)

comment:1 Changed 8 years ago by Mario


you can try following patch :-)

---	(revision 5)
+++	(revision 6)
@@ -134,6 +134,8 @@
                 'email' in req.session):
             for pattern in self.watchable_paths:
                 path = self.normalise_resource(req.path_info)
+                if path == 'ticketgantt' or path == 'ticketcalendar':
+                    break 
                 if re.match(pattern, path):
                     realm, _ = path.split('/', 1)
                     if '%s_VIEW'%realm.upper() not in req.perm:

comment:2 Changed 8 years ago by alephnull@…

Thank you. This certainly fixed the problem. However, I do have a follow-up question. Is it not conceivable that other plugins will introduce similar top-level URLs like these ? Or is it forbidden by trac policy ?

If the former, we should have a more generic form of excluding URLs, if the latter, then this ticket should be closed.

comment:3 Changed 8 years ago by Robert Corsaro

Resolution: fixed
Status: newclosed

(In [7544]) Fixes incompatibility with some plugin urls

Fixes #6598

comment:4 Changed 8 years ago by Robert Corsaro

(In [7570]) Fixes incompatibility with some plugin urls

Fixes #6598

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Robert Corsaro.
The resolution will be deleted.

Add Comment

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

Note: See TracTickets for help on using tickets.