From: David A. Velasco Date: Thu, 22 Nov 2012 12:35:07 +0000 (+0100) Subject: Removal of a folder (or file) not already existing in the server is considered a... X-Git-Tag: oc-android-1.4.3~88^2~2 X-Git-Url: http://git.linex4red.de/pub/Android/ownCloud.git/commitdiff_plain/60d0a121b7ccffd0d1668f771ba4061ab0254b2e?ds=inline;hp=--cc Removal of a folder (or file) not already existing in the server is considered a success --- 60d0a121b7ccffd0d1668f771ba4061ab0254b2e diff --git a/src/com/owncloud/android/operations/RemoveFileOperation.java b/src/com/owncloud/android/operations/RemoveFileOperation.java index d8809a69..bfc063f9 100644 --- a/src/com/owncloud/android/operations/RemoveFileOperation.java +++ b/src/com/owncloud/android/operations/RemoveFileOperation.java @@ -18,6 +18,7 @@ package com.owncloud.android.operations; +import org.apache.commons.httpclient.HttpStatus; import org.apache.jackrabbit.webdav.client.methods.DeleteMethod; import android.util.Log; @@ -81,7 +82,7 @@ public class RemoveFileOperation extends RemoteOperation { try { delete = new DeleteMethod(client.getBaseUri() + WebdavUtils.encodePath(mFileToRemove.getRemotePath())); int status = client.executeMethod(delete, REMOVE_READ_TIMEOUT, REMOVE_CONNECTION_TIMEOUT); - if (delete.succeeded()) { + if (delete.succeeded() || status == HttpStatus.SC_NOT_FOUND) { if (mFileToRemove.isDirectory()) { mDataStorageManager.removeDirectory(mFileToRemove, true, mDeleteLocalCopy); } else { @@ -89,7 +90,7 @@ public class RemoveFileOperation extends RemoteOperation { } } delete.getResponseBodyAsString(); // exhaust the response, although not interesting - result = new RemoteOperationResult(delete.succeeded(), status); + result = new RemoteOperationResult((delete.succeeded() || status == HttpStatus.SC_NOT_FOUND), status); Log.i(TAG, "Remove " + mFileToRemove.getRemotePath() + ": " + result.getLogMessage()); } catch (Exception e) {