[PLUGIN][VideoEncoder] Some videos don't have images (video stream), only audio, handle that

plugins/AudioEncoder/templates/audioEncoder/audioEncoderView.html.twig

@@ -1,3 +1,4 @@
+{% if attachment.getFilename() is not null %}
 <div>
   <figure>
     <audio class="u-audio" src="{{ attachment.getUrl() }}" controls>
@@ -11,3 +12,6 @@
     </figcaption>
   </figure>
 </div>
+{% else %}
+  {# Not stored locally. #}
+{% endif %}

plugins/StoreRemoteMedia/StoreRemoteMedia.php

@@ -69,36 +69,31 @@ class StoreRemoteMedia extends Plugin
         '.*', // Default to allowing any host
     ];
     public array $domain_blacklist = [];
-    // Whether to maintain a copy of the original media or only a thumbnail of it
-    public bool $store_original = false;
-    public ?int $thumbnail_width;
-    public ?int $thumbnail_height;
-    public ?int $max_size;
-    public ?bool $smart_crop;
 
+    // Whether to maintain a copy of the original media or only a thumbnail of it
     private function getStoreOriginal(): bool
     {
-        return $this->store_original ?? Common::config('plugin_store_remote_media', 'store_original');
+        return Common::config('plugin_store_remote_media', 'store_original');
     }
 
     private function getThumbnailWidth(): int
     {
-        return $this->thumbnail_width ?? Common::config('thumbnail', 'width');
+        return Common::config('thumbnail', 'width');
     }
 
     private function getThumbnailHeight(): int
     {
-        return $this->thumbnail_height ?? Common::config('thumbnail', 'height');
+        return Common::config('thumbnail', 'height');
     }
 
-    private function getMaxSize(): int
+    private function getMaxFileSize(): int
     {
-        return $this->max_size ?? Common::config('plugin_store_remote_media', 'max_file_size');
+        return Common::config('plugin_store_remote_media', 'max_file_size');
     }
 
     private function getSmartCrop(): bool
     {
-        return $this->smart_crop ?? Common::config('plugin_store_remote_media', 'smart_crop');
+        return Common::config('plugin_store_remote_media', 'smart_crop');
     }
 
     /**

plugins/VideoEncoder/VideoEncoder.php

@@ -54,7 +54,8 @@ class VideoEncoder extends Plugin
         return '1.0.0';
     }
 
-    public static function shouldHandle (string $mimetype): bool {
+    public static function shouldHandle(string $mimetype): bool
+    {
         return GSFile::mimetypeMajor($mimetype) === 'video' || $mimetype === 'image/gif';
     }
 
@@ -107,10 +108,12 @@ class VideoEncoder extends Plugin
         ]);
 
         $metadata = $ffprobe->streams($file->getRealPath()) // extracts streams informations
-                            ->videos()                      // filters video streams
+            ->videos()                      // filters video streams
-                            ->first();                      // returns the first video stream
+            ->first();                      // returns the first video stream
-        $width  = $metadata->get('width');
+        if (!\is_null($metadata)) {
-        $height = $metadata->get('height');
+            $width  = $metadata->get('width');
+            $height = $metadata->get('height');
+        }
 
         return true;
     }

plugins/VideoEncoder/templates/videoEncoder/videoEncoderView.html.twig

@@ -1,13 +1,23 @@
-<div>
+{% if attachment.getFilename() is not null %}
-    <figure>
+    <div>
-        <video class="u-video" src="{{ attachment.getUrl() }}" controls poster="{{ attachment.getThumbnailUrl('medium')}}">
+        <figure>
-        </video>
+            <video
-        <figcaption>
+                {% if attachment.getWidth() is not null %}
-            {% if attachment.getFilename() is not null %}
+                    class="u-video"
-                <a href="{{ path('attachment_show', {'id': attachment.getId()}) }}">{{ attachment.getBestTitle(note) }}</a>
+                {% else %}
-            {% else %}
+                    class="u-audio"
-                {{ attachment.getBestTitle(note) }}
+                {% endif %}
-            {% endif %}
+                src="{{ attachment.getUrl() }}" controls
-        </figcaption>
+                {% if attachment.getWidth() is not null %}
-    </figure>
+                    poster="{{ attachment.getThumbnailUrl('medium')}}"
-</div>
+                {% endif %}
+            >
+            </video>
+            <figcaption>
+                    <a href="{{ path('attachment_show', {'id': attachment.getId()}) }}">{{ attachment.getBestTitle(note) }}</a>
+            </figcaption>
+        </figure>
+    </div>
+{% else %}
+{# Not stored locally. #}
+{% endif %}