Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gfed’s picture

harris_david’s picture

I tried this patch after experiencing something that appeared to be the same issue. In the CKEditor, I can add a youtube video, but with I hit the submit button it does nothing. However, the patch did not work for me. The error I'm getting (before and after the patch) is this:

TypeError: 'undefined' is not an object (evaluating 'imgElement.attributes')

hamrant’s picture

Version: 7.x-1.x-dev » 7.x-1.16
Issue summary: View changes
FileSize
559 bytes

I have the same problem and the patch also did not work for me. Here's my solution to the problem.

hamrant’s picture

Tested on ckeditor 7.x-1.16 + media 7.x-1.5 + media_oembed 7.x-2.3

podarok’s picture

Version: 7.x-1.16 » 7.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

#3 looks good. Let's have it.

andypost’s picture

+1 to RTBC

junaidpv’s picture

@harris_david may be you need to install media_wysiwyg module.

junaidpv’s picture

Here is the improved patch. jQuery's `is()` function can have comma separated tag names. Consider the newly added tags while stripping also. Updated nearby comments as well.

jcisio’s picture

Status: Reviewed & tested by the community » Closed (won't fix)

Media integration has been moved to another module.

junaidpv’s picture

For those who wondering, the new integration module to glue media with ckeditor is media_ckeditor.