Minor improvements in video attachments

OpenVK · Feb 7, 2023 · a9d021b · a9d021b
1 parent 7ed870c
commit a9d021b
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 7 deletions.
diff --git a/Web/Models/VideoDrivers/VideoDriver.php b/Web/Models/VideoDrivers/VideoDriver.php
@@ -14,5 +14,5 @@ abstract function getThumbnailURL(): string;
 
     abstract function getURL(): string;
 
-    abstract function getEmbed(): string;
+    abstract function getEmbed(string $w = 600, string $h = 340): string;
 }
diff --git a/Web/Models/VideoDrivers/YouTubeVideoDriver.php b/Web/Models/VideoDrivers/YouTubeVideoDriver.php
@@ -13,12 +13,12 @@ function getURL(): string
         return "https://youtu.be/$this->id";
     }
 
-    function getEmbed(): string
+    function getEmbed(string $w = "600", string $h = "340"): string
     {
         return <<<CODE
         <iframe
-               width="600"
-               height="340"
+               width="$w"
+               height="$h"
                src="https://www.youtube-nocookie.com/embed/$this->id"
                frameborder="0"
                sandbox="allow-same-origin allow-scripts allow-popups"

diff --git a/Web/Presenters/templates/components/attachment.xml b/Web/Presenters/templates/components/attachment.xml
@@ -9,9 +9,19 @@
         </a>
     {/if}
 {elseif $attachment instanceof \openvk\Web\Models\Entities\Video}
-    <div class="bsdn media" data-name="{$attachment->getName()}" data-author="{$attachment->getOwner()->getCanonicalName()}">
-        <video class="media" src="{$attachment->getURL()}"></video>
-    </div>
+    {if $attachment->getType() === 0}
+        <div class="bsdn media" data-name="{$attachment->getName()}" data-author="{$attachment->getOwner()->getCanonicalName()}">
+            <video class="media" src="{$attachment->getURL()}"></video>
+        </div>
+    {else}
+        {var $driver = $attachment->getVideoDriver()}
+        {if !$driver}
+            <span style="color:red;">{_version_incompatibility}</span>
+        {else}
+            {$driver->getEmbed("100%")|noescape}
+        {/if}
+    {/if}
+
     <div class="video-wowzer">
         <img src="/assets/packages/static/openvk/img/videoico.png" />
         <a href="/video{$attachment->getPrettyId()}">{$attachment->getName()}</a>