From 7e9d6173c07396ccac19803140efa77de74c5f2e Mon Sep 17 00:00:00 2001
From: Ariel Ashri <arieliens@gmail.com>
Date: Sun, 1 Oct 2023 15:11:12 +0300
Subject: [PATCH 1/4] CLOUDINARY-469 - change version pattern to v1.19.90

---
 composer.json  | 2 +-
 etc/module.xml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/composer.json b/composer.json
index 1a0f919..880b1aa 100644
--- a/composer.json
+++ b/composer.json
@@ -2,7 +2,7 @@
     "name": "cloudinary/cloudinary",
     "description": "Cloudinary Magento 2 Integration.",
     "type": "magento2-module",
-    "version": "1.19.8.1",
+    "version": "1.19.90",
     "license": "MIT",
     "require": {
         "cloudinary/cloudinary_php": ">=2.7 <=2.11.0"
diff --git a/etc/module.xml b/etc/module.xml
index dbb10f4..7de462a 100644
--- a/etc/module.xml
+++ b/etc/module.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-  <module name="Cloudinary_Cloudinary" setup_version="1.19.8.1">
+  <module name="Cloudinary_Cloudinary" setup_version="1.19.90">
       <sequence>
           <module name="Magento_ProductVideo"/>
           <module name="Magento_PageBuilder"/>

From 35fd1126eca6e4d6bd8ebee378cbd375fdce5227 Mon Sep 17 00:00:00 2001
From: Ariel Ashri <arieliens@gmail.com>
Date: Mon, 27 Nov 2023 12:11:43 +0200
Subject: [PATCH 2/4] CLOUDINARY-476 - fix bug: Product Videos (Vimeo) -
 MAG-847

---
 composer.json                       |  2 +-
 etc/csp_whitelist.xml               |  2 +
 etc/module.xml                      |  2 +-
 view/frontend/web/js/load-player.js | 98 ++++++++++++++---------------
 4 files changed, 52 insertions(+), 52 deletions(-)

diff --git a/composer.json b/composer.json
index 880b1aa..2f347c6 100644
--- a/composer.json
+++ b/composer.json
@@ -2,7 +2,7 @@
     "name": "cloudinary/cloudinary",
     "description": "Cloudinary Magento 2 Integration.",
     "type": "magento2-module",
-    "version": "1.19.90",
+    "version": "1.19.91",
     "license": "MIT",
     "require": {
         "cloudinary/cloudinary_php": ">=2.7 <=2.11.0"
diff --git a/etc/csp_whitelist.xml b/etc/csp_whitelist.xml
index 2e5afe9..bdb7949 100644
--- a/etc/csp_whitelist.xml
+++ b/etc/csp_whitelist.xml
@@ -6,6 +6,8 @@
                 <value id="cloudinary_main" type="host">cloudinary.com</value>
                 <value id="cloudinary_wildcard" type="host">*.cloudinary.com</value>
                 <value id="cloudinary_cloudflare_cdnjs" type="host">cdnjs.cloudflare.com</value>
+                <value id="youtube_scripts" type="host">*.youtube.com</value>
+                <value id="vimeo_scripts" type="host">*.vimeo.com</value>
             </values>
         </policy>
         <policy id="frame-src">
diff --git a/etc/module.xml b/etc/module.xml
index 7de462a..a2927dd 100644
--- a/etc/module.xml
+++ b/etc/module.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-  <module name="Cloudinary_Cloudinary" setup_version="1.19.90">
+  <module name="Cloudinary_Cloudinary" setup_version="1.19.91">
       <sequence>
           <module name="Magento_ProductVideo"/>
           <module name="Magento_PageBuilder"/>
diff --git a/view/frontend/web/js/load-player.js b/view/frontend/web/js/load-player.js
index 8be12c8..0ed669a 100644
--- a/view/frontend/web/js/load-player.js
+++ b/view/frontend/web/js/load-player.js
@@ -10,8 +10,8 @@
 
 define(
     [
-      'jquery',
-      'jquery-ui-modules/widget'
+        'jquery',
+        'jquery-ui-modules/widget'
     ],
     function ($) {
         'use strict';
@@ -59,19 +59,19 @@ define(
                  */
                 _create: function () {
                     switch (this.element.data('type')) {
-                    case 'youtube':
-                        this.element.videoYoutube();
-                        this._player = this.element.data('mageVideoYoutube');
+                        case 'youtube':
+                            this.element.videoYoutube();
+                            this._player = this.element.data('mageVideoYoutube');
                             break;
-                    case 'vimeo':
-                        this.element.videoVimeo();
-                        this._player = this.element.data('mageVideoVimeo');
+                        case 'vimeo':
+                            this.element.videoVimeo();
+                            this._player = this.element.data('mageVideoVimeo');
                             break;
-                    case 'cloudinary':
-                        this.element.videoCloudinary();
-                        this._player = this.element.data('mageVideoCloudinary');
+                        case 'cloudinary':
+                            this.element.videoCloudinary();
+                            this._player = this.element.data('mageVideoCloudinary');
                             break;
-                    default:
+                        default:
                             throw {
                                 name: 'Video Error',
                                 message: 'Unknown video type',
@@ -82,7 +82,7 @@ define(
                                 toString: function () {
                                     return this.name + ': ' + this.message;
                                 }
-                        };
+                            };
                     }
                 },
 
@@ -226,11 +226,11 @@ define(
                                              */
                                             onStateChange: function (data) {
                                                 switch (window.parseInt(data.data, 10)) {
-                                                case 1:
-                                                    self._playing = true;
+                                                    case 1:
+                                                        self._playing = true;
                                                         break;
-                                                default:
-                                                    self._playing = false;
+                                                    default:
+                                                        self._playing = false;
                                                         break;
                                                 }
 
@@ -339,7 +339,7 @@ define(
                 _create: function () {
                     var timestamp,
                         additionalParams = '',
-                        src;
+                        src, id;
 
                     this._initialize();
                     timestamp = new Date().getTime();
@@ -357,27 +357,25 @@ define(
                         this._code +
                         timestamp +
                         additionalParams;
+                    id = 'vimeo' + this._code + timestamp;
                     this.element.append(
                         $('<iframe/>')
-                        .attr('frameborder', 0)
-                        .attr('id', 'vimeo' + this._code + timestamp)
-                        .attr('width', this._width)
-                        .attr('height', this._height)
-                        .attr('src', src)
-                        .attr('webkitallowfullscreen', '')
-                        .attr('mozallowfullscreen', '')
-                        .attr('allowfullscreen', '')
-                        .attr('referrerPolicy', 'origin')
+                            .attr('frameborder', 0)
+                            .attr('id', id)
+                            .attr('width', this._width)
+                            .attr('height', this._height)
+                            .attr('src', src)
+                            .attr('webkitallowfullscreen', '')
+                            .attr('mozallowfullscreen', '')
+                            .attr('allowfullscreen', '')
+                            .attr('referrerPolicy', 'origin')
                     );
-                    this._player = window.$f(this.element.children(':first')[0]);
 
+                    this._player = new Vimeo.Player(this.element.children(':first')[0]);
                     // Froogaloop throws error without a registered ready event
-                    this._player.addEvent(
-                        'ready',
-                        function (id) {
-                            $('#' + id).closest('.fotorama__stage__frame').addClass('fotorama__product-video--loaded');
-                        }
-                    );
+                    this._player.ready().then(function () {
+                        $('#' + id).closest('.fotorama__stage__frame').addClass('fotorama__product-video--loaded');
+                    });
                 },
 
                 /**
@@ -426,22 +424,22 @@ define(
                     var elem = this.element;
                     elem.append(
                         $('<iframe/>')
-                        .attr('frameborder', 0)
-                        .attr('id', 'cloudinary' + this._code + (new Date().getTime()))
-                        .attr('class', 'cld-video-player')
-                        .attr('width', this._width)
-                        .attr('height', this._height)
-                        .attr('src', this._videoUrl.replace(/(^\w+:|^)/, ''))
-                        .attr('webkitallowfullscreen', '')
-                        .attr('mozallowfullscreen', '')
-                        .attr('allowfullscreen', '')
-                        .attr('referrerPolicy', 'origin')
-                        .on(
-                            "load",
-                            function () {
-                                elem.parent('.fotorama__stage__frame').addClass('fotorama__product-video--loaded');
-                            }
-                        )
+                            .attr('frameborder', 0)
+                            .attr('id', 'cloudinary' + this._code + (new Date().getTime()))
+                            .attr('class', 'cld-video-player')
+                            .attr('width', this._width)
+                            .attr('height', this._height)
+                            .attr('src', this._videoUrl.replace(/(^\w+:|^)/, ''))
+                            .attr('webkitallowfullscreen', '')
+                            .attr('mozallowfullscreen', '')
+                            .attr('allowfullscreen', '')
+                            .attr('referrerPolicy', 'origin')
+                            .on(
+                                "load",
+                                function () {
+                                    elem.parent('.fotorama__stage__frame').addClass('fotorama__product-video--loaded');
+                                }
+                            )
                     );
 
                 },

From 4164ce6a4562e2b64d1445838100e0fea27d4994 Mon Sep 17 00:00:00 2001
From: Ariel Ashri <arieliens@gmail.com>
Date: Wed, 6 Dec 2023 15:13:13 +0200
Subject: [PATCH 3/4] CLOUDINARY-479 - Queue not processing videos - The
 Cloudname is null by default, so all videos are replaced with image. (fixed)

---
 Model/Configuration.php | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/Model/Configuration.php b/Model/Configuration.php
index d5c5e52..4ff36d1 100644
--- a/Model/Configuration.php
+++ b/Model/Configuration.php
@@ -376,6 +376,14 @@ public function getFormatsToPreserve()
         return ['png', 'webp', 'gif', 'svg'];
     }
 
+    /**
+     * @return array
+     */
+    public function getSupportedVideoFormats()
+    {
+        return ['mp4', 'webm', 'ogv', 'mov', 'wmv'];
+    }
+
     /**
      * @param  string $file
      * @return string
@@ -703,6 +711,10 @@ public function parseCloudinaryUrl($url, $publicId = null)
         $slug = \array_shift($_url);
         $parsed["type"] = ($parsed["cloudName"] && $slug  === "video") ? "video" : "image";
 
+        if (isset($parsed['extension'])) {
+            $parsed['type'] = (in_array($parsed['extension'], $this->getSupportedVideoFormats())) ? 'video' : 'image';
+        }
+
         $slug = \array_shift($_url);
         $parsed["transformations_string"] = ($slug === 'upload' ? '' : $slug) . implode('/', $_url);
 

From bb949921fabaa6f8c30986765550bb7e279e45e3 Mon Sep 17 00:00:00 2001
From: Ariel Ashri <arieliens@gmail.com>
Date: Mon, 11 Dec 2023 14:32:34 +0200
Subject: [PATCH 4/4] Updating module.xml + composer.json version signature

---
 composer.json  | 2 +-
 etc/module.xml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/composer.json b/composer.json
index 2f347c6..f68ee55 100644
--- a/composer.json
+++ b/composer.json
@@ -2,7 +2,7 @@
     "name": "cloudinary/cloudinary",
     "description": "Cloudinary Magento 2 Integration.",
     "type": "magento2-module",
-    "version": "1.19.91",
+    "version": "1.19.92",
     "license": "MIT",
     "require": {
         "cloudinary/cloudinary_php": ">=2.7 <=2.11.0"
diff --git a/etc/module.xml b/etc/module.xml
index a2927dd..e7c9b4e 100644
--- a/etc/module.xml
+++ b/etc/module.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0"?>
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-  <module name="Cloudinary_Cloudinary" setup_version="1.19.91">
+  <module name="Cloudinary_Cloudinary" setup_version="1.19.92">
       <sequence>
           <module name="Magento_ProductVideo"/>
           <module name="Magento_PageBuilder"/>