Change level on important logs (#472)

hborisoff · web-flow · commit c9e97f94e968 · 2021-09-27T17:31:11.000+03:00
diff --git a/AndroidSDKCore/src/main/java/com/leanplum/Leanplum.java b/AndroidSDKCore/src/main/java/com/leanplum/Leanplum.java
@@ -751,7 +751,7 @@ public void response(JSONObject response) {
     request.onError(new Request.ErrorCallback() {
       @Override
       public void error(Exception e) {
-        Log.d("Failed to receive start response");
+        Log.i("Failed to receive start response");
         handleStartResponse(null);
       }
     });
@@ -1685,7 +1685,7 @@ public static void trackPurchase(final String event, double value, String curren
       Map<String, ?> params) {
     try {
       if (TextUtils.isEmpty(event)) {
-        Log.d("Failed to trackPurchase, event name is null");
+        Log.i("Failed to trackPurchase, event name is null");
       }
 
       final Map<String, String> requestArgs = new HashMap<>();
@@ -1746,7 +1746,7 @@ public static void trackGooglePlayPurchase(String item, long priceMicros, String
   public static void trackGooglePlayPurchase(String eventName, String item, long priceMicros,
       String currencyCode, String purchaseData, String dataSignature, Map<String, ?> params) {
     if (TextUtils.isEmpty(eventName)) {
-      Log.d("Failed to trackGooglePlayPurchase, event name is null");
+      Log.i("Failed to trackGooglePlayPurchase, event name is null");
     }
 
     final Map<String, String> requestArgs = new HashMap<>();
@@ -2137,7 +2137,7 @@ public static void setIsTestModeEnabled(boolean isTestModeEnabled) {
    */
   public static String pathForResource(String filename) {
     if (TextUtils.isEmpty(filename)) {
-      Log.d("pathForResource - Empty filename parameter provided.");
+      Log.i("pathForResource - Empty filename parameter provided.");
       return null;
     }
 
diff --git a/AndroidSDKCore/src/main/java/com/leanplum/LeanplumInboxMessage.java b/AndroidSDKCore/src/main/java/com/leanplum/LeanplumInboxMessage.java
@@ -82,7 +82,7 @@ public String getImageFilePath() {
       return new File(path).getAbsolutePath();
     }
     if (!LeanplumInbox.getInstance().isInboxImagePrefetchingEnabled()) {
-      Log.d("Inbox Message image path is null because you're calling disableImagePrefetching. " +
+      Log.i("Inbox Message image path is null because you're calling disableImagePrefetching. " +
           "Consider using imageURL method or remove disableImagePrefetching.");
     }
     return null;
@@ -115,7 +115,7 @@ public JSONObject getData() {
           CollectionUtil.uncheckedCast(getContext().objectNamed(Constants.Keys.DATA));
       object = JsonConverter.mapToJsonObject(mapData);
     } catch (Throwable t) {
-      Log.d("Unable to parse JSONObject for Data field of inbox message.");
+      Log.i("Unable to parse JSONObject for Data field of inbox message.");
     }
     return object;
   }
diff --git a/AndroidSDKCore/src/main/java/com/leanplum/internal/ActionManager.java b/AndroidSDKCore/src/main/java/com/leanplum/internal/ActionManager.java
@@ -172,7 +172,7 @@ public boolean onResponse(ActionContext actionContext) {
                 .invoke(new Object(), context, messageId);
             boolean didCancel = existingEta > Clock.getInstance().currentTimeMillis();
             if (didCancel) {
-              Log.d("Cancelled notification");
+              Log.i("Cancelled notification");
             }
             return didCancel;
           } catch (Throwable throwable) {
diff --git a/AndroidSDKFcm/src/main/java/com/leanplum/LeanplumFirebaseServiceHandler.java b/AndroidSDKFcm/src/main/java/com/leanplum/LeanplumFirebaseServiceHandler.java
@@ -71,7 +71,7 @@ public void onMessageReceived(RemoteMessage remoteMessage, Context context) {
         notification.putString(Keys.CHANNEL_INTERNAL_KEY, channel);
         LeanplumPushService.handleNotification(context, notification);
       }
-      Log.d("Received push notification message: %s", messageMap.toString());
+      Log.i("Received push notification message: %s", messageMap.toString());
     } catch (Throwable t) {
       Log.exception(t);
     }
diff --git a/AndroidSDKMiPush/src/main/java/com/leanplum/LeanplumMiPushHandler.java b/AndroidSDKMiPush/src/main/java/com/leanplum/LeanplumMiPushHandler.java
@@ -60,7 +60,7 @@ public void onReceivePassThroughMessage(Context context, MiPushMessage message)
     if (context == null || message == null)
       return;
 
-    Log.d("Received MiPush data message %s: %s", message.getMessageId(), getContentLog(message));
+    Log.i("Received MiPush data message %s: %s", message.getMessageId(), getContentLog(message));
   }
 
   /**
@@ -71,7 +71,7 @@ public void onNotificationMessageClicked(Context context, MiPushMessage message)
       return;
     }
 
-    Log.d("MiPush notification clicked %s: %s", message.getMessageId(), getContentLog(message));
+    Log.i("MiPush notification clicked %s: %s", message.getMessageId(), getContentLog(message));
 
     try {
       Map<String, String> messageMap = parsePayload(message.getContent());
@@ -95,7 +95,7 @@ public void onNotificationMessageArrived(Context context, MiPushMessage message)
     if (message == null)
       return;
 
-    Log.d("Received MiPush notification message %s: %s",
+    Log.i("Received MiPush notification message %s: %s",
         message.getMessageId(), getContentLog(message));
 
     try {
diff --git a/AndroidSDKPush/src/main/java/com/leanplum/LeanplumCloudMessagingProvider.java b/AndroidSDKPush/src/main/java/com/leanplum/LeanplumCloudMessagingProvider.java
@@ -49,7 +49,7 @@ public void setRegistrationId(String registrationId) {
       return;
     }
 
-    Log.d("Registering for %s push notifications with ID %s", getType(), registrationId);
+    Log.i("Registering for %s push notifications with ID %s", getType(), registrationId);
 
     if (!registrationId.equals(getRegistrationId())) {
       storeRegistrationId(registrationId);
diff --git a/AndroidSDKPush/src/main/java/com/leanplum/LeanplumNotificationHelper.java b/AndroidSDKPush/src/main/java/com/leanplum/LeanplumNotificationHelper.java
@@ -502,7 +502,7 @@ static Bitmap getBigPictureBitmap(Context context, String imageUrl) {
     if (!TextUtils.isEmpty(imageUrl) && Build.VERSION.SDK_INT >= 16) {
       bigPicture = BitmapUtil.getScaledBitmap(context, imageUrl);
       if (bigPicture == null) {
-        Log.d("Failed to download image for push notification: %s", imageUrl);
+        Log.i("Failed to download image for push notification: %s", imageUrl);
       }
     }
     return bigPicture;
diff --git a/AndroidSDKPush/src/main/java/com/leanplum/LeanplumPushService.java b/AndroidSDKPush/src/main/java/com/leanplum/LeanplumPushService.java
@@ -497,7 +497,7 @@ public static Map<String, Object> parseNotificationBundle(Bundle notificationBun
 
       return arguments;
     } catch (Throwable ignored) {
-      Log.d("Failed to parse notification bundle.");
+      Log.i("Failed to parse notification bundle.");
     }
     return null;
   }
@@ -512,12 +512,12 @@ public static Map<String, Object> parseNotificationBundle(Bundle notificationBun
    */
   public static Bundle preHandlePushNotification(Context context, Intent intent) {
     if (intent == null) {
-      Log.d("Unable to pre handle push notification, Intent is null.");
+      Log.i("Unable to pre handle push notification, Intent is null.");
       return null;
     }
     Bundle notification = intent.getExtras();
     if (notification == null) {
-      Log.d("Unable to pre handle push notification, extras are null.");
+      Log.i("Unable to pre handle push notification, extras are null.");
       return null;
     }
     return notification;
diff --git a/AndroidSDKPush/src/main/java/com/leanplum/internal/LeanplumLocalPushHelper.java b/AndroidSDKPush/src/main/java/com/leanplum/internal/LeanplumLocalPushHelper.java
@@ -138,7 +138,7 @@ static boolean scheduleLocalPush(ActionContext actionContext, String messageId,
       editor.putLong(String.format(Constants.Defaults.LOCAL_NOTIFICATION_KEY, messageId), eta);
       SharedPreferencesUtil.commitChanges(editor);
 
-      Log.d("Scheduling local notification.");
+      Log.i("Scheduling local notification.");
       return true;
     } catch (Throwable t) {
       Log.exception(t);
diff --git a/AndroidSDKPush/src/main/java/com/leanplum/internal/LeanplumManifestHelper.java b/AndroidSDKPush/src/main/java/com/leanplum/internal/LeanplumManifestHelper.java
@@ -68,7 +68,7 @@ public static boolean enableServiceAndStart(Context context, PackageManager pack
     try {
       context.startService(new Intent(context, clazz));
     } catch (Throwable t) {
-      Log.d("Could not start service for class: %s", clazz.getName());
+      Log.i("Could not start service for class: %s", clazz.getName());
       return false;
     }
     return true;
@@ -92,7 +92,7 @@ public static boolean enableComponent(Context context, PackageManager packageMan
       packageManager.setComponentEnabledSetting(new ComponentName(context, clazz),
           PackageManager.COMPONENT_ENABLED_STATE_ENABLED, PackageManager.DONT_KILL_APP);
     } catch (Throwable t) {
-      Log.d("Could not enable component: %s", clazz.getName());
+      Log.i("Could not enable component: %s", clazz.getName());
       return false;
     }
     return true;

Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ public String getImageFilePath() {`
`82`	`82`	`return new File(path).getAbsolutePath();`
`83`	`83`	`}`
`84`	`84`	`if (!LeanplumInbox.getInstance().isInboxImagePrefetchingEnabled()) {`
`85`		`- Log.d("Inbox Message image path is null because you're calling disableImagePrefetching. " +`
	`85`	`+ Log.i("Inbox Message image path is null because you're calling disableImagePrefetching. " +`
`86`	`86`	`"Consider using imageURL method or remove disableImagePrefetching.");`
`87`	`87`	`}`
`88`	`88`	`return null;`
`@@ -115,7 +115,7 @@ public JSONObject getData() {`
`115`	`115`	`CollectionUtil.uncheckedCast(getContext().objectNamed(Constants.Keys.DATA));`
`116`	`116`	`object = JsonConverter.mapToJsonObject(mapData);`
`117`	`117`	`} catch (Throwable t) {`
`118`		`- Log.d("Unable to parse JSONObject for Data field of inbox message.");`
	`118`	`+ Log.i("Unable to parse JSONObject for Data field of inbox message.");`
`119`	`119`	`}`
`120`	`120`	`return object;`
`121`	`121`	`}`
Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ public boolean onResponse(ActionContext actionContext) {`
`172`	`172`	`.invoke(new Object(), context, messageId);`
`173`	`173`	`boolean didCancel = existingEta > Clock.getInstance().currentTimeMillis();`
`174`	`174`	`if (didCancel) {`
`175`		`- Log.d("Cancelled notification");`
	`175`	`+ Log.i("Cancelled notification");`
`176`	`176`	`}`
`177`	`177`	`return didCancel;`
`178`	`178`	`} catch (Throwable throwable) {`
Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ public void onMessageReceived(RemoteMessage remoteMessage, Context context) {`
`71`	`71`	`notification.putString(Keys.CHANNEL_INTERNAL_KEY, channel);`
`72`	`72`	`LeanplumPushService.handleNotification(context, notification);`
`73`	`73`	`}`
`74`		`- Log.d("Received push notification message: %s", messageMap.toString());`
	`74`	`+ Log.i("Received push notification message: %s", messageMap.toString());`
`75`	`75`	`} catch (Throwable t) {`
`76`	`76`	`Log.exception(t);`
`77`	`77`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public void setRegistrationId(String registrationId) {`
`49`	`49`	`return;`
`50`	`50`	`}`
`51`	`51`
`52`		`- Log.d("Registering for %s push notifications with ID %s", getType(), registrationId);`
	`52`	`+ Log.i("Registering for %s push notifications with ID %s", getType(), registrationId);`
`53`	`53`
`54`	`54`	`if (!registrationId.equals(getRegistrationId())) {`
`55`	`55`	`storeRegistrationId(registrationId);`
Original file line number	Diff line number	Diff line change
`@@ -502,7 +502,7 @@ static Bitmap getBigPictureBitmap(Context context, String imageUrl) {`
`502`	`502`	`if (!TextUtils.isEmpty(imageUrl) && Build.VERSION.SDK_INT >= 16) {`
`503`	`503`	`bigPicture = BitmapUtil.getScaledBitmap(context, imageUrl);`
`504`	`504`	`if (bigPicture == null) {`
`505`		`- Log.d("Failed to download image for push notification: %s", imageUrl);`
	`505`	`+ Log.i("Failed to download image for push notification: %s", imageUrl);`
`506`	`506`	`}`
`507`	`507`	`}`
`508`	`508`	`return bigPicture;`
Original file line number	Diff line number	Diff line change
`@@ -497,7 +497,7 @@ public static Map<String, Object> parseNotificationBundle(Bundle notificationBun`
`497`	`497`
`498`	`498`	`return arguments;`
`499`	`499`	`} catch (Throwable ignored) {`
`500`		`- Log.d("Failed to parse notification bundle.");`
	`500`	`+ Log.i("Failed to parse notification bundle.");`
`501`	`501`	`}`
`502`	`502`	`return null;`
`503`	`503`	`}`
`@@ -512,12 +512,12 @@ public static Map<String, Object> parseNotificationBundle(Bundle notificationBun`
`512`	`512`	`*/`
`513`	`513`	`public static Bundle preHandlePushNotification(Context context, Intent intent) {`
`514`	`514`	`if (intent == null) {`
`515`		`- Log.d("Unable to pre handle push notification, Intent is null.");`
	`515`	`+ Log.i("Unable to pre handle push notification, Intent is null.");`
`516`	`516`	`return null;`
`517`	`517`	`}`
`518`	`518`	`Bundle notification = intent.getExtras();`
`519`	`519`	`if (notification == null) {`
`520`		`- Log.d("Unable to pre handle push notification, extras are null.");`
	`520`	`+ Log.i("Unable to pre handle push notification, extras are null.");`
`521`	`521`	`return null;`
`522`	`522`	`}`
`523`	`523`	`return notification;`