From 0042fadcb0aa909bc28e7b2551c9464bd1383c04 Mon Sep 17 00:00:00 2001
From: Qstick <qstick@gmail.com>
Date: Wed, 8 Apr 2020 21:31:47 -0400
Subject: [PATCH] Fixed: Don't try to render quality when it's null

Co-Authored-By: Mark McDowall <markus101@users.noreply.github.com>
---
 frontend/src/Movie/MovieQuality.js           |  4 ++++
 frontend/src/Store/Actions/releaseActions.js | 16 +++++++---------
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/frontend/src/Movie/MovieQuality.js b/frontend/src/Movie/MovieQuality.js
index f54dbea5b..818bbfc5a 100644
--- a/frontend/src/Movie/MovieQuality.js
+++ b/frontend/src/Movie/MovieQuality.js
@@ -45,6 +45,10 @@ function MovieQuality(props) {
     kind = kinds.INVERSE;
   }
 
+  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 24e869e55..537ef278e 100644
--- a/frontend/src/Store/Actions/releaseActions.js
+++ b/frontend/src/Store/Actions/releaseActions.js
@@ -233,17 +233,15 @@ 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;
   },