diff --git a/Libraries/Image/ImageProps.js b/Libraries/Image/ImageProps.js index 8f8512e06cb861..c7bf077c1f3190 100644 --- a/Libraries/Image/ImageProps.js +++ b/Libraries/Image/ImageProps.js @@ -21,9 +21,8 @@ export type ImageLoadEvent = SyntheticEvent< source: $ReadOnly<{| width: number, height: number, - url: string, + uri: string, |}>, - uri?: string, // Only on Android |}>, >; diff --git a/Libraries/Image/RCTImageView.mm b/Libraries/Image/RCTImageView.mm index fca211934d49ab..c0b23f78f5ee0e 100644 --- a/Libraries/Image/RCTImageView.mm +++ b/Libraries/Image/RCTImageView.mm @@ -41,9 +41,9 @@ static BOOL RCTShouldReloadImageForSizeChange(CGSize currentSize, CGSize idealSi static NSDictionary *onLoadParamsForSource(RCTImageSource *source) { NSDictionary *dict = @{ + @"uri": source.request.URL.absoluteString, @"width": @(source.size.width), @"height": @(source.size.height), - @"url": source.request.URL.absoluteString, }; return @{ @"source": dict }; } diff --git a/RNTester/js/examples/Image/ImageExample.js b/RNTester/js/examples/Image/ImageExample.js index da56f1865f6455..32b9c58db999c4 100644 --- a/RNTester/js/examples/Image/ImageExample.js +++ b/RNTester/js/examples/Image/ImageExample.js @@ -76,7 +76,7 @@ class NetworkImageCallbackExample extends React.Component< } onLoad={event => { if (event.nativeEvent.source) { - const url = event.nativeEvent.source.url; + const url = event.nativeEvent.source.uri; this._loadEventFired( `✔ onLoad (+${new Date() - mountTime}ms) for URL ${url}`, ); @@ -127,7 +127,7 @@ class NetworkImageCallbackExample extends React.Component< onLoad={event => { // Currently this image source feature is only available on iOS. if (event.nativeEvent.source) { - const url = event.nativeEvent.source.url; + const url = event.nativeEvent.source.uri; this._loadEventFired( `✔ (prefetched) onLoad (+${new Date() - mountTime}ms) for URL ${url}`, diff --git a/ReactAndroid/src/main/java/com/facebook/react/views/image/ImageLoadEvent.java b/ReactAndroid/src/main/java/com/facebook/react/views/image/ImageLoadEvent.java index 271f6838e8c547..6e900a432aaed4 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/views/image/ImageLoadEvent.java +++ b/ReactAndroid/src/main/java/com/facebook/react/views/image/ImageLoadEvent.java @@ -107,14 +107,10 @@ public void dispatch(RCTEventEmitter rctEventEmitter) { switch (mEventType) { case ON_LOAD: eventData = Arguments.createMap(); - // TODO: Remove this (to be less redundant and to be consistent with iOS). - eventData.putString("uri", mSourceUri); eventData.putMap("source", createEventDataSource()); break; case ON_ERROR: eventData = Arguments.createMap(); - // TODO: Remove this (to be less redundant and to be consistent with iOS). - eventData.putString("uri", mSourceUri); eventData.putString("error", mErrorMessage); break; } @@ -124,9 +120,9 @@ public void dispatch(RCTEventEmitter rctEventEmitter) { private WritableMap createEventDataSource() { WritableMap source = Arguments.createMap(); + source.putString("uri", mSourceUri); source.putDouble("width", mWidth); source.putDouble("height", mHeight); - source.putString("url", mSourceUri); return source; } }