Joomla! Issue Tracker - J!Tracker

Download
Launch
Feeling Lucky

[#336] - Graphcomposer

enhancement Success

User tests: Successful: Unsuccessful:

avatar elkuku
elkuku
14 Apr 2014

Since clue/graph-composer#12 has been merged, we might now integrate it to our tool chain.
I'm not quite sure where an image like this might be useful...., but maybe there is more than one nerd (like me) that get's excited examining this huge thingy ?

dependencies

It would be nice to "clear" this graph a bit, maybe excluding the dev dependencies and "reorder" the elements, but that's for another PR ;)

Requirements

Usage

  • ./bin/jtracker make composergraph
    • Generate the graph with the default format (SVG) and open it using the default system application.
  • ./bin/jtracker make composergraph --format=[format]
    • Generate the graph with the given format (e.g. svg, svgz, png, jpeg, ...) and open it using the default system application.
  • ./bin/jtracker make composergraph --file=[path/to/file]
    • Generate the graph and save it to the given file taking the format from the file extension.
  • ./bin/jtracker make composergraph --file=[path/to/file] --format=[format]
    • Generate the graph and save it to the given file with the given format (e.g. svg, svgz, png, jpeg, ...).

BTW: I first heard about graphcomposer in an article about PHPStorm's file watchers

avatar elkuku elkuku - open - 14 Apr 2014
avatar b2z
b2z - comment - 21 Apr 2014

Ok it's working, but I want to clear one thing. If I run
make composergraph --file=/vagrant/logs --format=jpg

Then I got:

Format has been set to svg
Format has been set to jpg
The file has been written to /vagrant/logs/graph-composer.svg

So if the file=directory then you def file name to graph-composer.svg:

if (is_dir($filePath))
{
    // If a directory is given, use a default file name.
    $filePath = rtrim($filePath, '/') . '/graph-composer.svg';
}

But later you did not override the file extension, because $filePath is not changed. Is it expected behaviour? :)

avatar elkuku
elkuku - comment - 21 Apr 2014

Is it expected behaviour? :)

No it's a bug, well spotted - should be fixed ;)

avatar b2z b2z - change - 22 Apr 2014
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2014-04-22 20:11:16
avatar b2z b2z - close - 22 Apr 2014
avatar b2z b2z - reference | 7844582 - 22 Apr 14
avatar b2z b2z - merge - 22 Apr 2014
avatar b2z b2z - close - 22 Apr 2014
avatar b2z b2z - head_ref_deleted - 22 Apr 2014

Add a Comment

Login with GitHub to post a comment