Merge pull request Tencent#468 from passerbyloo/master

fix: request image for asset memory leaks
wsl-victor · Dec 27, 2018 · 90b9c98 · 90b9c98
2 parents bcd6789 + d3b456b
commit 90b9c98
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/QMUIKit/QMUIComponents/AssetLibrary/QMUIAsset.m b/QMUIKit/QMUIComponents/AssetLibrary/QMUIAsset.m
@@ -124,9 +124,9 @@ - (NSInteger)requestOriginImageWithCompletion:(void (^)(UIImage *result, NSDicti
     PHImageRequestOptions *imageRequestOptions = [[PHImageRequestOptions alloc] init];
     imageRequestOptions.networkAccessAllowed = YES; // 允许访问网络
     imageRequestOptions.progressHandler = phProgressHandler;
-    return [[[QMUIAssetsManager sharedInstance] phCachingImageManager] requestImageForAsset:_phAsset targetSize:PHImageManagerMaximumSize contentMode:PHImageContentModeDefault options:imageRequestOptions resultHandler:^(UIImage *result, NSDictionary *info) {
+    return [[[QMUIAssetsManager sharedInstance] phCachingImageManager] requestImageDataForAsset:_phAsset options:imageRequestOptions resultHandler:^(NSData * _Nullable imageData, NSString * _Nullable dataUTI, UIImageOrientation orientation, NSDictionary * _Nullable info) {
         if (completion) {
-            completion(result, info);
+            completion([UIImage imageWithData:imageData], info);
         }
     }];
 }