-
Notifications
You must be signed in to change notification settings - Fork 174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PLAT-22530: support captions in Simulive #10302
Conversation
get the entry's captionAssets and put it in the serveFlavor's json for the Simulive flow
use languageCodeManager to get the language code
…d include the captionAssets inside
change getClipData to receive asset instead of flavorAsset , and change hasAudio to it's opposite - addSilence (put true in case of asset different than flavorAsset)
…default language for flavorAsset without language specified (will add the captions to the json also in case of undefined language)
remove mergeAssets function as not needed (peers don't return null) avoid insert caption to json in case of 'und' language
@@ -1427,15 +1427,15 @@ public static function getPlaylistDataFromEntries($entries, $flavorParamsIds, $c | |||
return array($entryIds, $durations, $mediaEntry, $captionFiles); | |||
} | |||
|
|||
protected static function getCaptionFilesForEntryIds($entryIds, $captionLanguages) | |||
public static function getCaptionFilesForEntryIds($entryIds, $captionLanguages, $allLanguages = false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to add another flag to the function, we can set that $captionLanguages is default to null, and if it is not set that we grab all langueges
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm removing this change, it isn't needed anymore (was needed before and forgot to remove it)
alpha/apps/kaltura/modules/extwidget/actions/serveFlavorAction.class.php
Show resolved
Hide resolved
remove unnecessary array_merge in serveFlavorAction remove change of getCaptionFilesForEntryIds (as not used anymore)
reviewed |
get the entry's captionAssets and put it in the serveFlavor's json for the Simulive flow