From 91023875735830ba44027353e8ad355012212328 Mon Sep 17 00:00:00 2001
From: celestora <kitsuruko@gmail.com>
Date: Tue, 7 Feb 2023 22:39:07 +0200
Subject: [PATCH] Minor improvements in video attachments

---
 Web/Models/VideoDrivers/VideoDriver.php          |  2 +-
 Web/Models/VideoDrivers/YouTubeVideoDriver.php   |  6 +++---
 Web/Presenters/templates/Videos/View.xml         |  2 +-
 .../templates/components/attachment.xml          | 16 +++++++++++++---
 4 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/Web/Models/VideoDrivers/VideoDriver.php b/Web/Models/VideoDrivers/VideoDriver.php
index 141e738b..04c4d23a 100644
--- a/Web/Models/VideoDrivers/VideoDriver.php
+++ b/Web/Models/VideoDrivers/VideoDriver.php
@@ -14,5 +14,5 @@ abstract class VideoDriver
     
     abstract function getURL(): string;
     
-    abstract function getEmbed(): string;
+    abstract function getEmbed(int $w = 600, int $h = 340): string;
 }
diff --git a/Web/Models/VideoDrivers/YouTubeVideoDriver.php b/Web/Models/VideoDrivers/YouTubeVideoDriver.php
index 93aee0e5..a4bc58c7 100644
--- a/Web/Models/VideoDrivers/YouTubeVideoDriver.php
+++ b/Web/Models/VideoDrivers/YouTubeVideoDriver.php
@@ -13,12 +13,12 @@ final class YouTubeVideoDriver extends VideoDriver
         return "https://youtu.be/$this->id";
     }
     
-    function getEmbed(): string
+    function getEmbed(int $w = 600, int $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/Videos/View.xml b/Web/Presenters/templates/Videos/View.xml
index 4d5aebf3..fd2d268f 100644
--- a/Web/Presenters/templates/Videos/View.xml
+++ b/Web/Presenters/templates/Videos/View.xml
@@ -21,7 +21,7 @@
             {if !$driver}
                 Эта видеозапись не поддерживается в вашей версии OpenVK.
             {else}
-                {$driver->getEmbed()|noescape}
+                {$driver->getEmbed(-1, -1)|noescape}
             {/if}
         {/if}
     </center>
diff --git a/Web/Presenters/templates/components/attachment.xml b/Web/Presenters/templates/components/attachment.xml
index 3b5f9915..dccca5c8 100644
--- 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()|noescape}
+        {/if}
+    {/if}
+
     <div class="video-wowzer">
         <img src="/assets/packages/static/openvk/img/videoico.png" />
         <a href="/video{$attachment->getPrettyId()}">{$attachment->getName()}</a>