File browsing integration with CKEditor seems to be close, but missing a critical step in both the beta2 and dev releases.

I have tried this with CKEditor 1.2 and dev. I've also tried applying this patch, but no luck -- http://drupal.org/node/698452

Anyhow, 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 shown in the patch above, the function of concern seems to be webfm_popup.sendtocaller in webfm_popup.js.

I have noticed there seems to be a workaround for this by using the WYSIWYG module -- any suggestions along those lines are welcome, although I would find it preferable not have to bog things down with extra modules.

Comments

nhck’s picture

nhck commented
Category: bug » support
Status: Active » Closed (works as designed)

Dear notasheep,
thank you for reporting this and helping to make webfm better. However consider this:

The integration with CKEeditor only works if caller parameters of the editor is passed correctly. Please check with whatever CKEditor-Module you are using if its actually doing this. We changed this process in order to eventually accommodate more editors in the future. This is also noted in bold in the release notes for both the beta* and the dev release:
Note: The call for fckeditor/ckeditor changed they must now provide the parameter caller=EDITORNAME

As for the WYSIWYG-module: This is the recommended way - it provides hooks for us to do everything correctly and in order to provide and enhanced user experience with multiple editors.

If you find this related to your editor-module you might like to move this issue or open an new one over there.

Thank you.

notasheep’s picture

notasheep commented

Thanks for the help, nhck. I did install CKeditor through WYSIWYG, following the directions at http://drupal.org/node/932926 -- and the WebFM integration works correctly.

However, CKeditor module offers extra functionality that I would like to keep (such as fine-grained control of which fields to include/exclude rich text editing for various profiles) so it would be great to get this resolved. Since it seems to be an issue with the call made by CKeditor to WebFM, I've opened up an issue on their queue: http://drupal.org/node/1022208

I'd encourage anyone who's knowledgeable about how to resolve this to head over there and get things rolling. It seems like it may be a relatively easy fix -- I'm just not familiar enough with either of these modules to know where to start.