I get an error on pages with wysiwyg editors. The javascript library file is not found.

In the function advagg_process_js there is a check for not preprocessing certain files. The file tiny_mce.js should not be preprocessed. The url that is built is not correct. On line 861 the javascript line is nuilt with file_create_url and the resulting url is:

/sites/default/files/sites/all/libraries/tinymce/jscripts/tiny_mce/tiny_mce.js?T
and it should be off course:
/sites/all/libraries/tinymce/jscripts/tiny_mce/tiny_mce.js?T

CommentFileSizeAuthor
#3 advagg-fix-cdn-1077388-3.patch4.73 KBmikeytown2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pgrond’s picture

pgrond CreditAttribution: pgrond commented

Why do you use file_create_url and not just url() ? (solves it for me, at least in this case)

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented

file_create_url is used because of the CDN module... looks like I should only use it if CDN is running.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Active » Fixed
FileSize
advagg-fix-cdn-1077388-3.patch4.73 KB

This patch has been committed.

pgrond’s picture

pgrond CreditAttribution: pgrond commented

Thanks, that works for me.

Status: Fixed » Closed (fixed)

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