TinyMCE 3 with WYSIWYG API [#391000]

Attached is a patch that adds a button to TinyMCE 3 used with WYSIWYG API. It's based on the patch in #300643: Video Filter TinyMCE Plugin for 5.x. It's a very basic function right now, that only adds an input field. Later we could add width and height support here.

There's also some minor fixes that slipped in the patch, such as Godtube changed it's name to Tangled.

Instructions:

Apply the patch first. This creates a new folder called tinymce (not sure what the best name for that is though) and some files in there, and alters the module and inc files.
After that, unzip the images.zip archive and put it in the tinymce folder. I've used an icon from http://drupal.org/project/icon, since FamFam isn't GPL.

Comment	File	Size	Author
#7	video_filter-391000-tinymce_plugin.patch	6.35 KB	mrfelton
#7
#6	drupal_391000_video_filter_tinymce_0.patch	10.94 KB	sirkitree
#6
#4	drupal_391000_video_filter_tinymce_0.patch	10.92 KB	sirkitree
#4
#3	drupal_391000_video_filter_tinymce.patch	11.5 KB	helmo
#3
	images.zip	1.01 KB	blackdog
	video_filter_tinymce.patch	13.64 KB	blackdog

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

XerraX CreditAttribution: XerraX commented 8 March 2009 at 20:02

Status:

Needs review

» Needs work

doesnt work with TinyMCE 3.2.2, input stays blank if video filter button is enabled, just like the flash button:
http://drupal.org/node/379182#comment-1295726

Comment #2

TyraelTLK CreditAttribution: TyraelTLK commented 30 March 2009 at 12:32

subscribing

Comment #3

helmo CreditAttribution: helmo commented 28 May 2009 at 07:24

Status:

Needs work

» Needs review

File	Size
drupal_391000_video_filter_tinymce.patch	11.5 KB

Nice patch, although a patch against a clean version would have been nicer ;)

XerraX: It works fine with my TinyMCE 3.2.2.3, you tend to be suffering from a more general wysiwyg vs. TinyMCE problem.
I had a similar problem when I tested the combination a while back, but the solutions provided in #379182 are what solved it for me.

Attached is a clean patch against the latest Development snapshot

Comment #4

sirkitree CreditAttribution: sirkitree commented 29 May 2009 at 00:48

Version:

6.x-2.4

» 6.x-2.x-dev

File	Size
drupal_391000_video_filter_tinymce_0.patch	10.92 KB

Patch was a bit messy - cleaned it up properly and should now apply cleanly.

Please note that you still must download the images.zip to your video_filter/tinymce folder and extract it.

After the application of the patch, first add this filter to your desired input format (regular video_filter instructions) and then in WYSIWYG API settings (/admin/settings/wysiwyg/profile) Edit the input format that you enabled video_filter upon and under the 'Buttons and plugins' fieldset, check off the new 'Video Filter' option.

This patch saved me at least 4 hours of work, thank you very, very much!

Comment #5

helmo CreditAttribution: helmo commented 16 June 2009 at 09:06

To work properly with version 2.x of the wysiwyg API I had to add one line

--- video_filter.module (revision x)
+++ video_filter.module (working copy)
@@ -263,6 +263,7 @@
'path' => drupal_get_path('module', 'video_filter') . '/tinymce/editor_plugin.js',
'buttons' => array('videofilter' => t('Video Filter')),
'url' => 'http://drupal.org/project/video_filter',
+ 'load' => TRUE,
),
);
}

Comment #6

sirkitree CreditAttribution: sirkitree commented 14 July 2009 at 22:34

File	Size
drupal_391000_video_filter_tinymce_0.patch	10.94 KB

The location of the plugins has also changed to /sites/all/libraries

Update to patch.

Comment #7

mrfelton CreditAttribution: mrfelton commented 20 July 2009 at 15:41

File	Size
video_filter-391000-tinymce_plugin.patch	6.35 KB

With the latest code in -dev, applying this patch results in a file with a missing curly bracket, and hense, a fatal error. This patch includes the curly bracket!