Modify

Opened 8 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

Description

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)

xhtml-graphviz.py.patch (1.5 KB) - added by seniorquico@… 8 years ago.
Patch file fixing error in ticket

Download all attachments as: .zip

Change History (4)

Changed 8 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.

Cheers

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

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.