@npo/player 1.27.6 → 1.27.7

package/lib/npoplayer.js

@@ -114,12 +114,12 @@ export default class NpoPlayer {
                     customData5: this.version
                 }
             };
+            this.npoPlayerServices.handleStreamOptions(this.playerContext);
             void this.playerContext.player.createUIManager(this.playerContext, this.variant);
             await this.playerContext.player?.load(this.sourceConfig);
             this.npoPlayerServices.startPlayerTracker({
                 playerContext: this.playerContext,
-                source: source,
-                duration: undefined
+                source: source
             });
         }
         else if (sourceIsJWTToken) {
@@ -158,6 +158,7 @@ export default class NpoPlayer {
                 return;
             }
             this.sourceConfig = await playerAction.processSourceConfig(this.npoPlayerServices, source, options.sourceConfig ?? {}, this.streamObject, drmType && drmType.length > 0 ? profile.drm : undefined, this.streamOptions, this.version, this.npoTag?.npoTagInstance);
+            this.npoPlayerServices.handleStreamOptions(this.playerContext);
             await this.npoPlayerServices.verifyDRM(this.playerContext, payload);
             const streamDuration = _streamObject.metadata.duration;
             this.npoPlayerServices.startPlayerTracker({
@@ -169,7 +170,6 @@ export default class NpoPlayer {
         else {
             await this.npoPlayerServices.handleError(this.playerContext, 500);
         }
-        this.npoPlayerServices.handleStreamOptions(this.playerContext);
         this.npoPlayerServices.setupNicamKijkwijzerIcons(this.playerContext);
         setupMediaSessionActionHandlers(this.player, this.sourceConfig, this.streamObject);
         this.hidePlayNextScreen();

package/lib/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@npo/player",
-  "version": "1.27.6",
+  "version": "1.27.7",
   "description": "NPO Player",
   "author": "Publieke Omroep <player@npo.nl>",
   "contributors": [

package/lib/services/npoPlayerAPI/npoPlayerAPI.js

@@ -172,6 +172,7 @@ export class NpoPlayerAPI {
             playerContext.npoPlayer.uiManager = uiManager;
             playerContext.npoPlayer.variant = variant;
             npoPlayerServices.addUivisiblityHandlers(playerContext);
+            npoPlayerServices.setupNicamKijkwijzerIcons(playerContext);
             npoPlayerServices.showNicamAfterUiDelay(playerContext);
         }
     }

package/lib/services/streamoptionsHandlers/streamOptionsHandler.js

@@ -70,7 +70,8 @@ export function setupAutoplay(playerContext) {
     };
     const doAutoPlay = async () => {
         player.off(NpoPlayerEvent.Ready, handleAutoPlay);
+        player.off(NpoPlayerEvent.SourceLoaded, handleAutoPlay);
         await player.play();
     };
-    player.on(NpoPlayerEvent.Ready, handleAutoPlay);
+    player.on(NpoPlayerEvent.SourceLoaded, handleAutoPlay);
 }

package/lib/services/streamoptionsHandlers/streamOptionsHandler.test.js

@@ -178,7 +178,7 @@ describe('handleStreamOptions', () => {
                 player: mockNpoPlayerAPI
             });
             setupAutoplay(playerContextMock);
-            expect(playerContextMock.player.on).toHaveBeenCalledWith(NpoPlayerEvent.Ready, expect.any(Function));
+            expect(playerContextMock.player.on).toHaveBeenCalledWith(NpoPlayerEvent.SourceLoaded, expect.any(Function));
             const handleAutoPlay = playerContextMock.player.on.mock.calls[0][1];
             await handleAutoPlay();
             expect(playerContextMock.player.play).toHaveBeenCalled();

package/lib/services/trackingHandlers/playerTrackerStart.js

@@ -17,7 +17,7 @@ export const startPlayerTracker = function ({ playerContext, source, duration })
     }) || 'unknown';
     npoPlayer.streamTracker = initStreamTracker({
         playerContext: playerContext,
-        duration: getStreamDurationInSeconds({ duration: playerDuration }),
+        duration: getStreamDurationInSeconds({ duration: playerDuration, durationIsInMs: !!duration }),
         source: analyticsPrid
     });
     bindPlayerEvents(playerContext);

package/lib/services/trackingHandlers/playerTrackerStart.test.js

@@ -22,31 +22,32 @@ jest.mock('./eventLogging', () => ({
 }));
 describe('startPlayerTracker', () => {
     let playerContextMock;
+    let playerMock;
     beforeEach(() => {
+        playerMock = { getDuration: jest.fn().mockReturnValue(200) };
         playerContextMock = createPlayerContextMock({
             npoPlayer: {
-                npoTag: undefined,
-                npoTagInitialisation: undefined,
-                npoTagInstance: undefined,
-                npoTagPageTracker: undefined
+                player: playerMock,
+                sourceConfig: {},
+                streamObject: {}
             }
         });
         jest.clearAllMocks();
     });
-    it('should initialize the stream tracker with correct parameters', () => {
+    it('should initialize the stream tracker with the provided duration', () => {
         const source = 'some-source';
         const duration = 150;
         getAnalyticsPrid.mockReturnValue('some-prid');
         getStreamDurationInSeconds.mockReturnValue(duration);
-        startPlayerTracker({
-            playerContext: playerContextMock,
-            source,
-            duration
-        });
+        startPlayerTracker({ playerContext: playerContextMock, source, duration });
         expect(getAnalyticsPrid).toHaveBeenCalledWith({
             source,
-            sourceConfig: undefined,
-            streamObject: undefined
+            sourceConfig: {},
+            streamObject: {}
+        });
+        expect(getStreamDurationInSeconds).toHaveBeenCalledWith({
+            duration,
+            durationIsInMs: true
         });
         expect(initStreamTracker).toHaveBeenCalledWith({
             playerContext: playerContextMock,
@@ -56,4 +57,20 @@ describe('startPlayerTracker', () => {
         expect(bindPlayerEvents).toHaveBeenCalledWith(playerContextMock);
         expect(logEvent).toHaveBeenCalledWith({ playerContext: playerContextMock, event: 'load' });
     });
+    it('should fallback to player.getDuration() if duration is not provided', () => {
+        const source = 'some-source';
+        const fallbackDuration = 200;
+        getAnalyticsPrid.mockReturnValue('some-prid');
+        getStreamDurationInSeconds.mockReturnValue(fallbackDuration);
+        startPlayerTracker({ playerContext: playerContextMock, source });
+        expect(getStreamDurationInSeconds).toHaveBeenCalledWith({
+            duration: fallbackDuration,
+            durationIsInMs: false
+        });
+        expect(initStreamTracker).toHaveBeenCalledWith({
+            playerContext: playerContextMock,
+            duration: fallbackDuration,
+            source: 'some-prid'
+        });
+    });
 });

package/lib/src/types/interfaces.d.ts

@@ -234,8 +234,8 @@ export type LocalStorageData = Partial<Record<LocalStorageValues, string | numbe
 export type NPOGoogleCastRemoteControlConfig = GoogleCastRemoteControlConfig;
 export interface PlayerTrackerParams {
     playerContext: PlayerContext;
-    duration: number | undefined;
-    source: string | undefined;
+    duration?: number;
+    source?: string;
 }
 export interface LogEventParams {
     playerContext: PlayerContext;

package/lib/types/interfaces.d.ts

@@ -234,8 +234,8 @@ export type LocalStorageData = Partial<Record<LocalStorageValues, string | numbe
 export type NPOGoogleCastRemoteControlConfig = GoogleCastRemoteControlConfig;
 export interface PlayerTrackerParams {
     playerContext: PlayerContext;
-    duration: number | undefined;
-    source: string | undefined;
+    duration?: number;
+    source?: string;
 }
 export interface LogEventParams {
     playerContext: PlayerContext;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@npo/player",
-  "version": "1.27.6",
+  "version": "1.27.7",
   "description": "NPO Player",
   "author": "Publieke Omroep <player@npo.nl>",
   "contributors": [