Problem with “File set to webp”

  • Resolved Florian

    (@flocco21)


    4 years, 9 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)

    4 years, 8 months ago

    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)

    4 years, 8 months ago

    @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)

    4 years, 8 months ago

    Hi,

    I will wait for the Update.

    Im from Germany

    Best regards
    Florian

    Plugin Author rosell.dk

    (@roselldk)

    4 years, 8 months ago

    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.