removing page id check in pageChanged

Author: emawby

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

@@ -133,7 +133,6 @@ public String getClickName() {
         return clickName;
     }
 
-    @NonNull
     String getPageId() {
         return pageId;
     }

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

@@ -286,9 +286,6 @@ void onPageChanged(@NonNull final OSInAppMessage message, @NonNull final JSONObj
         if (message.isPreview) {
             return;
         }
-        if (message.getViewedPageIds().contains(newPage.getPageId())) {
-            return;
-        }
         fireRESTCallForPageChange(message, newPage);
     }
 
@@ -427,7 +424,6 @@ private void fireRESTCallForPageChange(@NonNull final OSInAppMessage message, @N
 
         final String pageId = page.getPageId();
 
-
         // Never send multiple page impressions for the same message UUID unless that page change is from an IAM with redisplay
         if (message.getViewedPageIds().contains(pageId))
             return;

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSInAppMessagePage.kt

@@ -16,7 +16,7 @@ class OSInAppMessagePage constructor(jsonObject: JSONObject) {
         pageIndex = jsonObject.optString(PAGE_INDEX, null)
     }
 
-    fun toJSONObject(): JSONObject? {
+    fun toJSONObject(): JSONObject {
         val mainObj = JSONObject()
         try {
             mainObj.put(PAGE_ID, pageId)