From e5fe405cd1319c8f696cd0c85f711cccf8e634c3 Mon Sep 17 00:00:00 2001
From: oscnord <6131004+oscnord@users.noreply.github.com>
Date: Mon, 11 Dec 2023 18:15:37 +0100
Subject: [PATCH] fix: allow seeking after a stream has reached the end

---
 README.md                 |  8 ++++++--
 src/media-event-filter.ts | 11 ++++++++++-
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 3d937ee5..b33998ae 100644
--- a/README.md
+++ b/README.md
@@ -18,8 +18,6 @@ Does not support native HTML5 MSE controls (`<video controls>`). It works, but e
 
 Does not support re-using the same video element.
 
-Does not support restarting the video after end of stream has been reached.
-
 ## Usage
 
 `npm install @eyevinn/media-event-filter`
@@ -316,6 +314,12 @@ A timeupdate event
 
 Can not trigger before loaded.
 
+### ended
+
+The player has reached the end of a stream.
+
+To allow restarting the stream after the end of stream has been reached set `allowResumeAfterEnded` to `true`.
+
 ## Contributing
 
 Contributions to improve compatibility with or add support for different engines, tracking solutions, and browsers are welcome.
diff --git a/src/media-event-filter.ts b/src/media-event-filter.ts
index 350a6297..e3e1a1f3 100644
--- a/src/media-event-filter.ts
+++ b/src/media-event-filter.ts
@@ -27,6 +27,7 @@ type TFilteredMediaEventCallback = (event: FilteredMediaEvent) => void;
 type TMediaEventFilterOptions = {
   videoElement: HTMLVideoElement;
   callback: TFilteredMediaEventCallback;
+  allowResumeAfterEnded?: boolean;
 };
 
 type TCallback = () => void;
@@ -101,6 +102,7 @@ export type TMediaEventFilter = {
 export const getMediaEventFilter = ({
   videoElement,
   callback,
+  allowResumeAfterEnded = false,
 }: TMediaEventFilterOptions): TMediaEventFilter => {
   let ratechangeBufferTimeout: number | null = null;
 
@@ -114,7 +116,8 @@ export const getMediaEventFilter = ({
     ...initialState,
   };
 
-  const isNotReady = (): boolean => state.loading || state.ended;
+  const isNotReady = (): boolean =>
+    allowResumeAfterEnded ? state.loading : state.loading || state.ended;
 
   const onCanPlayThrough = (): void => {
     if (!state.loading) {
@@ -153,6 +156,12 @@ export const getMediaEventFilter = ({
     // playback should be ready before reacting to "seeking" event (e.g. shaka jumps)
     if (isNotReady()) return;
 
+    // If stream has reached the end and an onSeeking has been triggered we want to set the ended state to false
+    // This can only happen if allowResumeAfterEnded is set to true (otherwise isNotReady is true)
+    if (state.ended) {
+      state.ended = false;
+    }
+
     // end ongoing buffering, this enables trackers to report
     // the buffer duration before starting the seek.
     if (state.buffering) callback(FilteredMediaEvent.BUFFERED);