Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
since adding support for high resolution thumbnails, the low resolution thumbnails no longer work.
need to fix this here.
Comment | File | Size | Author |
---|---|---|---|
#6 | low_resolution_thumbnail-2886546-6.patch | 2.74 KB | emerham |
Comments
Comment #2
afsolano CreditAttribution: afsolano commentedHi,
Here is one solution for this problem. After try to get the maxresdefault preview in getOriginalThumbnailPath function, if the response is a 404 error, try again but this time with hqdefault preview. In theory this preview (hqdefault) always exists for a video.
Sorry for my english.
Comment #3
emerham CreditAttribution: emerham as a volunteer commentedDid a quick test on a site, looks like its working great.
Marking for review.
Comment #4
joseph.olstadCan we take the current protocol instead of hard coding http? What if site is https ?
Comment #5
emerham CreditAttribution: emerham as a volunteer commentedWhy not just always use https?
Comment #6
emerham CreditAttribution: emerham as a volunteer commentedtest this on a few sites. should have most traffic to youtube on SSL only, and low res thumbs are working
Comment #7
joseph.olstadok, that way it should work on both http and https sites without complaints.
Comment #9
joseph.olstadThanks @emerham ,
fixed in 7.x-3.x dev