Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Contributor Mark Wilkinson

    (@wpmarkuk)

    Hi – can you provide a link to a page on your site which triggers this error please?

    Thread Starter cybeardjm

    (@didierjm)

    @wpmarkuk

    This post contains the reference to the error shown above:

    https://amf.didiermary.fr/empire-bakuba-papy-tex-livre-or/

    As you can see, and I just noticed it, it’s for a “broken” video (deleted…). I have 2100+ posts containing videos on one of my blogs, so this happens “quite a lot”…

    • This reply was modified 4 months ago by cybeardjm.
    Plugin Contributor Mark Wilkinson

    (@wpmarkuk)

    I am not quite sure why the error occurs. In my testing environment, the getimagesizes() function just returns false for that default YouTube thumbnail and I get no warning logs in the error log.

    Does the plugin work correctly for valid Youtube videos?

    (feeling a bit guilty as this was my code…)

    I’ve done some digging – if the fopen wrapper is disabled then you’ll get this error, and whether fopen is available will depend on the web host.

    For the small performance enhancement that height and width provide, it may be better to remove the getfilesize() call.

    Thread Starter cybeardjm

    (@didierjm)

    @wpmarkuk – yeah, it works fine with valid videos.

    It appears, I have plenty (50 or so) of “broken” ones on both blogs, all calling the “broken thumbnail” mqdefault.jpg…

    Plugin Contributor Mark Wilkinson

    (@wpmarkuk)

    This should be corrected in the 1.3.5 release as I have removed the getfilesize() function as recommended by @mattrad

    Thread Starter cybeardjm

    (@didierjm)

    @wpmarkuk cool, thx.

    But you know what, this could be used as part of a useful plugin (or an option to this plugin) to detect broken videos…

    I have a lot of videos embedded in 2 blogs and I’ve been checking regularly the errors to detect videos that were removed by YT.

    Sincerely
    DJM

Viewing 7 replies - 1 through 7 (of 7 total)
  • You must be logged in to reply to this topic.