diff --git a/library/src/androidTest/java/com/owncloud/android/FileIT.java b/library/src/androidTest/java/com/owncloud/android/FileIT.java index 2cadacc7c5..1c75ea5b9b 100644 --- a/library/src/androidTest/java/com/owncloud/android/FileIT.java +++ b/library/src/androidTest/java/com/owncloud/android/FileIT.java @@ -126,7 +126,7 @@ public void testShareViaLinkSharees() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); // verify RemoteOperationResult result = new ReadFolderRemoteOperation("/").execute(client); @@ -167,7 +167,7 @@ public void testShareToGroupSharees() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); // verify RemoteOperationResult result = new ReadFolderRemoteOperation("/").execute(client); @@ -202,7 +202,7 @@ public void testOneSharees() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); // verify RemoteOperationResult result = new ReadFolderRemoteOperation("/").execute(client); @@ -239,7 +239,7 @@ public void testTwoShareesOnParent() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); assertTrue(new CreateShareRemoteOperation(path, ShareType.USER, @@ -247,7 +247,7 @@ public void testTwoShareesOnParent() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); // verify RemoteOperationResult result = new ReadFolderRemoteOperation("/").execute(client); @@ -287,7 +287,7 @@ public void testTwoSharees() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); assertTrue(new CreateShareRemoteOperation(path, ShareType.USER, @@ -295,7 +295,7 @@ public void testTwoSharees() { false, "", OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + .execute(client).isSuccess()); // verify RemoteOperationResult result = new ReadFolderRemoteOperation(path).execute(client); diff --git a/library/src/androidTest/java/com/owncloud/android/lib/common/operations/CreateShareIT.java b/library/src/androidTest/java/com/owncloud/android/lib/common/operations/CreateShareIT.java index d4a327a643..fe9e5cc9e8 100644 --- a/library/src/androidTest/java/com/owncloud/android/lib/common/operations/CreateShareIT.java +++ b/library/src/androidTest/java/com/owncloud/android/lib/common/operations/CreateShareIT.java @@ -56,7 +56,7 @@ public void testCreatePublicShareSuccessful() { "", false, "", - 1).execute(client); + 1).execute(client); assertTrue(result.isSuccess()); } @@ -67,7 +67,7 @@ public void testCreatePublicShareFailure() { "", false, "", - 1).execute(client); + 1).execute(client); assertFalse(result.isSuccess()); assertEquals(ResultCode.FILE_NOT_FOUND, result.getCode()); @@ -83,7 +83,7 @@ public void testCreatePrivateShareWithUserSuccessful() { "admin", false, "", - 31).execute(client); + 31).execute(client); assertTrue(result.isSuccess()); } @@ -97,7 +97,7 @@ public void testCreatePrivateShareWithUserNotExists() { "no_exist", false, "", - 31).execute(client); + 31).execute(client); assertFalse(result.isSuccess()); // TODO 404 is File not found, but actually it is "user not found" @@ -114,7 +114,7 @@ public void testCreatePrivateShareWithFileNotExists() { "admin", false, "", - 31).execute(client); + 31).execute(client); assertFalse(result.isSuccess()); assertEquals(ResultCode.FILE_NOT_FOUND, result.getCode()); } @@ -129,7 +129,7 @@ public void testCreatePrivateShareWithGroupSuccessful() { "admin", false, "", - 1).execute(client); + 1).execute(client); assertTrue(result.isSuccess()); } @@ -143,7 +143,7 @@ public void testCreatePrivateShareWithNonExistingGroupSharee() { "no_exist", false, "", - 31).execute(client); + 31).execute(client); assertFalse(result.isSuccess()); // TODO 404 is File not found, but actually it is "user not found" @@ -160,7 +160,7 @@ public void testCreatePrivateShareWithNonExistingFile() { "admin", false, "", - 31).execute(client); + 31).execute(client); assertFalse(result.isSuccess()); assertEquals(ResultCode.FILE_NOT_FOUND, result.getCode()); } @@ -189,7 +189,7 @@ public void testCreateFederatedShareWithNonExistingSharee() { "no_exist@" + serverUri2, false, "", - 31).execute(client); + 31).execute(client); assertFalse("sharee doesn't exist in an existing remote server", result.isSuccess()); assertEquals("sharee doesn't exist in an existing remote server, forbidden", @@ -206,7 +206,7 @@ public void testCreateFederatedShareWithNonExistingRemoteServer() { "no_exist", false, "", - 31).execute(client); + 31).execute(client); assertFalse(result.isSuccess()); // TODO expected: but was: assertEquals("remote server doesn't exist", ResultCode.SHARE_FORBIDDEN, result.getCode()); @@ -222,7 +222,7 @@ public void testCreateFederatedShareWithNonExistingFile() { "admin@" + serverUri2, false, "", - 31).execute(client); + 31).execute(client); assertFalse("file doesn't exist", result.isSuccess()); assertEquals("file doesn't exist", ResultCode.FILE_NOT_FOUND, result.getCode()); diff --git a/library/src/androidTest/java/com/owncloud/android/lib/common/operations/GetSharesIT.java b/library/src/androidTest/java/com/owncloud/android/lib/common/operations/GetSharesIT.java index 5a80617087..55d72ca411 100644 --- a/library/src/androidTest/java/com/owncloud/android/lib/common/operations/GetSharesIT.java +++ b/library/src/androidTest/java/com/owncloud/android/lib/common/operations/GetSharesIT.java @@ -36,7 +36,7 @@ public void testGetShares() { "", false, "", - 1).execute(client).isSuccess()); + 1).execute(client).isSuccess()); assertTrue(new CreateFolderRemoteOperation("/2/", true).execute(client).isSuccess()); assertTrue(new CreateShareRemoteOperation("/2/", @@ -44,7 +44,7 @@ public void testGetShares() { "", false, "", - 1).execute(client).isSuccess()); + 1).execute(client).isSuccess()); RemoteOperationResult> result = new GetSharesRemoteOperation().execute(client); assertTrue(result.isSuccess()); diff --git a/library/src/androidTest/java/com/owncloud/android/lib/common/operations/RemoveShareIT.java b/library/src/androidTest/java/com/owncloud/android/lib/common/operations/RemoveShareIT.java index 70171e560e..f7986b37d8 100644 --- a/library/src/androidTest/java/com/owncloud/android/lib/common/operations/RemoveShareIT.java +++ b/library/src/androidTest/java/com/owncloud/android/lib/common/operations/RemoveShareIT.java @@ -42,7 +42,7 @@ public void testRemoveShare() throws IOException { ShareType.PUBLIC_LINK, "", false, - "", 1).execute(client); + "", 1).execute(client); assertTrue(result.isSuccess()); diff --git a/library/src/androidTest/java/com/owncloud/android/lib/resources/files/SearchRemoteOperationIT.java b/library/src/androidTest/java/com/owncloud/android/lib/resources/files/SearchRemoteOperationIT.java index f16c50e8da..4004ae1bcf 100644 --- a/library/src/androidTest/java/com/owncloud/android/lib/resources/files/SearchRemoteOperationIT.java +++ b/library/src/androidTest/java/com/owncloud/android/lib/resources/files/SearchRemoteOperationIT.java @@ -177,7 +177,7 @@ public void favoriteFiles() throws IOException { client.getUserId(), false, "", - 31).execute(client2) + 31).execute(client2) .isSuccess() ); diff --git a/library/src/androidTest/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperationIT.java b/library/src/androidTest/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperationIT.java index 58ed2ac252..2d173ad2a6 100644 --- a/library/src/androidTest/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperationIT.java +++ b/library/src/androidTest/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperationIT.java @@ -53,30 +53,30 @@ public void searchSharedFiles() { // share folder to user "admin" assertTrue(new CreateShareRemoteOperation("/shareToAdmin/", - ShareType.USER, - "admin", - false, - "", - OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER) - .execute(client).isSuccess()); + ShareType.USER, + "admin", + false, + "", + OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER) + .execute(client).isSuccess()); // share folder via public link assertTrue(new CreateShareRemoteOperation("/shareViaLink/", - ShareType.PUBLIC_LINK, - "", - true, - "", - OCShare.READ_PERMISSION_FLAG) - .execute(client).isSuccess()); + ShareType.PUBLIC_LINK, + "", + true, + "", + OCShare.READ_PERMISSION_FLAG) + .execute(client).isSuccess()); // share folder to group assertTrue(new CreateShareRemoteOperation("/shareToGroup/", - ShareType.GROUP, - "users", - false, - "", - OCShare.NO_PERMISSION) - .execute(client).isSuccess()); + ShareType.GROUP, + "users", + false, + "", + OCShare.NO_PERMISSION) + .execute(client).isSuccess()); // share folder to circle // get share @@ -169,12 +169,12 @@ public void sharedWithMe() { // share folder to user "admin" assertTrue(new CreateFolderRemoteOperation("/shareToAdmin/", true).execute(client).isSuccess()); assertTrue(new CreateShareRemoteOperation("/shareToAdmin/", - ShareType.USER, - "admin", - false, - "", - OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER) - .execute(client).isSuccess()); + ShareType.USER, + "admin", + false, + "", + OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER) + .execute(client).isSuccess()); // Expect one file shared by me, no file shared with me result = sut.execute(client); @@ -196,12 +196,12 @@ public void sharedWithMe() { // share folder to previous user assertTrue(new CreateFolderRemoteOperation("/shareToUser/", true).execute(clientUser1).isSuccess()); assertTrue(new CreateShareRemoteOperation("/shareToUser/", - ShareType.USER, - client.getCredentials().getUsername(), - false, - "", - OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER) - .execute(clientUser1).isSuccess()); + ShareType.USER, + client.getCredentials().getUsername(), + false, + "", + OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER) + .execute(clientUser1).isSuccess()); // Expect one file shared by me, one file shared with me result = sut.execute(client); @@ -225,7 +225,7 @@ public void favorites() { "", OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER, true) - .execute(client); + .execute(client); assertTrue(createResult.isSuccess()); @@ -243,7 +243,7 @@ public void favorites() { "", OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER, true) - .execute(client); + .execute(client); assertTrue(createResult.isSuccess()); @@ -269,7 +269,7 @@ public void noFavorite() { "", OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER, true) - .execute(client); + .execute(client); assertTrue(createResult.isSuccess()); @@ -299,7 +299,7 @@ public void favorite() { "", OCShare.MAXIMUM_PERMISSIONS_FOR_FOLDER, true) - .execute(client); + .execute(client); assertTrue(createResult.isSuccess()); diff --git a/library/src/main/java/com/owncloud/android/lib/resources/e2ee/UnlockFileRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/e2ee/UnlockFileRemoteOperation.java index db110881ef..b3036c9bdd 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/e2ee/UnlockFileRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/e2ee/UnlockFileRemoteOperation.java @@ -55,7 +55,7 @@ protected RemoteOperationResult run(OwnCloudClient client) { int status = client.executeMethod(deleteMethod, SYNC_READ_TIMEOUT, SYNC_CONNECTION_TIMEOUT); result = new RemoteOperationResult<>(status == HttpStatus.SC_OK, deleteMethod); - + client.exhaustResponse(deleteMethod.getResponseBodyAsStream()); } catch (Exception e) { result = new RemoteOperationResult<>(e); diff --git a/library/src/main/java/com/owncloud/android/lib/resources/files/ExistenceCheckRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/files/ExistenceCheckRemoteOperation.java index 32270bc7a4..dac141ca5a 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/files/ExistenceCheckRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/files/ExistenceCheckRemoteOperation.java @@ -24,12 +24,14 @@ * @author David A. Velasco */ public class ExistenceCheckRemoteOperation extends RemoteOperation { - - /** Maximum time to wait for a response from the server in MILLISECONDs. */ + + /** + * Maximum time to wait for a response from the server in MILLISECONDs. + */ public static final int TIMEOUT = 50000; private static final String TAG = ExistenceCheckRemoteOperation.class.getSimpleName(); - + private String mPath; private boolean mSuccessIfAbsent; @@ -63,7 +65,7 @@ public ExistenceCheckRemoteOperation(String remotePath, Context context, boolean } @Override - protected RemoteOperationResult run(OwnCloudClient client) { + protected RemoteOperationResult run(OwnCloudClient client) { RemoteOperationResult result = null; HeadMethod head = null; boolean previousFollowRedirects = client.isFollowRedirects(); diff --git a/library/src/main/java/com/owncloud/android/lib/resources/files/RenameFileRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/files/RenameFileRemoteOperation.java index ec3300f0ec..79d92543bd 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/files/RenameFileRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/files/RenameFileRemoteOperation.java @@ -76,7 +76,7 @@ protected RemoteOperationResult run(OwnCloudClient client) { // check if a file with the new name already exists RemoteOperationResult existenceResult = new ExistenceCheckRemoteOperation(mNewRemotePath, false) - .execute(client); + .execute(client); if (existenceResult.isSuccess()) { return new RemoteOperationResult(ResultCode.INVALID_OVERWRITE); } diff --git a/library/src/main/java/com/owncloud/android/lib/resources/shares/CreateShareRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/shares/CreateShareRemoteOperation.java index 0ef9a46e98..3515c0be94 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/shares/CreateShareRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/shares/CreateShareRemoteOperation.java @@ -166,7 +166,7 @@ protected RemoteOperationResult> run(OwnCloudClient client) { String response = post.getResponseBodyAsString(); ShareToRemoteOperationResultParser parser = new ShareToRemoteOperationResultParser( - new ShareXMLParser() + new ShareXMLParser() ); parser.setOneOrMoreSharesRequired(true); parser.setServerBaseUri(client.getBaseUri()); diff --git a/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesForFileRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesForFileRemoteOperation.java index b579e65ba9..620a8fd1f9 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesForFileRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesForFileRemoteOperation.java @@ -68,8 +68,8 @@ protected RemoteOperationResult run(OwnCloudClient client) { get.setQueryString(new NameValuePair[]{ new NameValuePair(PARAM_PATH, mRemoteFilePath), new NameValuePair(PARAM_RESHARES, String.valueOf(mReshares)), - new NameValuePair(PARAM_SUBFILES, String.valueOf(mSubfiles)) //, - //new NameValuePair("shared_with_me", "true") + new NameValuePair(PARAM_SUBFILES, String.valueOf(mSubfiles)) //, + //new NameValuePair("shared_with_me", "true") }); get.addRequestHeader(OCS_API_HEADER, OCS_API_HEADER_VALUE); diff --git a/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperation.java index 5e5090a666..586b7fde09 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/shares/GetSharesRemoteOperation.java @@ -10,6 +10,8 @@ */ package com.owncloud.android.lib.resources.shares; +import static com.owncloud.android.lib.resources.shares.ShareUtils.INCLUDE_TAGS; + import com.owncloud.android.lib.common.OwnCloudClient; import com.owncloud.android.lib.common.operations.RemoteOperation; import com.owncloud.android.lib.common.operations.RemoteOperationResult; @@ -20,8 +22,6 @@ import java.util.List; -import static com.owncloud.android.lib.resources.shares.ShareUtils.INCLUDE_TAGS; - /** * Get the data from the server about ALL the known shares owned by the requester. */ diff --git a/library/src/main/java/com/owncloud/android/lib/resources/shares/UpdateShareRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/shares/UpdateShareRemoteOperation.java index 830a677249..5cbef96551 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/shares/UpdateShareRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/shares/UpdateShareRemoteOperation.java @@ -199,9 +199,9 @@ protected RemoteOperationResult> run(OwnCloudClient client) { put = new PutMethod(uriString); put.addRequestHeader(OCS_API_HEADER, OCS_API_HEADER_VALUE); put.setRequestEntity(new StringRequestEntity( - parameter.first + "=" + parameter.second, - ENTITY_CONTENT_TYPE, - ENTITY_CHARSET + parameter.first + "=" + parameter.second, + ENTITY_CONTENT_TYPE, + ENTITY_CHARSET )); status = client.executeMethod(put); diff --git a/library/src/main/java/com/owncloud/android/lib/resources/status/GetStatusRemoteOperation.java b/library/src/main/java/com/owncloud/android/lib/resources/status/GetStatusRemoteOperation.java index 423abcbe32..dbc8283786 100644 --- a/library/src/main/java/com/owncloud/android/lib/resources/status/GetStatusRemoteOperation.java +++ b/library/src/main/java/com/owncloud/android/lib/resources/status/GetStatusRemoteOperation.java @@ -172,9 +172,9 @@ private boolean tryConnection(OwnCloudClient client) { private boolean isOnline() { ConnectivityManager cm = (ConnectivityManager) mContext - .getSystemService(Context.CONNECTIVITY_SERVICE); + .getSystemService(Context.CONNECTIVITY_SERVICE); return cm != null && cm.getActiveNetworkInfo() != null - && cm.getActiveNetworkInfo().isConnectedOrConnecting(); + && cm.getActiveNetworkInfo().isConnectedOrConnecting(); } @Override