Video Filter TinyMCE Plugin

mpaler - August 27, 2008 - 15:25
Project:Video Filter
Version:5.x-2.4
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

Hi,

I've created a patch/plugin to add a button to TinyMCE wysiwyg editor to work with the video filter. This new TinyMCE button simply pops up a single field form that let's enduser input supported video urls and said url is converted and inserted in proper video filter format.

I am suggesting housing this plugin within the video filter module rather than creating an entirely separate module. This is a situation similar to the tinymce button provided by the img_assist module.

In both cases, the custom TinyMCE plugin relies on the hook_wysiwyg_plugin provided by the Wysiwyg Module (http://drupal.org/project/wysiwyg), the new and soon-to-be defacto controller for wysiwyg support in Drupal. This hook (essentially an api) allows third party modules (eg. Video Filter) to have accompanying editor plugins without having to modify the core code of the parent wyswyg modules (see for example (http://drupal.org/node/152046).

That means this patch/plugin won't work if you don't have TinyMCE enabled via the Wysiwyg Module.

----------------

In the patch, I'm suggesting adding a sample url to a codec. In my situation it helps so I can print out instructions. It might be useful for other situations...

$codecs['google'] = array(
'name' => t('Google Video'),
'sample_url' => 'http://video.google.com/videoplay?docid=-uN1qUeId',
'callback' => 'video_filter_google',
'regexp' => '/video\.google\.com\/videoplay\?docid=(\-?[0-9]+)/',
'ratio' => 400 / 326,
);

Attached is a patch file and a separate zip file that contains the actual tinymce pluging a folder drupalvideo which must be placed into your video_filter directory...

Overall, this patch/plugin is based on the img_assist tinymce plugin.

#1

mpaler - August 27, 2008 - 15:47

files attached...

AttachmentSize
video_filter.patch 6.47 KB
drupalvideo.zip 150 bytes

#2

neyoung - August 30, 2008 - 18:46

The drupalvideo.zip file seems to be empty. I'd love to experiment with this.

#3

mpaler - September 1, 2008 - 16:37

uugh. I was having problems with attachments that day for some reason. Try this one.

AttachmentSize
drupalvideo.zip 12.32 KB

#4

TyraelTLK - September 20, 2008 - 20:23

Hi, I'm getting:
warning: Missing argument 1 for video_filter_menu() in /DRUPALDIR/sites/default/modules/video_filter/video_filter.module on line 221.

(drupal 6.4)

#5

jdotti - October 21, 2008 - 13:20

I try it with drupal 5.X and it is working.

Thank you

JD

#6

jdotti - October 28, 2008 - 09:12
Status:active» reviewed & tested by the community

Update the satus

#7

mpaler - October 28, 2008 - 15:40

Drupal 6.4 > 5.x

;-)

#8

jdotti - November 22, 2008 - 12:54
Status:reviewed & tested by the community» needs work

Hi,

Now module got problems with {lang} tag since new wysiwyg dev version ...

JD

#9

blackdog - October 22, 2009 - 21:11
Status:needs work» won't fix

I won't add any new features to the 5.x branch, except codecs, so i'm afraid this won't get in. I did however commit a wysiwyg feature to the 6.x branch tonight.

#10

moksa - October 25, 2009 - 15:15

Hi,

why i can't patch ??

/ww2/web/club/sites/all/modules/video_filter# patch < video_filter_1.patch
zsh: correct 'patch' to '_patch' [nyae]? n
patching file video_filter.codecs.inc
Hunk #1 FAILED at 7.
Hunk #2 FAILED at 43.
Hunk #3 FAILED at 59.
3 out of 3 hunks FAILED -- saving rejects to file video_filter.codecs.inc.rej
patching file video_filter.module
Hunk #1 FAILED at 215.
1 out of 1 hunk FAILED -- saving rejects to file video_filter.module.rej

Any idea ?

++

#11

blackdog - October 25, 2009 - 20:42

The patch is outdated.

#12

moksa - October 26, 2009 - 13:16

It's a pity. Nothing replaces?

 
 

Drupal is a registered trademark of Dries Buytaert.