Don't hydrate push subscription status

Existing Problem:
- If on a previous session, `optOut()` was called, we will hydrate the push subscription with the status of `UNSUBSCRIBE(-2)` on the next session.
- However, we lose the true device-level subscription status in doing so.
- Then, when `optIn()` is called, we were using `UNSUBSCRIBE(-2)` along with `optedIn` to calculate the data to send in the Update Subscription request.
- This resulted in incorrect values being sent and reports of `optIn()` method not working.

Solution:
- We can still hydrate the push subscription model's `optedIn` property if we see either UNSUBSCRIBE(-2) or DISABLED_FROM_REST_API_DEFAULT_REASON(-30).
- However, when the subscription model store is replacing all of the subscriptions in the refresh user operation, keep the existing push subscription `status`.
- This way, when `optIn()` is called again, we can use device-level status with the `optedIn` boolean to calculate values to send to the server.

Author: nan-li

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/operations/impl/executors/RefreshUserOperationExecutor.kt

@@ -105,7 +105,7 @@ internal class RefreshUserOperationExecutor(
                             SubscriptionType.PUSH
                         }
                     }
-                subscriptionModel.optedIn = subscriptionModel.status != SubscriptionStatus.UNSUBSCRIBE
+                subscriptionModel.optedIn = subscriptionModel.status != SubscriptionStatus.UNSUBSCRIBE && subscriptionModel.status != SubscriptionStatus.DISABLED_FROM_REST_API_DEFAULT_REASON
                 subscriptionModel.sdk = subscription.sdk ?: ""
                 subscriptionModel.deviceOS = subscription.deviceOS ?: ""
                 subscriptionModel.carrier = subscription.carrier ?: ""

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/subscriptions/SubscriptionModel.kt

@@ -67,6 +67,9 @@ enum class SubscriptionStatus(val value: Int) {
     /** The subscription is not enabled due to an FCM authentication failed IOException */
     FIREBASE_FCM_ERROR_IOEXCEPTION_AUTHENTICATION_FAILED(-29),
 
+    /** The subscription is not enabled because the app has disabled the subscription via API */
+    DISABLED_FROM_REST_API_DEFAULT_REASON(-30),
+
     /** The subscription is not enabled due to some other (unknown locally) error */
     ERROR(9999),
     ;
@@ -79,6 +82,10 @@ enum class SubscriptionStatus(val value: Int) {
 }
 
 class SubscriptionModel : Model() {
+    /**
+     * Reflects user preference only, defaults true.
+     * The public API for [IPushSubscription.optedIn] considers this value AND permission.
+     */
     var optedIn: Boolean
         get() = getBooleanProperty(::optedIn.name)
         set(value) {
@@ -97,6 +104,13 @@ class SubscriptionModel : Model() {
             setStringProperty(::address.name, value)
         }
 
+    /**
+     * This reflects the "device-level" subscription status.
+     *
+     * For example, if [IPushSubscription.optOut] is called, the SDK sends UNSUBSCRIBE(-2) to the server.
+     * However, locally on the model, we still keep the existing status.
+     * It is necessary when [IPushSubscription.optIn] is called again to know the true device status.
+     */
     var status: SubscriptionStatus
         get() {
             if (!hasProperty(::status.name)) {

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/subscriptions/SubscriptionModelStore.kt

@@ -24,6 +24,7 @@ open class SubscriptionModelStore(prefs: IPreferencesService) : SimpleModelStore
                         model.deviceOS = existingPushModel.deviceOS
                         model.carrier = existingPushModel.carrier
                         model.appVersion = existingPushModel.appVersion
+                        model.status = existingPushModel.status
                     }
                     break
                 }