Skip Menu |

This queue is for tickets about the SQL-Translator CPAN distribution.

Report information
The Basics
Id: 65794
Status: rejected
Priority: 0/
Queue: SQL-Translator

People
Owner: Nobody in particular
Requestors: cvicente [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.11007
Fixed in: (no value)



Subject: Graphviz graph shows unconnected nodes
Hi, Thanks for this awesome module. I use it in this other project in which we were having issues with the generated schema graph. I'm copying the thread and the patch sent to me. The patch is against 0.11003-1, but I just checked and it looks like the line is the same in the latest version. Let me know if you need any further information. Regards, cv On Thu, Nov 04, 2010 at 10:52:05AM +0000, Brian Candler wrote: Show quoted text
> > I found one specific problem with the current auto-generated schema
picture: Show quoted text
> > if you look at the graphviz source, it has a load of fake, unconnected > > 'node' entries. These are rendered as a long horizontal line of
unconnected Show quoted text
> > boxes, squeezing the rest of the diagram to the right-hand edge.
I fixed this one: it seems to be a bug in SQL::Translator (Ubuntu installs 0.11003-1 which I believe is the most recent) --- /usr/share/perl5/SQL/Translator/Producer/GraphViz.pm.orig 2010-11-05 18:29:05.585271435 +0000 +++ /usr/share/perl5/SQL/Translator/Producer/GraphViz.pm 2010-11-05 18:42:14.637242388 +0000 @@ -519,7 +519,7 @@ $node_args->{cluster} = $cluster_name; } - $gv->add_node(qq["$table_name"], %$node_args); + $gv->add_node($table_name, %$node_args); debug("Processing table '$table_name'"); That gets rid of all the spurious nodes.
This was long fixed as RT#21065 and released as 0.10000