From 9b644a361fb2b608c1059e5362952a262224c506 Mon Sep 17 00:00:00 2001
From: leila-gu <77660332+leila-gu@users.noreply.github.com>
Date: Wed, 18 May 2022 14:02:14 +0900
Subject: [PATCH] improve: update the content provider's init-order to high
 value (SDKCF-5292)

---
 README.md                              | 15 ++++++++-------
 sdk-utils/src/main/AndroidManifest.xml |  5 ++++-
 2 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/README.md b/README.md
index 5883c4a..fcdfa91 100644
--- a/README.md
+++ b/README.md
@@ -26,22 +26,20 @@ dependency {
 
 ### 2. Set initialization order (optional)
 
-You only need to set this if you are facing an Exception when trying to use features of this SDK at launch time.
-
-This SDK uses a `ContentProvider` in order to automatically initialize. If you are also using a ContentProvider in your App
-or you are developing an SDK which uses a ContentProvider to automatically initialize, then you may need to manually set the
+⚠️  This SDK uses a `ContentProvider` in order to automatically initialize. It is not recommended to override the default value set for `initOrder`. 
+However, you may need to set it if you are facing an Exception when trying to use features of this SDK at launch time. If you are also using a ContentProvider in your App or you are developing an SDK which uses a ContentProvider to automatically initialize, then you may need to manually set the
 `initOrder` for this SDK so that it is initialized first. This can be set in `AndroidManifest.xml`:
 
 ```xml
   <application>
-    <!-- The `initOrder` for this SDK is set to `99` by default. -->
-    <!-- Set it to something higher to make it initialize before your own ContentProvider. -->
+    <!-- The `initOrder` for this SDK is set to `9999` by default. -->
+    <!-- ⚠️ It is not recommended to override the `initOrder`, however, if you need to manually set it, make sure that it is initialized before your own ContentProvider. -->
     <provider
       tools:replace="android:initOrder"
       android:name="com.rakuten.tech.mobile.sdkutils.SdkUtilsInitProvider"
       android:authorities="${applicationId}.SdkUtilsInitProvider"
       android:exported="false"
-      android:initOrder="100" />
+      android:initOrder="9999" />
   </application>
 ```
 
@@ -227,6 +225,9 @@ val retrofit = Retrofit.Builder().build("your_baseUrl", okHttpClient gsonConvert
 
 ## Changelog
 
+### v1.2.0 (in progress)
+* SDKCF-5292: Set initOrder of the content provider to a high value to make sure that it is initialized before the host app ContentProvider.
+
 ### v1.1.0 (2022-03-17)
 
 * SDKCF-4887: Added NetworkUtil for checking network connectivity and capabilities. Please see [usage](#networking-utility) section for details.
diff --git a/sdk-utils/src/main/AndroidManifest.xml b/sdk-utils/src/main/AndroidManifest.xml
index 63dd096..a02f0a2 100644
--- a/sdk-utils/src/main/AndroidManifest.xml
+++ b/sdk-utils/src/main/AndroidManifest.xml
@@ -3,10 +3,13 @@
   xmlns:android="http://schemas.android.com/apk/res/android">
 
   <application>
+    <!-- It is not recommended to override the `initOrder`, however,
+    if you need to manually set it, make sure that it is initialized before your own
+    ContentProvider. -->
     <provider
       android:name="com.rakuten.tech.mobile.sdkutils.SdkUtilsInitProvider"
       android:authorities="${applicationId}.SdkUtilsInitProvider"
       android:exported="false"
-      android:initOrder="99" />
+      android:initOrder="9999" />
   </application>
 </manifest>
\ No newline at end of file