I have updated the module from 2.1 to 2.2

I have now several errors in the logs:

Nachricht Warning: file_get_contents(http://www.xxx/sites/default/files/js/wysiwyg/wysiwyg_ckeditor_uWMoMQ7qlhtyf-cFJlMTazsAxhCeS88weKBiAtXujAQ.js) [function.file-get-contents]: failed to open stream: no suitable wrapper could be found in drupal_build_js_cache() (Zeile 4846 von /xxxx/www/includes/common.inc).

and

Nachricht Warning: file_get_contents() [function.file-get-contents]: http:// wrapper is disabled in the server configuration by allow_url_fopen=0 in drupal_build_js_cache() (Zeile 4846 von /is/xxxx/www/includes/common.inc).

Files: 
CommentFileSizeAuthor
#4 wysiwyg-1802394-4.patch468 bytesTwoD

Comments

Title:Warning: file_get_contents from 2.1 to 2.2Warning: file_get_contents from 7.2.1 to 7.2.2

sub

This happens because Wysiwyg now dynamically generates a small script file and loads it before the library scripts to be able to set things like an editor's base path properties.

The generated file in itself does not cause the problem, but it does not get included properly in the aggregated script file Drupal creates for the optimization. The generated file is placed in public://js/wysiwyg/wysiwyg_EDITOR_HASH.js and Drupal should have recognized that it is a local file instead of trying to fetch it via HTTP from itself (which isn't allowed since your server has allow_url_fopen=0.

A quick workaround, with the implied performance penalties, would be to disable JavaScript Optimization on the Performance page while we're looking into this.

Note: The warning is annoying but mostly harmless. It should only be able to cause any problems when using editors that have been loaded via AJAX, like with the Panels in-place editor, because Wysiwyg is no longer able to tell the editor where to load its internal files from.
Update: This would also cause problems in locale module since it tries to parse script files as well.

Status:Active» Needs review
StatusFileSize
new468 bytes

Here's a simple fix.
There was no need for the file_create_url() call because drupal_add_js() supports stream wrappers.

Can you please verify this change removes the warnings for you too?

The warnings are gone, thank you!

Status:Needs review» Fixed

Thanks for reporting, reviewing, and testing! Committed to 7.x-2.x.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status:Fixed» Closed (fixed)

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

Title:Warning: file_get_contents from 7.2.1 to 7.2.2Warning: file_get_contents from 7.x-2.1 to 7.x-2.2

Updated the somewhat misleading version numbers in the title.

Status:Closed (fixed)» Active

Can we please get another official release of the module that includes this fix?

Status:Active» Closed (fixed)

Please look for issues like #1838100: Time for a 7.x-2.3 release? rather than re-opening individual ones.

Status:Closed (fixed)» Active

Hmm. I've updated to the dev version but I still get this problem (base path not set to the correct location of CKEDITOR).

I'm using aggregation and my app uses push/pop state to change the URL. CKEDITOR seems to take the latest URL as the base path, which is daft.

Hope setting this to "active" was the right thing to do. Perhaps you think it's a separate issue? I note the comment in the code "this won't work with ajax" line 387 of wysiwgy.module.

Status:Active» Closed (fixed)

Hmm. Sorry, don't think it's a bug in this module. Something breaks when you turn aggregation on with ckeditor.

Can confirm this issue with 8.x-2.2 when JS aggregation is enabled.