From 91de7ff11c86c3c4b2ab0a9cf25fb6ca6ef742f8 Mon Sep 17 00:00:00 2001
From: Mark McDowall <mark@mcdowall.ca>
Date: Thu, 2 Apr 2020 17:24:18 -0700
Subject: [PATCH] Fixed: Don't try to render quality when it's null

Fixes #3649
---
 frontend/src/Episode/EpisodeQuality.js       |  4 ++++
 frontend/src/Store/Actions/releaseActions.js | 17 ++++++++---------
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/frontend/src/Episode/EpisodeQuality.js b/frontend/src/Episode/EpisodeQuality.js
index d7ca6b46c..d75b1287c 100644
--- a/frontend/src/Episode/EpisodeQuality.js
+++ b/frontend/src/Episode/EpisodeQuality.js
@@ -35,6 +35,10 @@ function EpisodeQuality(props) {
     isCutoffNotMet
   } = props;
 
+  if (!quality) {
+    return null;
+  }
+
   return (
     <Label
       className={className}
diff --git a/frontend/src/Store/Actions/releaseActions.js b/frontend/src/Store/Actions/releaseActions.js
index 7e4688c92..ef9b46532 100644
--- a/frontend/src/Store/Actions/releaseActions.js
+++ b/frontend/src/Store/Actions/releaseActions.js
@@ -261,17 +261,16 @@ export const reducers = createHandleActions({
     const guid = payload.guid;
     const newState = Object.assign({}, state);
     const items = newState.items;
-
-    // Return early if there aren't any items (the user closed the modal)
-    if (!items.length) {
-      return;
-    }
-
     const index = items.findIndex((item) => item.guid === guid);
-    const item = Object.assign({}, items[index], payload);
 
-    newState.items = [...items];
-    newState.items.splice(index, 1, item);
+    // Don't try to update if there isnt a matching item (the user closed the modal)
+
+    if (index >= 0) {
+      const item = Object.assign({}, items[index], payload);
+
+      newState.items = [...items];
+      newState.items.splice(index, 1, item);
+    }
 
     return newState;
   },