Merge pull request #848 from Iterable/MOB-10410-TargetSDK31

[MOB-10410] - Making SDK Target SDK 31 compliant
Iterable · Dec 12, 2024 · 8fdec83 · 8fdec83
2 parents c80a471 + 3aa3e19
commit 8fdec83
Show file tree

Hide file tree

Showing 11 changed files with 65 additions and 6 deletions.
diff --git a/app/build.gradle b/app/build.gradle
@@ -17,7 +17,7 @@ android {
     defaultConfig {
         applicationId "com.iterable.iterableapi.testapp"
         minSdkVersion 16
-        targetSdkVersion 27
+        targetSdkVersion 31
         versionCode 1
         versionName "1.0"
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -10,12 +10,23 @@
         android:theme="@style/AppTheme">
         <activity
             android:name="com.iterable.androidsdk.MainActivity"
+            android:exported="true"
             android:label="@string/app_name"
             android:theme="@style/AppTheme.NoActionBar">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
+
                 <category android:name="android.intent.category.LAUNCHER" />
             </intent-filter>
         </activity>
+        <activity
+            android:name="androidx.test.core.app.InstrumentationActivityInvoker$BootstrapActivity"
+            android:exported="true" />
+        <activity
+            android:name="androidx.test.core.app.InstrumentationActivityInvoker$EmptyActivity"
+            android:exported="true" />
+        <activity
+            android:name="androidx.test.core.app.InstrumentationActivityInvoker$EmptyFloatingActivity"
+            android:exported="true" />
     </application>
 </manifest>
diff --git a/iterableapi-ui/build.gradle b/iterableapi-ui/build.gradle
@@ -8,7 +8,7 @@ android {
 
     defaultConfig {
         minSdkVersion 16
-        targetSdkVersion 28
+        targetSdkVersion 31
         vectorDrawables.useSupportLibrary = true
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
     }

diff --git a/iterableapi-ui/src/main/res/layout-v21/banner_view.xml b/iterableapi-ui/src/main/res/layout-v21/banner_view.xml
@@ -7,6 +7,7 @@
     android:layout_gravity="center_vertical"
     android:layout_marginTop="8dp"
     android:layout_marginStart="16dp"
+    android:layout_marginLeft="16dp"
     android:layout_marginBottom="12dp"
     app:cardCornerRadius="8dp"
     app:cardElevation="0dp"
@@ -18,6 +19,7 @@
         android:layout_width="240dp"
         android:layout_height="wrap_content"
         android:layout_marginStart="16dp"
+        android:layout_marginLeft="16dp"
         android:layout_marginTop="16dp"
         android:ellipsize="end"
         android:maxLines="2"
@@ -34,6 +36,7 @@
         android:layout_width="240dp"
         android:layout_height="wrap_content"
         android:layout_marginStart="16dp"
+        android:layout_marginLeft="16dp"
         android:layout_marginTop="8dp"
         android:ellipsize="end"
         android:maxLines="2"
@@ -74,6 +77,8 @@
             android:ellipsize="end"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="8dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="8dp"
             android:paddingTop="0dp"
             android:paddingBottom="0dp"
             android:paddingStart="12dp"

diff --git a/iterableapi-ui/src/main/res/layout-v21/card_view.xml b/iterableapi-ui/src/main/res/layout-v21/card_view.xml
@@ -40,6 +40,8 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="16dp"
             android:layout_marginTop="16dp"
             android:ellipsize="end"
             android:maxLines="2"
@@ -56,6 +58,8 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="16dp"
             android:layout_marginTop="8dp"
             android:ellipsize="end"
             android:maxLines="2"
@@ -85,6 +89,8 @@
             android:ellipsize="end"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="8dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="8dp"
             android:paddingTop="0dp"
             android:paddingBottom="0dp"
             android:paddingStart="12dp"

diff --git a/iterableapi-ui/src/main/res/layout/banner_view.xml b/iterableapi-ui/src/main/res/layout/banner_view.xml
@@ -7,6 +7,7 @@
     android:layout_gravity="center_vertical"
     android:layout_marginTop="8dp"
     android:layout_marginStart="16dp"
+    android:layout_marginLeft="16dp"
     android:layout_marginBottom="12dp"
     app:cardCornerRadius="8dp"
     app:cardElevation="0dp"
@@ -18,6 +19,7 @@
         android:layout_width="240dp"
         android:layout_height="wrap_content"
         android:layout_marginStart="16dp"
+        android:layout_marginLeft="16dp"
         android:layout_marginTop="16dp"
         android:ellipsize="end"
         android:maxLines="2"
@@ -34,6 +36,7 @@
         android:layout_width="240dp"
         android:layout_height="wrap_content"
         android:layout_marginStart="16dp"
+        android:layout_marginLeft="16dp"
         android:layout_marginTop="8dp"
         android:ellipsize="end"
         android:maxLines="2"
@@ -51,6 +54,7 @@
         android:layout_height="80dp"
         android:layout_marginTop="16dp"
         android:layout_marginEnd="16dp"
+        android:layout_marginRight="16dp"
         android:contentDescription=""
         app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintTop_toTopOf="parent" />
@@ -72,6 +76,7 @@
             android:background="@drawable/primary_banner_button_background"
             android:ellipsize="end"
             android:layout_marginStart="16dp"
+            android:layout_marginLeft="16dp"
             android:paddingTop="0dp"
             android:paddingBottom="0dp"
             android:paddingStart="12dp"
@@ -88,6 +93,7 @@
             android:layout_width="wrap_content"
             android:layout_height="36dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginRight="16dp"
             android:background="@drawable/secondary_banner_button_background"
             android:ellipsize="end"
             android:paddingTop="0dp"

diff --git a/iterableapi-ui/src/main/res/layout/card_view.xml b/iterableapi-ui/src/main/res/layout/card_view.xml
@@ -1,10 +1,11 @@
 <?xml version="1.0" encoding="utf-8"?>
-<androidx.constraintlayout.widget.ConstraintLayout
-    xmlns:android="http://schemas.android.com/apk/res/android"
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
+    xmlns:tools="http://schemas.android.com/tools"
     android:layout_gravity="center_vertical"
+    android:layout_marginStart="16dp"
     android:layout_marginTop="8dp"
     android:layout_marginLeft="16dp"
     android:layout_marginBottom="12dp"
@@ -40,6 +41,8 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="16dp"
             android:layout_marginTop="16dp"
             android:ellipsize="end"
             android:maxLines="2"
@@ -56,6 +59,8 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="16dp"
             android:layout_marginTop="8dp"
             android:ellipsize="end"
             android:maxLines="2"
@@ -82,6 +87,8 @@
             android:layout_height="36dp"
             android:layout_marginStart="16dp"
             android:layout_marginEnd="8dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="8dp"
             android:background="@drawable/primary_card_button_background"
             android:ellipsize="end"
             android:paddingStart="8dp"
@@ -92,13 +99,16 @@
             android:stateListAnimator="@null"
             android:textAllCaps="false"
             android:textColor="@color/white"
-            app:cornerRadius="32dp" />
+            app:cornerRadius="32dp"
+            tools:ignore="Suspicious0dp"/>
 
         <Button
             android:id="@+id/embedded_message_second_button"
             android:layout_width="0dp"
             android:layout_height="36dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginLeft="16dp"
+            android:layout_marginRight="16dp"
             android:paddingTop="0dp"
             android:paddingBottom="0dp"
             android:layout_weight="1"

diff --git a/iterableapi-ui/src/main/res/layout/notification_view.xml b/iterableapi-ui/src/main/res/layout/notification_view.xml
@@ -8,6 +8,7 @@
     android:layout_marginTop="8dp"
     android:layout_marginBottom="12dp"
     android:layout_marginStart="16dp"
+    android:layout_marginLeft="16dp"
     app:cardCornerRadius="8dp"
     app:cardElevation="0dp"
     android:background="@drawable/notification_view_border"
@@ -18,6 +19,8 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_marginStart="16dp"
+        android:layout_marginLeft="16dp"
+        android:layout_marginRight="16dp"
         android:layout_marginTop="16dp"
         android:layout_marginEnd="16dp"
         android:ellipsize="end"
@@ -36,6 +39,8 @@
         android:layout_marginStart="16dp"
         android:layout_marginTop="8dp"
         android:layout_marginEnd="16dp"
+        android:layout_marginLeft="16dp"
+        android:layout_marginRight="16dp"
         android:ellipsize="end"
         android:maxLines="2"
         android:text=""
@@ -62,6 +67,7 @@
             android:layout_width="wrap_content"
             android:layout_height="36dp"
             android:layout_marginStart="16dp"
+            android:layout_marginLeft="16dp"
             android:textColor="@color/notification_text_color"
             android:background="@drawable/primary_notification_button_background"
             android:ellipsize="end"
@@ -80,6 +86,7 @@
             android:layout_width="wrap_content"
             android:layout_height="36dp"
             android:layout_marginEnd="16dp"
+            android:layout_marginRight="16dp"
             android:textColor="@color/notification_text_color"
             android:background="@drawable/secondary_notification_button_background"
             android:ellipsize="end"

diff --git a/iterableapi/build.gradle b/iterableapi/build.gradle
@@ -17,7 +17,7 @@ android {
 
     defaultConfig {
         minSdkVersion 16
-        targetSdkVersion 27
+        targetSdkVersion 31
 
         buildConfigField "String", "ITERABLE_SDK_VERSION", "\"3.5.4\""
 

diff --git a/iterableapi/src/main/AndroidManifest.xml b/iterableapi/src/main/AndroidManifest.xml
@@ -2,6 +2,8 @@
     xmlns:tools="http://schemas.android.com/tools">
 
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
+    <uses-permission android:name="android.permission.BROADCAST_CLOSE_SYSTEM_DIALOGS"
+        tools:ignore="ProtectedPermissions" />
 
     <application>
         <!--FCM-->
@@ -29,6 +31,16 @@
             android:excludeFromRecents="true"
             android:theme="@style/TrampolineActivity.Transparent"/>
 
+        <activity
+            android:name="androidx.test.core.app.InstrumentationActivityInvoker$BootstrapActivity"
+            android:exported="true" />
+        <activity
+            android:name="androidx.test.core.app.InstrumentationActivityInvoker$EmptyActivity"
+            android:exported="true" />
+        <activity
+            android:name="androidx.test.core.app.InstrumentationActivityInvoker$EmptyFloatingActivity"
+            android:exported="true" />
+
         <uses-library android:name="androidx.security" android:required="false" />
     </application>
 

diff --git a/iterableapi/src/main/java/com/iterable/iterableapi/IterableTaskStorage.java b/iterableapi/src/main/java/com/iterable/iterableapi/IterableTaskStorage.java
@@ -1,5 +1,6 @@
 package com.iterable.iterableapi;
 
+import android.annotation.SuppressLint;
 import android.content.ContentValues;
 import android.content.Context;
 import android.database.Cursor;
@@ -184,6 +185,7 @@ IterableTask getTask(String id) {
         return task;
     }
 
+    @SuppressLint("Range")
     private IterableTask createTaskFromCursor(Cursor cursor) {
         String id, name;
         IterableTaskType type = null;