Add reload image if the URL on change.

Author: bullinnyc

README.md

@@ -47,13 +47,11 @@ CachedAsyncImage(
 **Note:** The default value is `0`, e.g. is no count limit and is no total cost limit.
 
 ```swift
-.onAppear {
-    // Set image cache limit.
-    TemporaryImageCache.shared.setCacheLimit(
-        countLimit: 1000, // 1000 items
-        totalCostLimit: 1024 * 1024 * 200 // 200 MB
-    )
-}
+// Set image cache limit.
+TemporaryImageCache.shared.setCacheLimit(
+    countLimit: 1000, // 1000 items
+    totalCostLimit: 1024 * 1024 * 200 // 200 MB
+)
 ```
 
 ## Requirements

Sources/CachedAsyncImage/Services/ImageLoader.swift

@@ -16,7 +16,6 @@ final class ImageLoader: ObservableObject {
 
     // MARK: - Private Properties
 
-    private let url: URL?
     private let networkManager: NetworkManagerProtocol
     private let imageCache = TemporaryImageCache.shared
 
@@ -29,11 +28,7 @@ final class ImageLoader: ObservableObject {
 
     // MARK: - Initializers
 
-    init(
-        url: URL?,
-        networkManager: NetworkManagerProtocol
-    ) {
-        self.url = url
+    init(networkManager: NetworkManagerProtocol) {
         self.networkManager = networkManager
     }
 
@@ -45,9 +40,11 @@ final class ImageLoader: ObservableObject {
 
     // MARK: - Public Methods
 
-    func fetchImage() {
+    func fetchImage(from url: String) {
         guard !isLoading else { return }
 
+        let url = URL(string: url)
+        
         if let url = url, let cachedImage = imageCache[url] {
             image = cachedImage
             return
@@ -61,7 +58,7 @@ final class ImageLoader: ObservableObject {
                     self?.start()
                 },
                 receiveOutput: { [weak self] in
-                    self?.cache($0)
+                    self?.cache(url: url, image: $0)
                 },
                 receiveCompletion: { [weak self] _ in
                     self?.finish()
@@ -87,7 +84,7 @@ final class ImageLoader: ObservableObject {
         isLoading = false
     }
 
-    private func cache(_ image: UIImage?) {
+    private func cache(url: URL?, image: UIImage?) {
         guard let url = url else { return }
         image.map { imageCache[url] = $0 }
     }

Sources/CachedAsyncImage/Services/NetworkManager.swift

@@ -53,8 +53,7 @@ final class NetworkManager: NetworkManagerProtocol {
 
                 #if DEBUG
                     let message = """
-                    ****
-                    CachedAsyncImage response.
+                    **** CachedAsyncImage response.
                     From: \(urlResponse.url?.absoluteString ?? "")
                     Status code: \(urlResponse.statusCode)
                     """

Sources/CachedAsyncImage/Views/CachedAsyncImage.swift

@@ -16,15 +16,17 @@ public struct CachedAsyncImage: View {
 
     // MARK: - Private Properties
 
+    private let url: String
     private let placeholder: (() -> any View)?
     private let image: (UIImage) -> any View
 
     // MARK: - Body
 
     public var body: some View {
         content
+            .onChange(of: url, perform: { imageLoader.fetchImage(from: $0) })
             .onAppear {
-                imageLoader.fetchImage()
+                imageLoader.fetchImage(from: url)
             }
     }
 
@@ -40,12 +42,10 @@ public struct CachedAsyncImage: View {
         image: @escaping (UIImage) -> any View
     ) {
         _imageLoader = StateObject(
-            wrappedValue: ImageLoader(
-                url: URL(string: url),
-                networkManager: NetworkManager.shared
-            )
+            wrappedValue: ImageLoader(networkManager: NetworkManager.shared)
         )
 
+        self.url = url
         self.placeholder = placeholder
         self.image = image
     }

Tests/CachedAsyncImageTests/ImageLoaderTests.swift

@@ -27,21 +27,20 @@ final class ImageLoaderTests: XCTestCase {
 
     func testFetchImage_WithCachedImage() {
         // Given
-        let imageUrl = URL(string: "https://example.com/image.jpg")
+        let url = "https://example.com/image.jpg"
         let networkManager = sut.networkManager
         let imageCache = sut.imageCache
         let cachedImage = RM.image("backToTheFuture")
 
-        let imageLoader = ImageLoader(
-            url: imageUrl,
-            networkManager: networkManager
-        )
+        let imageLoader = ImageLoader(networkManager: networkManager)
 
         // When
-        guard let imageUrl = imageUrl else { fatalError("Bad URL or nil.") }
+        guard let imageUrl = URL(string: url) else {
+            fatalError("Bad URL or nil.")
+        }
 
         imageCache[imageUrl] = cachedImage
-        imageLoader.fetchImage()
+        imageLoader.fetchImage(from: url)
 
         // Then
         XCTAssertNotNil(imageLoader.image, "Image should be not nil.")
@@ -56,21 +55,20 @@ final class ImageLoaderTests: XCTestCase {
 
     func testFetchImage_WithoutCachedImage() {
         // Given
-        let imageUrl = URL(string: "https://example.com/image.jpg")
+        let url = "https://example.com/image.jpg"
         let networkManager = sut.networkManager
         let imageCache = sut.imageCache
         imageCache.removeCache()
 
-        let imageLoader = ImageLoader(
-            url: imageUrl,
-            networkManager: networkManager
-        )
+        let imageLoader = ImageLoader(networkManager: networkManager)
 
         // When
-        imageLoader.fetchImage()
+        imageLoader.fetchImage(from: url)
 
         // Then
-        guard let imageUrl = imageUrl else { fatalError("Bad URL or nil.") }
+        guard let imageUrl = URL(string: url) else {
+            fatalError("Bad URL or nil.")
+        }
 
         XCTAssertNil(imageCache[imageUrl], "Image cache should be nil.")