When media module inserts mediaelement into a wysiwyg we return an audio element, and so the insert fails.
We should do what the media_youtube module does and react to that.

CommentFileSizeAuthor
#3 mediaelement-1462854-media-insert.patch4.4 KBsteven jones
#1 mediaelement-1462854-media-insert.patch4.37 KBsteven jones

Comments

steven jones’s picture

steven jones commented
Status: Active » Needs review
StatusFileSize
new mediaelement-1462854-media-insert.patch4.37 KB
steven jones’s picture

steven jones commented
Status: Needs review » Needs work

This patch causes a warning, and thus needs a little tweak.

steven jones’s picture

steven jones commented
Status: Needs work » Needs review
StatusFileSize
new mediaelement-1462854-media-insert.patch4.4 KB

I think this should be enough, sometimes the $entity is NULL for some reason.

dpfitzsi’s picture

dpfitzsi commented
Status: Needs review » Needs work

Tried to install this patch to work with Media 1.3 (seems that this issue mostly involves the 1.x version of media).

I'm receiving this error when I try the patch:

Notice: Undefined variable: element in mediaelement_field_formatter_view() (line 148 ...

jnettik’s picture

jnettik
Primary language English
Location Denver, CO
commented
Issue summary: View changes
Status: Needs work » Closed (outdated)