Merge pull request #10 from dream11/fix/on-error-empty-null-source

fix: on error empty null source [DylanVann#1028]

README.md

@@ -183,7 +183,8 @@ e.g. `onLoad={e => console.log(e.nativeEvent.width, e.nativeEvent.height)}`
 
 ### `onError?: () => void`
 
-Called on an image fetching error.
+Called on an image fetching error or when source is malformed (i.e: empty or null).
+
 
 ---
 

android/src/main/java/com/dylanvann/fastimage/FastImageViewWithUrl.java

@@ -73,6 +73,14 @@ public void onAfterUpdate(
 
             // Clear the image.
             setImageDrawable(null);
+
+            ThemedReactContext context = (ThemedReactContext) getContext();
+            RCTEventEmitter eventEmitter = context.getJSModule(RCTEventEmitter.class);
+            int viewId = getId();
+            WritableMap event = new WritableNativeMap();
+            event.putString("message", "Image source uri cannot be empty or null:" + mSource);
+            eventEmitter.receiveEvent(viewId, REACT_ON_ERROR_EVENT, event);
+
             return;
         }
 
@@ -84,7 +92,7 @@ public void onAfterUpdate(
             RCTEventEmitter eventEmitter = context.getJSModule(RCTEventEmitter.class);
             int viewId = getId();
             WritableMap event = new WritableNativeMap();
-            event.putString("message", "Invalid source prop:" + mSource);
+            event.putString("message", "Invalid source prop: " + mSource);
             eventEmitter.receiveEvent(viewId, REACT_ON_ERROR_EVENT, event);
 
             // Cancel existing requests.

src/index.tsx

@@ -14,7 +14,7 @@ import {
     Platform,
     AccessibilityProps,
     ViewProps,
-    ColorValue, Text,
+    ColorValue
 } from 'react-native'
 
 export type ResizeMode = 'contain' | 'cover' | 'stretch' | 'center'