I've got a Video field. The video field allows me to upload a Video thumbnail. I'm using Views 3 to select that field and in the field formatter I can choose Video Player or Video Thumbnail. This works just fine normally, but when I'm generating the output as a JSON document the field renders as the video URL instead of the thumbnail URL.
Only local images are allowed.

When I use the Video Thumbnail formatter in a non-JSON view, it renders the thumbnail image just fine.

Drupal 7.18
Views 7.x-3.5
Views Datasource 7.x-1.x-dev
Video 7.x-2.1

CommentFileSizeAuthor
#7 1880610-7_video_thumbnail.patch1.31 KBzeip
#4 views_datasource-Video_Thumbnail_JSON_rendering_wrong_url-1880610_1.patch1.09 KBsanjeev.chadha
#3 views_datasource-Video_Thumbnail_JSON_rendering_wrong_url-1880610.patch1.21 KBArnion
#2 views_datasource-Video_Thumbnail_JSON_rendering_wrong_url-1880610.patch1.19 KBArnion

Comments

kennywyland’s picture

kennywyland commented

Hm, the image of my config doesn't seem to be working, let me try again:

Screenshot of my config

Arnion’s picture

Arnion commented
StatusFileSize
new views_datasource-Video_Thumbnail_JSON_rendering_wrong_url-1880610.patch1.19 KB

I hope this serves you.

Arnion’s picture

Arnion commented
StatusFileSize
new views_datasource-Video_Thumbnail_JSON_rendering_wrong_url-1880610.patch1.21 KB

Sorry, a little mistake. I upload the patch again.

sanjeev.chadha’s picture

sanjeev.chadha commented
Issue summary: View changes
StatusFileSize
new views_datasource-Video_Thumbnail_JSON_rendering_wrong_url-1880610_1.patch1.09 KB

Issue was observed while selecting style image option as None (original video thumbnail)
Updated the latest patch with ternary condition to suit image style as well as default image.

goblin29’s picture

goblin29 commented
Status: Active » Needs review
rjdjohnston’s picture

rjdjohnston commented

Can we get this merged? Big issue for video users.

#3 and #4 worked beautifully

zeip’s picture

zeip commented
StatusFileSize
new 1880610-7_video_thumbnail.patch1.31 KB

Attached is a patch combining the changes in #3 and #4.

  • ZeiP committed ff53c0e on 7.x-1.x 
    Issue #1880610 by Arnion, sanjeev.chadha, ZeiP: Video Thumbnail JSON...
zeip’s picture

zeip commented
Status: Needs review » Fixed

Committed the fix to 7.x-1.x. Thanks everyone!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.