Use the overloading mechanism instead of the optional parameters.

DineshSolanki · Dec 21, 2023 · 60a9dcd · 60a9dcd
1 parent 758fe70
commit 60a9dcd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/FoliCon/Models/Data/PickedListItem.cs b/FoliCon/Models/Data/PickedListItem.cs
@@ -10,9 +10,9 @@ public string FolderName
         set => SetProperty(ref _folderName, value);
     }
 
-    public PickedListItem(string title, string year, string rating, string folder = "", string folderName = "", string poster= "") 
+    public PickedListItem(string title, string year, string rating, string folder, string folderName, string poster) 
         : base(title, year, rating, "", poster, folder)
     {
         FolderName = folderName;
-    }
+    } 
 }
diff --git a/FoliCon/ViewModels/MainWindowViewModel.cs b/FoliCon/ViewModels/MainWindowViewModel.cs
@@ -533,7 +533,11 @@ private void ProcessProfessionalMode()
         StatusBarProperties.AppStatus = "Searching";
         _dialogService.ShowProSearchResult(SelectedFolder, Fnames, _pickedListDataTable, _imgDownloadList,
             _dArtObject, _ => { });
-        if (_pickedListDataTable.Count <= 0) return;
+        if (_pickedListDataTable.Count <= 0)
+        {
+            return;
+        }
+
         Logger.Debug("ProcessProfessionalMode: found {_pickedListDataTable.Rows.Count} results, adding to final list");
         FinalListViewData.Data.AddRange(_pickedListDataTable);
         StatusBarProperties.ProcessedFolder = _pickedListDataTable.Count;