Fixed issue with empty maxwidth oembed param

Author: alexbech

CHANGELOG.md

@@ -1,5 +1,9 @@
 # ToolMate Changelog
 
+## 1.0.1 - 2020-12-14
+### Fixed
+- Fixed issue with empty maxwidth oembed param
+
 ## 1.0.0 - 2020-05-09
 
 ### Added

composer.json

@@ -2,7 +2,7 @@
     "name": "vaersaagod/toolmate",
     "description": "Is that a tool in your pocket, or are you just happy to see me, mate?",
     "type": "craft-plugin",
-    "version": "1.0.0",
+    "version": "1.0.1",
     "keywords": [
         "craft",
         "cms",
@@ -28,8 +28,8 @@
     ],
     "require": {
         "craftcms/cms": "^3.1.0",
-        "matthiasmullie/minify": "^1.3.0",        
-        "voku/html-min": "^4.4.0"        
+        "matthiasmullie/minify": "^1.3.0",
+        "voku/html-min": "^4.4.0"
     },
     "autoload": {
         "psr-4": {

src/services/EmbedService.php

@@ -75,10 +75,6 @@ public function getVideoEmbed(string $videoUrl, array $params = []): array
         // set the semi-ubiquitous parameters
         $maxWidth = isset($params['max_width']) ? '&maxwidth=' . $params['max_width'] : '';
         $maxHeight = isset($params['max_height']) ? '&maxheight=' . $params['max_height'] : '';
-        if (empty($maxHeight)) // correct for a bug in YouTube response if only maxheight is set and the video is over 612px wide
-        {
-            $maxHeight = '&maxheight=' . $maxWidth;
-        }
         $wmode_param = isset($params['wmode']) ? '&wmode=' . $params['wmode'] : '';
         $url .= $maxWidth . $maxHeight . $wmode_param;