Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Support Michael Beckwith

    (@tw2113)

    The BenchPresser

    The existence of those images is all dependent on Youtube itself, and is nothing we actively control.

    As is in the latest version, these are the lines that determine which gets used.

    $file_headers = get_headers( 'https://img.youtube.com/vi/' . $youtube_id . '/maxresdefault.jpg' );
    $is_404 = $file_headers[0] == 'HTTP/1.0 404 Not Found' || false !== strpos( $file_headers[0], '404 Not Found' );
    $video_thumbnail_url = $is_404 ? 'https://img.youtube.com/vi/' . $youtube_id . '/maxresdefault.jpg' : 'https://img.youtube.com/vi/' . $youtube_id . '/hqdefault.jpg';

    Unless I’m mistaken, every video should have at least the hqdefault.jpg, much like your example above.

    Thread Starter Gerald

    (@gerital)

    The problem is that it works with v1.0.1 but stopped working with the new version. Still didn’t find out what’s the problem, could you try with the above example if it works for? Maybe it’s again something with server configuration (as the problem I had with allow_url_fopen)

    Plugin Support Michael Beckwith

    (@tw2113)

    The BenchPresser

    I’m not able to fully test, due to being out in the middle of the ocean and the ship I’m on has restrictions on internet content types.

    That said, I did look things over with a finer-tooth comb and did see that our thumbnail selection does look like it was incorrectly mixed up. It was basically saying “if this is a 404 error, use it anyway. Else, use the regular image”. This was not part that I had changed myself for the 1.0.2 update, it was one that was made already but had not been released yet. Not deflecting blame, just stating that multiple hands have been on the plugin.

    If you would please, I have pushed up what will be 1.0.3 to GitHub, and would appreciate some testing/confirmation if you are willing.

    https://github.com/WebDevStudios/Automatic-Featured-Images-from-Videos/archive/master.zip

    Plugin Support Michael Beckwith

    (@tw2113)

    The BenchPresser

    Any changes with this one gerital?

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘v 1.0.2 doesn't work for HQ/HD videos’ is closed to new replies.