From 11767828cf768e37e3d924e5109614c49b9f068b Mon Sep 17 00:00:00 2001
From: Matchu <matchu1993@gmail.com>
Date: Fri, 8 Jan 2021 03:37:56 -0800
Subject: [PATCH] Disable PosePicker while saved outfit is loading

Oops, if you try to show PosePicker before we have a species/color ready, it sends a bad GraphQL request. No visible user impact, just an unnecessary network call and an error in the console! This happens when you're loading an outfit by ID.

Here, we hide PosePicker if there's no species/color ready yet. This stops the extra request from firing!
---
 src/app/WardrobePage/OutfitControls.js | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/src/app/WardrobePage/OutfitControls.js b/src/app/WardrobePage/OutfitControls.js
index a64fdf7..aa4ba74 100644
--- a/src/app/WardrobePage/OutfitControls.js
+++ b/src/app/WardrobePage/OutfitControls.js
@@ -189,17 +189,19 @@ function OutfitControls({
                 />
               </DarkMode>
             </Box>
-            <Flex flex="1 1 0" align="center" pl="4">
-              <PosePicker
-                speciesId={outfitState.speciesId}
-                colorId={outfitState.colorId}
-                pose={outfitState.pose}
-                appearanceId={outfitState.appearanceId}
-                dispatchToOutfit={dispatchToOutfit}
-                onLockFocus={onLockFocus}
-                onUnlockFocus={onUnlockFocus}
-              />
-            </Flex>
+            {outfitState.speciesId && outfitState.colorId && (
+              <Flex flex="1 1 0" align="center" pl="4">
+                <PosePicker
+                  speciesId={outfitState.speciesId}
+                  colorId={outfitState.colorId}
+                  pose={outfitState.pose}
+                  appearanceId={outfitState.appearanceId}
+                  dispatchToOutfit={dispatchToOutfit}
+                  onLockFocus={onLockFocus}
+                  onUnlockFocus={onUnlockFocus}
+                />
+              </Flex>
+            )}
           </Flex>
         </Box>
       )}