Modify

Opened 9 years ago

Closed 8 years ago

Last modified 7 years ago

#119 closed defect (fixed)

class diagram are not displayed

Reported by: anonymous Owned by: jparks
Priority: high Component: DoxygenPlugin
Severity: major Keywords:
Cc: serge@… blackhex@… stefan@… Trac Release: 0.9

Description

class diagram are not displayed, the <img> source attribute may not be valid in the include context...

Attachments (2)

doxygentrac.diff (626 bytes) - added by Blackhex 9 years ago.
Image handling patch
doxygentrac.2.diff (6.8 KB) - added by Blackhex 9 years ago.
Image handling patch

Download all attachments as: .zip

Change History (11)

comment:1 Changed 9 years ago by anonymous

  • Cc serge@… added

comment:2 Changed 9 years ago by Blackhex

There was no request handling for any other types than text/html. I'd added this handling but only by openning ie. image file and writing it to request object. I'm new in Track and I did't find out how to send any image to request object more effectively. Here is a diff.

comment:3 Changed 9 years ago by Blackhex

  • Cc serge@… added; serge@… removed

Changed 9 years ago by Blackhex

Image handling patch

comment:4 Changed 9 years ago by Blackhex

Another doxgentrac.diff update. Added Doxygen's search.php searching handling.

Changed 9 years ago by Blackhex

Image handling patch

comment:5 Changed 9 years ago by SunSailor

Added an alternative implementation of this patch, which solves an incompatibility with the reg.args[] not to string conversion. See ticket #194 for the patch and more information.

comment:6 Changed 9 years ago by stefan@…

  • Cc serge@… added; serge@… removed

After trying the provided patches here, which didn't worked for me I created a workaround: In the doxygen generated sources I added a prefix of "/doxygen" to any image <img> tag using a sed script. Also I changed the "nav" CSS class to "doxnav". In the apache configuration I added a LocationMatch directive to match any /doxygen/*.(css|png) as a static page. After this I merged the CSS of doxygen 1.4.6 into a single CSS and removed/changed any style which conflicted with Trac. Now doxygen is fully integrated into Trac including doxygen own styles!

I wanted to attach a tar archive, but I get an Trac error stating can't create unique path name.

comment:7 Changed 8 years ago by maxpag@…

  • Trac Release set to 0.9

Is there any news about this?
I have tried the second patch without success. I can see in the apache logs that the images are not delivered (code 500), but I don't get further messages.
The last solution (using sed to modify doxygen output and changing css toi fix things up) seems interesting, but I am not enough confident with CSS to foresee a quick job.

I'm using:

  • Fedora Core 3 (kernel 2.6.9-1.667)
  • Apache 2.0.53-3.3
  • Python 2.3.4-13.1
  • trac 0.9.4-1.1.fc3.rf
  • subversion subversion-1.2.1-0.1.1.fc3.rf

Thank you in advance

comment:8 Changed 8 years ago by Blackhex

I'll try investigate why is in my environment /doxygen/ path prepended automatically and modify doxygentrac.2.diff accordingly. It's a pitty that there is noone who maintains this plugin and real author didn't respond to my requests.

comment:9 Changed 8 years ago by Blackhex

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

Should be fixed in changeset 871.

Add Comment

Modify Ticket

Action
as closed .
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.