There's a few issues I've seen so far:

  1. The wrong thumbnail is shown as being selected for a given video.
  2. The "Refresh" button doesn't work. It looks to still be using the old #ahah APIs instead of what's needed for D7.
  3. Clicking a thumbnail doesn't appear to save it as the promoThumbnail for the video.
CommentFileSizeAuthor
#3 ooyala_backport_thumbs-1287972.patch1.43 KBquicksketch
#2 ooyala_refresh_d7-1287972.patch10.1 KBquicksketch
#1 ooyala_refresh_d7-1287972.patch7.19 KBquicksketch

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Needs review
StatusFileSize
new ooyala_refresh_d7-1287972.patch7.19 KB

Here's a patch that at least fixes #2 and gets the thumbnails refreshing properly with D7 AJAX. I'll work on fixing the rest of the issues also.

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new ooyala_refresh_d7-1287972.patch10.1 KB

This patch fixes the file saving so that new thumbnails are immediately updated.

quicksketch’s picture

quicksketch
he/him
commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Needs review » Fixed
StatusFileSize
new ooyala_backport_thumbs-1287972.patch1.43 KB

Committed to 7.x-2.x.

I also committed a partial backport to D6-2.x that ensures we always have 5 thumbnails.

Status: Fixed » Closed (fixed)

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