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 8 years ago.
Image handling patch
doxygentrac.2.diff (6.8 KB) - added by Blackhex 8 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 8 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 8 years ago by Blackhex

  • Cc serge@… added; serge@… removed

Changed 8 years ago by Blackhex

Image handling patch

comment:4 Changed 8 years ago by Blackhex

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

Changed 8 years ago by Blackhex

Image handling patch

comment:5 Changed 8 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 8 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 .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from jparks. Next status will be '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.