From bf47b93b09571cb1d5f81499a5c537383c611459 Mon Sep 17 00:00:00 2001
From: yambenari24 <yam.benari@kaltura.com>
Date: Tue, 28 Jan 2025 17:08:26 +0200
Subject: [PATCH 1/4] fix(SUP-46065): Player shows VR button based on "360" (or
 variations) tag

---
 src/kaltura-player.ts | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/kaltura-player.ts b/src/kaltura-player.ts
index 5b5c2a5b5..191c93e41 100644
--- a/src/kaltura-player.ts
+++ b/src/kaltura-player.ts
@@ -131,12 +131,16 @@ export class KalturaPlayer extends FakeEventTarget {
       })
     );
     this._cuepointManager = new CuePointManager(this);
+
+    const vrTag = this.getVrTag(options.plugins);
+
     this._provider = new Provider(
       Utils.Object.mergeDeep(options.provider, {
         logger: { getLogger, LogLevel },
         referrer: getOriginalRequestReferrer()
       }),
-      __VERSION__
+      __VERSION__,
+      vrTag
     );
     this._playlistManager = new PlaylistManager(this, options);
     Object.values(CoreEventType).forEach((coreEvent) => this._eventManager.listen(this._localPlayer, coreEvent, (e) => this.dispatchEvent(e)));
@@ -535,6 +539,14 @@ export class KalturaPlayer extends FakeEventTarget {
     return this._localPlayer.isVr();
   }
 
