Opened 14 years ago

Last modified 14 years ago

#889 closed defect

Clicking "Help" throws a Python error — at Version 7

Reported by: base Owned by: Martin Svensson
Priority: minor Milestone: BASE 2.6
Component: web Version: 2.5.0
Keywords: Cc:

Description (last modified by Martin Svensson)

http://base2.thep.lu.se/demo/ BASE 2.5.0beta (build #3933; schema #45)

To reproduce the error, simply: Click on Help, Help.

Traceback (most recent call last):

File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 356, in dispatch_request

dispatcher.dispatch(req)

File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 224, in dispatch

resp = chosen_handler.process_request(req)

File "/usr/lib/python2.4/site-packages/trac/web/chrome.py", line 234, in process_request

raise HTTPNotFound('File %s not found', filename)

HTTPException: 404 Not Found (File doc/docbook/html/index.html not found)

Thanks,

Jay Hannah - jay@… http://clab.ist.unomaha.edu/CLAB/index.php/User:Jhannah

Note! Existing servers has to be updated manually. Consult ticket#930 and documentation: Server configurations to know how.

Change History (7)

comment:1 Changed 14 years ago by Nicklas Nordborg

Milestone: BASE 2.6

comment:2 Changed 14 years ago by Martin Svensson

Owner: changed from everyone to Martin Svensson
Status: newassigned

comment:3 Changed 14 years ago by Martin Svensson

Resolution: fixed
Status: assignedclosed

(In [4140]) Fixes #889 Clicking "Help" throws a Python error. Changed an invalid link

comment:4 Changed 14 years ago by Nicklas Nordborg

Resolution: fixed
Status: closedreopened

I think the fix only works for new installations. It will not update the link on existing installations. There are two approaches:

  1. Fix the code so that it updates the link in existing installation.
  2. Document this issue in this ticket and tell user how to manually change the link and what the correct link should be.

comment:5 Changed 14 years ago by Martin Svensson

A suggestion to cover both.

  1. Similar option like there is for help texts will be put into base.config (server.configurations.update = true|false). Default value for this option will be false.
  2. Documentation - See ticket#930

comment:6 Changed 14 years ago by Nicklas Nordborg

A: is not a very good solution. Users will probably not want to loose their custom made "About the server" text and other settings just to fix the bad link. It will require more work to restore those texts than to manually fix the link as in B.

B: There should be a note in this ticket!

Personally I think B is good enough. Just put the correct link in the description part of the ticket (not as a comment since that list is growing large).

comment:7 Changed 14 years ago by Martin Svensson

Description: modified (diff)

Added reference to documentation ticket

Note: See TracTickets for help on using tickets.