Skip to content

Commit 6f5c8c8

Browse files
authored
Merge pull request #6172 from thornbill/fix-playback-subscriber-bind
2 parents 1cfaf1e + 49433cd commit 6f5c8c8

File tree

1 file changed

+24
-24
lines changed

1 file changed

+24
-24
lines changed

src/apps/stable/features/playback/utils/playbackSubscriber.ts

+24-24
Original file line numberDiff line numberDiff line change
@@ -39,35 +39,35 @@ export interface PlaybackSubscriber {
3939
}
4040

4141
export abstract class PlaybackSubscriber {
42-
private player: Plugin | undefined;
42+
protected player: Plugin | undefined;
4343

4444
private readonly playbackManagerEvents = {
45-
[PlaybackManagerEvent.PlaybackCancelled]: this.onPlaybackCancelled,
46-
[PlaybackManagerEvent.PlaybackError]: this.onPlaybackError,
47-
[PlaybackManagerEvent.PlaybackStart]: this.onPlaybackStart,
48-
[PlaybackManagerEvent.PlaybackStop]: this.onPlaybackStop,
49-
[PlaybackManagerEvent.PlayerChange]: this.onPlayerChange,
50-
[PlaybackManagerEvent.ReportPlayback]: this.onReportPlayback
45+
[PlaybackManagerEvent.PlaybackCancelled]: this.onPlaybackCancelled?.bind(this),
46+
[PlaybackManagerEvent.PlaybackError]: this.onPlaybackError?.bind(this),
47+
[PlaybackManagerEvent.PlaybackStart]: this.onPlaybackStart?.bind(this),
48+
[PlaybackManagerEvent.PlaybackStop]: this.onPlaybackStop?.bind(this),
49+
[PlaybackManagerEvent.PlayerChange]: this.onPlayerChange?.bind(this),
50+
[PlaybackManagerEvent.ReportPlayback]: this.onReportPlayback?.bind(this)
5151
};
5252

5353
private readonly playerEvents = {
54-
[PlayerEvent.Error]: this.onPlayerError,
55-
[PlayerEvent.FullscreenChange]: this.onPlayerFullscreenChange,
56-
[PlayerEvent.ItemStarted]: this.onPlayerItemStarted,
57-
[PlayerEvent.ItemStopped]: this.onPlayerItemStopped,
58-
[PlayerEvent.MediaStreamsChange]: this.onPlayerMediaStreamsChange,
59-
[PlayerEvent.Pause]: this.onPlayerPause,
60-
[PlayerEvent.PlaybackStart]: this.onPlayerPlaybackStart,
61-
[PlayerEvent.PlaybackStop]: this.onPlayerPlaybackStop,
62-
[PlayerEvent.PlaylistItemAdd]: this.onPlayerPlaylistItemAdd,
63-
[PlayerEvent.PlaylistItemMove]: this.onPlayerPlaylistItemMove,
64-
[PlayerEvent.PlaylistItemRemove]: this.onPlayerPlaylistItemRemove,
65-
[PlayerEvent.RepeatModeChange]: this.onPlayerRepeatModeChange,
66-
[PlayerEvent.ShuffleModeChange]: this.onPlayerShuffleModeChange,
67-
[PlayerEvent.Stopped]: this.onPlayerStopped,
68-
[PlayerEvent.TimeUpdate]: this.onPlayerTimeUpdate,
69-
[PlayerEvent.Unpause]: this.onPlayerUnpause,
70-
[PlayerEvent.VolumeChange]: this.onPlayerVolumeChange
54+
[PlayerEvent.Error]: this.onPlayerError?.bind(this),
55+
[PlayerEvent.FullscreenChange]: this.onPlayerFullscreenChange?.bind(this),
56+
[PlayerEvent.ItemStarted]: this.onPlayerItemStarted?.bind(this),
57+
[PlayerEvent.ItemStopped]: this.onPlayerItemStopped?.bind(this),
58+
[PlayerEvent.MediaStreamsChange]: this.onPlayerMediaStreamsChange?.bind(this),
59+
[PlayerEvent.Pause]: this.onPlayerPause?.bind(this),
60+
[PlayerEvent.PlaybackStart]: this.onPlayerPlaybackStart?.bind(this),
61+
[PlayerEvent.PlaybackStop]: this.onPlayerPlaybackStop?.bind(this),
62+
[PlayerEvent.PlaylistItemAdd]: this.onPlayerPlaylistItemAdd?.bind(this),
63+
[PlayerEvent.PlaylistItemMove]: this.onPlayerPlaylistItemMove?.bind(this),
64+
[PlayerEvent.PlaylistItemRemove]: this.onPlayerPlaylistItemRemove?.bind(this),
65+
[PlayerEvent.RepeatModeChange]: this.onPlayerRepeatModeChange?.bind(this),
66+
[PlayerEvent.ShuffleModeChange]: this.onPlayerShuffleModeChange?.bind(this),
67+
[PlayerEvent.Stopped]: this.onPlayerStopped?.bind(this),
68+
[PlayerEvent.TimeUpdate]: this.onPlayerTimeUpdate?.bind(this),
69+
[PlayerEvent.Unpause]: this.onPlayerUnpause?.bind(this),
70+
[PlayerEvent.VolumeChange]: this.onPlayerVolumeChange?.bind(this)
7171
};
7272

7373
constructor(

0 commit comments

Comments
 (0)