fix(app, ios): avoid photo API not present on Catalyst (#4328)

invertase · Sep 30, 2020 · 86f1f63 · 86f1f63 · vercel · Sep 30, 2020
1 parent cbd1eba
commit 86f1f63
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/app/ios/RNFBApp/RNFBUtilsModule.m b/packages/app/ios/RNFBApp/RNFBUtilsModule.m
@@ -52,7 +52,7 @@ + (NSString *)valueForKey:(NSString *)key fromQueryItems:(NSArray *)queryItems {
 }
 
 + (PHAsset *)fetchAssetForPath:(NSString *)localFilePath {
-  PHAsset *asset;
+  PHAsset *asset = nil;
 
   if ([localFilePath hasPrefix:@"assets-library://"] || [localFilePath hasPrefix:@"ph://"]) {
     if ([localFilePath hasPrefix:@"assets-library://"]) {
@@ -63,9 +63,10 @@ + (PHAsset *)fetchAssetForPath:(NSString *)localFilePath {
           NSLog(@"'assets-library://' & 'ph://' URLs are not supported in Catalyst-based targets or iOS 12 and higher; returning nil (future warnings will be suppressed)");
           hasWarned = YES;
         }
-        asset = nil;
       } else {
+        #if (!TARGET_OS_MACCATALYST)
         asset = [[PHAsset fetchAssetsWithALAssetURLs:@[localFile] options:nil] firstObject];
+        #endif
       }
     } else {
       NSString *assetId = [localFilePath substringFromIndex:@"ph://".length];