Problem with “File set to webp”

  • Resolved Florian

    (@flocco21)


    5 years, 8 months ago

    Hello,

    Recently I have a little problem with the plugin.

    I convert all pictures as picture.webp – I have set it that way. (File set to webp).

    Recently, however, all pictures are saved as picture.jpg.webp – but I don’t know why.

    This is a problem for me because I need to override some files in my workflow as WebP (without JPG).

    Does anyone know why this is?

    The page I need help with: [log in to see the link]

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author rosell.dk

    (@roselldk)

    Hi,

    Sorry for the delayed response. I have been vacationing.

    Yes, I can reproduce the bug.
    Thanks for notifying!

    I have fixed the bug in the development branch.
    I shall have this fix part of the next bug-fix release (0.14.22)

    If you don’t want to wait for that release, you can easily fix it manually.

    Line 85-88 in plugins/webp-express/lib/classes/ConverterHelperIndependent.php says:

    
$destination = preg_replace('/\\.(jpe?g|png)$/', '', $source) . '.webp';
// TODO: make this check work with symlinks
//$destination = SanityCheck::absPathIsInDocRoot($source . '.webp');
$destination = $source . '.webp';

    The last three of these lines must be removed. They probably got copied in there by mistake.

    Plugin Author rosell.dk

    (@roselldk)

    @flocco21: I’d like to credit you in the release notes for reporting this. When doing this, I usually write the country the person is from. I like the fact that people all around the world are participating in making this plugin better ??

    So, are you ok with being credited, and are you willing to reveal the country of your residence/origin/whatever for this purpose?

    Thread Starter Florian

    (@flocco21)

    Hi,

    I will wait for the Update.

    Im from Germany

    Best regards
    Florian

    Plugin Author rosell.dk

    (@roselldk)

    Update is here.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Problem with “File set to webp”’ is closed to new replies.