Add Localizable attribute and improve encapsulation

Added the `[Localizable(false)]` attribute to various classes to exclude them from localization. Improved encapsulation by modifying access modifiers for properties and fields, ensuring private or protected access where appropriate. These changes help in better resource management and code maintainability.

FoliCon/Models/Constants/GlobalVariables.cs

@@ -17,7 +17,7 @@ public static IconOverlay IconOverlayType()
         };
     }
 
-    public static readonly string MediaInfoFile = "info.folicon";
+    public const string MediaInfoFile = "info.folicon";
 
     private static string IconOverlayTypeString =>
         Services.Tracker.Store.GetData("PosterIconConfigViewModel")["p.IconOverlay"].ToString();

FoliCon/Modules/Configuration/AssemblyInfo.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.Configuration;
 
+[Localizable(false)]
 internal static class AssemblyInfo
 {
     public static string GetVersion()

FoliCon/Modules/Convertor/BoolToPermissionTextConvertor.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.Convertor;
 
+[Localizable(false)]
 public class BoolToPermissionTextConverter : IValueConverter
 {
     public object Convert(object value, Type targetType, object parameter, CultureInfo culture)

FoliCon/Modules/Convertor/ImageCacheConverter.cs

@@ -4,6 +4,7 @@
 /// Converts an image path to a BitmapImage with caching to improve performance and thread safety.
 /// CREDIT: https://stackoverflow.com/a/37652158/8076598
 /// </summary>
+[Localizable(false)]
 public class ImageCacheConverter : IValueConverter
 {
     public object Convert(object value, Type targetType, object parameter, CultureInfo culture)

FoliCon/Modules/DeviantArt/DArt.cs

@@ -2,29 +2,30 @@
 
 namespace FoliCon.Modules.DeviantArt;
 
+[Localizable(false)]
 public class DArt : BindableBase
 {
     private string _clientAccessToken;
     private string _clientSecret;
-    private string _clientId;
+    private readonly string _clientId;
 
     private readonly MemoryCache _cache = new(new MemoryCacheOptions());
 
     private static readonly JsonSerializerSettings SerializerSettings = new() { NullValueHandling = NullValueHandling.Ignore };
-    
-    public string ClientId
+
+    private string ClientId
     {
         get => _clientId;
-        set => SetProperty(ref _clientId, value);
+        init => SetProperty(ref _clientId, value);
     }
 
-    public string ClientSecret
+    private string ClientSecret
     {
         get => _clientSecret;
-        set => SetProperty(ref _clientSecret, value);
+        init => SetProperty(ref _clientSecret, value);
     }
 
-    public string ClientAccessToken
+    private string ClientAccessToken
     {
         get => _clientAccessToken;
         set => SetProperty(ref _clientAccessToken, value);

FoliCon/Modules/Extension/BindingPathExtensions.cs

@@ -2,6 +2,7 @@
 
 namespace FoliCon.Modules.Extension;
 
+[Localizable(false)]
 public static class BindingPathExtensions
 {
     private static readonly ImageCacheConverter ImageCacheConverter = new ();

FoliCon/Modules/Extension/DialogServiceExtensions.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.Extension;
 
+[Localizable(false)]
 public static class DialogServiceExtensions
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/Modules/Extension/Extensions.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.Extension;
 
+[Localizable(false)]
 public static class Extensions
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/Modules/Extension/StreamExtension.cs

@@ -5,6 +5,7 @@
 
 namespace FoliCon.Modules.Extension;
 
+[Localizable(false)]
 public static class StreamExtensions
 {
     private static readonly ReaderOptions ReaderOptions = new() { ArchiveEncoding = 

FoliCon/Modules/IGDB/IgdbDataTransformer.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.IGDB;
 
+[Localizable(false)]
 public class IgdbDataTransformer(ref List<PickedListItem> listDataTable, ref List<ImageToDownload> imgDownloadList)
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/Modules/IGDB/IgdbService.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.IGDB;
 
+[Localizable(false)]
 public class IgdbService(ref IGDBClient serviceClient)
 {
     private const string ServiceClientIsNotInitialized = "Service Client is not initialized.";
@@ -19,7 +20,7 @@ public async Task<ResultResponse> SearchGameAsync(string query)
         }
 
         var r = await _serviceClient.QueryAsync<Game>(IGDBClient.Endpoints.Games,
-            $"search \"{query}\"; fields artworks.image_id, name,first_release_date,total_rating,summary,cover.*;");
+            $"""search "{query}"; fields artworks.image_id, name,first_release_date,total_rating,summary,cover.*;""");
 
         return new ResultResponse
         {

FoliCon/Modules/LangProvider.cs

@@ -2,6 +2,7 @@
 
 namespace FoliCon.Properties.Langs;
 
+[Localizable(false)]
 public class LangProvider : INotifyPropertyChanged
 {
     private static string _cultureInfoStr;

FoliCon/Modules/Media/PngToIcoService.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.Media;
 
+[Localizable(false)]
 public static class PngToIcoService
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/Modules/TMDB/TMDBService.cs

@@ -3,6 +3,7 @@
 
 namespace FoliCon.Modules.TMDB;
 
+[Localizable(false)]
 internal class TmdbService
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
@@ -114,8 +115,8 @@ public async Task<ResultResponse> SearchWithParamsAsync(ParsedTitle parsedTitle,
     {
         Logger.Debug("Searching for {ParsedTitle} in {SearchMode}", parsedTitle, searchMode);
 
-        var mediaType = "";
-        object? searchResult = null;
+        string mediaType;
+        object? searchResult;
 
         switch (searchMode)
         {

FoliCon/Modules/UI/CustomMessageBox.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.UI;
 
+[Localizable(false)]
 internal static class CustomMessageBox
 {
     public static MessageBoxInfo Ask(string messageBoxText, string caption)

FoliCon/Modules/UI/FolderDragDropHelper.cs

@@ -59,7 +59,7 @@ private static void Control_DragOver(object sender, DragEventArgs e)
         var dt = e.Data.GetData(DataFormats.FileDrop);
         var data = (dt as Array)?.GetValue(0)?.ToString();
 
-        Logger.Trace("Control_DragOver: Data: {Data}", data ?? "null");
+        Logger.Trace("Control_DragOver: Data: {Data}", data);
 
         e.Effects = Directory.Exists(data) ? DragDropEffects.Link : DragDropEffects.None;
         e.Handled = true;

FoliCon/Modules/UI/HandyWindow.cs

@@ -3,6 +3,7 @@
 
 namespace FoliCon.Modules.UI;
 
+[Localizable(false)]
 public class HandyWindow : Window, IDialogWindow
 {
     static HandyWindow()

FoliCon/Modules/UI/ListViewClickSortBehavior.cs

@@ -62,7 +62,7 @@ private void OnClick(object sender, RoutedEventArgs e)
             ? Application.Current.Resources["HeaderTemplateArrowUp"] as DataTemplate
             : Application.Current.Resources["HeaderTemplateArrowDown"] as DataTemplate;
 
-        // Remove arrow from previously sorted header
+        // Remove arrow from the previously sorted header
         if (_lastHeaderClicked != null && _lastHeaderClicked != headerClicked)
         {
             _lastHeaderClicked.Column.HeaderTemplate = null;

FoliCon/Modules/utils/CultureUtils.cs

@@ -1,24 +1,25 @@
 namespace FoliCon.Modules.utils;
 
+[Localizable(false)]
 public static class CultureUtils
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
 
+    private static readonly Dictionary<Languages, string> LanguageCodes = new()
+    {
+        { Languages.English, "en-US" },
+        { Languages.Spanish, "es-MX" },
+        { Languages.Arabic, "ar-SA" },
+        { Languages.Russian, "ru-RU" },
+        { Languages.Hindi, "hi-IN" },
+        { Languages.Portuguese, "pt-PT" }
+    };
+
     public static CultureInfo GetCultureInfoByLanguage(Languages language)
     {
         Logger.Debug("Getting CultureInfo by Language: {Language}", language);
-
-        var languageCodes = new Dictionary<Languages, string>
-        {
-            { Languages.English, "en-US" },
-            { Languages.Spanish, "es-MX" },
-            { Languages.Arabic, "ar-SA" },
-            { Languages.Russian, "ru-RU" },
-            { Languages.Hindi, "hi-IN" },
-            { Languages.Portuguese, "pt-PT" }
-        };
-
-        var langCode = languageCodes.GetValueOrDefault(language, "en-US");
+
+        var langCode = LanguageCodes.GetValueOrDefault(language, "en-US");
         ConfigHelper.Instance.SetLang(langCode.Split("-")[0]);
 
         return new CultureInfo(langCode);

FoliCon/Modules/utils/DataUtils.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.utils;
 
+[Localizable(false)]
 public static class DataUtils
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
@@ -24,10 +25,10 @@ public static int GetResultCount(bool isPickedById, dynamic result, string searc
     /// <returns>Formatted Rating value.</returns>
     public static string FormatRating(double ratingInput)
     {
-        Logger.Debug("Start FormatRatingString() - Input received : {RatingInput}", ratingInput);
+        Logger.Trace("Start FormatRatingString() - Input received : {RatingInput}", ratingInput);
         var decimalPart = ratingInput % 1;
         var formattedRatingValue = decimalPart > 0 ? ratingInput.ToString("0.##") : ratingInput.ToString("0");
-        Logger.Debug("End FormatRatingString() - Formatted Rating : {FormattedRatingValue}", formattedRatingValue);
+        Logger.Trace("End FormatRatingString() - Formatted Rating : {FormattedRatingValue}", formattedRatingValue);
         return formattedRatingValue;
     }
 

FoliCon/Modules/utils/DialogUtils.cs

@@ -54,10 +54,7 @@ public static VistaFolderBrowserDialog NewFolderBrowserDialog(string description
         };
         return folderBrowser;
     }
-    public static VistaOpenFileDialog NewOpenFileDialog(string description)
-    {
-        return NewOpenFileDialog(description, "All files (*.*)|*.*");
-    }
+
     public static VistaOpenFileDialog NewOpenFileDialog(string description, string filter)
     {
         Logger.Debug("Creating New Open File Dialog");

FoliCon/Modules/utils/StaTask.cs

@@ -1,6 +1,7 @@
 namespace FoliCon.Modules.utils;
 
 //Taken from-https://stackoverflow.com/a/16722767/8076598
+[Localizable(false)]
 public static class StaTask
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/Modules/utils/TitleCleaner.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.Modules.utils;
 
+[Localizable(false)]
 internal static class TitleCleaner
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/ViewModels/AboutBoxViewModel.cs

@@ -1,5 +1,6 @@
 namespace FoliCon.ViewModels;
 
+[Localizable(false)]
 public class AboutBoxViewModel : BindableBase, IDialogAware
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

FoliCon/ViewModels/ApiConfigurationViewModel.cs

@@ -2,7 +2,7 @@
 
 public class ApiConfigurationViewModel : BindableBase, IDialogAware
 {
-    private string _title = "API Configuration";
+    private string _title = Lang.APIKeysConfiguration;
     private string _dartClient;
     private string _dartClientId;
     private string _tmdbKey;

FoliCon/ViewModels/CustomIconControlViewModel.cs

@@ -4,6 +4,7 @@
 
 namespace FoliCon.ViewModels;
 
+[Localizable(false)]
 public class CustomIconControlViewModel : BindableBase, IDialogAware, IFileDragDropTarget
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
@@ -54,7 +55,7 @@ public bool IsUndoEnable
         set => SetProperty(ref _isUndoEnable, value);
     }
 
-    public string SelectedDirectory
+    private string SelectedDirectory
     {
         get => _selectedDirectory;
         set
@@ -69,7 +70,7 @@ public string SelectedDirectory
         }
     }
 
-    public string SelectedIconsDirectory
+    private string SelectedIconsDirectory
     {
         get => _selectedIconsDirectory;
         set
@@ -88,17 +89,17 @@ public string SelectedIconsDirectory
     public ObservableCollection<string> Directories
     {
         get => _directories;
-        set => SetProperty(ref _directories, value);
+        private set => SetProperty(ref _directories, value);
     }
 
     public ObservableCollection<string> Icons
     {
         get => _icons;
-        set => SetProperty(ref _icons, value);
+        private set => SetProperty(ref _icons, value);
     }
     public int Index { get => _index; set => SetProperty(ref _index, value); }
     public int TotalIcons { get => _totalIcons; set => SetProperty(ref _totalIcons, value); }
-    public bool StopSearch { get => _stopSearch; set => SetProperty(ref _stopSearch, value); }
+    private bool StopSearch { get => _stopSearch; set => SetProperty(ref _stopSearch, value); }
     #region Declare Delegates
 
     public DelegateCommand LoadDirectory { get; set; }
@@ -201,7 +202,7 @@ protected virtual void CloseDialog(string parameter)
         RaiseRequestClose(new DialogResult(result));
     }
 
-    public virtual void RaiseRequestClose(IDialogResult dialogResult) =>
+    protected virtual void RaiseRequestClose(IDialogResult dialogResult) =>
         RequestClose?.Invoke(dialogResult);
 
     public virtual bool CanCloseDialog() => true;

FoliCon/ViewModels/DialogControlViewModel.cs

@@ -13,15 +13,15 @@ public class DialogControlViewModel : BindableBase, IDialogAware
     public string Message
     {
         get => _message;
-        set => SetProperty(ref _message, value);
+        private set => SetProperty(ref _message, value);
     }
 
     private string _title = "Notification";
 
     public string Title
     {
         get => _title;
-        set => SetProperty(ref _title, value);
+        private set => SetProperty(ref _title, value);
     }
 
     public event Action<IDialogResult> RequestClose;
@@ -39,7 +39,7 @@ protected virtual void CloseDialog(string parameter)
         RaiseRequestClose(new DialogResult(result));
     }
 
-    public virtual void RaiseRequestClose(IDialogResult dialogResult)
+    protected virtual void RaiseRequestClose(IDialogResult dialogResult)
     {
         RequestClose?.Invoke(dialogResult);
     }