From 46b0d1ceac3027dcd0107973adf4284be23523b8 Mon Sep 17 00:00:00 2001
From: Riley Testut <riley@rileytestut.com>
Date: Wed, 21 Sep 2022 17:31:23 -0500
Subject: [PATCH] Always displays PatreonViewController loading indicator when
 fetching patrons

Previously, we only showed the loading indicator if user had not yet cached any Friend Zone patrons.
---
 AltStore/Settings/PatreonViewController.swift | 31 +++++++++++++------
 1 file changed, 21 insertions(+), 10 deletions(-)

diff --git a/AltStore/Settings/PatreonViewController.swift b/AltStore/Settings/PatreonViewController.swift
index b0765fd8..6b1be872 100644
--- a/AltStore/Settings/PatreonViewController.swift
+++ b/AltStore/Settings/PatreonViewController.swift
@@ -238,7 +238,8 @@ private extension PatreonViewController
     
     @objc func didUpdatePatrons(_ notification: Notification)
     {
-        DispatchQueue.main.async {
+        DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) {
+            // Wait short delay before reloading or else footer won't properly update if it's already visible 🤷‍♂️
             self.collectionView.reloadData()
         }
     }
@@ -270,17 +271,27 @@ extension PatreonViewController
                 footerView.button.isHidden = false
                 //footerView.button.addTarget(self, action: #selector(PatreonViewController.fetchPatrons), for: .primaryActionTriggered)
                 
-                if self.patronsDataSource.itemCount > 0
+                switch AppManager.shared.updatePatronsResult
                 {
-                    footerView.button.isHidden = true
-                }
-                else
-                {
-                    switch AppManager.shared.updatePatronsResult
+                case .none: footerView.button.isIndicatingActivity = true
+                case .success?: footerView.button.isHidden = true
+                case .failure?:
+                    #if DEBUG
+                    let debug = true
+                    #else
+                    let debug = false
+                    #endif
+                    
+                    if self.patronsDataSource.itemCount == 0 || debug
                     {
-                    case .none: footerView.button.isIndicatingActivity = true
-                    case .success?: footerView.button.isHidden = true
-                    case .failure?: footerView.button.setTitle(NSLocalizedString("Error Loading Patrons", comment: ""), for: .normal)
+                        // Only show error message if there aren't any cached Patrons (or if this is a debug build).
+                        
+                        footerView.button.isHidden = false
+                        footerView.button.setTitle(NSLocalizedString("Error Loading Patrons", comment: ""), for: .normal)
+                    }
+                    else
+                    {
+                        footerView.button.isHidden = true
                     }
                 }