fix(FEC-10583): IMA image ad is not functional

Description of the Changes When user clicks through a linear ad, only pause it if it's a video ad Fixes FEC-10583
kaltura · Nov 3, 2021 · 28fb6c3 · 28fb6c3
1 parent 22411b0
commit 28fb6c3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/src/ima-state-machine.js b/src/ima-state-machine.js
@@ -218,11 +218,13 @@ function onAdStarted(options: Object, adEvent: any): void {
 function onAdClicked(options: Object, adEvent: any): void {
   this.logger.debug(adEvent.type.toUpperCase());
   if (this._currentAd.isLinear()) {
-    this._maybeIgnoreClickOnAd();
-    if (this._stateMachine.is(State.PLAYING)) {
-      this._adsManager.pause();
+    if (this._isVideoAd()) {
+      this._maybeIgnoreClickOnAd();
+      if (this._stateMachine.is(State.PLAYING)) {
+        this._adsManager.pause();
+      }
+      this._setToggleAdsCover(true);
     }
-    this._setToggleAdsCover(true);
   } else {
     if (!this.player.paused) {
       this.player.pause();

diff --git a/src/ima.js b/src/ima.js
@@ -1474,6 +1474,10 @@ class Ima extends BasePlugin implements IMiddlewareProvider, IAdsControllerProvi
   _playAdByConfig(): boolean {
     return !!(this.config.adTagUrl || this.config.adsResponse);
   }
+
+  _isVideoAd(): boolean {
+    return this._currentAd && this._currentAd.getContentType().startsWith('video');
+  }
 }
 
 export {Ima};