Skip to content

Commit 95afd52

Browse files
author
Ricardo Moutinho
committed
[upd] prevent release builds from printing social logs
1 parent 0e3d03a commit 95afd52

File tree

1 file changed

+12
-5
lines changed
  • oneallsdk/src/main/java/com/oneall/oneallsdk

1 file changed

+12
-5
lines changed

oneallsdk/src/main/java/com/oneall/oneallsdk/OALog.java

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
package com.oneall.oneallsdk;
22

3+
import com.logentries.android.AndroidLogger;
4+
35
import android.content.Context;
46
import android.util.Log;
57

6-
import com.logentries.android.AndroidLogger;
7-
88
/**
99
* Wrapper class for error logger, able to handle multiple types of logs: LogEntries, regular
1010
* Android log. The class is used internally by OneAll SDK and should not be used by the external
@@ -15,6 +15,7 @@ class OALog {
1515
// region Constants
1616

1717
private final static String TAG = "oneall";
18+
private final static boolean PRINT_LOGS = BuildConfig.DEBUG;
1819

1920
// endregion
2021

@@ -80,7 +81,9 @@ public static OALog getInstance() {
8081
* @param logMessage message to post
8182
*/
8283
public static void info(String logMessage) {
83-
Log.i(TAG, logMessage);
84+
if (PRINT_LOGS) {
85+
Log.i(TAG, logMessage);
86+
}
8487
if (getInstance().logger != null) {
8588
getInstance().logger.info(logMessage);
8689
}
@@ -92,7 +95,9 @@ public static void info(String logMessage) {
9295
* @param logMessage message to post
9396
*/
9497
public static void warn(String logMessage) {
95-
Log.w(TAG, logMessage);
98+
if (PRINT_LOGS) {
99+
Log.w(TAG, logMessage);
100+
}
96101
if (getInstance().logger != null) {
97102
getInstance().logger.warn(logMessage);
98103
}
@@ -104,7 +109,9 @@ public static void warn(String logMessage) {
104109
* @param logMessage message to post
105110
*/
106111
public static void error(String logMessage) {
107-
Log.e(TAG, logMessage);
112+
if(PRINT_LOGS) {
113+
Log.e(TAG, logMessage);
114+
}
108115
if (getInstance().logger != null) {
109116
getInstance().logger.error(logMessage);
110117
}

0 commit comments

Comments
 (0)