Skip to content

Commit 7c54348

Browse files
committed
Update external id method
* Implement method from demo app * Rename from onComplete to onSuccess
1 parent 32da4ec commit 7c54348

File tree

4 files changed

+11
-4
lines changed

4 files changed

+11
-4
lines changed

Examples/OneSignalDemo/app/src/main/java/com/onesignal/sdktest/util/Dialog.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ public void run() {
183183
private void updateExternalUserId(final DialogInterface dialog, final String externalUserId) {
184184
OneSignal.setExternalUserId(externalUserId, new OneSignal.OSExternalUserIdUpdateCompletionHandler() {
185185
@Override
186-
public void onComplete(JSONObject results) {
186+
public void onSuccess(JSONObject results) {
187187
// Default success to false until we know push came back successful
188188
boolean successful = false;
189189

@@ -210,6 +210,13 @@ public void onComplete(JSONObject results) {
210210
dialog.dismiss();
211211
}
212212

213+
@Override
214+
public void onFailure(OneSignal.ExternalIdError error) {
215+
OneSignal.onesignalLog(OneSignal.LOG_LEVEL.VERBOSE, "External user id set failed with error: " + error);
216+
callback.onFailure();
217+
dialog.dismiss();
218+
}
219+
213220
/**
214221
* Parse the results of the external user id completion callback and make sure push.success = true
215222
*/

OneSignalSDK/onesignal/src/main/java/com/onesignal/OneSignal.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,7 @@ public String getMessage() {
253253
}
254254

255255
public interface OSExternalUserIdUpdateCompletionHandler {
256-
void onComplete(JSONObject results);
256+
void onSuccess(JSONObject results);
257257
void onFailure(ExternalIdError error);
258258
}
259259

OneSignalSDK/onesignal/src/main/java/com/onesignal/OneSignalStateSynchronizer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ public void onComplete(String channel, boolean success) {
216216
@Override
217217
public void run() {
218218
if (completionHandler != null)
219-
completionHandler.onComplete(responses);
219+
completionHandler.onSuccess(responses);
220220
}
221221
});
222222
}

OneSignalSDK/unittest/src/test/java/com/test/onesignal/MainOneSignalClassRunner.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ private static OneSignal.OSNotificationOpenedHandler getNotificationOpenedHandle
219219
private static OneSignal.OSExternalUserIdUpdateCompletionHandler getExternalUserIdUpdateCompletionHandler() {
220220
return new OneSignal.OSExternalUserIdUpdateCompletionHandler() {
221221
@Override
222-
public void onComplete(JSONObject results) {
222+
public void onSuccess(JSONObject results) {
223223
lastExternalUserIdResponse = results;
224224
}
225225

0 commit comments

Comments
 (0)