fix(FEC-12562): Ignore ks configuration on kava plugin incase of ottA…

…nalytics exists on player (#574)

* if kava && ottAnalytics  exists remove the ks from kava plugin

.flowconfig

@@ -15,4 +15,6 @@ module.name_mapper='player-defaults' -> '<PROJECT_ROOT>/src/ovp/player-defaults'
 module.name_mapper='player-defaults' -> '<PROJECT_ROOT>/src/ott/player-defaults'
 module.name_mapper='poster' -> '<PROJECT_ROOT>/src/ovp/poster'
 module.name_mapper='poster' -> '<PROJECT_ROOT>/src/ott/poster'
+module.name_mapper='plugins-config-store' -> '<PROJECT_ROOT>/src/ovp/plugins/plugins-config-store'
+module.name_mapper='plugins-config-store' -> '<PROJECT_ROOT>/src/ott/plugins/plugins-config-store'
 esproposal.optional_chaining=enable

src/common/plugins/plugins-config-store.js

@@ -93,15 +93,16 @@ const removeUndefineds = (obj: Object = {}): Object =>
     return product;
   }, {});
 
-class PluginConfigStore {
+class BasePluginConfigStore {
+  _basePluginsConfig: dataStoreType;
   _config: dataStoreType;
 
   /**
    * constructor
    * @constructor
    */
   constructor() {
-    this._config = Utils.Object.copyDeep(defaultConfig);
+    this._basePluginsConfig = Utils.Object.copyDeep(defaultConfig);
   }
 
   /**
@@ -132,4 +133,4 @@ class PluginConfigStore {
   }
 }
 
-export {PluginConfigStore, templateRegex};
+export {BasePluginConfigStore, templateRegex};

src/common/plugins/plugins-config.js

@@ -1,5 +1,5 @@
 //@flow
-import {PluginConfigStore, templateRegex} from './plugins-config-store.js';
+import {PluginConfigStore, templateRegex} from 'plugins-config-store';
 import evaluate from '../utils/evaluate';
 import {getReferrer} from '../utils/kaltura-params';
 import {Utils} from '@playkit-js/playkit-js';

src/index.html

@@ -27,7 +27,6 @@
   } catch (e) {
     console.error(e.message);
   }
-
 </script>
 </body>
 </html>

src/ott/plugins/plugins-config-store.js

@@ -0,0 +1,21 @@
+//@flow
+
+import {BasePluginConfigStore, templateRegex} from '../../common/plugins/plugins-config-store';
+
+import {Utils} from '@playkit-js/playkit-js';
+
+class PluginConfigStore extends BasePluginConfigStore {
+  _ottPluginsConfig: {[pluginName: string]: Object} = {
+    kava: {
+      ks: ''
+    }
+  };
+
+  constructor() {
+    super();
+
+    this._config = Utils.Object.mergeDeep({}, this._basePluginsConfig, this._ottPluginsConfig);
+  }
+}
+
+export {PluginConfigStore, templateRegex};

src/ovp/plugins/plugins-config-store.js

@@ -0,0 +1,19 @@
+//@flow
+
+import {BasePluginConfigStore, templateRegex} from '../../common/plugins/plugins-config-store';
+
+import {Utils} from '@playkit-js/playkit-js';
+
+class PluginConfigStore extends BasePluginConfigStore {
+  _ovpPluginsConfig: {[pluginName: string]: Object} = {
+    ottAnalytics: {}
+  };
+
+  constructor() {
+    super();
+
+    this._config = Utils.Object.mergeDeep({}, this._basePluginsConfig, this._ovpPluginsConfig);
+  }
+}
+
+export {PluginConfigStore, templateRegex};

ts-typed/base-middleware.d.ts

@@ -1,5 +1,5 @@
 declare namespace KalturaPlayerTypes {
-    export interface BaseMiddleware {
-        callNext(next: Function): void;
-    }
+  export interface BaseMiddleware {
+    callNext(next: Function): void;
+  }
 }

ts-typed/engine-decorator.d.ts

@@ -1,12 +1,12 @@
 interface FakeEvent {
-    type: string;
+  type: string;
 }
 declare namespace KalturaPlayerTypes {
-    export interface IEngineDecorator {
-        dispatchEvent(event: FakeEvent): boolean;
-        active: boolean;
-    }
-    export interface IEngineDecoratorProvider {
-        getEngineDecorator(engine: any, dispatchEventHandler: Function): IEngineDecorator;
-    }
+  export interface IEngineDecorator {
+    dispatchEvent(event: FakeEvent): boolean;
+    active: boolean;
+  }
+  export interface IEngineDecoratorProvider {
+    getEngineDecorator(engine: any, dispatchEventHandler: Function): IEngineDecorator;
+  }
 }

types.d.ts

@@ -4,7 +4,7 @@ import * as preactLib from 'preact';
 declare module 'kaltura-player-js' {
   export const ui: typeof PlaykitUI;
   export class FakeEvent implements CustomEvent {
-    readonly payload: any
+    readonly payload: any;
     readonly AT_TARGET: number;
     readonly BUBBLING_PHASE: number;
     readonly CAPTURING_PHASE: number;
@@ -76,10 +76,10 @@ export interface Logger {
 }
 
 export interface EventTypes {
-  Core: {[event: string]: string},
-  UI: {[event: string]: string},
-  Cast: {[event: string]: string},
-  Playlist: {[event: string]: string}
+  Core: {[event: string]: string};
+  UI: {[event: string]: string};
+  Cast: {[event: string]: string};
+  Playlist: {[event: string]: string};
 }
 
 declare module PlaykitUI {

webpack.config.js

@@ -102,7 +102,8 @@ function createConfig(env, argv, target) {
         'playkit-js-providers': path.resolve(`./node_modules/playkit-js-providers/dist/playkit-${playerType}-provider`),
         'player-defaults': path.resolve(`./src/${playerType}/player-defaults`),
         'hls.js': path.resolve('./node_modules/hls.js/dist/hls.min.js'),
-        poster: path.resolve(`./src/${playerType}/poster`)
+        poster: path.resolve(`./src/${playerType}/poster`),
+        'plugins-config-store': path.resolve(`./src/${playerType}/plugins/plugins-config-store`)
       },
       modules: [path.resolve(__dirname, 'src'), 'node_modules']
     }