Ability to not show loading overlay when cached

Author: anconaesselmann

Sources/LoadableView/Protocols/LoadableView/BaseLoadableViewModel.swift

@@ -22,6 +22,8 @@ public protocol BaseLoadableViewModel: ObservableObject, AnyObject {
 
     func shouldAnimate(_ oldItem: Element?, newItem: Element) async -> Bool
 
+    func showLoadingStateWhenCached() -> Bool
+
     // MARK: - Don't implement
     func onAppear()
     func onDisappear()
@@ -102,4 +104,8 @@ public extension BaseLoadableViewModel {
     func onLoadingChange(isLoading: Bool) {
 
     }
+
+    func showLoadingStateWhenCached() -> Bool {
+        return true
+    }
 }

Sources/LoadableView/Protocols/LoadableView/IDedLoadableViewModel.swift

@@ -100,11 +100,17 @@ public extension IDedLoadableViewModel {
     }
 
     func refresh(showLoading: Bool) async throws {
-        setIsLoading(showLoading)
         do {
             guard let id = self.id else {
                 throw LoadableViewError.noId
             }
+            let combinedShowLoading: Bool
+            if let cache, cache.hasValue(forKey: id) {
+                combinedShowLoading = showLoadingStateWhenCached()
+            } else {
+                combinedShowLoading = showLoading
+            }
+            setIsLoading(combinedShowLoading)
             let item = try await load(id: id)
             cache?.insert(item, forKey: id)
             guard await shouldRefresh(viewState.loaded, newItem: item) else {

Sources/LoadableView/Protocols/ViewDataCache.swift

@@ -10,4 +10,5 @@ public protocol ViewDataCache<Key, Value> {
 
     func insert(_ value: Value, forKey key: Key)
     func value(forKey key: Key) -> Value?
+    func hasValue(forKey key: Key) -> Bool
 }