-
-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update SRF 'format=graph' to use GraphViz extension version 1.1+. #46
Conversation
Update SRF composer.json to suggest mediawiki/graphviz.
$result = renderGraphviz( $graphInput ); | ||
// Calls graphvizParserHook function from MediaWiki GraphViz extension | ||
global $wgParser; | ||
$result = GraphViz::graphvizParserHook( $graphInput, "", $wgParser ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use $GLOBALS['wgParser'] instead since we don't want to expose global $wgParser;
to other objects
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to other objects
??? The only difference is in the function scope.
I prefer using $GLOBALS, though in this case it is the last of our worries, and global $wg
is used all over the place anyway. So I'd not block this commit on that
Update SRF 'format=graph' to use GraphViz extension version 1.1+.
Release notes should be updated to state that format=graph is supported by PS : https://www.mediawiki.org/wiki/Extension:GraphViz still shows some |
Thanks for making |
@kghbln FYI |
Also update SRF composer.json to suggest mediawiki/graphviz.