From d597423a7ebf43c0cd34efabb3efc29cb55e298d Mon Sep 17 00:00:00 2001
From: albin-karlsson <55614148+albin-karlsson@users.noreply.github.com>
Date: Fri, 26 Apr 2024 21:11:51 +0200
Subject: [PATCH] Add max index limit to messages to avoid replaying
 interjection replies

---
 client/src/components/Output.jsx | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/client/src/components/Output.jsx b/client/src/components/Output.jsx
index 0d73d99..717fb1a 100644
--- a/client/src/components/Output.jsx
+++ b/client/src/components/Output.jsx
@@ -18,6 +18,7 @@ function Output({
   const [currentTextMessage, setCurrentTextMessage] = useState(null);
   const [currentAudioMessage, setCurrentAudioMessage] = useState(null);
   const [stopAudio, setStopAudio] = useState(false);
+  const [totalInterjections, setTotalInterjections] = useState(0);
 
   useEffect(() => {
     if (interjectionReplyRecieved) {
@@ -29,6 +30,8 @@ function Output({
       setCurrentTextMessage(textMessages[textMessages.length - 1]);
       setCurrentAudioMessage(audioMessages[audioMessages.length - 1]);
 
+      setTotalInterjections((prev) => prev + 1);
+
       onResetInterjectionReply();
     }
   }, [interjectionReplyRecieved]);
@@ -90,17 +93,24 @@ function Output({
       // Set isReady to true in the parent component to render the controls (for skipping forward et.c.)
       onIsReady();
 
-      setCurrentTextMessage((prev) => textMessage);
-      setCurrentAudioMessage((prev) => audioMessage);
+      setCurrentTextMessage(() => textMessage);
+      setCurrentAudioMessage(() => audioMessage);
     }
   }
 
   function proceedToNextMessage() {
+    // Reset the current message contents
     setCurrentTextMessage(() => null);
     setCurrentAudioMessage(() => null);
 
-    console.log("Current index: ", currentMessageIndex);
-    setCurrentMessageIndex((prev) => prev + 1);
+    // Update the index to the next message, ensuring it doesn't exceed the available range
+    // considering interjections that may reduce the number of messages to show
+    setCurrentMessageIndex((prev) => {
+      // Calculate the maximum index allowed based on the length of the messages and total interjections
+      const maxIndex = textMessages.length - totalInterjections - 1;
+      // Increment the index if it's within the bounds, otherwise keep it at the maximum allowed index
+      return prev < maxIndex ? prev + 1 : maxIndex;
+    });
   }
 
   function handleOnFinishedPlaying() {