Skip to content

Commit c9ac94a

Browse files
dmytrorykunfacebook-github-bot
authored andcommitted
Rename shouldNotify to shouldNotifyLoadEvents (#48100)
Summary: Pull Request resolved: #48100 This diff renames `shouldNotify` to `shouldNotifyLoadEvents` as it is named in the spec. Changelog: [Internal] Reviewed By: javache Differential Revision: D66769660 fbshipit-source-id: 64282c08ab82101d51dedb583e0c34476ed90eeb
1 parent f5506dc commit c9ac94a

File tree

5 files changed

+14
-13
lines changed

5 files changed

+14
-13
lines changed

packages/react-native/ReactCommon/react/renderer/components/image/ImageProps.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -107,14 +107,14 @@ ImageProps::ImageProps(
107107
"resizeMultiplier",
108108
sourceProps.resizeMultiplier,
109109
{})),
110-
shouldNotify(
110+
shouldNotifyLoadEvents(
111111
ReactNativeFeatureFlags::enableCppPropsIteratorSetter()
112-
? sourceProps.shouldNotify
112+
? sourceProps.shouldNotifyLoadEvents
113113
: convertRawProp(
114114
context,
115115
rawProps,
116-
"shouldNotify",
117-
sourceProps.shouldNotify,
116+
"shouldNotifyLoadEvents",
117+
sourceProps.shouldNotifyLoadEvents,
118118
{})),
119119
overlayColor(
120120
ReactNativeFeatureFlags::enableCppPropsIteratorSetter()
@@ -167,7 +167,7 @@ void ImageProps::setProp(
167167
RAW_SET_PROP_SWITCH_CASE_BASIC(internal_analyticTag);
168168
RAW_SET_PROP_SWITCH_CASE_BASIC(resizeMethod);
169169
RAW_SET_PROP_SWITCH_CASE_BASIC(resizeMultiplier);
170-
RAW_SET_PROP_SWITCH_CASE_BASIC(shouldNotify);
170+
RAW_SET_PROP_SWITCH_CASE_BASIC(shouldNotifyLoadEvents);
171171
RAW_SET_PROP_SWITCH_CASE_BASIC(overlayColor);
172172
RAW_SET_PROP_SWITCH_CASE_BASIC(fadeDuration);
173173
RAW_SET_PROP_SWITCH_CASE_BASIC(progressiveRenderingEnabled);

packages/react-native/ReactCommon/react/renderer/components/image/ImageProps.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ class ImageProps final : public ViewProps {
4141
std::string internal_analyticTag{};
4242
std::string resizeMethod{};
4343
Float resizeMultiplier{};
44-
bool shouldNotify{};
44+
bool shouldNotifyLoadEvents{};
4545
SharedColor overlayColor{};
4646
Float fadeDuration{};
4747
bool progressiveRenderingEnabled{};

packages/react-native/ReactCommon/react/renderer/components/image/ImageShadowNode.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ void ImageShadowNode::updateStateIfNeeded() {
3939
imageProps.resizeMethod,
4040
// TODO: should we resizeMultiplier * imageSource.scale ?
4141
imageProps.resizeMultiplier,
42-
imageProps.shouldNotify,
42+
imageProps.shouldNotifyLoadEvents,
4343
imageProps.overlayColor,
4444
imageProps.tintColor,
4545
imageProps.fadeDuration,

packages/react-native/ReactCommon/react/renderer/imagemanager/platform/android/react/renderer/imagemanager/ImageRequestParams.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class ImageRequestParams {
2424
ImageResizeMode resizeMode,
2525
std::string resizeMethod,
2626
Float resizeMultiplier,
27-
bool shouldNotify,
27+
bool shouldNotifyLoadEvents,
2828
SharedColor overlayColor,
2929
SharedColor tintColor,
3030
Float fadeDuration,
@@ -36,7 +36,7 @@ class ImageRequestParams {
3636
resizeMode(resizeMode),
3737
resizeMethod(std::move(resizeMethod)),
3838
resizeMultiplier(resizeMultiplier),
39-
shouldNotify(shouldNotify),
39+
shouldNotifyLoadEvents(shouldNotifyLoadEvents),
4040
overlayColor(overlayColor),
4141
tintColor(tintColor),
4242
fadeDuration(fadeDuration),
@@ -49,7 +49,7 @@ class ImageRequestParams {
4949
ImageResizeMode resizeMode{ImageResizeMode::Stretch};
5050
std::string resizeMethod{};
5151
Float resizeMultiplier{};
52-
bool shouldNotify{};
52+
bool shouldNotifyLoadEvents{};
5353
SharedColor overlayColor{};
5454
SharedColor tintColor{};
5555
Float fadeDuration{};
@@ -64,7 +64,7 @@ class ImageRequestParams {
6464
this->resizeMode,
6565
this->resizeMethod,
6666
this->resizeMultiplier,
67-
this->shouldNotify,
67+
this->shouldNotifyLoadEvents,
6868
this->overlayColor,
6969
this->tintColor,
7070
this->fadeDuration,
@@ -77,7 +77,7 @@ class ImageRequestParams {
7777
rhs.resizeMode,
7878
rhs.resizeMethod,
7979
rhs.resizeMultiplier,
80-
rhs.shouldNotify,
80+
rhs.shouldNotifyLoadEvents,
8181
rhs.overlayColor,
8282
rhs.tintColor,
8383
rhs.fadeDuration,

packages/react-native/ReactCommon/react/renderer/imagemanager/platform/android/react/renderer/imagemanager/conversions.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,8 @@ inline void serializeImageRequestParams(
6969
builder.putDouble(
7070
IS_KEY_RESIZE_MULTIPLIER, imageRequestParams.resizeMultiplier);
7171
builder.putBool(
72-
IS_KEY_SHOULD_NOTIFY_LOAD_EVENTS, imageRequestParams.shouldNotify);
72+
IS_KEY_SHOULD_NOTIFY_LOAD_EVENTS,
73+
imageRequestParams.shouldNotifyLoadEvents);
7374
if (isColorMeaningful(imageRequestParams.overlayColor)) {
7475
builder.putInt(
7576
IS_KEY_OVERLAY_COLOR, toAndroidRepr(imageRequestParams.overlayColor));

0 commit comments

Comments
 (0)