feat: Removing deprecations - Part 1

MIGRATING.md

@@ -0,0 +1,333 @@
+# Migration Guides
+
+## Migrating from versions < 6.0.0
+
+This guide covers the breaking changes introduced in mParticle Android SDK 6.0.0 and how to migrate your code to the new APIs.
+
+---
+
+### Removed `isAndroidIdDisabled()` / `androidIdDisabled(boolean)`
+
+The `isAndroidIdDisabled()` method on `MParticle` and `MParticleOptions`, and the `androidIdDisabled(boolean)` builder method on `MParticleOptions.Builder`, have been removed. These were replaced by inverted equivalents with clearer semantics.
+
+Replace `isAndroidIdDisabled()` with `isAndroidIdEnabled()` and invert the logic:
+
+Java:
+
+```java
+// Before
+if (MParticle.isAndroidIdDisabled()) { ... }
+if (options.isAndroidIdDisabled()) { ... }
+
+// After
+if (!MParticle.isAndroidIdEnabled()) { ... }
+if (!options.isAndroidIdEnabled()) { ... }
+```
+
+Kotlin:
+
+```kotlin
+// Before
+if (MParticle.isAndroidIdDisabled()) { ... }
+if (options.isAndroidIdDisabled()) { ... }
+
+// After
+if (!MParticle.isAndroidIdEnabled()) { ... }
+if (!options.isAndroidIdEnabled()) { ... }
+```
+
+Replace `androidIdDisabled(boolean)` with `androidIdEnabled(boolean)` and invert the value:
+
+Java:
+
+```java
+// Before
+MParticleOptions.builder(context)
+    .androidIdDisabled(true)
+    .build();
+
+// After
+MParticleOptions.builder(context)
+    .androidIdEnabled(false)
+    .build();
+```
+
+Kotlin:
+
+```kotlin
+// Before
+MParticleOptions.builder(context)
+    .androidIdDisabled(true)
+    .build()
+
+// After
+MParticleOptions.builder(context)
+    .androidIdEnabled(false)
+    .build()
+```
+
+---
+
+### Removed `isAutoTrackingEnabled()`
+
+The `isAutoTrackingEnabled()` method on `MParticle` has been removed. This method always returned `false` and automatic screen tracking via activity lifecycle hooks has been removed.
+
+Remove any calls to `isAutoTrackingEnabled()`. If your code depends on this value, note that it was always `false` — no screen tracking was being performed automatically.
+
+Java:
+
+```java
+// Before
+if (MParticle.getInstance().isAutoTrackingEnabled()) {
+    // this block was never executed
+}
+
+// After
+// Remove the call entirely
+```
+
+Kotlin:
+
+```kotlin
+// Before
+if (MParticle.getInstance()?.isAutoTrackingEnabled == true) {
+    // this block was never executed
+}
+
+// After
+// Remove the call entirely
+```
+
+---
+
+### Removed `isProviderActive(int)`
+
+The `isProviderActive(int)` method on `MParticle` has been removed. It has been renamed to `isKitActive(int)`.
+
+Replace `isProviderActive(serviceProviderId)` with `isKitActive(serviceProviderId)`:
+
+Java:
+
+```java
+// Before
+boolean active = MParticle.getInstance().isProviderActive(MParticle.ServiceProviders.APPBOY);
+
+// After
+boolean active = MParticle.getInstance().isKitActive(MParticle.ServiceProviders.APPBOY);
+```
+
+Kotlin:
+
+```kotlin
+// Before
+val active = MParticle.getInstance()?.isProviderActive(MParticle.ServiceProviders.APPBOY)
+
+// After
+val active = MParticle.getInstance()?.isKitActive(MParticle.ServiceProviders.APPBOY)
+```
+
+---
+
+### Removed `MPEvent.setInfo()` / `getInfo()` / `Builder.info()`
+
+The `setInfo(Map)`, `getInfo()`, and `Builder.info(Map)` methods on `MPEvent` have been removed. They have been renamed to `setCustomAttributes()`, `getCustomAttributes()`, and `Builder.customAttributes()`.
+
+Replace usages with the new method names:
+
+Java:
+
+```java
+// Before
+MPEvent event = new MPEvent.Builder("Event Name", MParticle.EventType.Other)
+    .info(attributes)
+    .build();
+Map<String, String> info = event.getInfo();
+event.setInfo(attributes);
+
+// After
+MPEvent event = new MPEvent.Builder("Event Name", MParticle.EventType.Other)
+    .customAttributes(attributes)
+    .build();
+Map<String, ?> customAttributes = event.getCustomAttributes();
+event.setCustomAttributes(attributes);
+```
+
+Kotlin:
+
+```kotlin
+// Before
+val event = MPEvent.Builder("Event Name", MParticle.EventType.Other)
+    .info(attributes)
+    .build()
+val info = event.getInfo()
+event.setInfo(attributes)
+
+// After
+val event = MPEvent.Builder("Event Name", MParticle.EventType.Other)
+    .customAttributes(attributes)
+    .build()
+val customAttributes = event.getCustomAttributes()
+event.setCustomAttributes(attributes)
+```
+
+---
+
+### Removed `UserAttributeListener`
+
+The `com.mparticle.UserAttributeListener` interface has been removed. It has been replaced by `com.mparticle.TypedUserAttributeListener`.
+
+The key difference is that `onUserAttributesReceived` now receives `Map<String, Any?>` instead of `Map<String, String?>` for user attribute singles, allowing typed values (numbers, booleans) to be preserved rather than stringified. Additionally, the `mpid` parameter is non-nullable (`Long` instead of `Long?`).
+
+Java:
+
+```java
+// Before
+user.getUserAttributes(new UserAttributeListener() {
+    @Override
+    public void onUserAttributesReceived(
+        @Nullable Map<String, String> userAttributes,
+        @Nullable Map<String, List<String>> userAttributeLists,
+        @Nullable Long mpid
+    ) {
+        // handle attributes
+    }
+});
+
+// After
+user.getUserAttributes(new TypedUserAttributeListener() {
+    @Override
+    public void onUserAttributesReceived(
+        @NonNull Map<String, ?> userAttributes,
+        @NonNull Map<String, ? extends List<String>> userAttributeLists,
+        long mpid
+    ) {
+        // handle attributes — values may be String, Number, or null
+    }
+});
+```
+
+Kotlin:
+
+```kotlin
+// Before
+user.getUserAttributes(UserAttributeListener { userAttributes, userAttributeLists, mpid ->
+    // userAttributes: Map<String, String?>?
+})
+
+// After
+user.getUserAttributes(object : TypedUserAttributeListener {
+    override fun onUserAttributesReceived(
+        userAttributes: Map<String, Any?>,
+        userAttributeLists: Map<String, List<String?>?>,
+        mpid: Long
+    ) {
+        // userAttributes values may be String, Number, Boolean, or null
+    }
+})
+```
+
+Note: numeric and boolean attribute values are no longer automatically converted to strings. If you need string values, call `.toString()` on each value manually.
+
+---
+
+### Removed `UserAliasHandler` / `IdentityApiRequest.Builder.userAliasHandler()`
+
+The `com.mparticle.identity.UserAliasHandler` interface and the `IdentityApiRequest.Builder.userAliasHandler()` method have been removed. Use a success listener on the `BaseIdentityTask` returned by identity API calls instead.
+
+Java:
+
+```java
+// Before
+IdentityApiRequest request = IdentityApiRequest.withEmptyUser()
+    .email("user@example.com")
+    .userAliasHandler(new UserAliasHandler() {
+        @Override
+        public void onUserAlias(MParticleUser previousUser, MParticleUser newUser) {
+            // copy attributes from previousUser to newUser
+        }
+    })
+    .build();
+MParticle.getInstance().Identity().login(request);
+
+// After
+IdentityApiRequest request = IdentityApiRequest.withEmptyUser()
+    .email("user@example.com")
+    .build();
+MParticle.getInstance().Identity().login(request)
+    .addSuccessListener(new TaskSuccessListener() {
+        @Override
+        public void onSuccess(IdentityApiResult result) {
+            MParticleUser newUser = result.getUser();
+            MParticleUser previousUser = result.getPreviousUser();
+            if (previousUser != null) {
+                // copy attributes from previousUser to newUser
+            }
+        }
+    });
+```
+
+Kotlin:
+
+```kotlin
+// Before
+val request = IdentityApiRequest.withEmptyUser()
+    .email("user@example.com")
+    .userAliasHandler { previousUser, newUser ->
+        // copy attributes from previousUser to newUser
+    }
+    .build()
+MParticle.getInstance()?.Identity()?.login(request)
+
+// After
+val request = IdentityApiRequest.withEmptyUser()
+    .email("user@example.com")
+    .build()
+MParticle.getInstance()?.Identity()?.login(request)
+    ?.addSuccessListener { result ->
+        val newUser = result.user
+        val previousUser = result.previousUser
+        // copy attributes from previousUser to newUser
+    }
+```
+
+---
+
+### Removed `ConsentState.Builder.setCCPAConsent()` / `removeCCPAConsent()`
+
+The `setCCPAConsent(CCPAConsent)` and `removeCCPAConsent()` methods on `ConsentState.Builder` have been removed. They have been renamed to `setCCPAConsentState(CCPAConsent)` and `removeCCPAConsentState()`.
+
+Java:
+
+```java
+// Before
+ConsentState state = ConsentState.builder()
+    .setCCPAConsent(CCPAConsent.builder(true).build())
+    .build();
+
+ConsentState.builder().removeCCPAConsent();
+
+// After
+ConsentState state = ConsentState.builder()
+    .setCCPAConsentState(CCPAConsent.builder(true).build())
+    .build();
+
+ConsentState.builder().removeCCPAConsentState();
+```
+
+Kotlin:
+
+```kotlin
+// Before
+val state = ConsentState.builder()
+    .setCCPAConsent(CCPAConsent.builder(true).build())
+    .build()
+
+ConsentState.builder().removeCCPAConsent()
+
+// After
+val state = ConsentState.builder()
+    .setCCPAConsentState(CCPAConsent.builder(true).build())
+    .build()
+
+ConsentState.builder().removeCCPAConsentState()
+```

