• Resolved oriflame23

    (@oriflame23)


    Please add support for WEBP. files. Error: File not supported wp-content/plugins/lightbox-photoswipe/src/LightboxPhotoSwipe/ExifHelper.php:16″

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Author Arno Welzel

    (@awelzel)

    Webp is supported. This is bug, since Exif can not be read from webp. I will fix this as soon as possible.

    Plugin Author Arno Welzel

    (@awelzel)

    There is now a Codeberg issue as well to fix this: https://codeberg.org/askaaron/lightbox-photoswipe/issues/115

    Plugin Author Arno Welzel

    (@awelzel)

    Also keep in mind, that the message this a warning, not a hard error which would stop the code. The code which tries to read the EXIF data using @exif_read_data(). And if @exif_read_data() can not read the file, it will just ignore it, since the @ in front of the function call tells PHP not to output any warning message even if the call will trigger a warning due to a runtime problem (also see PHP: Error Control – Manual).

    In fact it should still work even if there is no fix to avoid calling this function for webp files, you just see a warning in the log. Also do not configure the server to treat warnings as hard errors – this should only be done on a development system.

    • This reply was modified 1 week, 1 day ago by Arno Welzel.
    Thread Starter oriflame23

    (@oriflame23)

    Can the patch simply block this call for WebP?

    Plugin Author Arno Welzel

    (@awelzel)

    Yes a patch will block this as it is already the case for other files like SVG, GIF etc. – I will provide an update as soon as possible. In the past there was experimentel webp support implemented as PHP code in the EXIF helper which had been removed due to massive performance issues.

    Thread Starter oriflame23

    (@oriflame23)

    In general, this is the best plugin today! Thank you very much!

    Plugin Author Arno Welzel

    (@awelzel)

    Version 5.8.5 will fix this warning. However due to the new release policy of WordPress, it will take 24 hours before this version is available.

    • This reply was modified 1 week, 1 day ago by Arno Welzel.
    Thread Starter oriflame23

    (@oriflame23)

    Ок! Thank you very much!

Viewing 8 replies - 1 through 8 (of 8 total)

You must be logged in to reply to this topic.