Remove interceptor and fix some errors related with shares

Author: abelgardep

owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/http/HttpClient.java

@@ -26,7 +26,6 @@
 
 import android.content.Context;
 
-import com.owncloud.android.lib.common.http.interceptors.HttpInterceptor;
 import com.owncloud.android.lib.common.network.AdvancedX509TrustManager;
 import com.owncloud.android.lib.common.network.NetworkUtils;
 import okhttp3.Cookie;
@@ -56,7 +55,6 @@
  */
 public class HttpClient {
     private static OkHttpClient sOkHttpClient;
-    private static HttpInterceptor sOkHttpInterceptor;
     private static Context sContext;
     private static HashMap<String, List<Cookie>> sCookieStore = new HashMap<>();
 
@@ -112,7 +110,6 @@ public List<Cookie> loadForRequest(HttpUrl url) {
                 };
 
                 OkHttpClient.Builder clientBuilder = new OkHttpClient.Builder()
-                        .addInterceptor(getOkHttpInterceptor())
                         .protocols(Arrays.asList(Protocol.HTTP_1_1))
                         .readTimeout(HttpConstants.DEFAULT_DATA_TIMEOUT, TimeUnit.MILLISECONDS)
                         .writeTimeout(HttpConstants.DEFAULT_DATA_TIMEOUT, TimeUnit.MILLISECONDS)
@@ -132,13 +129,6 @@ public List<Cookie> loadForRequest(HttpUrl url) {
         return sOkHttpClient;
     }
 
-    private static HttpInterceptor getOkHttpInterceptor() {
-        if (sOkHttpInterceptor == null) {
-            sOkHttpInterceptor = new HttpInterceptor();
-        }
-        return sOkHttpInterceptor;
-    }
-
     public Context getContext() {
         return sContext;
     }

owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/http/interceptors/HttpInterceptor.java

@@ -20,7 +20,7 @@ abstract class HttpBaseMethod constructor(url: URL) {
     var request: Request
     var requestBody: RequestBody? = null
     lateinit var response: Response
-    var responseBodyString: String? = null
+    private var responseBodyString: String? = null
 
     var call: Call? = null
 

owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/http/methods/HttpBaseMethod.kt

@@ -289,8 +289,7 @@ class OCDavResource(
     @Throws(IOException::class, HttpException::class, DavException::class)
     fun propfind(
         depth: Int,
-        vararg reqProp:
-        Property.Name,
+        vararg reqProp: Property.Name,
         listOfHeaders: HashMap<String, String?>?,
         callback: DavResponseCallback,
         rawCallback: (response: Response) -> Unit

owncloudComLibrary/src/main/java/com/owncloud/android/lib/common/http/methods/webdav/OCDavResource.kt

@@ -131,7 +131,7 @@ class CreateRemoteShareOperation(
                 parser.oneOrMoreSharesRequired = true
                 parser.ownCloudVersion = client.ownCloudVersion
                 parser.serverBaseUri = client.baseUri
-                result = parser.parse(postMethod.responseBodyString)
+                result = parser.parse(postMethod.getResponseBodyAsString())
 
                 if (result.isSuccess && retrieveShareDetails) {
                     // retrieve more info - POST only returns the index of the new share
@@ -143,7 +143,7 @@ class CreateRemoteShareOperation(
                 }
 
             } else {
-                result = parser.parse(postMethod.responseBodyString)
+                result = parser.parse(postMethod.getResponseBodyAsString())
             }
 
         } catch (e: Exception) {

owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/shares/CreateRemoteShareOperation.kt

@@ -98,7 +98,7 @@ class GetRemoteShareesOperation
             getMethod.addRequestHeader(OCS_API_HEADER, OCS_API_HEADER_VALUE)
 
             val status = client.executeHttpMethod(getMethod)
-            val response = getMethod.responseBodyString
+            val response = getMethod.getResponseBodyAsString()
 
             if (isSuccess(status)) {
                 Timber.d("Successful response: $response")

owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/shares/GetRemoteShareesOperation.kt

@@ -86,7 +86,7 @@ class GetRemoteSharesForFileOperation(
                 )
                 parser.ownCloudVersion = client.ownCloudVersion
                 parser.serverBaseUri = client.baseUri
-                result = parser.parse(getMethod.responseBodyString)
+                result = parser.parse(getMethod.getResponseBodyAsString())
 
                 if (result.isSuccess) {
                     Timber.d("Got ${result.data.shares.size} shares")

owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/shares/GetRemoteSharesForFileOperation.kt

@@ -73,7 +73,7 @@ class RemoveRemoteShareOperation(private val remoteShareId: Long) : RemoteOperat
                 val parser = ShareToRemoteOperationResultParser(
                     ShareXMLParser()
                 )
-                result = parser.parse(deleteMethod.responseBodyString)
+                result = parser.parse(deleteMethod.getResponseBodyAsString())
 
                 Timber.d("Unshare $remoteShareId: ${result.logMessage}")