Modify

Opened 19 years ago

Closed 19 years ago

Last modified 9 years ago

#119 closed defect (fixed)

class diagram are not displayed

Reported by: anonymous Owned by: Jason Parks
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 Radek Bartoň 19 years ago.
Image handling patch
doxygentrac.2.diff (6.8 KB) - added by Radek Bartoň 19 years ago.
Image handling patch

Download all attachments as: .zip

Change History (12)

comment:1 Changed 19 years ago by anonymous

Cc: serge@… added; anonymous removed

comment:2 Changed 19 years ago by Radek Bartoň

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 19 years ago by Radek Bartoň

Cc: serge@… added; serge@… removed

Changed 19 years ago by Radek Bartoň

Attachment: doxygentrac.diff added

Image handling patch

comment:4 Changed 19 years ago by Radek Bartoň

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

Changed 19 years ago by Radek Bartoň

Attachment: doxygentrac.2.diff added

Image handling patch

comment:5 Changed 19 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 19 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 19 years ago by maxpag@…

Trac Release: 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 19 years ago by Radek Bartoň

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 19 years ago by Radek Bartoň

Resolution: fixed
Status: newclosed

Should be fixed in changeset 871.

comment:10 Changed 9 years ago by Ryan J Ollos

Cc: serge@… blackhex@… stefan@… added; serge@… removed

Modify Ticket

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