Plans for drupal 7 port?

I've no plans yet. I'd be delighted if you went for it!

Woohoo! I'll review it soon and get back to you.

Download from DRUPAL-6--1 branch instead of HEAD.

Are there any further plans to update this? Looks like a lot of functions in Drupal core have changed since this last patch was posted, and the module no longer appears to work with the 7.0 release.

subscribing!.. Anybody?

The patch in #8 was not enough - I had to replace every instance of

file_directory_path()

with

file_default_scheme() . ':/'

Once that was done, it appeared to work just fine, but I'm only using it to build node reference graphs.

Here are the changes I mentioned in #12, combined with the original patch from #8 to make a new patch.

+++ b/*.info
@@ -3,4 +3,5 @@ description = Produces a pseudo-UML class diagram of the content types using Gra
+files[] = cck_schema.module
\ No newline at end of file

All (?) .info files

+++ b/graphviz_filter.module
@@ -119,25 +115,24 @@ function graphviz_filter_file_download($file) {
+	file_delete($file->filename);

whitespace

I'm working on #1209526: Add support for graphviz_filter module. Will come back on this on.

Ok, tabs swapped for spaces in the module, and newlines appended to all info files. How's that?

Notice: Undefined index: PATH in _graphviz_filter_filter_settings() (line 147 of /Users/clemens/Sites/d.o.clone/downloads/graphviz_filter/graphviz_filter.module).

Trying to set the path fails.

Hum, I'm afraid that's in the part of the code I didn't touch, and I don't know how to fix it. Furthermore, that line is not touched at all by the patch that I submitted, and comes from the original code, so I'm really not sure how to proceed. Sorry.

+++ b/graphviz_filter.module
@@ -58,30 +58,25 @@ class Functor {
+function graphviz_filter_filter_info() {

Could use default settings.

http://api.drupal.org/api/drupal/modules--filter--filter.api.php/functio...

+++ b/graphviz_filter.module
@@ -58,30 +58,25 @@ class Functor {
+function _graphviz_filter_filter_process($text, $format) {
+  return preg_replace_callback(GRAPHVIZ_REGEX, Functor::create_functor('_graphviz_filter_filterproc', $format), $text);
 }

$format is not used by graphviz_filter_filterproc

function graphviz_filter_filterproc($text) {
  // Parse arguments.
  list($formats, $args) = _graphviz_filter_formats();
  _graphviz_filter_parse_args($text, $args);

Is that functor really necessary?

  $form['graphviz_filter']['graphviz_filter_dot_path'] = array(
    '#type' => 'textfield',
    '#title' => t('Path of Graphviz commands'),
    '#default_value' => $filter->settings['graphviz_filter']['graphviz_filter_dot_path'],
    '#description' => t('Specify the base path of the Graphviz commands (`dot`, `neato`, etc). 
                         Leave blank if these commands are to be found on the system PATH which, on this server, is set to: %path',
                         array('%path' => isset($_ENV['PATH']) ? $_ENV['PATH'] : '- empty -')
                       ),
  );

I had to replace variable_get by $filter->settings[...]

My PATH is empty so changed the $_ENV

function _graphviz_filter_filter_settings($form, &$form_state, $filter, $format, $defaults, $filters) {
  //dsm(func_get_args(), __FUNCTION__);
  dsm($filter, __FUNCTION__);

function args changed according to http://api.drupal.org/api/drupal/modules--filter--filter.api.php/functio... states

Hope you could apply these throughout the module ... I'll have code for graphapi :-(

As I've only been making rather superficial code changes and I'm not currently up to speed on the Drupal 7 API, I'm not really competent to answer these questions and make these changes. I'm happy to tweak and test patches, but that's all I can do right now.

I think in graphviz-filter-d7-v4.patch line 148 should be:

+  file_prepare_directory(file_build_uri($outdir), FILE_CREATE_DIRECTORY);

instead of:

+  file_prepare_directory($outdir, FILE_CREATE_DIRECTORY);

subscribe

subscribe --- SUPER

Committed to new D7 branch. Fixed some of the concerns in #18 and #20. Please keep this issue closed and open new issues as you encounter them.

Thanks to all contributors!