swift/Beam
swift/Beam
1
0
Fork 0
mirror of https://github.com/element-hq/element-x-ios.git synced 2025-03-10 21:39:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Upgrade Kingfisher to fix a bug that prevented GIFs from being tapped. (#3326)

Browse Source 
* Upgrade Kingfisher to fix a bug the prevented GIFs from being tapped.

https://github.com/onevcat/Kingfisher/issues/2295

* Fix tests.
This commit is contained in:
Doug 2024-09-25 13:10:02 +01:00 committed by GitHub
parent 864bd23d35
commit af8c16150b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
5 changed files with 9 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ElementX.xcodeproj/project.pbxproj
View File

@ -7839,7 +7839,7 @@
repositoryURL = "https://github.com/onevcat/Kingfisher";
requirement = {
kind = upToNextMinorVersion;
minimumVersion = 7.6.0;
minimumVersion = 8.0.3;
};
};
D5F7D47BBAAE0CF1DDEB3034 /* XCRemoteSwiftPackageReference "DeviceKit" */ = {

4
ElementX.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
View File

@ -86,8 +86,8 @@
"kind" : "remoteSourceControl",
"location" : "https://github.com/onevcat/Kingfisher",
"state" : {
"revision" : "af4be924ad984cf4d16f4ae4df424e79a443d435",
"version" : "7.6.2"
"revision" : "33696a71dd4d71f1b0f781ade11a48ba5549581a",
"version" : "8.0.3"
}
},
{

16
ElementX/Sources/Services/Media/Provider/MediaProvider.swift
View File

@ -38,8 +38,8 @@ struct MediaProvider: MediaProviderProtocol {
}
let cacheKey = cacheKeyForURL(source.url, size: size)
if case let .success(cacheResult) = await imageCache.retrieveImage(forKey: cacheKey),
if let cacheResult = try? await imageCache.retrieveImage(forKey: cacheKey, options: nil),
let image = cacheResult.image {
return .success(image)
}
@ -57,7 +57,7 @@ struct MediaProvider: MediaProviderProtocol {
return .failure(.invalidImageData)
}
imageCache.store(image, forKey: cacheKey)
try await imageCache.store(image, forKey: cacheKey)
return .success(image)
} catch {
@ -149,13 +149,3 @@ struct MediaProvider: MediaProviderProtocol {
}
}
}
private extension ImageCache {
func retrieveImage(forKey key: String) async -> Result<ImageCacheResult, KingfisherError> {
await withCheckedContinuation { continuation in
retrieveImage(forKey: key) { result in
continuation.resume(returning: result)
}
}
}
}

3
UnitTests/Sources/MediaProvider/MockImageCache.swift
View File

@ -5,7 +5,7 @@
// Please see LICENSE in the repository root for full details.
//
@testable import ElementX
import Kingfisher
@testable import Kingfisher
import UIKit
class MockImageCache: ImageCache {
@ -35,5 +35,6 @@ class MockImageCache: ImageCache {
callbackQueue: CallbackQueue = .untouch,
completionHandler: ((CacheStoreResult) -> Void)? = nil) {
storedImages[key] = image
completionHandler?(.init(memoryCacheResult: .success(()), diskCacheResult: .success(())))
}
}

2
project.yml
View File

@ -106,7 +106,7 @@ packages:
minorVersion: 4.2.0
Kingfisher:
url: https://github.com/onevcat/Kingfisher
minorVersion: 7.6.0
minorVersion: 8.0.3
KZFileWatchers:
url: https://github.com/krzysztofzablocki/KZFileWatchers
minorVersion: 1.2.0