Fix few more null checks/refs

U-C-S · Aug 25, 2024 · 3649eef · 3649eef
1 parent 52f60f3
commit 3649eef
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 11 deletions.
diff --git a/Source/Hurl.Library/Models/Ruleset.cs b/Source/Hurl.Library/Models/Ruleset.cs
@@ -7,7 +7,7 @@ public class Ruleset
     [JsonIgnore]
     public Guid Id { get; set; } = Guid.NewGuid();
 
-    public List<string> Rules { get; set; }
+    public List<string> Rules { get; set; } = [];
 
     public string RulesetName { get; set; }
 

diff --git a/Source/Hurl.Settings/ViewModels/StoreRulesetViewModel.cs b/Source/Hurl.Settings/ViewModels/StoreRulesetViewModel.cs
@@ -21,7 +21,7 @@ public class StoreRulesetViewModel : ObservableObject
     public StoreRulesetViewModel()
     {
         Id = Guid.NewGuid();
-        Browsers = State.Settings.GetBrowsers()
+        Browsers = State.Settings.Browsers
             .Select(x => x.Name)
             .ToList();
         Rules = [];
@@ -30,7 +30,7 @@ public StoreRulesetViewModel()
     public StoreRulesetViewModel(Ruleset set)
     {
         Id = set.Id;
-        Browsers = State.Settings.GetBrowsers()
+        Browsers = State.Settings.Browsers
             .Select(x => x.Name)
             .ToList();
         Name = set?.RulesetName;
@@ -44,9 +44,9 @@ public StoreRulesetViewModel(Ruleset set)
         if (set?.AltLaunchIndex is int altLaunchIndex)
         {
             List<string> altLaunchList = ["< None >"];
-            var x = State.Settings.GetBrowsers()[SelectedBrowser]
-                .AlternateLaunches
-                .Select(x => x.ItemName)
+            var x = State.Settings.Browsers[SelectedBrowser]
+                ?.AlternateLaunches
+                ?.Select(x => x.ItemName)
                 .ToList();
             altLaunchList.AddRange(x);
             AltLaunches = altLaunchList;
@@ -101,8 +101,7 @@ public Ruleset ToRuleSet()
         {
             Id = Id,
             RulesetName = Name ?? "",
-            // TODO: dont do this, do proper validation
-            BrowserName = Browsers.ElementAtOrDefault(SelectedBrowser),
+            BrowserName = Browsers[SelectedBrowser],
             Rules = Rules.Select(x => x.ToString()).ToList(),
             AltLaunchIndex = SelectedAltLaunch > 0 ? SelectedAltLaunch - 1 : null
         };

diff --git a/Source/Hurl.Settings/Views/RulesetPage.xaml.cs b/Source/Hurl.Settings/Views/RulesetPage.xaml.cs
@@ -133,20 +133,20 @@ private async void ViewRuleset(object sender, RoutedEventArgs e)
 
     private void ClickMoveUpRuleset(object sender, RoutedEventArgs e)
     {
-        var Id = (Guid)(sender as MenuFlyoutItem).Tag;
+        var Id = (Guid)(sender as MenuFlyoutItem)!.Tag;
 
         ViewModel.MoveRulesetUp(Id);
     }
     private void ClickMoveDownRuleset(object sender, RoutedEventArgs e)
     {
-        var Id = (Guid)(sender as MenuFlyoutItem).Tag;
+        var Id = (Guid)(sender as MenuFlyoutItem)!.Tag;
 
         ViewModel.MoveRulesetDown(Id);
     }
 
     private void ClickDeleteRuleset(object sender, RoutedEventArgs e)
     {
-        var Id = (Guid)(sender as MenuFlyoutItem).Tag;
+        var Id = (Guid)(sender as MenuFlyoutItem)!.Tag;
 
         ViewModel.DeleteRuleset(Id);
     }