Fix pref saving (model validation)

Author: webprofusion-chrisc

src/Certify.UI.Shared/Controls/Settings/General.xaml

@@ -165,6 +165,11 @@
                                     Margin="0,4,0,0"
                                     Content="Enable Modern PFX Alg (OpenSSL 3.0+ compatibility etc)"
                                     IsChecked="{Binding Prefs.UseModernPFXAlgs}" />
+                                <CheckBox
+                                    x:Name="DisableARIChecks"
+                                    Margin="0,4,0,0"
+                                    Content="Disable ACME ARI (don't use renewal dates suggested by CA)"
+                                    IsChecked="{Binding Prefs.DisableARIChecks}" />
 
                             </StackPanel>
 

src/Certify.UI.Shared/ViewModel/AppViewModel/AppViewModel.Settings.cs

@@ -4,6 +4,7 @@
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
+using System.Windows;
 using Certify.Client;
 using Certify.Models;
 using Certify.Models.Config.Migration;
@@ -132,8 +133,25 @@ public virtual async Task SavePreferences()
             await _prefLock.WaitAsync(500);
             try
             {
+                // set datastoreid and feature flags to pass model validation
+                if (Preferences.ConfigDataStoreConnectionId == null)
+                {
+                    Preferences.ConfigDataStoreConnectionId = "0";
+                }
+
+                if (Preferences.FeatureFlags == null)
+                {
+
+                    Preferences.FeatureFlags = [];
+                }
+
                 await _certifyClient.SetPreferences(Preferences);
             }
+            catch (ServiceCommsException ex)
+            {
+                MessageBox.Show($"Unable to save preferences. {ex?.Message}.", "Save Preferences Error", MessageBoxButton.OK, MessageBoxImage.Error);
+
+            }
             catch
             {
                 Debug.WriteLine("Pref wait lock exceeded");