Fixed LeakCanary flavor lint errors

smarek · smarek · commit d81e442a3bdb · 2015-07-12T15:03:49.000+02:00
diff --git a/sample/build.gradle b/sample/build.gradle
@@ -35,7 +35,7 @@ android {
         warningsAsErrors true
         quiet false
         showAll true
-        disable 'OldTargetApi', 'LongLogTag'
+        disable 'OldTargetApi', 'UnusedAttribute', 'LongLogTag'
     }
 
     packagingOptions {
diff --git a/sample/src/main/AndroidManifest.xml b/sample/src/main/AndroidManifest.xml
@@ -9,6 +9,7 @@
 
     <application
         android:name=".SampleApplication"
+        android:allowBackup="true"
         android:icon="@drawable/ic_launcher"
         android:label="@string/app_name"
         android:theme="@style/AppTheme">
diff --git a/sample/src/main/java/com/loopj/android/http/sample/JsonStreamerSample.java b/sample/src/main/java/com/loopj/android/http/sample/JsonStreamerSample.java
@@ -18,6 +18,7 @@
 
 package com.loopj.android.http.sample;
 
+import android.text.TextUtils;
 import android.util.Log;
 
 import com.loopj.android.http.AsyncHttpClient;
@@ -87,7 +88,7 @@ public boolean isRequestHeadersAllowed() {
 
     protected JSONObject getBodyTextAsJSON() {
         String bodyText = getBodyText();
-        if (bodyText != null && !bodyText.isEmpty()) {
+        if (bodyText != null && TextUtils.isEmpty(bodyText)) {
             try {
                 return new JSONObject(bodyText);
             } catch (JSONException e) {
diff --git a/sample/src/withLeakCanary/java/com/loopj/android/http/sample/SampleApplication.java b/sample/src/withLeakCanary/java/com/loopj/android/http/sample/SampleApplication.java

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ android {`
`35`	`35`	`warningsAsErrors true`
`36`	`36`	`quiet false`
`37`	`37`	`showAll true`
`38`		`- disable 'OldTargetApi', 'LongLogTag'`
	`38`	`+ disable 'OldTargetApi', 'UnusedAttribute', 'LongLogTag'`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`packagingOptions {`