Modify

Opened 9 years ago

Closed 7 months ago

#1564 closed defect (fixed)

Links on doxygen Main Page resulted in call of trac search

Reported by: anonymous Owned by: cboos
Priority: normal Component: DoxygenPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

Problem was, that if I clicked for example 'Classes' Link, the plugin searched for a file /path/to/doxygen/classes.html instead of /path/to/doxygen/html/classes.html. The result was that it didn't find the page and rerouted to the search.

I had to apply the following patch (for the 0.10 version).

Note: I did NOT test what happens if the 'html_output' option is not set in 'trac.ini'.

--- doxygentrac.py.orig 2007-05-23 09:42:35.750206000 +0200
+++ doxygentrac.py  2007-05-23 09:43:07.942571000 +0200
@@ -263,7 +263,7 @@
                                                    self.html_output,
                                                    file or self.index])
             else:
-                doc = ''
+                doc = self.html_output

         def lookup(file, category='undefined'):
             """Build (full path, relative link) and check if path exists."""

Attachments (0)

Change History (11)

comment:1 Changed 9 years ago by matthijs

I've been experiencing the same problem and can confirm that the proposed fix works for me as well.

comment:2 Changed 9 years ago by Blackhex

  • Owner changed from Blackhex to cboos

comment:3 follow-ups: Changed 9 years ago by jfemiani

Same problem.

I dont know python, so just used

[doxygen]
path = \path\to\doxygen\output\html
html_output = ./

in trac.ini

Any chance of resolving this issue and committing the changes you folks have made?

comment:4 Changed 9 years ago by Blackhex

  • Owner changed from cboos to Blackhex
  • Status changed from new to assigned

OK, I'll test patch tomorrow and if it will be working I'll commit it...

comment:5 Changed 9 years ago by Blackhex

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

Done with r2651.

comment:6 Changed 9 years ago by gerard.breiner@…

  • Resolution fixed deleted
  • Status changed from closed to reopened

Hello,

I've got the version tracdoxygen 0.10 with the hack r2651 but I have a similar problem with my links.

My links seems to be correct (ie "path/doxygen/html/index.html when I focus the link "Main Page") but when I click on this link I get a search page for "index.html".

My context doxygen in trac.ini is

[doxygen] path = /home/trac/nulltimate/doxygen index = index.html wiki_index = DoxyGen default_documentation = doxygen #html_output = html ext = 'py'

Would you have an idea please ?

Thanks in advance

Gerard

comment:7 Changed 9 years ago by Blackhex

  • Owner changed from Blackhex to cboos
  • Status changed from reopened to new

Cboos should be more competent to answer now...

comment:8 in reply to: ↑ 3 Changed 7 years ago by lucasrangit

Replying to john.femiani@asu.edu:

Same problem.

I dont know python, so just used

[doxygen]
path = \path\to\doxygen\output\html
html_output = ./

in trac.ini

Any chance of resolving this issue and committing the changes you folks have made?

That fixed it for me as well. Using TracDoxygen-0.11.0.2 and the following trac.ini configuration I get similar behavior until I use the configuration described by John.

[doxygen]
path = /var/local/doxygen
html_output = html
default_documentation = project

comment:9 Changed 7 years ago by lucasrangit

And I'm using Trac 0.11.1

comment:10 in reply to: ↑ 3 Changed 4 years ago by anonymous

  • Trac Release changed from 0.10 to 0.12

Replying to john.femiani@asu.edu:

Same problem.

I dont know python, so just used

[doxygen]
path = \path\to\doxygen\output\html
html_output = ./

in trac.ini

Any chance of resolving this issue and committing the changes you folks have made?

Thanks man! The search feature was not working for me. Basically, I was not able to access any of the files inside folders. This fixed it as well. Thanks a ton!

comment:11 Changed 7 months ago by Committo-Ergo-Sum

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

In 15358:

DoxygenPlugin: new implementation of IRequestHandler methods. In particular, the "path" parameter in the query-string is droped. This fixes #772 #951 #962 #1564 and #2702 who complain about the security vulnerability it may contain, and other wrong path generations.

Add Comment

Modify Ticket

Action
as closed The owner will remain cboos.
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.