use webp based previews in the assetmanager if possible

agentejo · Apr 8, 2021 · 689ca6d · 689ca6d
1 parent d09ffd9
commit 689ca6d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/modules/Cockpit/Controller/RestApi.php b/modules/Cockpit/Controller/RestApi.php
@@ -220,7 +220,8 @@ public function image() {
         $mime = $this->param('mime', null);
 
         if ($mime == 'auto' && strpos($this->app->request->headers['Accept'] ?? '', 'image/webp') !== false) {
-            $mime = 'image/webp';
+            $gdinfo = \gd_info();
+            $mime = isset($gdinfo['WebP Support']) && $gdinfo['WebP Support'] ? 'image/webp' : 'auto';
         }
 
         $options = [

diff --git a/modules/Cockpit/Controller/Utils.php b/modules/Cockpit/Controller/Utils.php
@@ -16,15 +16,23 @@ public function thumb_url() {
 
         \session_write_close(); // improve concurrency loading
 
+        $mime = $this->param('mime', 'auto');
+
+        if ($mime == 'auto' && strpos($this->app->request->headers['Accept'] ?? '', 'image/webp') !== false) {
+            $gdinfo = \gd_info();
+            $mime = isset($gdinfo['WebP Support']) && $gdinfo['WebP Support'] ? 'image/webp' : 'auto';
+        }
+
+
         $options = [
             'src' => $this->param('src', false),
             'fp' => $this->param('fp', null),
             'mode' => $this->param('m', 'thumbnail'),
-            'mime' => $this->param('mime', null),
+            'mime' => $mime,
             'filters' => (array) $this->param('f', []),
             'width' => intval($this->param('w', null)),
             'height' => intval($this->param('h', null)),
-            'quality' => intval($this->param('q', 85)),
+            'quality' => intval($this->param('q', 80)),
             'rebuild' => intval($this->param('r', false)),
             'base64' => intval($this->param('b64', false)),
             'output' => intval($this->param('o', false)),