From 3aa3e19d8579733aae5ed5183252122e222ac4aa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9CAkshay?= <“ayyanchira.akshay@gmail.com”>
Date: Thu, 12 Dec 2024 14:42:27 -0800
Subject: [PATCH] [MOB-10410] - Making SDK Target SDK 31 compliant

Only bare minimum changes added
---
 app/build.gradle                                 |  2 +-
 app/src/main/AndroidManifest.xml                 | 11 +++++++++++
 iterableapi-ui/build.gradle                      |  2 +-
 .../src/main/res/layout-v21/banner_view.xml      |  5 +++++
 .../src/main/res/layout-v21/card_view.xml        |  6 ++++++
 .../src/main/res/layout/banner_view.xml          |  6 ++++++
 iterableapi-ui/src/main/res/layout/card_view.xml | 16 +++++++++++++---
 .../src/main/res/layout/notification_view.xml    |  7 +++++++
 iterableapi/build.gradle                         |  2 +-
 iterableapi/src/main/AndroidManifest.xml         | 12 ++++++++++++
 .../iterableapi/IterableTaskStorage.java         |  2 ++
 11 files changed, 65 insertions(+), 6 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index 083e1ecef..899da28b3 100644
--- 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
index d59b671b6..a3609abf3 100644
--- 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>
\ No newline at end of file
diff --git a/iterableapi-ui/build.gradle b/iterableapi-ui/build.gradle
index 7ea701b66..0721ee026 100644
--- 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
index bd29e47a2..9f5fff581 100644
--- 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
index a3abcd64f..ad8ddcf28 100644
--- 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
index 9cc691d7b..2988c4f28 100644
--- 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
index ec1d54049..479dfcbc2 100644
--- 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
index 8757e3aea..081512931 100644
--- 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
index aaf669b3b..98003f97b 100644
--- 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
index 65cedb161..1fa8bec96 100644
--- 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
index e9ae5f7b7..0cce48b73 100644
--- 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;