Chore: Updates

Author: BornToBeRoot

Source/NETworkManager/ViewModels/SettingsProfilesViewModel.cs

@@ -48,6 +48,9 @@ public string Location
     /// </summary>
     public bool IsLocationManagedByPolicy => !string.IsNullOrWhiteSpace(PolicyManager.Current?.Profiles_FolderLocation);
 
+    /// <summary>
+    /// Private field of <see cref="IsLocationChanged" /> property.
+    /// </summary>
     private bool _isLocationChanged;
 
     /// <summary>
@@ -66,6 +69,9 @@ public bool IsLocationChanged
         }
     }
 
+    /// <summary>
+    /// Private field of <see cref="IsDefaultLocation" /> property.
+    /// </summary>
     private bool _isDefaultLocation;
 
     /// <summary>
@@ -84,8 +90,14 @@ public bool IsDefaultLocation
         }
     }
 
+    /// <summary>
+    /// Private field of <see cref="ProfileFiles" /> property.
+    /// </summary>
     private readonly ICollectionView _profileFiles;
 
+    /// <summary>
+    /// Gets the collection view of profile files.
+    /// </summary>    
     public ICollectionView ProfileFiles
     {
         get => _profileFiles;
@@ -99,8 +111,15 @@ private init
         }
     }
 
+    /// <summary>
+    /// Private field of <see cref="SelectedProfileFile" /> property.
+    /// </summary>
+
     private ProfileFileInfo _selectedProfileFile;
 
+    /// <summary>
+    /// Gets or sets the currently selected profile file information.
+    /// </summary>    
     public ProfileFileInfo SelectedProfileFile
     {
         get => _selectedProfileFile;
@@ -114,8 +133,15 @@ public ProfileFileInfo SelectedProfileFile
         }
     }
 
+    /// <summary>
+    /// Private field of <see cref="IsDailyBackupEnabled" /> property.
+    /// </summary>
     private bool _isDailyBackupEnabled;
 
+
+    /// <summary>
+    /// Gets or sets a value indicating whether daily backups are enabled.
+    /// </summary>
     public bool IsDailyBackupEnabled
     {
         get => _isDailyBackupEnabled;
@@ -132,8 +158,14 @@ public bool IsDailyBackupEnabled
         }
     }
 
+    /// <summary>
+    /// Private field of <see cref="MaximumNumberOfBackups" /> property.
+    /// </summary>
     private int _maximumNumberOfBackups;
 
+    /// <summary>
+    /// Gets or sets the maximum number of backups to keep.
+    /// </summary>
     public int MaximumNumberOfBackups
     {
         get => _maximumNumberOfBackups;
@@ -153,6 +185,9 @@ public int MaximumNumberOfBackups
 
     #region Constructor, LoadSettings
 
+    /// <summary>
+    /// Initializes a new instance of the <see cref="SettingsProfilesViewModel" /> class and loads the current profile files.
+    /// </summary>    
     public SettingsProfilesViewModel()
     {
         _isLoading = true;
@@ -168,6 +203,9 @@ public SettingsProfilesViewModel()
         _isLoading = false;
     }
 
+    /// <summary>
+    /// Load view specific settings.
+    /// </summary>
     private void LoadSettings()
     {
         Location = ProfileManager.GetProfilesFolderLocation();
@@ -179,14 +217,6 @@ private void LoadSettings()
     #endregion
 
     #region ICommands & Actions
-
-    public ICommand OpenLocationCommand => new RelayCommand(_ => OpenLocationAction());
-
-    private static void OpenLocationAction()
-    {
-        Process.Start("explorer.exe", ProfileManager.GetProfilesFolderLocation());
-    }
-
     /// <summary>
     /// Gets the command that opens the location folder selection dialog.
     /// </summary>    
@@ -209,14 +239,16 @@ private void BrowseLocationFolderAction()
             Location = dialog.SelectedPath;
     }
 
+
     /// <summary>
-    /// Sets the location path based on the provided drag-and-drop input.
-    /// </summary>
-    /// <param name="path">The path to set as the location.</param>
-    public void SetLocationPathFromDragDrop(string path)
+    /// Gets the command that initiates the action to change the location.
+    /// </summary>    
+    public ICommand OpenLocationCommand => new RelayCommand(_ => OpenLocationAction());
+
+    private static void OpenLocationAction()
     {
-        Location = path;
-    }
+        Process.Start("explorer.exe", ProfileManager.GetProfilesFolderLocation());
+    }    
 
     /// <summary>
     /// Gets the command that initiates the action to change the location.
