It would be cool to display GraphViz generated workflow graphs on the workflow page.

CommentFileSizeAuthor
#1 graphviz-integration-1556598-1.patch4.55 KBzroger

Comments

zroger’s picture

zroger commented
Assigned: Unassigned » fmitchell
StatusFileSize
new graphviz-integration-1556598-1.patch4.55 KB

and the patch.

zroger’s picture

zroger commented
Status: Active » Needs review

I forgot to mention, you'll need graphviz and PEAR Image_Graphviz installed for this to work. On a mac you this can be done like so:

brew install graphviz
pear install Image_GraphViz
fmitchell’s picture

fmitchell commented

apparently graphviz does not like php 5.3.x

Strict warning: Non-static method System::mktemp() should not be called statically, assuming $this from incompatible context in Image_GraphViz->saveParsedGraph() (line 1005 of /usr/local/Cellar/php/5.3.8/lib/php/Image/GraphViz.php).
Strict warning: Non-static method System::_parseArgs() should not be called statically, assuming $this from incompatible context in System->mktemp() (line 393 of /usr/local/Cellar/php/5.3.8/lib/php/System.php).
Strict warning: Non-static method Console_Getopt::getopt2() should not be called statically, assuming $this from incompatible context in System->_parseArgs() (line 84 of /usr/local/Cellar/php/5.3.8/lib/php/System.php).
Strict warning: Non-static method Console_Getopt::doGetopt() should not be called statically, assuming $this from incompatible context in Console_Getopt->getopt2() (line 75 of /usr/local/Cellar/php/5.3.8/lib/php/Console/Getopt.php).
Strict warning: Non-static method PEAR::isError() should not be called statically, assuming $this from incompatible context in Console_Getopt->doGetopt() (line 109 of /usr/local/Cellar/php/5.3.8/lib/php/Console/Getopt.php).

etc....

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented

XRef Graph API #1806520: How to provide support for graphviz (properly)

s.daniel’s picture

s.daniel commented

I use PHP 5.3.10-1ubuntu3.4 with Suhosin-Patch (cli) along with graphviz_filter and can't confirm the general issues with php 5.3.x

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
commented

I'm not sure but as state machine has views integration why not use Graph API as the workflow renderer.

Graph API needs nodes + links and can render to several 'graph' engines like thejit, d3, graph phyz and (almost) to graphviz_filter.

That would relieve state machine from integration with graphviz_filter.

I have to fix #1209526: Add support for graphviz_filter module quick but @s.Daniel provided for a simple patch.