added try catch to handle exceptions at mainWindow

DineshSolanki · Aug 5, 2021 · 317aacf · 317aacf
1 parent 38e0e30
commit 317aacf
Show file tree

Hide file tree

Showing 10 changed files with 86 additions and 87 deletions.
diff --git a/FoliCon/Models/BackdropSize.cs b/FoliCon/Models/BackdropSize.cs
@@ -1,10 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
-namespace FoliCon.Models
+namespace FoliCon.Models
 {
     public static class BackdropSize
     {

diff --git a/FoliCon/Models/Languages.cs b/FoliCon/Models/Languages.cs
@@ -1,10 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
-namespace FoliCon.Models
+namespace FoliCon.Models
 {
     public enum Languages
     {

diff --git a/FoliCon/Models/PosterSize.cs b/FoliCon/Models/PosterSize.cs
@@ -1,10 +1,4 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Text;
-using System.Threading.Tasks;
-
-namespace FoliCon.Models
+namespace FoliCon.Models
 {
     public static class PosterSize
     {

diff --git a/FoliCon/Modules/TMDB.cs b/FoliCon/Modules/TMDB.cs
@@ -150,6 +150,7 @@ public static ObservableCollection<ListItem> ExtractTvDetailsIntoListItem(Search
         /// <param name="result">Search Response</param>
         /// <param name="resultType">Type of search Response.</param>
         /// <param name="fullFolderPath">Full Path to the current Media Folder</param>
+        /// <param name="rating">Rating for media</param>
         /// TODO: Merge parameter response and resultType.
         public void ResultPicked(dynamic result, string resultType, string fullFolderPath, string rating = "")
         {

diff --git a/FoliCon/Modules/Util.cs b/FoliCon/Modules/Util.cs
@@ -531,5 +531,39 @@ public static void WriteApiConfiguration(string tmdbkey, string igdbClientId, st
             settings.DevClientSecret = dartClientSecret;
             settings.Save();
         }
+
+        public static CultureInfo GetCultureInfoByLanguage(Languages language)
+        {
+            CultureInfo cultureInfo;
+            switch (language)
+            {
+                case Languages.English:
+                    ConfigHelper.Instance.SetLang("en");
+                    cultureInfo = new CultureInfo("en-US");
+                    break;
+                case Languages.Spanish:
+                    ConfigHelper.Instance.SetLang("es");
+                    cultureInfo = new CultureInfo("es-MX");
+                    break;
+                case Languages.Arabic:
+                    ConfigHelper.Instance.SetLang("ar");
+                    cultureInfo = new CultureInfo("ar-SA");
+                    break;
+                case Languages.Russian:
+                    ConfigHelper.Instance.SetLang("ru");
+                    cultureInfo = new CultureInfo("ru-RU");
+                    break;
+                case Languages.Hindi:
+                    ConfigHelper.Instance.SetLang("hi");
+                    cultureInfo = new CultureInfo("hi-IN");
+                    break;
+                default:
+                    ConfigHelper.Instance.SetLang("en");
+                    cultureInfo = new CultureInfo("en-US");
+                    break;
+            }
+
+            return cultureInfo;
+        }
     }
 }
diff --git a/FoliCon/ViewModels/MainWindowViewModel.cs b/FoliCon/ViewModels/MainWindowViewModel.cs
@@ -11,8 +11,10 @@
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Data;
+using System.Globalization;
 using System.IO;
 using System.Net.NetworkInformation;
+using System.Threading;
 using HandyControl.Tools.Extension;
 
 namespace FoliCon.ViewModels
@@ -228,52 +230,61 @@ private void LoadMethod()
 
         private async void SearchAndMakeMethod()
         {
-            if (Directory.Exists(SelectedFolder))
-            {
-                Fnames = Util.GetFolderNames(SelectedFolder);
-            }
-            else
+            try
             {
-                MessageBox.Show(CustomMessageBox.Error(LangProvider.GetLang("FolderDoesNotExist"), LangProvider.GetLang("InvalidPath")));
-                return;
-            }
+                if (Directory.Exists(SelectedFolder))
+                {
+                    Fnames = Util.GetFolderNames(SelectedFolder);
+                }
+                else
+                {
+                    MessageBox.Show(CustomMessageBox.Error(LangProvider.GetLang("FolderDoesNotExist"), LangProvider.GetLang("InvalidPath")));
+                    return;
+                }
 
-            if (Fnames.Count != 0)
-            {
-                if (Util.IsNetworkAvailable())
+                if (Fnames.Count != 0)
                 {
-                    if (!IsObjectsInitialized)
+                    if (Util.IsNetworkAvailable())
                     {
-                        InitializeClientObjects();
-                        IsObjectsInitialized = true;
-                    }
+                        if (!IsObjectsInitialized)
+                        {
+                            InitializeClientObjects();
+                            IsObjectsInitialized = true;
+                        }
 
-                    StatusBarProperties.ResetData();
-                    FinalListViewData.Data.Clear();
-                    StatusBarProperties.TotalFolders = Fnames.Count;
-                    PrepareForSearch();
-                    if (IconMode == "Poster")
-                        await ProcessPosterModeAsync();
+                        StatusBarProperties.ResetData();
+                        FinalListViewData.Data.Clear();
+                        StatusBarProperties.TotalFolders = Fnames.Count;
+                        PrepareForSearch();
+                        if (IconMode == "Poster")
+                            await ProcessPosterModeAsync();
+                        else
+                        {
+                            ProcessProfessionalMode();
+                        }
+                        StatusBarProperties.TotalIcons = BusyIndicatorProperties.Max =
+                            StatusBarProperties.ProgressBarData.Max = _imgDownloadList.Count;
+                        BusyIndicatorProperties.Text = LangProvider.GetLang("DownloadingIconWithCount").Format(1, _imgDownloadList.Count);
+                        if (_imgDownloadList.Count > 0)
+                            await StartDownloadingAsync();
+                        else
+                            IsMakeEnabled = true;
+                    }
                     else
                     {
-                        ProcessProfessionalMode();
+                        MessageBox.Show(CustomMessageBox.Error(LangProvider.GetLang("NoInternet"), LangProvider.GetLang("NetworkError")));
                     }
-                    StatusBarProperties.TotalIcons = BusyIndicatorProperties.Max =
-                        StatusBarProperties.ProgressBarData.Max = _imgDownloadList.Count;
-                    BusyIndicatorProperties.Text = LangProvider.GetLang("DownloadingIconWithCount").Format(1, _imgDownloadList.Count);
-                    if (_imgDownloadList.Count > 0)
-                        await StartDownloadingAsync();
-                    else
-                        IsMakeEnabled = true;
                 }
                 else
                 {
-                    MessageBox.Show(CustomMessageBox.Error(LangProvider.GetLang("NoInternet"), LangProvider.GetLang("NetworkError")));
+                    MessageBox.Show(CustomMessageBox.Warning(LangProvider.GetLang("IconsAlready"), LangProvider.GetLang("FolderError")));
                 }
             }
-            else
+            catch (Exception e)
             {
-                MessageBox.Show(CustomMessageBox.Warning(LangProvider.GetLang("IconsAlready"), LangProvider.GetLang("FolderError")));
+                MessageBox.Show(CustomMessageBox.Error(e.Message, "Exception Occurred"));
+                StatusBarProperties.ResetData();
+                IsMakeEnabled = true;
             }
         }
 

diff --git a/FoliCon/ViewModels/PosterPickerViewModel.cs b/FoliCon/ViewModels/PosterPickerViewModel.cs
@@ -6,10 +6,8 @@
 using Prism.Mvvm;
 using Prism.Services.Dialogs;
 using System;
-using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Globalization;
-using System.Linq;
 using FoliCon.Properties.Langs;
 using TMDbLib.Objects.General;
 using TMDbLib.Objects.Search;

diff --git a/FoliCon/ViewModels/ProSearchResultViewModel.cs b/FoliCon/ViewModels/ProSearchResultViewModel.cs
@@ -92,12 +92,11 @@ private async void PrepareForSearch()
 
         private async Task Search(string query, int offset = 0)
         {
-            int lastIndex;
             Index = 0;
             while (true)
             {
                 var searchResult = await DArtObject.Browse(query, offset);
-                lastIndex = Index;
+                var lastIndex = Index;
                 if (searchResult.Results?.Length > 0)
                 {
                     TotalPosters = searchResult.Results.Length + offset;

diff --git a/FoliCon/ViewModels/SearchResultViewModel.cs b/FoliCon/ViewModels/SearchResultViewModel.cs
@@ -4,7 +4,6 @@
 using Prism.Mvvm;
 using Prism.Services.Dialogs;
 using System;
-using System.Collections;
 using System.Collections.Generic;
 using System.Globalization;
 using System.Windows.Input;

diff --git a/FoliCon/Views/MainWindow.xaml.cs b/FoliCon/Views/MainWindow.xaml.cs
@@ -42,35 +42,10 @@ private void CmbLanguage_OnSelectionChanged(object sender, SelectionChangedEvent
             if (CmbLanguage.SelectedItem is null)
                 return;
             var selectedLanguage = (Languages)CmbLanguage.SelectedValue;
-            switch (selectedLanguage)
-            {
-                case Languages.English:
-                    ConfigHelper.Instance.SetLang("en");
-                    LangProvider.Culture = new CultureInfo("en-US");
-                    break;
-                case Languages.Spanish:
-                    ConfigHelper.Instance.SetLang("es");
-                    LangProvider.Culture = new CultureInfo("es-MX");
-                    break;
-                case Languages.Arabic:
-                    ConfigHelper.Instance.SetLang("ar");
-                    LangProvider.Culture = new CultureInfo("ar-SA");
-                    break;
-                case Languages.Russian:
-                    ConfigHelper.Instance.SetLang("ru");
-                    LangProvider.Culture = new CultureInfo("ru-RU");
-                    break;
-                case Languages.Hindi:
-                    ConfigHelper.Instance.SetLang("hi");
-                    LangProvider.Culture = new CultureInfo("hi-IN");
-                    break;
-                default:
-                    ConfigHelper.Instance.SetLang("en");
-                    LangProvider.Culture = new CultureInfo("en-US");
-                    break;
-            }
-            Thread.CurrentThread.CurrentCulture = LangProvider.Culture;
-            Thread.CurrentThread.CurrentUICulture = LangProvider.Culture;
+            var cultureInfo = Util.GetCultureInfoByLanguage(selectedLanguage);
+            LangProvider.Culture = cultureInfo;
+            Thread.CurrentThread.CurrentCulture = cultureInfo;
+            Thread.CurrentThread.CurrentUICulture = cultureInfo;
             Kernel32.SetThreadUILanguage((ushort)Thread.CurrentThread.CurrentUICulture.LCID);
             if (FinalList is not null)
             {