@@ -238,13 +270,8 @@ private async Task ChangeLocationAction()
         if (!result)
             return;
 
-        // Save settings at the current location before changing it to prevent
-        // unintended saves to the new location (e.g., triggered by background timer or the app close & restart).
-        SettingsManager.Save();
-
         // Set new location in SettingsInfo
         SettingsManager.Current.Profiles_FolderLocation = Location;
-        SettingsManager.Save();
 
         // Restart the application
         (Application.Current.MainWindow as MainWindow)?.RestartApplication();
@@ -270,14 +297,9 @@ private async Task RestoreDefaultLocationActionAsync()
         if (!result)
             return;
 
-        // Save settings at the current location before changing it to prevent
-        // unintended saves to the new location (e.g., triggered by background timer or the app close & restart).
-        SettingsManager.Save();
-
         // Clear custom location to revert to default
         SettingsManager.Current.Profiles_FolderLocation = null;
-        SettingsManager.Save();
-
+        
         // Restart the application
         (Application.Current.MainWindow as MainWindow)?.RestartApplication();
     }
@@ -551,4 +573,15 @@ await DialogHelper.ShowMessageAsync(Application.Current.MainWindow,
     }
 
     #endregion
+
+    #region Methods
+    /// <summary>
+    /// Sets the location path based on the provided drag-and-drop input.
+    /// </summary>
+    /// <param name="path">The path to set as the location.</param>
+    public void SetLocationPathFromDragDrop(string path)
+    {
+        Location = path;
+    }
+    #endregion
 }

Source/NETworkManager/ViewModels/SettingsSettingsViewModel.cs

@@ -159,8 +159,8 @@ public SettingsSettingsViewModel()
     }
 
     /// <summary>
