Take the extension from options into account when saving the image

[mjauvin]

ref. octobercms/library#546

[mjauvin]

@LukeTowers This is required to allow converting images to other formats (e.g. webp). Otherwise the resulting file will have the correct extension but the wrong image format.

[mjauvin]

In the mean time, the following code is required:

        Event::listen('system.resizer.processResize', function ($resizer, $tempPath) {
            // Get the resizing configuration
            $config = $resizer->getConfig();
            $options = array_get($config, 'options', []);

            list($base, $ext) = explode('.', $tempPath);
            $newPath = $base . '.' . array_get($options, 'extension', $ext);

            // Resize the image
            $resizedImageContents = \Winter\Storm\Database\Attach\Resizer::open($tempPath)
                ->resize($config['width'], $config['height'], $options)
                ->save($newPath);

            if ($newPath != $tempPath) {
                File::move($newPath, $tempPath);
            }
            
            // Prevent any other resizing replacer logic from running
            return true;
        });

[github-actions]

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days.
If this is still being worked on, please respond and we will re-open this pull request.
If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

[github-actions]

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days.
If this is still being worked on, please respond and we will re-open this pull request.
If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

[github-actions]

This pull request will be closed and archived in 3 days, as there has been no activity in the last 60 days.
If this is still being worked on, please respond and we will re-open this pull request.
If this pull request is critical to your business, consider joining the Premium Support Program where a Service Level Agreement is offered.

[mjauvin]

Any concerns for merging this @LukeTowers?

[mjauvin]

Closed in favor of #74