From 742781ce9de8e708b30e4981f6156b63f4fe7053 Mon Sep 17 00:00:00 2001
From: Nan <nanftw@gmail.com>
Date: Thu, 12 Aug 2021 10:53:04 -0700
Subject: [PATCH 1/2] Add `device_type` to `sendReceiveReceipt`

- get deviceType from `OSUtils.getDeviceType()`
---
 .../main/java/com/onesignal/OSReceiveReceiptController.java | 3 ++-
 .../main/java/com/onesignal/OSReceiveReceiptRepository.java | 6 ++++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptController.java b/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptController.java
index c51138c52d..2095a97cc9 100644
--- a/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptController.java
+++ b/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptController.java
@@ -54,6 +54,7 @@ synchronized public static OSReceiveReceiptController getInstance() {
     void sendReceiveReceipt(final CallbackToFutureAdapter.Completer<ListenableWorker.Result> callbackCompleter, @NonNull final String notificationId) {
         final String appId = OneSignal.appId == null || OneSignal.appId.isEmpty() ? OneSignal.getSavedAppId() : OneSignal.appId;
         final String playerId = OneSignal.getUserId();
+        final int deviceType = new OSUtils().getDeviceType();
 
         if (!remoteParamController.isReceiveReceiptEnabled()) {
             OneSignal.Log(OneSignal.LOG_LEVEL.DEBUG, "sendReceiveReceipt disable");
@@ -64,7 +65,7 @@ void sendReceiveReceipt(final CallbackToFutureAdapter.Completer<ListenableWorker
         Runnable receiveReceiptRunnable = new Runnable() {
             @Override
             public void run() {
-                repository.sendReceiveReceipt(appId, playerId, notificationId, new OneSignalRestClient.ResponseHandler() {
+                repository.sendReceiveReceipt(appId, playerId, deviceType, notificationId, new OneSignalRestClient.ResponseHandler() {
                     @Override
                     void onSuccess(String response) {
                         OneSignal.Log(OneSignal.LOG_LEVEL.DEBUG, "Receive receipt sent for notificationID: " + notificationId);
diff --git a/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptRepository.java b/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptRepository.java
index 86d63122e0..73d872a9b5 100644
--- a/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptRepository.java
+++ b/OneSignalSDK/onesignal/src/main/java/com/onesignal/OSReceiveReceiptRepository.java
@@ -36,12 +36,14 @@ class OSReceiveReceiptRepository {
 
     private static final String APP_ID = "app_id";
     private static final String PLAYER_ID = "player_id";
+    private static final String DEVICE_TYPE = "device_type";
 
-    void sendReceiveReceipt(@NonNull String appId, @NonNull String playerId, @NonNull String notificationId, @NonNull OneSignalRestClient.ResponseHandler responseHandler) {
+    void sendReceiveReceipt(@NonNull String appId, @NonNull String playerId,  @NonNull int deviceType, @NonNull String notificationId, @NonNull OneSignalRestClient.ResponseHandler responseHandler) {
         try {
             JSONObject jsonBody = new JSONObject()
                     .put(APP_ID, appId)
-                    .put(PLAYER_ID, playerId);
+                    .put(PLAYER_ID, playerId)
+                    .put(DEVICE_TYPE, deviceType);
 
             OneSignalRestClient.put("notifications/" + notificationId + "/report_received", jsonBody, responseHandler);
         } catch (JSONException e) {

From cdf0e7495e82cb30888b140bdaa509eedb5f31a1 Mon Sep 17 00:00:00 2001
From: Nan <nanftw@gmail.com>
Date: Thu, 12 Aug 2021 12:19:46 -0700
Subject: [PATCH 2/2] add `device_type` to
 `shouldSendReceivedReceiptWhenEnabled` test

- put "device_type" of 1 (ANDROID) to the payload
---
 .../java/com/test/onesignal/GenerateNotificationRunner.java  | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/OneSignalSDK/unittest/src/test/java/com/test/onesignal/GenerateNotificationRunner.java b/OneSignalSDK/unittest/src/test/java/com/test/onesignal/GenerateNotificationRunner.java
index 637ee38cbd..526ba37289 100644
--- a/OneSignalSDK/unittest/src/test/java/com/test/onesignal/GenerateNotificationRunner.java
+++ b/OneSignalSDK/unittest/src/test/java/com/test/onesignal/GenerateNotificationRunner.java
@@ -1268,7 +1268,10 @@ public void shouldSendReceivedReceiptWhenEnabled() throws Exception {
       assertReportReceivedAtIndex(
          2,
          "UUID",
-         new JSONObject().put("app_id", appId).put("player_id", ShadowOneSignalRestClient.pushUserId)
+         new JSONObject()
+                 .put("app_id", appId)
+                 .put("player_id", ShadowOneSignalRestClient.pushUserId)
+                 .put("device_type", 1)
       );
    }