From 27d7b4bb1e040ccdbc568462e271e0f0afd3b333 Mon Sep 17 00:00:00 2001 From: Yair Ansbacher Date: Sun, 16 Jun 2019 17:43:18 +0300 Subject: [PATCH] fix: bumper and ima-dai integration (#245) set bumper config ``` disableMediaPreload: true, position: [0] ``` to avoid conflicts with dai plugin --- src/common/utils/setup-helpers.js | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/src/common/utils/setup-helpers.js b/src/common/utils/setup-helpers.js index b3ffa40b6..6cbfae597 100644 --- a/src/common/utils/setup-helpers.js +++ b/src/common/utils/setup-helpers.js @@ -254,6 +254,7 @@ function getDefaultOptions(options: PartialKPOptionsObject): KPOptionsObject { configureVrDefaultOptions(defaultOptions); configureLGTVDefaultOptions(defaultOptions); configureDAIDefaultOptions(defaultOptions); + configureBumperDefaultOptions(defaultOptions); configureExternalStreamRedirect(defaultOptions); maybeSetDefaultUiComponents(defaultOptions); return defaultOptions; @@ -367,6 +368,27 @@ function configureDAIDefaultOptions(options: KPOptionsObject): void { } } +/** + * Sets default config option for bumper plugin when ima-dai enabled + * @private + * @param {KPOptionsObject} options - kaltura player options + * @returns {void} + */ +function configureBumperDefaultOptions(options: KPOptionsObject): void { + const bumperPlugin = Utils.Object.getPropertyPath(options, 'plugins.bumper'); + const daiPlugin = Utils.Object.getPropertyPath(options, 'plugins.imadai'); + if (bumperPlugin && !bumperPlugin.disable && daiPlugin && !daiPlugin.disable) { + Utils.Object.mergeDeep(options, { + plugins: { + bumper: { + position: [0], + disableMediaPreload: true + } + } + }); + } +} + /** * Transform options structure from legacy structure to new structure. * @private