add notification permission response handler

Added new public method overload for promptForPushNotifications to
accept a new optional PromptForPushNotificationPermissionResponseHandler
type. This fires when the user accepts or declines the notification
permission, includes handling of the fallbackToSettings logic as well.

Author: jkasten2

Examples/OneSignalDemo/app/src/main/java/com/onesignal/sdktest/model/MainActivityViewModel.java

@@ -560,8 +560,17 @@ private void setupSubscriptionSwitch() {
 
         subscriptionSwitch.setOnClickListener(v -> {
             OneSignal.disablePush(!subscriptionSwitch.isChecked());
-            if (subscriptionSwitch.isChecked())
-                OneSignal.promptForPushNotifications(true);
+            if (subscriptionSwitch.isChecked()) {
+                OneSignal.promptForPushNotifications(
+                        true,
+                        new OneSignal.PromptForPushNotificationPermissionResponseHandler() {
+                            @Override
+                            public void response(boolean accepted) {
+                                System.out.println("APP promptForPushNotifications:" + this + ":accepted: " + accepted);
+                            }
+                        }
+                );
+            }
         });
     }
 

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

@@ -28,19 +28,41 @@
 package com.onesignal
 
 import android.os.Build
+import androidx.annotation.ChecksSdkIntAtLeast
 
 object NotificationPermissionController : PermissionsActivity.PermissionCallback {
     private const val PERMISSION_TYPE = "NOTIFICATION"
     private const val ANDROID_PERMISSION_STRING = "android.permission.POST_NOTIFICATIONS"
 
+    private var callback: OneSignal.PromptForPushNotificationPermissionResponseHandler? = null
+
     init {
         PermissionsActivity.registerAsCallback(PERMISSION_TYPE, this)
     }
 
-    fun prompt(fallbackToSettings: Boolean) {
-        // TODO: Android 13 Beta 1 reports as 32 instead of 33, update to 33 once Google fixes this
-        if (Build.VERSION.SDK_INT < 32)
+    @ChecksSdkIntAtLeast(api = 33)
+    val supportsNativePrompt =
+        Build.VERSION.SDK_INT > 32 &&
+        OSUtils.getTargetSdkVersion(OneSignal.appContext) > 32
+
+    fun prompt(
+        fallbackToSettings: Boolean,
+        callback: OneSignal.PromptForPushNotificationPermissionResponseHandler?,
+    ) {
+        if (callback != null) this.callback = callback
+
+        if (notificationsEnabled()) {
+            fireCallBack(true)
+            return
+        }
+
+        if (!supportsNativePrompt) {
+            if (fallbackToSettings)
+                showFallbackAlertDialog()
+            else
+                fireCallBack(false)
             return
+        }
 
         PermissionsActivity.startPrompt(
             fallbackToSettings,
@@ -52,14 +74,21 @@ object NotificationPermissionController : PermissionsActivity.PermissionCallback
 
     override fun onAccept() {
         OneSignal.refreshNotificationPermissionState()
+        fireCallBack(true)
     }
 
     override fun onReject(fallbackToSettings: Boolean) {
-        if (fallbackToSettings) showFallbackAlertDialog()
+        val fallbackShown =
+            if (fallbackToSettings)
+                showFallbackAlertDialog()
+            else
+                false
+        if (!fallbackShown) fireCallBack(false)
     }
 
-    private fun showFallbackAlertDialog() {
-        val activity = OneSignal.getCurrentActivity() ?: return
+    // Returns true if dialog was shown
+    private fun showFallbackAlertDialog(): Boolean {
+        val activity = OneSignal.getCurrentActivity() ?: return false
         AlertDialogPrepromptForAndroidSettings.show(
             activity,
             activity.getString(R.string.notification_permission_name_for_title),
@@ -69,8 +98,22 @@ object NotificationPermissionController : PermissionsActivity.PermissionCallback
                     NavigateToAndroidSettingsForNotifications.show(activity)
                 }
                 override fun onDecline() {
+                    fireCallBack(false)
                 }
             }
         )
+        return true
     }
+
+    // Fires callback and clears it to ensure it is only called once.
+    private fun fireCallBack(accepted: Boolean) {
+        callback?.response(accepted)
+        callback = null
+    }
+
+    fun onAppForegrounded() {
+        fireCallBack(notificationsEnabled())
+    }
+
+    private fun notificationsEnabled() = OSUtils.areNotificationsEnabled(OneSignal.appContext)
 }

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

@@ -349,6 +349,13 @@ public interface PostNotificationResponseHandler {
       void onFailure(JSONObject response);
    }
 
+   /**
+    * Fires when the User accepts or declines the notification permission prompt.
+    */
+   public interface PromptForPushNotificationPermissionResponseHandler {
+      void response(boolean accepted);
+   }
+
    interface EntryStateListener {
       // Fire with the last appEntryState that just ended.
       void onEntryStateChange(AppEntryAction appEntryState);
@@ -1400,6 +1407,7 @@ static void onAppFocus() {
       }
 
       LocationController.onFocusChange();
+      NotificationPermissionController.INSTANCE.onAppForegrounded();
 
       if (OSUtils.shouldLogMissingAppIdError(appId))
          return;
@@ -2856,7 +2864,23 @@ public static void promptForPushNotifications() {
     *                           settings if they have declined before.
     */
    public static void promptForPushNotifications(boolean fallbackToSettings) {
-      NotificationPermissionController.INSTANCE.prompt(fallbackToSettings);
+      promptForPushNotifications(fallbackToSettings, null);
+   }
+
+   /**
+    * On Android 13 shows the system notification permission prompt to enable displaying
+    * notifications. This is required for apps that target Android API level 33 / "Tiramisu"
+    * to subscribe the device for push notifications.
+    *
+    * @param fallbackToSettings whether to show a Dialog to direct users to the App's notification
+    *                           settings if they have declined before.
+    * @param handler fires when the user declines ore accepts the notification permission prompt.
+    */
+   public static void promptForPushNotifications(
+      boolean fallbackToSettings,
+      @Nullable PromptForPushNotificationPermissionResponseHandler handler
+   ) {
+      NotificationPermissionController.INSTANCE.prompt(fallbackToSettings, handler);
    }
 
    /**