(Originally posted on the WebFM issue queue: http://drupal.org/node/1021026 )

File browsing integration with WebFM seems to be close, but missing a critical step in both the 1.2 and dev releases. I have tried each of these with WebFM beta1, beta2, and dev.

The problem occurs at the step after choosing 'Send to rich text editor' in WebFM -- when the file URL should be passed to CKEditor but instead nothing is passed.

As of the beta* and latest dev releases of WebFM, the release notes have the following notice in bold:

Note: The call for fckeditor/ckeditor changed they must now provide the paramter caller=EDITORNAME

It seems like this should be a very simple fix, but I'm not familiar enough with either of these modules to know where to start.

Comments

notasheep’s picture

notasheep commented

OK -- seems I should have trusted myself more. Here's what I did to fix it:

In ckeditor.module, on line 767, change from:
$settings[$textarea_id]['filebrowserBrowseUrl'] = $host ."index.php?q=webfm_popup";
to:
$settings[$textarea_id]['filebrowserBrowseUrl'] = $host ."index.php?q=webfm_popup&caller=ckeditor";

I'm still relatively inexperienced, so I don't know how to write a patch for this -- but at least I found the fix.

dczepierga’s picture

dczepierga commented
Status: Active » Fixed

I commit changes to CVS.

Please check the latest dev release and let me know if you notice any poblems with it. Remember to clear Drupal cache and browsers cache before testing.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

activelink’s picture

activelink commented

This issue is still a problem with the latest modules for Drupal 6, and thanks to notasheep's post above I found the answer, but there is a more graceful way to implement the fix, in the Advanced Settings of your CKEditor profile you can add the following to the Custom javascript configuration:

filebrowserBrowseUrl = Drupal.settings.basePath + '?q=webfm_popup&caller=ckeditor';

This achieves the same result without hacking any modules.