Looks like in the update from 1.x to 2.x the configuration form code for providing the thumbnail option was lost. This also causes problems with saving images to filefield as it attempts to save the thumbnail to the field without an extension.

Attached patch adds the code back in.

CommentFileSizeAuthor
thumbnail_format.patch1.1 KBcivicpixel

Comments

arthurf’s picture

arthurf commented

My thinking here was that jpeg is really the only viable format to output- of course, it wasn't passing the extension correctly. If you look at: http://www.ffmpeg.org/faq.html#SEC15 it appears that you have to switch codecs to do png (and I didn't even explore gif). Since we're taking frames from a video, I think it's reasonable to assume that jpeg is the output of choice. There are questions about the license of jpeg, but I'm not going there :)

I've committed a fix for the file extension to mm_ffmpeg_video_thumbnail() which should hopefully solve this.

arthurf’s picture

arthurf commented
Status: Needs review » Closed (fixed)