Opened 7 years ago

Closed 6 years ago

#1598 closed defect (fixed)

XHTML Incompatibilities

Reported by: seniorquico@… Owned by: cboos
Priority: normal Component: GraphvizPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11


graphviz output that's linked to an image map produces invalid XHTML. The problems are:

  • "area" tag is not closed; should be <area ... />
  • "id" attributes must be unique; duplicated in "area" and "img" tags
  • "id" attributes must begin with a letter; generated "sha_key" has possibility of beginning with a number; static letter should be prefixed to "sha_key"

The first bullet is a problem with graphviz's output. My environment: Win 2k3 Enterprise, Apache 2.2.3, Python 2.5.1, mod_python 3.3.1, graphviz 2.12 (pre-built binary), Trac 0.11dev. I've attached a patch that fixed the output for me.

Please note: this is the first time I've ever coded in Python. So...sorry if something is wrong or if it's messy. I just look at this as a pointer to what could be done. I hope it helps someone out.

Attachments (1) (1.5 KB) - added by seniorquico@… 7 years ago.
Patch file fixing error in ticket

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by seniorquico@…

Patch file fixing error in ticket

comment:1 Changed 6 years ago by franck34

I think you have to report this bug on graphviz mailing list, it's not related to the plugin.
Tool used to generated the sitemap is from graphviz.


Note: i'll try to ask to grapviz team myself, i'm on the ml

comment:2 Changed 6 years ago by cboos

  • Owner changed from pkropf to cboos

comment:3 Changed 6 years ago by cboos

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

Should be fixed by [4401] and [4402].

Add Comment

Modify Ticket

as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from cboos. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.