diff --git a/owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/users/GetRemoteUserQuotaOperation.kt b/owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/users/GetRemoteUserQuotaOperation.kt index 11c60a03..e2e00230 100644 --- a/owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/users/GetRemoteUserQuotaOperation.kt +++ b/owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/users/GetRemoteUserQuotaOperation.kt @@ -47,32 +47,31 @@ import kotlin.math.roundToLong * @author David González Verdugo */ class GetRemoteUserQuotaOperation : RemoteOperation() { - override fun run(client: OwnCloudClient): RemoteOperationResult { - lateinit var result: RemoteOperationResult + override fun run(client: OwnCloudClient): RemoteOperationResult = try { val propfindMethod = PropfindMethod( URL(client.userFilesWebDavUri.toString()), DavConstants.DEPTH_0, DavUtils.getQuotaPropSet() ) - val status = client.executeHttpMethod(propfindMethod) - if (isSuccess(status)) { - val remoteQuota = readData(propfindMethod.root.properties) - result = RemoteOperationResult(ResultCode.OK).apply { - data = remoteQuota + with(client.executeHttpMethod(propfindMethod)) { + if (isSuccess(this)) { + RemoteOperationResult(ResultCode.OK).apply { + data = readData(propfindMethod.root.properties) + }.also { + Timber.i("Get quota completed: ${it.data} and message: ${it.logMessage}") + } + } else { // synchronization failed + RemoteOperationResult(propfindMethod).also { + Timber.e("Get quota without success: ${it.logMessage}") + } } - Timber.i("Get quota completed: ${result.data} and message: ${result.logMessage}") - - } else { // synchronization failed - result = RemoteOperationResult(propfindMethod) - Timber.e("Get quota without success: ${result.logMessage}") } } catch (e: Exception) { - result = RemoteOperationResult(e) - Timber.e(result.exception, "Get quota: ${result.logMessage}") + RemoteOperationResult(e).also { + Timber.e(it.exception, "Get quota: ${it.logMessage}") + } } - return result - } private fun isSuccess(status: Int) = status == HttpConstants.HTTP_MULTI_STATUS || status == HttpConstants.HTTP_OK