From e9c9a5c0b7afc3abd2587775ee1726b9e6e87c3e Mon Sep 17 00:00:00 2001
From: JohnMark Sill <johnmarksilly@gmail.com>
Date: Thu, 13 Jan 2022 00:42:58 -0600
Subject: [PATCH] fix: video not replaced if a new event was queued

---
 web/src/components/HistoryViewer.jsx | 34 ++++++++++++++++------------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/web/src/components/HistoryViewer.jsx b/web/src/components/HistoryViewer.jsx
index 558b5bff6..69b61a4e1 100644
--- a/web/src/components/HistoryViewer.jsx
+++ b/web/src/components/HistoryViewer.jsx
@@ -1,5 +1,5 @@
 import { Fragment, h } from 'preact';
-import { useEffect, useRef, useState } from 'preact/hooks';
+import { useCallback, useEffect, useRef, useState } from 'preact/hooks';
 import { useApiHost, useEvents } from '../api';
 import { useSearchString } from '../hooks/useSearchString';
 import { Next } from '../icons/Next';
@@ -60,6 +60,24 @@ export default function HistoryViewer({ camera }) {
     setCurrentEventIndex(currentEvent.index + 1);
   };
 
+  const RenderVideo = useCallback(() => {
+    return (
+      <video
+        ref={videoRef}
+        onTimeUpdate={handleTimeUpdate}
+        onPause={handlePaused}
+        poster={`${apiHost}/api/events/${currentEvent.id}/snapshot.jpg`}
+        preload='none'
+        playsInline
+        controls
+      >
+        <source
+          src={`${apiHost}/api/${camera}/start/${currentEvent.start_time}/end/${currentEvent.end_time}/clip.mp4`}
+        />
+      </video>
+    );
+  }, [currentEvent, apiHost, camera]);
+
   return (
     <Fragment>
       {currentEvent && (
@@ -71,19 +89,7 @@ export default function HistoryViewer({ camera }) {
               objectLabel={currentEvent.label}
               className='mb-2'
             />
-            <video
-              ref={videoRef}
-              onTimeUpdate={handleTimeUpdate}
-              onPause={handlePaused}
-              poster={`${apiHost}/api/events/${currentEvent.id}/snapshot.jpg`}
-              preload='none'
-              playsInline
-              controls
-            >
-              <source
-                src={`${apiHost}/api/${camera}/start/${currentEvent.start_time}/end/${currentEvent.end_time}/clip.mp4`}
-              />
-            </video>
+            <RenderVideo />
           </div>
         </Fragment>
       )}