Skip to content

Commit 4ced96b

Browse files
author
Rodrigo Gomez Palacio
authored
Merge pull request #1133 from OneSignal/null-check-fix
Outcome Event Null Check Fix
2 parents 2c78d41 + f33cc5b commit 4ced96b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

android/src/main/java/com/geektime/rnonesignalandroid/RNOneSignal.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -536,7 +536,7 @@ public void sendOutcome(final String name, final Callback callback) {
536536
OneSignal.sendOutcome(name, new OutcomeCallback() {
537537
@Override
538538
public void onSuccess(OSOutcomeEvent outcomeEvent) {
539-
if (outcomeEvent) {
539+
if (outcomeEvent != null) {
540540
try {
541541
callback.invoke(RNUtils.jsonToWritableMap(outcomeEvent.toJSONObject()));
542542
} catch (JSONException e) {
@@ -555,7 +555,7 @@ public void sendUniqueOutcome(final String name, final Callback callback) {
555555
OneSignal.sendUniqueOutcome(name, new OutcomeCallback() {
556556
@Override
557557
public void onSuccess(OSOutcomeEvent outcomeEvent) {
558-
if (outcomeEvent) {
558+
if (outcomeEvent != null) {
559559
try {
560560
callback.invoke(RNUtils.jsonToWritableMap(outcomeEvent.toJSONObject()));
561561
} catch (JSONException e) {
@@ -574,7 +574,7 @@ public void sendOutcomeWithValue(final String name, final float value, final Cal
574574
OneSignal.sendOutcomeWithValue(name, value, new OutcomeCallback() {
575575
@Override
576576
public void onSuccess(OSOutcomeEvent outcomeEvent) {
577-
if (outcomeEvent) {
577+
if (outcomeEvent != null) {
578578
try {
579579
callback.invoke(RNUtils.jsonToWritableMap(outcomeEvent.toJSONObject()));
580580
} catch (JSONException e) {

0 commit comments

Comments
 (0)