Change getDownloadURL to return url rather than object

Author: chrisbianca

android/src/main/java/io/fullstack/firestack/storage/FirestackStorage.java

@@ -115,47 +115,13 @@ public void getDownloadURL(final String path,
         .addOnSuccessListener(new OnSuccessListener<Uri>() {
           @Override
           public void onSuccess(Uri uri) {
-            final WritableMap res = Arguments.createMap();
-
-            res.putString("status", "success");
-            res.putString("bucket", FirebaseStorage.getInstance().getApp().getOptions().getStorageBucket());
-            res.putString("fullPath", uri.toString());
-            res.putString("path", uri.getPath());
-            res.putString("url", uri.toString());
-
-            reference.getMetadata()
-                .addOnSuccessListener(new OnSuccessListener<StorageMetadata>() {
-                  @Override
-                  public void onSuccess(final StorageMetadata storageMetadata) {
-                    Log.d(TAG, "getMetadata success " + storageMetadata);
-
-                    res.putMap("metadata", getMetadataAsMap(storageMetadata));
-                    res.putString("name", storageMetadata.getName());
-                    res.putString("url", storageMetadata.getDownloadUrl().toString());
-                    callback.invoke(null, res);
-                  }
-                })
-                .addOnFailureListener(new OnFailureListener() {
-                  @Override
-                  public void onFailure(@NonNull Exception exception) {
-                    Log.e(TAG, "Failure in download " + exception);
-                    final int errorCode = 1;
-                    callback.invoke(makeErrorPayload(errorCode, exception));
-                  }
-                });
-
+            callback.invoke(null, uri.toString());
           }
         })
         .addOnFailureListener(new OnFailureListener() {
           @Override
           public void onFailure(@NonNull Exception exception) {
-            Log.e(TAG, "Failed to download file " + exception.getMessage());
-
-            WritableMap err = Arguments.createMap();
-            err.putString("status", "error");
-            err.putString("description", exception.getLocalizedMessage());
-
-            callback.invoke(err);
+            callback.invoke(makeErrorPayload(1, exception));
           }
         });
   }

ios/Firestack/FirestackStorage.m

@@ -57,12 +57,7 @@ - (dispatch_queue_t)methodQueue
                                   };
             callback(@[evt]);
         } else {
-            NSDictionary *resp = @{
-                                   @"status": @"success",
-                                   @"url": [URL absoluteString],
-                                   @"path": [URL path]
-                                   };
-            callback(@[[NSNull null], resp]);
+            callback(@[[NSNull null], [URL absoluteString]]);
         }
     }];
 }