Closed (fixed)
Project:
Wysiwyg
Version:
7.x-2.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 May 2013 at 11:35 UTC
Updated:
25 Feb 2014 at 14:44 UTC
Jump to comment: Most recent
I have downloaded tinymce and ckeditor (latest versions as of today and neither is being recognised by WYSIWYG module. I have tried various fgets() patches in the.inc files to no avail.
Strict warning: Only variables should be passed by reference in wysiwyg_openwysiwyg_settings() (line 103 of /var/www/drupal-examples/sites/all/modules/wysiwyg/editors/openwysiwyg.inc).
Comments
Comment #1
Syntapse commentednicedit and openwysiwyg install without issue. lots of patches and tips online that dont work. would appreciate assistance to get this working... im running Drupal 7.22 trying to load latest versions of tinymce and ckeditor through wysiwyg module.
note: tinymce unzips to variations on directory and filenames specified on drupal/wysiwyg/config. tried changing directory and filenames to match wysiwyg requirements... no joy.
Comment #2
Syntapse commentedfixed in latest wysiwyg dev version
Comment #3
Syntapse commentedfixed
Comment #4
zestudio.net commentedDoesn't work for me, drupal 7, tinymce 4.0.16
Fixed by adding this code in tiny_mce.js, copied from the same file further :
majorVersion:"4",minorVersion:"0.16",releaseDate:"2014-01-31"
at the top of the script near "4.0.16 (2014-01-31)"
->
// 4.0.16 (2014-01-31) majorVersion:"4",minorVersion:"0.16",releaseDate:"2014-01-31"
!function(e,t){"use strict";function n(e,t) ............
The "wysiwyg_tinymce_version" function in sites\all\modules\wysiwyg\editors\tinymce.inc is looking for this version in the first 200 chars, but this code version is further than that position.
So I copied it in the first comment :
// 4.0.16 (2014-01-31) majorVersion:"4",minorVersion:"0.16",releaseDate:"2014-01-31"
works well now
Comment #5
kochertj commentedThe tip about spelling out the version of tinymce really helped with the recognition problem:
'// 4.0.22 (2014-04-16) majorVersion:"4",minorVersion:"0.22",releaseDate:"2014-04-16"' versus just '// 4.0.22 (2014-04-16)'
Comment #6
GrantI commentedI had the same issue with version 4.2.7 and the following code pasted in to the first line of tinymc.js (which I renamed to tiny_mc.js in order to following the WYSIWYG file path/name). This code replaces the first line start //
'// 4.2.7 (2014-04-16) majorVersion:"4",minorVersion:"2.7",releaseDate:"2014-04-16"' versus just '// 4.0.22 (2014-04-16)'