Browse Source

Code analytic fixes

Signed-off-by: alperozturk <alper_ozturk@proton.me>
alperozturk 1 năm trước cách đây
mục cha
commit
f6ee552bf9

+ 1 - 1
app/src/main/java/com/nextcloud/client/database/NextcloudDatabase.kt

@@ -68,7 +68,7 @@ import com.owncloud.android.db.ProviderMeta
         AutoMigration(from = 71, to = 72),
         AutoMigration(from = 72, to = 73),
         AutoMigration(from = 73, to = 74, spec = DatabaseMigrationUtil.ResetCapabilitiesPostMigration::class),
-        AutoMigration(from = 74, to = 75),
+        AutoMigration(from = 74, to = 75)
     ],
     exportSchema = true
 )

+ 2 - 3
app/src/main/java/com/owncloud/android/datamodel/FileDataStorageManager.java

@@ -37,7 +37,6 @@ import android.os.Build;
 import android.os.RemoteException;
 import android.provider.MediaStore;
 import android.text.TextUtils;
-import android.util.Log;
 
 import com.google.gson.Gson;
 import com.google.gson.JsonSyntaxException;
@@ -463,7 +462,7 @@ public class FileDataStorageManager {
         cv.put(ProviderTableMeta.FILE_REMOTE_ID, fileOrFolder.getRemoteId());
         cv.put(ProviderTableMeta.FILE_LOCAL_ID, fileOrFolder.getLocalId());
         cv.put(ProviderTableMeta.FILE_FAVORITE, fileOrFolder.isFavorite());
-        cv.put(ProviderTableMeta.FILE_HIDDEN, fileOrFolder.getHidden());
+        cv.put(ProviderTableMeta.FILE_HIDDEN, fileOrFolder.shouldHide());
         cv.put(ProviderTableMeta.FILE_UNREAD_COMMENTS_COUNT, fileOrFolder.getUnreadCommentsCount());
         cv.put(ProviderTableMeta.FILE_OWNER_ID, fileOrFolder.getOwnerId());
         cv.put(ProviderTableMeta.FILE_OWNER_DISPLAY_NAME, fileOrFolder.getOwnerDisplayName());
@@ -503,7 +502,7 @@ public class FileDataStorageManager {
         cv.put(ProviderTableMeta.FILE_LOCKED, file.isLocked());
         final FileLockType lockType = file.getLockType();
         cv.put(ProviderTableMeta.FILE_LOCK_TYPE, lockType != null ? lockType.getValue() : -1);
-        cv.put(ProviderTableMeta.FILE_HIDDEN, file.getHidden());
+        cv.put(ProviderTableMeta.FILE_HIDDEN, file.shouldHide());
         cv.put(ProviderTableMeta.FILE_LOCK_OWNER, file.getLockOwnerId());
         cv.put(ProviderTableMeta.FILE_LOCK_OWNER_DISPLAY_NAME, file.getLockOwnerDisplayName());
         cv.put(ProviderTableMeta.FILE_LOCK_OWNER_EDITOR, file.getLockOwnerEditor());

+ 1 - 9
app/src/main/java/com/owncloud/android/datamodel/OCFile.java

@@ -28,7 +28,6 @@ import android.net.Uri;
 import android.os.Parcel;
 import android.os.Parcelable;
 import android.text.TextUtils;
-import android.util.Log;
 
 import com.owncloud.android.R;
 import com.owncloud.android.lib.common.network.WebdavEntry;
@@ -369,13 +368,6 @@ public class OCFile implements Parcelable, Comparable<OCFile>, ServerFileInterfa
         }
     }
 
-    public String getFileNameWithExtension(int fileNameLength) {
-        String fileName = getFileName();
-        String shortFileName = fileName.substring(0, Math.min(fileName.length(), fileNameLength));
-        String extension = "." + fileName.substring(fileName.lastIndexOf('.') + 1);
-        return shortFileName + extension;
-    }
-
     /**
      * The path, where the file is stored locally
      *
@@ -804,7 +796,7 @@ public class OCFile implements Parcelable, Comparable<OCFile>, ServerFileInterfa
         return this.favorite;
     }
 
-    public boolean getHidden() {
+    public boolean shouldHide() {
         return this.hidden;
     }
 

+ 1 - 1
app/src/main/java/com/owncloud/android/operations/SynchronizeFileOperation.java

@@ -254,7 +254,7 @@ public class SynchronizeFileOperation extends SyncOperation {
                     } else {
                         // TODO CHECK: is this really useful in some point in the code?
                         mServerFile.setFavorite(mLocalFile.isFavorite());
-                        mServerFile.setHidden(mLocalFile.getHidden());
+                        mServerFile.setHidden(mLocalFile.shouldHide());
                         mServerFile.setLastSyncDateForData(mLocalFile.getLastSyncDateForData());
                         mServerFile.setStoragePath(mLocalFile.getStoragePath());
                         mServerFile.setParentId(mLocalFile.getParentId());

+ 1 - 2
app/src/main/java/com/owncloud/android/ui/adapter/OCFileListAdapter.java

@@ -92,7 +92,6 @@ import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 import java.util.Set;
-import java.util.stream.Collectors;
 
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
@@ -950,7 +949,7 @@ public class OCFileListAdapter extends RecyclerView.Adapter<RecyclerView.ViewHol
         listOfHiddenFiles.clear();
 
         mFiles.forEach(file -> {
-            if (file.getHidden()) {
+            if (file.shouldHide()) {
                 listOfHiddenFiles.add(file.getFileName());
             }
         });