[#2722] use already referenced non-null variable mainFragmentViewModel

TeamAmaze · Jan 1, 2025 · 255f9f0 · 255f9f0
1 parent 77f5ee6
commit 255f9f0
Showing 1 changed file with 7 additions and 8 deletions.
diff --git a/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/LoadFilesListTask.java b/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/LoadFilesListTask.java
@@ -202,7 +202,7 @@ public LoadFilesListTask(
     if (list != null
         && !(openmode == OpenMode.CUSTOM
             && (("5").equals(path) || ("6").equals(path) || ("7").equals(path)))) {
-      postListCustomPathProcess(list, mainFragment);
+      postListCustomPathProcess(list, mainFragmentViewModel);
     }
 
     return new Pair<>(openmode, list);
@@ -301,13 +301,12 @@ private List<LayoutElementParcelable> getCachedMediaList(
   }
 
   private void postListCustomPathProcess(
-      @NonNull List<LayoutElementParcelable> list, @NonNull MainFragment mainFragment) {
+      @NonNull List<LayoutElementParcelable> list,
+      @NonNull MainFragmentViewModel mainFragmentViewModel) {
 
     SortType sortType = SortHandler.getSortType(context.get(), path);
 
-    MainFragmentViewModel viewModel = mainFragment.getMainFragmentViewModel();
-
-    if (viewModel == null) {
+    if (mainFragmentViewModel == null) {
       LOG.error("MainFragmentViewModel is null, this is a bug");
       return;
     }
@@ -322,13 +321,13 @@ private void postListCustomPathProcess(
       }
 
       if (layoutElementParcelable.isDirectory) {
-        viewModel.incrementFolderCount();
+        mainFragmentViewModel.incrementFolderCount();
       } else {
-        viewModel.incrementFileCount();
+        mainFragmentViewModel.incrementFileCount();
       }
     }
 
-    Collections.sort(list, new FileListSorter(viewModel.getDsort(), sortType));
+    Collections.sort(list, new FileListSorter(mainFragmentViewModel.getDsort(), sortType));
   }
 
   private @Nullable LayoutElementParcelable createListParcelables(HybridFileParcelable baseFile) {