Browse Source

codacy: Avoid really long methods.

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Andy Scherzinger 5 years ago
parent
commit
5a1b5ed5cf

+ 17 - 9
src/main/java/com/owncloud/android/MainApp.java

@@ -113,7 +113,7 @@ import static com.owncloud.android.ui.activity.ContactsPreferenceActivity.PREFER
 
 /**
  * Main Application of the project
- * <p>
+ *
  * Contains methods to build the "static" strings. These strings were before constants in different classes
  */
 public class MainApp extends MultiDexApplication implements HasAndroidInjector {
@@ -122,6 +122,7 @@ public class MainApp extends MultiDexApplication implements HasAndroidInjector {
     public static final OwnCloudVersion MINIMUM_SUPPORTED_SERVER_VERSION = OwnCloudVersion.nextcloud_12;
 
     private static final String TAG = MainApp.class.getSimpleName();
+    public static final String DOT = ".";
 
     private static Context mContext;
 
@@ -233,11 +234,7 @@ public class MainApp extends MultiDexApplication implements HasAndroidInjector {
 
         insertConscrypt();
 
-        SecurityKeyManager securityKeyManager = SecurityKeyManager.getInstance();
-        SecurityKeyManagerConfig config = new SecurityKeyManagerConfig.Builder()
-            .setEnableDebugLogging(BuildConfig.DEBUG)
-            .build();
-        securityKeyManager.init(this, config);
+        initSecurityKeyManager();
 
         registerActivityLifecycleCallbacks(new ActivityInjector());
 
@@ -307,7 +304,10 @@ public class MainApp extends MultiDexApplication implements HasAndroidInjector {
             .build()
             .schedule();
 
-        // register global protection with pass code
+        registerGlobalPassCodeProtection();
+    }
+
+    private void registerGlobalPassCodeProtection() {
         registerActivityLifecycleCallbacks(new ActivityLifecycleCallbacks() {
 
             @Override
@@ -350,6 +350,14 @@ public class MainApp extends MultiDexApplication implements HasAndroidInjector {
         });
     }
 
+    private void initSecurityKeyManager() {
+        SecurityKeyManager securityKeyManager = SecurityKeyManager.getInstance();
+        SecurityKeyManagerConfig config = new SecurityKeyManagerConfig.Builder()
+            .setEnableDebugLogging(BuildConfig.DEBUG)
+            .build();
+        securityKeyManager.init(this, config);
+    }
+
     public static void initContactsBackup(UserAccountManager accountManager) {
         ArbitraryDataProvider arbitraryDataProvider = new ArbitraryDataProvider(mContext.getContentResolver());
         Account[] accounts = accountManager.getAccounts();
@@ -368,9 +376,9 @@ public class MainApp extends MultiDexApplication implements HasAndroidInjector {
             Conscrypt.Version version = Conscrypt.version();
             Log_OC.i(TAG, "Using Conscrypt/"
                 + version.major()
-                + "."
+                + DOT
                 + version.minor()
-                + "." + version.patch()
+                + DOT + version.patch()
                 + " for TLS");
             SSLEngine engine = SSLContext.getDefault().createSSLEngine();
             Log_OC.i(TAG, "Enabled protocols: " + Arrays.toString(engine.getEnabledProtocols()) + " }");

+ 58 - 49
src/main/java/com/owncloud/android/datamodel/FileDataStorageManager.java

@@ -38,6 +38,7 @@ import android.text.TextUtils;
 import com.google.gson.Gson;
 import com.google.gson.JsonSyntaxException;
 import com.owncloud.android.MainApp;
+import com.owncloud.android.db.ProviderMeta;
 import com.owncloud.android.db.ProviderMeta.ProviderTableMeta;
 import com.owncloud.android.lib.common.network.WebdavEntry;
 import com.owncloud.android.lib.common.operations.RemoteOperationResult;
@@ -54,6 +55,8 @@ import com.owncloud.android.utils.FileStorageUtils;
 import com.owncloud.android.utils.MimeType;
 import com.owncloud.android.utils.MimeTypeUtil;
 
+import org.jetbrains.annotations.NotNull;
+
 import java.io.File;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -1920,8 +1923,50 @@ public class FileDataStorageManager {
     public OCCapability saveCapabilities(OCCapability capability) {
 
         // Prepare capabilities data
+        ContentValues cv = createContentValues(account.name, capability);
+
+        if (capabilityExists(account.name)) {
+            if (getContentResolver() != null) {
+                getContentResolver().update(ProviderTableMeta.CONTENT_URI_CAPABILITIES, cv,
+                        ProviderTableMeta.CAPABILITIES_ACCOUNT_NAME + "=?",
+                        new String[]{account.name});
+            } else {
+                try {
+                    getContentProviderClient().update(ProviderTableMeta.CONTENT_URI_CAPABILITIES,
+                            cv, ProviderTableMeta.CAPABILITIES_ACCOUNT_NAME + "=?",
+                            new String[]{account.name});
+                } catch (RemoteException e) {
+                    Log_OC.e(TAG, FAILED_TO_INSERT_MSG + e.getMessage(), e);
+                }
+            }
+        } else {
+            Uri result_uri = null;
+            if (getContentResolver() != null) {
+                result_uri = getContentResolver().insert(
+                        ProviderTableMeta.CONTENT_URI_CAPABILITIES, cv);
+            } else {
+                try {
+                    result_uri = getContentProviderClient().insert(
+                            ProviderTableMeta.CONTENT_URI_CAPABILITIES, cv);
+                } catch (RemoteException e) {
+                    Log_OC.e(TAG, FAILED_TO_INSERT_MSG + e.getMessage(), e);
+                }
+            }
+            if (result_uri != null) {
+                long new_id = Long.parseLong(result_uri.getPathSegments()
+                        .get(1));
+                capability.setId(new_id);
+                capability.setAccountName(account.name);
+            }
+        }
+
+        return capability;
+    }
+
+    @NotNull
+    private ContentValues createContentValues(String accountName, OCCapability capability) {
         ContentValues cv = new ContentValues();
-        cv.put(ProviderTableMeta.CAPABILITIES_ACCOUNT_NAME, account.name);
+        cv.put(ProviderTableMeta.CAPABILITIES_ACCOUNT_NAME, accountName);
         cv.put(ProviderTableMeta.CAPABILITIES_VERSION_MAYOR, capability.getVersionMayor());
         cv.put(ProviderTableMeta.CAPABILITIES_VERSION_MINOR, capability.getVersionMinor());
         cv.put(ProviderTableMeta.CAPABILITIES_VERSION_MICRO, capability.getVersionMicro());
@@ -1931,28 +1976,28 @@ public class FileDataStorageManager {
         cv.put(ProviderTableMeta.CAPABILITIES_CORE_POLLINTERVAL, capability.getCorePollInterval());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_API_ENABLED, capability.getFilesSharingApiEnabled().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_ENABLED,
-                capability.getFilesSharingPublicEnabled().getValue());
+               capability.getFilesSharingPublicEnabled().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_PASSWORD_ENFORCED,
-                capability.getFilesSharingPublicPasswordEnforced().getValue());
+               capability.getFilesSharingPublicPasswordEnforced().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_ASK_FOR_OPTIONAL_PASSWORD,
                capability.getFilesSharingPublicAskForOptionalPassword().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_EXPIRE_DATE_ENABLED,
-                capability.getFilesSharingPublicExpireDateEnabled().getValue());
+               capability.getFilesSharingPublicExpireDateEnabled().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_EXPIRE_DATE_DAYS,
-                capability.getFilesSharingPublicExpireDateDays());
+               capability.getFilesSharingPublicExpireDateDays());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_EXPIRE_DATE_ENFORCED,
-                capability.getFilesSharingPublicExpireDateEnforced().getValue());
+               capability.getFilesSharingPublicExpireDateEnforced().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_SEND_MAIL,
-                capability.getFilesSharingPublicSendMail().getValue());
+               capability.getFilesSharingPublicSendMail().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_PUBLIC_UPLOAD,
-                capability.getFilesSharingPublicUpload().getValue());
+               capability.getFilesSharingPublicUpload().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_USER_SEND_MAIL,
-                capability.getFilesSharingUserSendMail().getValue());
+               capability.getFilesSharingUserSendMail().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_RESHARING, capability.getFilesSharingResharing().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_FEDERATION_OUTGOING,
-                capability.getFilesSharingFederationOutgoing().getValue());
+               capability.getFilesSharingFederationOutgoing().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_SHARING_FEDERATION_INCOMING,
-                capability.getFilesSharingFederationIncoming().getValue());
+               capability.getFilesSharingFederationIncoming().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_FILES_BIGFILECHUNKING, capability.getFilesBigFileChunking().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_FILES_UNDELETE, capability.getFilesUndelete().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_FILES_VERSIONING, capability.getFilesVersioning().getValue());
@@ -1972,7 +2017,7 @@ public class FileDataStorageManager {
         cv.put(ProviderTableMeta.CAPABILITIES_ACTIVITY, capability.getActivity().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_RICHDOCUMENT, capability.getRichDocuments().getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_RICHDOCUMENT_MIMETYPE_LIST,
-                TextUtils.join(",", capability.getRichDocumentsMimeTypeList()));
+               TextUtils.join(",", capability.getRichDocumentsMimeTypeList()));
         cv.put(ProviderTableMeta.CAPABILITIES_RICHDOCUMENT_OPTIONAL_MIMETYPE_LIST,
                TextUtils.join(",", capability.getRichDocumentsOptionalMimeTypeList()));
         cv.put(ProviderTableMeta.CAPABILITIES_RICHDOCUMENT_DIRECT_EDITING, capability.getRichDocumentsDirectEditing()
@@ -1980,43 +2025,7 @@ public class FileDataStorageManager {
         cv.put(ProviderTableMeta.CAPABILITIES_RICHDOCUMENT_TEMPLATES, capability.getRichDocumentsTemplatesAvailable()
             .getValue());
         cv.put(ProviderTableMeta.CAPABILITIES_RICHDOCUMENT_PRODUCT_NAME, capability.getRichDocumentsProductName());
-
-        if (capabilityExists(account.name)) {
-            if (getContentResolver() != null) {
-                getContentResolver().update(ProviderTableMeta.CONTENT_URI_CAPABILITIES, cv,
-                        ProviderTableMeta.CAPABILITIES_ACCOUNT_NAME + "=?",
-                        new String[]{account.name});
-            } else {
-                try {
-                    getContentProviderClient().update(ProviderTableMeta.CONTENT_URI_CAPABILITIES,
-                            cv, ProviderTableMeta.CAPABILITIES_ACCOUNT_NAME + "=?",
-                            new String[]{account.name});
-                } catch (RemoteException e) {
-                    Log_OC.e(TAG, FAILED_TO_INSERT_MSG + e.getMessage(), e);
-                }
-            }
-        } else {
-            Uri result_uri = null;
-            if (getContentResolver() != null) {
-                result_uri = getContentResolver().insert(
-                        ProviderTableMeta.CONTENT_URI_CAPABILITIES, cv);
-            } else {
-                try {
-                    result_uri = getContentProviderClient().insert(
-                            ProviderTableMeta.CONTENT_URI_CAPABILITIES, cv);
-                } catch (RemoteException e) {
-                    Log_OC.e(TAG, FAILED_TO_INSERT_MSG + e.getMessage(), e);
-                }
-            }
-            if (result_uri != null) {
-                long new_id = Long.parseLong(result_uri.getPathSegments()
-                        .get(1));
-                capability.setId(new_id);
-                capability.setAccountName(account.name);
-            }
-        }
-
-        return capability;
+        return cv;
     }
 
     private boolean capabilityExists(String accountName) {

+ 20 - 35
src/main/java/com/owncloud/android/operations/UploadFileOperation.java

@@ -675,24 +675,7 @@ public class UploadFileOperation extends SyncOperation {
                 result = new RemoteOperationResult(ResultCode.UNKNOWN_ERROR);
             }
 
-            if (result.isSuccess()) {
-                Log_OC.i(TAG, "Upload of " + mFile.getStoragePath() + " to " + mFile.getRemotePath() + ": " +
-                        result.getLogMessage());
-            } else {
-                if (result.getException() != null) {
-                    if (result.isCancelled()) {
-                        Log_OC.w(TAG, "Upload of " + mFile.getStoragePath() + " to " + mFile.getRemotePath() +
-                                ": " + result.getLogMessage());
-                    } else {
-                        Log_OC.e(TAG, "Upload of " + mFile.getStoragePath() + " to " + mFile.getRemotePath() +
-                                ": " + result.getLogMessage(), result.getException());
-                    }
-
-                } else {
-                    Log_OC.e(TAG, "Upload of " + mFile.getStoragePath() + " to " + mFile.getRemotePath() +
-                            ": " + result.getLogMessage());
-                }
-            }
+            logResult(result, mFile.getStoragePath(), mFile.getRemotePath());
         }
 
         if (result.isSuccess()) {
@@ -886,23 +869,7 @@ public class UploadFileOperation extends SyncOperation {
                 result = new RemoteOperationResult(ResultCode.UNKNOWN_ERROR);
             }
 
-            if (result.isSuccess()) {
-                Log_OC.i(TAG, "Upload of " + mOriginalStoragePath + " to " + mRemotePath + ": " +
-                        result.getLogMessage());
-            } else {
-                if (result.getException() != null) {
-                    if (result.isCancelled()) {
-                        Log_OC.w(TAG, "Upload of " + mOriginalStoragePath + " to " + mRemotePath +
-                                ": " + result.getLogMessage());
-                    } else {
-                        Log_OC.e(TAG, "Upload of " + mOriginalStoragePath + " to " + mRemotePath +
-                                ": " + result.getLogMessage(), result.getException());
-                    }
-                } else {
-                    Log_OC.e(TAG, "Upload of " + mOriginalStoragePath + " to " + mRemotePath +
-                            ": " + result.getLogMessage());
-                }
-            }
+            logResult(result, mOriginalStoragePath, mRemotePath);
         }
 
         if (result.isSuccess()) {
@@ -919,6 +886,24 @@ public class UploadFileOperation extends SyncOperation {
         return result;
     }
 
+    private void logResult(RemoteOperationResult result, String sourcePath, String targetPath) {
+        if (result.isSuccess()) {
+            Log_OC.i(TAG, "Upload of " + sourcePath + " to " + targetPath + ": " + result.getLogMessage());
+        } else {
+            if (result.getException() != null) {
+                if (result.isCancelled()) {
+                    Log_OC.w(TAG, "Upload of " + sourcePath + " to " + targetPath + ": "
+                        + result.getLogMessage());
+                } else {
+                    Log_OC.e(TAG, "Upload of " + sourcePath + " to " + targetPath + ": "
+                        + result.getLogMessage(), result.getException());
+                }
+            } else {
+                Log_OC.e(TAG, "Upload of " + sourcePath + " to " + targetPath + ": " + result.getLogMessage());
+            }
+        }
+    }
+
     private RemoteOperationResult copyFile(File originalFile, String expectedPath) throws OperationCancelledException,
             IOException {
         if (mLocalBehaviour == FileUploader.LOCAL_BEHAVIOUR_COPY && !mOriginalStoragePath.equals(expectedPath)) {

+ 46 - 43
src/main/java/com/owncloud/android/ui/activity/DrawerActivity.java

@@ -29,6 +29,7 @@ package com.owncloud.android.ui.activity;
 import android.accounts.Account;
 import android.accounts.AccountManager;
 import android.accounts.AccountManagerFuture;
+import android.app.Activity;
 import android.content.Context;
 import android.content.Intent;
 import android.content.res.Configuration;
@@ -396,30 +397,7 @@ public abstract class DrawerActivity extends ToolbarActivity
         setDrawerMenuItemChecked(menuItem.getItemId());
 
         if (menuItem.getGroupId() == R.id.drawer_menu_accounts) {
-            switch (menuItem.getItemId()) {
-                case R.id.drawer_menu_account_add:
-                    boolean isProviderOrOwnInstallationVisible = getResources()
-                        .getBoolean(R.bool.show_provider_or_own_installation);
-
-                    if (isProviderOrOwnInstallationVisible) {
-                        Intent firstRunIntent = new Intent(getApplicationContext(), FirstRunActivity.class);
-                        firstRunIntent.putExtra(FirstRunActivity.EXTRA_ALLOW_CLOSE, true);
-                        startActivity(firstRunIntent);
-                    } else {
-                        createAccount(false);
-                    }
-                    break;
-
-                case R.id.drawer_menu_account_manage:
-                    Intent manageAccountsIntent = new Intent(getApplicationContext(), ManageAccountsActivity.class);
-                    startActivityForResult(manageAccountsIntent, ACTION_MANAGE_ACCOUNTS);
-                    break;
-
-                default:
-                    accountClicked(menuItem.getItemId());
-                    break;
-            }
-
+            handleAccountItemClick(menuItem);
             return;
         }
 
@@ -459,39 +437,28 @@ public abstract class DrawerActivity extends ToolbarActivity
                 showFiles(true);
                 break;
             case R.id.nav_uploads:
-                Intent uploadListIntent = new Intent(getApplicationContext(), UploadListActivity.class);
-                uploadListIntent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
-                startActivity(uploadListIntent);
+                startActivity(UploadListActivity.class, Intent.FLAG_ACTIVITY_CLEAR_TOP);
                 break;
             case R.id.nav_trashbin:
-                Intent trashbinIntent = new Intent(getApplicationContext(), TrashbinActivity.class);
-                trashbinIntent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
-                startActivity(trashbinIntent);
+                startActivity(TrashbinActivity.class, Intent.FLAG_ACTIVITY_CLEAR_TOP);
                 break;
             case R.id.nav_activity:
-                Intent activityIntent = new Intent(getApplicationContext(), ActivitiesActivity.class);
-                activityIntent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
-                startActivity(activityIntent);
+                startActivity(ActivitiesActivity.class, Intent.FLAG_ACTIVITY_CLEAR_TOP);
                 break;
             case R.id.nav_notifications:
-                Intent notificationsIntent = new Intent(getApplicationContext(), NotificationsActivity.class);
-                startActivity(notificationsIntent);
+                startActivity(NotificationsActivity.class);
                 break;
             case R.id.nav_synced_folders:
-                Intent syncedFoldersIntent = new Intent(getApplicationContext(), SyncedFoldersActivity.class);
-                startActivity(syncedFoldersIntent);
+                startActivity(SyncedFoldersActivity.class);
                 break;
             case R.id.nav_contacts:
-                Intent contactsIntent = new Intent(getApplicationContext(), ContactsPreferenceActivity.class);
-                startActivity(contactsIntent);
+                startActivity(ContactsPreferenceActivity.class);
                 break;
             case R.id.nav_settings:
-                Intent settingsIntent = new Intent(getApplicationContext(), SettingsActivity.class);
-                startActivity(settingsIntent);
+                startActivity(SettingsActivity.class);
                 break;
             case R.id.nav_participate:
-                Intent participateIntent = new Intent(getApplicationContext(), ParticipateActivity.class);
-                startActivity(participateIntent);
+                startActivity(ParticipateActivity.class);
                 break;
             case R.id.nav_logout:
                 mCheckedMenuItem = -1;
@@ -522,6 +489,42 @@ public abstract class DrawerActivity extends ToolbarActivity
         }
     }
 
+    private void startActivity(Class<? extends Activity> activity) {
+        startActivity(new Intent(getApplicationContext(), activity));
+    }
+
+    private void startActivity(Class<? extends Activity> activity, int flags) {
+        Intent intent = new Intent(getApplicationContext(), activity);
+        intent.setFlags(flags);
+        startActivity(intent);
+    }
+
+    private void handleAccountItemClick(MenuItem menuItem) {
+        switch (menuItem.getItemId()) {
+            case R.id.drawer_menu_account_add:
+                boolean isProviderOrOwnInstallationVisible = getResources()
+                    .getBoolean(R.bool.show_provider_or_own_installation);
+
+                if (isProviderOrOwnInstallationVisible) {
+                    Intent firstRunIntent = new Intent(getApplicationContext(), FirstRunActivity.class);
+                    firstRunIntent.putExtra(FirstRunActivity.EXTRA_ALLOW_CLOSE, true);
+                    startActivity(firstRunIntent);
+                } else {
+                    createAccount(false);
+                }
+                break;
+
+            case R.id.drawer_menu_account_manage:
+                Intent manageAccountsIntent = new Intent(getApplicationContext(), ManageAccountsActivity.class);
+                startActivityForResult(manageAccountsIntent, ACTION_MANAGE_ACCOUNTS);
+                break;
+
+            default:
+                accountClicked(menuItem.getItemId());
+                break;
+        }
+    }
+
     private void startPhotoSearch(MenuItem menuItem) {
         SearchEvent searchEvent = new SearchEvent("image/%",
                                                   SearchRemoteOperation.SearchType.PHOTO_SEARCH,