refactor for code quality

Author: DineshSolanki

FoliCon/Models/Constants/MediaTypes.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Models.Constants;
 
+[Localizable(false)]
 public static class MediaTypes
 {
     public const string Game = "Game";

FoliCon/Modules/TMDB/TmdbDataTransformer.cs

@@ -269,7 +269,11 @@ public void ResultPicked(dynamic result, string resultType, string fullFolderPat
 
     private static string PrepareRating(string resultType, string rating, dynamic result)
     {
-        if (!string.IsNullOrWhiteSpace(rating) || resultType == MediaTypes.Collection) return rating;
+        if (!string.IsNullOrWhiteSpace(rating) || resultType == MediaTypes.Collection)
+        {
+            return rating;
+        }
+
         Logger.Debug("Rating is null or empty, getting rating from result");
         rating = result.VoteAverage.ToString(CultureInfo.InvariantCulture);
         Logger.Debug("Rating: {Rating}", rating);
@@ -286,6 +290,7 @@ private string PickResult(dynamic result,
         string localPosterPath)
     {
         id = 0;
+        var mediaType = resultType;
         switch (resultType)
         {
             case MediaTypes.Tv:
@@ -312,11 +317,12 @@ private string PickResult(dynamic result,
                     break;
                 }
             case MediaTypes.Mtv:
-                resultType = SelectMtvType(result, fullFolderPath, rating, isPickedById, out id, folderName, localPosterPath);
+                mediaType = SelectMtvType(result, fullFolderPath, rating, isPickedById, out id, folderName, localPosterPath);
                 break;
+            default: throw new InvalidDataException($"Invalid Result Type: {resultType}");
         }
 
-        return resultType;
+        return mediaType;
     }
 
     private string SelectMtvType(dynamic result, string fullFolderPath, string rating, bool isPickedById, out int id, string folderName, string localPosterPath)

FoliCon/Modules/utils/FileUtils.cs

@@ -324,7 +324,12 @@ public static string GetResourcePath(string resource)
 
     }
 
-    public static async Task CheckForUpdate(bool onlyShowIfUpdateAvailable = false)
+    public static async Task CheckForUpdate()
+    {
+        await CheckForUpdate(false);
+    }
+    
+    public static async Task CheckForUpdate(bool onlyShowIfUpdateAvailable)
     {
         Logger.Debug("Checking for Update");
 

FoliCon/ViewModels/PosterPickerViewModel.cs

@@ -236,7 +236,11 @@ private void ProcessImages(IEnumerable<IImage> images)
         {
             Index = item.Index + 1;
             TryLoadImage(item.Value);
-            if (!StopSearch) continue;
+            if (!StopSearch)
+            {
+                continue;
+            }
+
             Logger.Trace("Stop Search is true, breaking loop");
             break;
         }