Setting convert images to "no" doesn't do anything

[CyberSecutor]

When setting the "convert images" option in NextGenImages to No, it keeps converting the images to webp.

When looking at the code, the config setting allowImageCreation() is never used.

    public function allowImageCreation(): bool
    {
        return (bool)$this->getValue('yireo_nextgenimages/settings/convert_images');
    }

[jissereitsma]

Good point. This was indeed added but never implemented. I've implemented the method in the WebP module version 0.12.4, even though I think it requires further finetuning.

I'll close this issue because the feature is now there. But if you bump into issues with it, do feel free to reopen this ticket again.

[CyberSecutor]

Thank you Jisse,

I've tested it and it works as expected.