Modify

Opened 16 years ago

Closed 16 years ago

#3276 closed defect (fixed)

Admin page not displayed on 0.11 final

Reported by: anonymous Owned by: Colin Guthrie
Priority: high Component: WorkLogPlugin
Severity: normal Keywords:
Cc: denis.kostousov@… Trac Release: 0.11

Description

WorklogPlugin works, but no admin config panel is present. An admin entry is present, but clicking it results in worklog page.

Workaround: in webadminui.py, change type to something other than worklog (like workls as a test). It seems that match_request() in webui.py (or something like that)is catching the admin link.

Sorry - this is not a proper fix. I don't know trac or worklogplugin very well (but know both better now).

Attachments (1)

adminmenu.patch (708 bytes) - added by denis.kostousov@… 16 years ago.
I think this patch can solve the problem temporary

Download all attachments as: .zip

Change History (7)

comment:1 Changed 16 years ago by dsteven@…

I've done some more digging and have a better fix (sorry - no patch, just an edit instruction):

weblog/webui.py

Line 84 (ish)

def match_request(self, req):

if re.search('/worklog'...

replace if re.search line with:

re.match(r'/worklog/?$', req.path_info) is not None:

Like in trac/ticket/web_ui.py

Seems to fix it. Changing the webadmin type dissociates the config and the worklog, and fixes the admin display, but does not allow the options to be actioned.

Hope this helps, and I hope this plugin willbe the help to me that is seems. Nice work.

comment:2 Changed 16 years ago by Colin Guthrie

Status: newassigned

That's strange! I don't get this bug :s

I'm not 100% sure how python's work, but in most regular expressions you can use a caret ^ to mean "start of the line".

So (seeing as I can't replicate here!), can you tell me if a line like:

re.search('^/worklog........

work?

This change should be in line 31, 85, 96 and 100.

I'm not 100% sure how this will work if trac is installed in a subdirectory but I can test that if you can confirm it fixes your problem.

Cheers :)

PS I've got some big changes coming up for this plugin, but I need to discuss some of the details with other Trac people as it would go some way to breaking compatibility with TimingAndEstimationPlugin and I want to make sure that we can move together in some way (perhaps via a common third plugin....)

comment:3 Changed 16 years ago by anonymous

I'll try - my other fix breaks the worklog/users/xxxx pages

My trac URL us http://host/trac/projects/trac-instance-name using mod_python. I'm just setting up a clean place to test/replicate.

comment:4 Changed 16 years ago by anonymous

Thanks. re.search('/worklog.... on the line suggested has fixed it. works a treat.

FYI - setup is all latest (non dev releases) of download sites:

  • fedora 9 i386
  • trac 0.11 final
  • Genshi 0.5
  • python 2.5
  • mod_python

Changed 16 years ago by denis.kostousov@…

Attachment: adminmenu.patch added

I think this patch can solve the problem temporary

comment:5 Changed 16 years ago by anonymous

Cc: denis.kostousov@… added; anonymous removed

comment:6 Changed 16 years ago by Colin Guthrie

Resolution: fixed
Status: assignedclosed

I think this was fixed in [5022].

If it's not please reopen.

Sorry for the delays in addressing this :( (I've been over worked - ooh the irony ;))

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Colin Guthrie.
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.