Ver código fonte

Validate empty list check in getFilesFromIDs

Neeraj Gupta 3 anos atrás
pai
commit
86842ad2f6
2 arquivos alterados com 4 adições e 4 exclusões
  1. 4 1
      lib/db/files_db.dart
  2. 0 3
      lib/services/deduplication_service.dart

+ 4 - 1
lib/db/files_db.dart

@@ -875,6 +875,10 @@ class FilesDB {
   }
   }
 
 
   Future<Map<int, File>> getFilesFromIDs(List<int> ids) async {
   Future<Map<int, File>> getFilesFromIDs(List<int> ids) async {
+    final result = <int, File>{};
+    if (ids.isEmpty) {
+      return result;
+    }
     String inParam = "";
     String inParam = "";
     for (final id in ids) {
     for (final id in ids) {
       inParam += "'" + id.toString() + "',";
       inParam += "'" + id.toString() + "',";
@@ -886,7 +890,6 @@ class FilesDB {
       where: '$columnUploadedFileID IN ($inParam)',
       where: '$columnUploadedFileID IN ($inParam)',
     );
     );
     final files = _convertToFiles(results);
     final files = _convertToFiles(results);
-    final result = <int, File>{};
     for (final file in files) {
     for (final file in files) {
       result[file.uploadedFileID] = file;
       result[file.uploadedFileID] = file;
     }
     }

+ 0 - 3
lib/services/deduplication_service.dart

@@ -19,9 +19,6 @@ class DeduplicationService {
   Future<List<DuplicateFiles>> getDuplicateFiles() async {
   Future<List<DuplicateFiles>> getDuplicateFiles() async {
     try {
     try {
       DuplicateFilesResponse dupes = await _fetchDuplicateFileIDs();
       DuplicateFilesResponse dupes = await _fetchDuplicateFileIDs();
-      if (dupes.duplicates.isEmpty) {
-        return <DuplicateFiles>[];
-      }
       final ids = <int>[];
       final ids = <int>[];
       for (final dupe in dupes.duplicates) {
       for (final dupe in dupes.duplicates) {
         ids.addAll(dupe.fileIDs);
         ids.addAll(dupe.fileIDs);