From b39269d92618ded45c608744d632fe7e80d3813d Mon Sep 17 00:00:00 2001
From: alperozturk <alper_ozturk@proton.me>
Date: Thu, 12 Dec 2024 16:04:42 +0100
Subject: [PATCH] fix code analytics

Signed-off-by: alperozturk <alper_ozturk@proton.me>
---
 .../com/nextcloud/client/jobs/BackgroundJobManager.kt    | 2 +-
 .../nextcloud/client/jobs/BackgroundJobManagerImpl.kt    | 4 +++-
 .../nextcloud/client/jobs/download/FileDownloadHelper.kt | 4 ++--
 .../java/com/nextcloud/client/jobs/sync/SyncWorker.kt    | 2 +-
 .../android/operations/SynchronizeFolderOperation.java   | 9 +--------
 5 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManager.kt b/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManager.kt
index 6c967e1441..d1731b12ff 100644
--- a/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManager.kt
+++ b/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManager.kt
@@ -172,5 +172,5 @@ interface BackgroundJobManager {
     fun startPeriodicallyOfflineOperation()
     fun scheduleInternal2WaySync(intervalMinutes: Long)
     fun cancelAllFilesDownloadJobs()
-    fun syncFolder(filePaths: List<String>)
+    fun syncFolder(files: List<OCFile>)
 }
diff --git a/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt b/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt
index be7d333b61..24badc052b 100644
--- a/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt
+++ b/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt
@@ -715,7 +715,9 @@ internal class BackgroundJobManagerImpl(
         workManager.enqueueUniquePeriodicWork(JOB_INTERNAL_TWO_WAY_SYNC, ExistingPeriodicWorkPolicy.UPDATE, request)
     }
 
-    override fun syncFolder(filePaths: List<String>) {
+    override fun syncFolder(files: List<OCFile>) {
+        val filePaths = files.map { it.decryptedRemotePath }
+
         val data = Data.Builder()
             .putStringArray(SyncWorker.FILE_PATHS, filePaths.toTypedArray())
             .build()
diff --git a/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadHelper.kt b/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadHelper.kt
index 347d9bc152..7a263dc335 100644
--- a/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadHelper.kt
+++ b/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadHelper.kt
@@ -138,7 +138,7 @@ class FileDownloadHelper {
         )
     }
 
-    fun syncFolder(filePaths: List<String>) {
-        backgroundJobManager.syncFolder(filePaths)
+    fun syncFolder(files: List<OCFile>) {
+        backgroundJobManager.syncFolder(files)
     }
 }
diff --git a/app/src/main/java/com/nextcloud/client/jobs/sync/SyncWorker.kt b/app/src/main/java/com/nextcloud/client/jobs/sync/SyncWorker.kt
index 215421ef16..8b822d8894 100644
--- a/app/src/main/java/com/nextcloud/client/jobs/sync/SyncWorker.kt
+++ b/app/src/main/java/com/nextcloud/client/jobs/sync/SyncWorker.kt
@@ -59,7 +59,7 @@ class SyncWorker(
             val filePaths = inputData.getStringArray(FILE_PATHS)
 
             if (filePaths.isNullOrEmpty()) {
-                return@withContext Result.failure()
+                return@withContext Result.success()
             }
 
             val fileDataStorageManager = FileDataStorageManager(user, context.contentResolver)
diff --git a/app/src/main/java/com/owncloud/android/operations/SynchronizeFolderOperation.java b/app/src/main/java/com/owncloud/android/operations/SynchronizeFolderOperation.java
index 4e3b0fa984..2514801a50 100644
--- a/app/src/main/java/com/owncloud/android/operations/SynchronizeFolderOperation.java
+++ b/app/src/main/java/com/owncloud/android/operations/SynchronizeFolderOperation.java
@@ -40,7 +40,6 @@ import java.util.List;
 import java.util.Map;
 import java.util.Vector;
 import java.util.concurrent.atomic.AtomicBoolean;
-import java.util.function.Consumer;
 
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
@@ -501,13 +500,7 @@ public class SynchronizeFolderOperation extends SyncOperation {
                 Log_OC.d(TAG, "Exception caught at startDirectDownloads" + e);
             }
         } else {
-            final var filePaths = new ArrayList<String>();
-            mFilesForDirectDownload.forEach(file -> filePaths.add(file.getDecryptedRemotePath()));
-            if (filePaths.isEmpty()) {
-                return;
-            }
-
-            fileDownloadHelper.syncFolder(filePaths);
+            fileDownloadHelper.syncFolder(mFilesForDirectDownload);
         }
     }