+  public getVrTag(config: PluginsConfig): string | null {
+    const vrPlugin = config['vr'];
+    if (vrPlugin) {
+      return vrPlugin.tag ?? '360';
+    }
+    return null;
+  }
+
   public toggleVrStereoMode(): void {
     const vrPlugin: BasePlugin | undefined = this._pluginManager.get('vr');
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment

From c1697a69ed8ad2edcf1a7e482f30cb5e566b8ce9 Mon Sep 17 00:00:00 2001
From: yambenari24 <yam.benari@kaltura.com>
Date: Thu, 30 Jan 2025 18:51:09 +0200
Subject: [PATCH 2/4] fix(SUP-46065): Player shows VR button based on "360" (or
 variations) tag

---
 demo/player-ovp.html  | 67 ++++++++++++++++++++++++++++++++-----------
 src/kaltura-player.ts |  6 ++++
 2 files changed, 57 insertions(+), 16 deletions(-)

diff --git a/demo/player-ovp.html b/demo/player-ovp.html
index 997d39b3d..54942a201 100644
--- a/demo/player-ovp.html
+++ b/demo/player-ovp.html
@@ -8,26 +8,61 @@
     <title>Player</title>
     <link rel="stylesheet" type="text/css" href="./style.css" />
     <script src="./kaltura-ovp-player.js" type="text/javascript"></script>
+    <script src = "http://localhost:8081/playkit.js"></script>
+    <script src = "http://localhost:8082/playkit-ui-managers.js"></script>
+    <script src = "http://localhost:8083/playkit-ovp-provider.js"></script>
+    <script src = "http://localhost:8084/playkit-vr.js"></script>
+<!--    <script src = "http://localhost:8083/playkit-timeline.js"></script>-->
+<!--    <script src = "http://localhost:8087/playkit-ivq.js"></script>-->
+<!--    <script src = "http://localhost:8088/playkit-ui.js"></script>-->
+<!--    <script src = "http://localhost:8083/playkit-vr.js"></script>-->
+<!--    <script src = "http://localhost:8084/playkit-ovp-provider.js"></script>-->
+    <!--    <script src="https://cdn.jsdelivr.net/npm/@playkit-js/playkit-js-kaltura-cuepoints@3.0.16/dist/playkit-kaltura-cuepoints.js"></script>-->
+    <!--    <script src = "http://localhost:8083/playkit-ui.js"></script>>-->
+    <!--    <script type="text/javascript" src="http://localhost:8084/playkit-transcript.js"></script>-->
+    <!--    <script src = "http://localhost:8085/playkit-dual-screen.js"></script>-->
+    <!--    <script src = "http://localhost:8084/playkit-kaltura-live.js"></script>-->
+    <!--    <script src = "http://localhost:8085/playkit-dual-screen.js"></script>-->
+    <!--    <script src = "http://localhost:8086/playkit-hotspots.js"></script>-->
+    <!--    <script src = "http://localhost:8087/playkit-ui.js"></script>-->
+    <!--<script src='http://localhost:8081/playkit-ui-managers.js'></script> -->
+    <!--<script src = "http://localhost:8081/playkit-visibility.js"></script> -->
+    <!-- <script type="text/javascript" src="http://localhost:8082/playkit-transcript.js"></script> -->
+    <!-- <script src="https://cdn.jsdelivr.net/npm/@playkit-js/playkit-js-kaltura-cuepoints@3.0.16/dist/playkit-kaltura-cuepoints.js"></script> -->
+    <!-- <script src="http://localhost:8081/playkit-info.js"></script> -->
   </head>
   <body>
-    <div id="player-placeholder"></div>
-    <script type="text/javascript">
-      const config = {
-        log: {
-          level: 'DEBUG'
+  <div id="player-placeholder"></div>
+  <div id="player-placeholder-1"></div>
+  <script type="text/javascript">
+    const config = {
+      captions: {
+        lang: "en"
+      },
+      log: {
+        level: 'DEBUG'
+      },
+      targetId: 'player-placeholder',
+      provider: {
+        partnerId: 6146202,
+        env: {
         },
-        targetId: 'player-placeholder',
-        provider: {
-          partnerId: 242,
-          env: {
-            cdnUrl: 'https://cfvod.nvq2.ovp.kaltura.com',
-            serviceUrl: 'https://api.nvq2.ovp.kaltura.com/api_v3'
-          }
-        },
-      };
+      },
+      plugins: {
+        vr: {},
+        uiManagers: {},
+        // ivq: {},
+        // timeline: {},
+        // kalturaCuepoints: {},
+        //   'playkit-js-transcript': {},
+        //  dualscreen: {},
+        //   qna: {},
+        //   "playkit-js-hotspots": {},
+      }
+    };
 
-      const player = KalturaPlayer.setup(config);
-      player.loadMedia({ entryId: '0_bfj8lg0f' });
+    const player = KalturaPlayer.setup(config);
+    player.loadMedia({ entryId: '1_7rotjo8c' });
     </script>
   </body>
 </html>
diff --git a/src/kaltura-player.ts b/src/kaltura-player.ts
index 191c93e41..803ed6a26 100644
--- a/src/kaltura-player.ts
+++ b/src/kaltura-player.ts
@@ -142,6 +142,7 @@ export class KalturaPlayer extends FakeEventTarget {
       __VERSION__,
       vrTag
     );
+
     this._playlistManager = new PlaylistManager(this, options);
     Object.values(CoreEventType).forEach((coreEvent) => this._eventManager.listen(this._localPlayer, coreEvent, (e) => this.dispatchEvent(e)));
 
@@ -496,6 +497,11 @@ export class KalturaPlayer extends FakeEventTarget {
     return this._localPlayer.isInPictureInPicture();
   }
 
+  public foo(playerConfig): void {
+    const sources = Utils.Object.mergeDeep({}, playerConfig.sources, this._localPlayer.sources);
+    this._provider.setSourcesObject(sources, )
+  }
+
   public isPictureInPictureSupported(): boolean {
     return this._localPlayer.isPictureInPictureSupported();
   }

From ab8941dd9bffe0b03447ef0e50c239b701a115ab Mon Sep 17 00:00:00 2001
From: yambenari24 <yam.benari@kaltura.com>
Date: Sun, 2 Feb 2025 14:32:39 +0200
Subject: [PATCH 3/4] fix(SUP-46065): Player shows VR button based on "360" (or
 variations) tag

---
 src/kaltura-player.ts | 18 +++---------------
 1 file changed, 3 insertions(+), 15 deletions(-)

diff --git a/src/kaltura-player.ts b/src/kaltura-player.ts
index 803ed6a26..db160ae38 100644
--- a/src/kaltura-player.ts
+++ b/src/kaltura-player.ts
@@ -132,15 +132,12 @@ export class KalturaPlayer extends FakeEventTarget {
     );
     this._cuepointManager = new CuePointManager(this);
 
-    const vrTag = this.getVrTag(options.plugins);
-
     this._provider = new Provider(
       Utils.Object.mergeDeep(options.provider, {
         logger: { getLogger, LogLevel },
         referrer: getOriginalRequestReferrer()
       }),
-      __VERSION__,
-      vrTag
+      __VERSION__
     );
 
     this._playlistManager = new PlaylistManager(this, options);
@@ -497,9 +494,8 @@ export class KalturaPlayer extends FakeEventTarget {
     return this._localPlayer.isInPictureInPicture();
   }
 
-  public foo(playerConfig): void {
-    const sources = Utils.Object.mergeDeep({}, playerConfig.sources, this._localPlayer.sources);
-    this._provider.setSourcesObject(sources, )
+  public _updatePlayerVrPluginIsOn(vrTag: string): void {
+    this._provider._updatePlayerVrPluginIsOn(vrTag);
   }
 
   public isPictureInPictureSupported(): boolean {
@@ -545,14 +541,6 @@ export class KalturaPlayer extends FakeEventTarget {
     return this._localPlayer.isVr();
   }
 
-  public getVrTag(config: PluginsConfig): string | null {
-    const vrPlugin = config['vr'];
-    if (vrPlugin) {
-      return vrPlugin.tag ?? '360';
-    }
-    return null;
-  }
-
   public toggleVrStereoMode(): void {
     const vrPlugin: BasePlugin | undefined = this._pluginManager.get('vr');
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment

From 957facbc1011b29a7c4ac16ef76181c5807acaea Mon Sep 17 00:00:00 2001
From: yambenari24 <yam.benari@kaltura.com>
Date: Sun, 2 Feb 2025 14:54:29 +0200
Subject: [PATCH 4/4] Remove kaltura-ovp changes file from PR

---
 demo/player-ovp.html | 68 --------------------------------------------
 1 file changed, 68 deletions(-)
 delete mode 100644 demo/player-ovp.html

diff --git a/demo/player-ovp.html b/demo/player-ovp.html
deleted file mode 100644
index 54942a201..000000000
--- a/demo/player-ovp.html
+++ /dev/null
@@ -1,68 +0,0 @@
-<!doctype html>
-<html lang="en">
-  <head>
-    <meta charset="utf-8" />
-    <meta name="viewport" content="width=device-width, initial-scale=1" />
-    <meta name="theme-color" content="#000000" />
-    <meta name="description" content="Kaltura Player Demo Page" />
-    <title>Player</title>
-    <link rel="stylesheet" type="text/css" href="./style.css" />
-    <script src="./kaltura-ovp-player.js" type="text/javascript"></script>
-    <script src = "http://localhost:8081/playkit.js"></script>
-    <script src = "http://localhost:8082/playkit-ui-managers.js"></script>
-    <script src = "http://localhost:8083/playkit-ovp-provider.js"></script>
-    <script src = "http://localhost:8084/playkit-vr.js"></script>
-<!--    <script src = "http://localhost:8083/playkit-timeline.js"></script>-->
-<!--    <script src = "http://localhost:8087/playkit-ivq.js"></script>-->
-<!--    <script src = "http://localhost:8088/playkit-ui.js"></script>-->
-<!--    <script src = "http://localhost:8083/playkit-vr.js"></script>-->
-<!--    <script src = "http://localhost:8084/playkit-ovp-provider.js"></script>-->
-    <!--    <script src="https://cdn.jsdelivr.net/npm/@playkit-js/playkit-js-kaltura-cuepoints@3.0.16/dist/playkit-kaltura-cuepoints.js"></script>-->
-    <!--    <script src = "http://localhost:8083/playkit-ui.js"></script>>-->
-    <!--    <script type="text/javascript" src="http://localhost:8084/playkit-transcript.js"></script>-->
-    <!--    <script src = "http://localhost:8085/playkit-dual-screen.js"></script>-->
-    <!--    <script src = "http://localhost:8084/playkit-kaltura-live.js"></script>-->
-    <!--    <script src = "http://localhost:8085/playkit-dual-screen.js"></script>-->
-    <!--    <script src = "http://localhost:8086/playkit-hotspots.js"></script>-->
-    <!--    <script src = "http://localhost:8087/playkit-ui.js"></script>-->
-    <!--<script src='http://localhost:8081/playkit-ui-managers.js'></script> -->
-    <!--<script src = "http://localhost:8081/playkit-visibility.js"></script> -->
-    <!-- <script type="text/javascript" src="http://localhost:8082/playkit-transcript.js"></script> -->
-    <!-- <script src="https://cdn.jsdelivr.net/npm/@playkit-js/playkit-js-kaltura-cuepoints@3.0.16/dist/playkit-kaltura-cuepoints.js"></script> -->
-    <!-- <script src="http://localhost:8081/playkit-info.js"></script> -->
-  </head>
-  <body>
-  <div id="player-placeholder"></div>
-  <div id="player-placeholder-1"></div>
-  <script type="text/javascript">
-    const config = {
-      captions: {
-        lang: "en"
-      },
-      log: {
-        level: 'DEBUG'
-      },
-      targetId: 'player-placeholder',
-      provider: {
-        partnerId: 6146202,
-        env: {
-        },
-      },
-      plugins: {
-        vr: {},
-        uiManagers: {},
-        // ivq: {},
-        // timeline: {},
-        // kalturaCuepoints: {},
-        //   'playkit-js-transcript': {},
-        //  dualscreen: {},
-        //   qna: {},
-        //   "playkit-js-hotspots": {},
-      }
-    };
-
-    const player = KalturaPlayer.setup(config);
-    player.loadMedia({ entryId: '1_7rotjo8c' });
-    </script>
-  </body>
-</html>