-    /// Loads the application settings from the current settings folder location.
-    /// </summary>    
+    /// Load view specific settings.
+    /// </summary>
     private void LoadSettings()
     {
         Location = SettingsManager.GetSettingsFolderLocation();
@@ -172,36 +172,6 @@ private void LoadSettings()
     #endregion
 
     #region ICommands & Actions
-
-    /// <summary>
-    /// Gets the command that opens a location when executed.
-    /// </summary>    
-    public ICommand OpenLocationCommand => new RelayCommand(_ => OpenLocationAction());
-
-    /// <summary>
-    /// Opens the settings folder location in Windows Explorer.
-    /// </summary>    
-    private static void OpenLocationAction()
-    {
-        Process.Start("explorer.exe", SettingsManager.GetSettingsFolderLocation());
-    }
-
-    /// <summary>
-    /// Gets the command that resets the application settings to their default values.
-    /// </summary>    
-    public ICommand ResetSettingsCommand => new RelayCommand(_ => ResetSettingsAction());
-
-    /// <summary>
-    /// Resets the application settings to their default values.
-    /// </summary>    
-    private void ResetSettingsAction()
-    {
-        ResetSettings().ConfigureAwait(false);
-    }
-
-    #endregion
-
-    #region Methods
     /// <summary>
     /// Gets the command that opens the location folder selection dialog.
     /// </summary>    
@@ -210,10 +180,7 @@ private void ResetSettingsAction()
     /// <summary>
     /// Opens a dialog that allows the user to select a folder location and updates the Location property with the
     /// selected path if the user confirms the selection.
-    /// </summary>
-    /// <remarks>If the Location property is set to a valid directory path, it is pre-selected in the dialog.
-    /// This method does not return a value and is intended for use in a user interface context where folder selection
-    /// is required.</remarks>
+    /// </summary>    
     private void BrowseLocationFolderAction()
     {
         using var dialog = new System.Windows.Forms.FolderBrowserDialog();
@@ -228,12 +195,16 @@ private void BrowseLocationFolderAction()
     }
 
     /// <summary>
-    /// Sets the location path based on the provided drag-and-drop input.
-    /// </summary>
-    /// <param name="path">The path to set as the location. This value cannot be null or empty.</param>
-    public void SetLocationPathFromDragDrop(string path)
+    /// Gets the command that opens a location when executed.
+    /// </summary>    
+    public ICommand OpenLocationCommand => new RelayCommand(_ => OpenLocationAction());
+
+    /// <summary>
+    /// Opens the settings folder location in Windows Explorer.
+    /// </summary>    
+    private static void OpenLocationAction()
     {
-        Location = path;
+        Process.Start("explorer.exe", SettingsManager.GetSettingsFolderLocation());
     }
 
     /// <summary>
@@ -242,12 +213,8 @@ public void SetLocationPathFromDragDrop(string path)
     public ICommand ChangeLocationCommand => new RelayCommand(_ => ChangeLocationAction().ConfigureAwait(false));
 
     /// <summary>
-    /// Prompts the user to confirm and then changes the location of the application's settings folder.
+    /// Prompts the user to confirm and then changes the location of the profiles folder.
     /// </summary>
-    /// <remarks>This method displays a confirmation dialog to the user before changing the settings folder
-    /// location. If the user confirms, it saves the current settings, updates the settings folder location, and
-    /// restarts the application to apply the changes. No action is taken if the user cancels the confirmation
-    /// dialog.</remarks>
     /// <returns>A task that represents the asynchronous operation.</returns>
     private async Task ChangeLocationAction()
     {
@@ -309,19 +276,21 @@ private async Task RestoreDefaultLocationActionAsync()
     }
 
     /// <summary>
-    /// Resets the application settings to their default values and restarts the application after user confirmation.
+    /// Gets the command that resets the application settings to their default values.
+    /// </summary>    
+    public ICommand ResetSettingsCommand => new RelayCommand(_ => ResetSettingsAction().ConfigureAwait(false));
+
+    /// <summary>
+    /// Resets the application settings to their default values.
     /// </summary>
-    /// <remarks>Displays a confirmation dialog to the user before proceeding. If the user confirms, the
-    /// settings are reinitialized to their defaults and the application is restarted. No action is taken if the user
-    /// cancels the confirmation dialog.</remarks>
     /// <returns>A task that represents the asynchronous operation.</returns>
-    private async Task ResetSettings()
+    private async Task ResetSettingsAction()
     {
         var result = await DialogHelper.ShowConfirmationMessageAsync(Application.Current.MainWindow,
-            Strings.ResetSettingsQuestion,
-            Strings.SettingsAreResetAndApplicationWillBeRestartedMessage,
-            ChildWindowIcon.Question,
-            Strings.Reset);
+           Strings.ResetSettingsQuestion,
+           Strings.SettingsAreResetAndApplicationWillBeRestartedMessage,
+           ChildWindowIcon.Question,
+           Strings.Reset);
 
         if (!result)
             return;
@@ -332,5 +301,17 @@ private async Task ResetSettings()
         // Restart the application
         (Application.Current.MainWindow as MainWindow)?.RestartApplication();
     }
+
+    #endregion
+
+    #region Methods
+    /// <summary>
+    /// Sets the location path based on the provided drag-and-drop input.
+    /// </summary>
+    /// <param name="path">The path to set as the location.</param>
+    public void SetLocationPathFromDragDrop(string path)
+    {
+        Location = path;
+    }       
     #endregion
 }