Skip to content

Commit

Permalink
fix(FEC-14093): rapt is not loading on latest player (#834)
Browse files Browse the repository at this point in the history
### Description of the Changes

**bugfix - regression**

**Issue:**
since the latest player changes, while loading rapt media, the player is
not being displayed on the page (blank space).
- as part of the latest changes in bottom-bar comp, we register a new
service in the constructor, which uses the `serviceProvider` member of
kaltura player
- rapt renders the bottom-bar component before kaltura-player finished
initializing - `serviceProvider` is `undefined`, which causes a runtime
error

**Fix:**
move `serviceProvider` initialization before initializing the ui.

#### Resolves FEC-14093
  • Loading branch information
lianbenjamin authored Aug 7, 2024
1 parent 8cc4785 commit e8c3e4e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/kaltura-player.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,13 +122,13 @@ export class KalturaPlayer extends FakeEventTarget {
this._localPlayer = loadPlayer(noSourcesOptions);
this._controllerProvider = new ControllerProvider(this._pluginManager);
this._viewabilityManager = new ViewabilityManager(this.config.viewability);
this._serviceProvider = new ServiceProvider(this);
this._uiWrapper = new UIWrapper(
this,
Utils.Object.mergeDeep(options, {
ui: { logger: { getLogger, LogLevel } }
})
);
this._serviceProvider = new ServiceProvider(this);
this._cuepointManager = new CuePointManager(this);
this._provider = new Provider(
Utils.Object.mergeDeep(options.provider, {
Expand Down

0 comments on commit e8c3e4e

Please sign in to comment.