Asset Buttons does not appear in TinyMCE editor version 3. They do appear on the TinyMCE profile page though.

Firebug has presented following JS errors when navigating to page with TinyMCE editor:

1. 
tinyMCE.importPluginLanguagePack is not a function
[Break on this error] tinyMCE.importPluginLanguagePack('drupalasset', 'en');

sites/all/modules/tinymce/tinymce/jscripts/tiny_mce/plugins/drupalasset/editor_plugin.js


2. 
tinyMCE.getEditorId is not a function
(no name)()asset_tinymce_hel... (line 17)
e(Object type=click target=a#wysiwyg4body)jquery.js (line 2)
e("click", [Object type=click target=a#wysiwyg4body], a#wysiwyg4body javascript:mceToggle...)jquery.js (line 2)
e("click", undefined)jquery.js (line 2)
e([a#wysiwyg4body javascript:mceToggle...], function(), ["click"])jquery.js (line 2)
e(function(), ["click"])jquery.js (line 2)
e()jquery.js (line 2)
e(undefined)jquery.js (line 2)
(no name)()asset_tinymce_hel... (line 13)
e([a#asset-link-edit-body.asset-textarea-link textarea, a#asset-link-edit-log.asset-textarea-link textarea], function(), undefined)jquery.js (line 2)
e(function(), undefined)jquery.js (line 2)
(no name)()asset_tinymce_hel... (line 6)
e()jquery.js (line 2)
[Break on this error] if(tinyMCE.getEditorId(textarea_id) == null){

sites/all/modules/asset/tinymce/asset_tinymce_helper.js
CommentFileSizeAuthor
#16 drupalasset.zip10.42 KBjide

Comments

seutje’s picture

seutje commented

the "getEditorId is not a function" can easily be resolves as u will see in this post: http://drupal.org/node/215580

still looking for a way to fix the "importPluginLanguagePack is not a function", coz it's killing my drupalbreak :(

wmostrey’s picture

wmostrey commented

Here is a patch for the getEditorId issue: http://drupal.org/files/issues/asset_tinymce_helper.js_3.patch.

I'll cook something up for the other issue.

oskarb’s picture

oskarb commented

I confirm that the "getEditorId issue" is resolved. I would be greateful for the other issue too :)

wmostrey’s picture

wmostrey commented
Version: 5.x-1.0-beta4 » 5.x-1.x-dev
Status: Active » Fixed

This has been fixed on dev in http://drupal.org/cvs?commit=132111.

marcingy’s picture

marcingy commented
Status: Fixed » Needs work

This patch works for tinymce 3 but breaks support in tinymce 2 as the function being called does not exist.

wmostrey’s picture

wmostrey commented

Hi Marc, indeed it does. This is in the same way that the dev version supports imagecache2 but not imagecache1 anymore. There's no way to keep track of older versions. In the official release of rc2 there will be a warning message about this.

marcingy’s picture

marcingy commented

yes but the issue is that technically drupal 5 version of tinymce is only 2.1. This patch should only apply to D6 branch when it is release. This is only my thinking of course. See this issue with regards issues http://drupal.org/node/234380. Given that this patch hasn't been commited my understand is D5 = 2.1 D6=3.

I'll happily roll a patch based on the solution provided on the above issue so as it can be committed.

wmostrey’s picture

wmostrey commented

Thanks a lot for pointing that out. I'll submit a patch to TinyMCE and hope it gets submitted soon.

1kenthomas’s picture

1kenthomas commented

subscribe(me);

1kenthomas’s picture

1kenthomas commented

Also note: TinyMCE does not seem to be maintained (per sun). Under D5, I am running v3 to maintain compatibility with jQuery 1.2.6 and IMCE: I do not see a path to that under TinyMCE v2 (perhaps sun's wysiwyg editor, but it does not run for me on a cursory attempt).

wmostrey’s picture

wmostrey commented

Hey Kenneth, what exactly do you mean? So you're running v3 with which patch then? By default the TinyMCE module for Drupal 5 only supports TinyMCE v2, right?

1kenthomas’s picture

1kenthomas commented

Hey Wim,

I've been running TinyMCE 3.x for about 6 months: via http://drupal.org/node/234380 at this point (gosh). Also see http://drupal.org/node/235972#comment-971092 : I actually have to exclude a lot more textbox types at this point...

wmostrey’s picture

wmostrey commented

I'll revert the patch then and continue support for TinyMCE for release candidate 2 as long as TinyMCE 3 isn't stable for Drupal 5.

wmostrey’s picture

wmostrey commented
Category: bug » feature
Status: Needs work » Postponed

Revert is done: http://drupal.org/cvs?commit=135295

Waiting for complete TinymCE3 support for Drupal 5. This will be included in the Drupal 6 port of course.

rarps’s picture

rarps commented

I have been using the Tinytinymce module, insted of the Tinymce module. Tinytinymce does support Tinymce 3 perfectly on Drupal 5. Acctually I find the Tinytinymce module way better (quicker to install and configure) than the traditional tinymce module.

So it is not correct to assume that Tinymce 3 is not supported on Drupal 5, since it depends on which module you are using to integrate Tinymce on Drupal.

jide’s picture

jide commented
StatusFileSize
new drupalasset.zip10.42 KB

I rewrote the TinyMCE plugin for TinyMCE 3 (i had a situation where there was TinyMCE module for D5 with TinyMCE 3). Maybe this can be useful to someone.

freddyseubert’s picture

freddyseubert commented

Hey jide!

First: Thanks for updating the plugin! I have one issue: In line 77 of editor_plugin.js it says: url : '/index.php?q=asset/wizard/tinymce' + query, ... That is causing an error "Not Found - The requested URL /index.php was not found on this server." if I click on the drupalasset-icon... I have clean urls activated, working on safari 3.2.1 on OS X 10.5.6 with drupal 5.14, Asset 5.x-1.0, TinyMCE 5.x-1.9 and TinyMCE 3.2.3.1...

Got an idea? Thanks :)

edit: it works if I change the above mentioned line to url : 'http://my.complete.path/to_drupal_directory/asset/wizard/tinymce' + query, - so "it works" means: the 404 error is gone and the asset GUI shows up. But it's in the false "mode" (I do not know how to explain that)... I can't choose a file to insert, I only can edit my files, delete them and so on. :/

wmostrey’s picture

wmostrey commented
Status: Postponed » Closed (won't fix)

TinyMCE has been replaced by Wysiwyg API.