android-core/src/androidTest/kotlin/com.mparticle/MPUserTest.kt

@@ -4,7 +4,6 @@ import com.mparticle.internal.AccessUtils
 import com.mparticle.testutils.BaseCleanStartedEachTest
 import org.junit.Test
 import kotlin.test.assertEquals
-import kotlin.test.assertNotNull
 import kotlin.test.assertTrue
 
 class MPUserTest : BaseCleanStartedEachTest() {
@@ -34,26 +33,6 @@ class MPUserTest : BaseCleanStartedEachTest() {
             this.userAttributes.put("fooNull", null)
             android_test_hack()
 
-            getUserAttributes(
-                object : UserAttributeListener {
-                    override fun onUserAttributesReceived(
-                        userAttributes: Map<String, String?>?,
-                        userAttributeLists: Map<String, List<String?>>?,
-                        mpid: Long?,
-                    ) {
-                        assertNotNull(userAttributes)
-                        assertEquals(6, userAttributes.size)
-                        assertEquals("bar", userAttributes["foo"])
-                        assertEquals("123", userAttributes["fooInt"])
-                        assertEquals("12345", userAttributes["fooLong"])
-                        assertEquals("10.15", userAttributes["fooDouble"])
-                        assertEquals("-10", userAttributes["fooNegInt"])
-                        assertEquals("-1010", userAttributes["fooNegLong"])
-                        assertEquals(null, userAttributes["fooNull"])
-                    }
-                },
-            )
-
             getUserAttributes(
                 object : TypedUserAttributeListener {
                     override fun onUserAttributesReceived(