flywithu/android-library
1
0
Fork 0
mirror of https://github.com/owncloud/android-library.git synced 2025-10-31 02:17:41 +00:00
Code Issues Releases Wiki Activity

Apply code review suggestions

Browse Source
This commit is contained in:
agarcia 2020-08-05 14:00:48 +02:00 committed by Abel García de Prada
parent cd94e39b7f
commit 5e478036ae
2 changed files with 30 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/files/CreateRemoteFolderOperation.kt
View File

@ -56,6 +56,7 @@ class CreateRemoteFolderOperation(
result = createParentFolder(FileUtils.getParentPath(remotePath), client) result = createParentFolder(FileUtils.getParentPath(remotePath), client)
if (result.isSuccess) { if (result.isSuccess) {
// Second and last try
result = createFolder(client) result = createFolder(client)
} }
} }
@ -65,25 +66,32 @@ class CreateRemoteFolderOperation(
private fun createFolder(client: OwnCloudClient): RemoteOperationResult<Unit> { private fun createFolder(client: OwnCloudClient): RemoteOperationResult<Unit> {
var result: RemoteOperationResult<Unit> var result: RemoteOperationResult<Unit>
try { try {
val webDavUri = if (isChunksFolder) client.uploadsWebDavUri else client.userFilesWebDavUri val webDavUri = if (isChunksFolder) {
val mkcol = MkColMethod(URL(webDavUri.toString() + WebdavUtils.encodePath(remotePath))).apply { client.uploadsWebDavUri
} else {
client.userFilesWebDavUri
}
val mkCol = MkColMethod(
URL(webDavUri.toString() + WebdavUtils.encodePath(remotePath))
).apply {
setReadTimeout(READ_TIMEOUT, TimeUnit.SECONDS) setReadTimeout(READ_TIMEOUT, TimeUnit.SECONDS)
setConnectionTimeout(CONNECTION_TIMEOUT, TimeUnit.SECONDS) setConnectionTimeout(CONNECTION_TIMEOUT, TimeUnit.SECONDS)
} }
val status = client.executeHttpMethod(mkcol) val status = client.executeHttpMethod(mkCol)
result = result =
if (status == HttpConstants.HTTP_CREATED) { if (status == HttpConstants.HTTP_CREATED) {
RemoteOperationResult<Unit>(ResultCode.OK) RemoteOperationResult(ResultCode.OK)
} else { } else {
RemoteOperationResult<Unit>(mkcol) RemoteOperationResult(mkCol)
} }
Timber.d("Create directory $remotePath: ${result.logMessage}") Timber.d("Create directory $remotePath: ${result.logMessage}")
client.exhaustResponse(mkcol.getResponseBodyAsStream()) client.exhaustResponse(mkCol.getResponseBodyAsStream())
} catch (e: Exception) { } catch (e: Exception) {
result = RemoteOperationResult<Unit>(e) result = RemoteOperationResult(e)
Timber.e(e, "Create directory $remotePath: ${result.logMessage}") Timber.e(e, "Create directory $remotePath: ${result.logMessage}")
} }
return result return result

18
owncloudComLibrary/src/main/java/com/owncloud/android/lib/resources/files/services/FileService.kt
View File

@ -28,8 +28,20 @@ import com.owncloud.android.lib.resources.Service
import com.owncloud.android.lib.resources.files.RemoteFile import com.owncloud.android.lib.resources.files.RemoteFile
interface FileService : Service { interface FileService : Service {
fun checkPathExistence(path: String, isUserLogged: Boolean): RemoteOperationResult<Boolean>
fun getUrlToOpenInWeb(openWebEndpoint: String, fileId: String): RemoteOperationResult<String> fun getUrlToOpenInWeb(openWebEndpoint: String, fileId: String): RemoteOperationResult<String>
fun createFolder(remotePath: String, createFullPath: Boolean, isChunkFolder: Boolean = false): RemoteOperationResult<Unit>
fun refreshFolder(remotePath: String): RemoteOperationResult<ArrayList<RemoteFile>> fun checkPathExistence(
path: String,
isUserLogged: Boolean
): RemoteOperationResult<Boolean>
fun createFolder(
remotePath: String,
createFullPath: Boolean,
isChunkFolder: Boolean = false
): RemoteOperationResult<Unit>
fun refreshFolder(
remotePath: String
): RemoteOperationResult<ArrayList<RemoteFile>>
} }