Merge pull request #754 from Flow-Launcher/fix_converter_casting_null

Dev branch: fix issue with converters casting to null values

Author: jjw24

Flow.Launcher/Converters/OpenResultHotkeyVisibilityConverter.cs

@@ -16,12 +16,10 @@ public class OpenResultHotkeyVisibilityConverter : IValueConverter
         public object Convert(object value, Type targetType, object parameter, CultureInfo culture)
         {
             var hotkeyNumber = int.MaxValue;
-             
-            if (value is ListBoxItem listBoxItem)
-            {
-                ListBox listBox = ItemsControl.ItemsControlFromItemContainer(listBoxItem) as ListBox;
+
+            if (value is ListBoxItem listBoxItem
+                && ItemsControl.ItemsControlFromItemContainer(listBoxItem) is ListBox listBox)
                 hotkeyNumber = listBox.ItemContainerGenerator.IndexFromContainer(listBoxItem) + 1;
-            }
 
             return hotkeyNumber <= MaxVisibleHotkeys ? Visibility.Visible : Visibility.Collapsed;
         }

Flow.Launcher/Converters/OrdinalConverter.cs

@@ -8,11 +8,9 @@ public class OrdinalConverter : IValueConverter
     {
         public object Convert(object value, System.Type targetType, object parameter, CultureInfo culture)
         {
-            if (value is ListBoxItem listBoxItem)
-            {
-                ListBox listBox = ItemsControl.ItemsControlFromItemContainer(listBoxItem) as ListBox;
+            if (value is ListBoxItem listBoxItem
+                && ItemsControl.ItemsControlFromItemContainer(listBoxItem) is ListBox listBox)
                 return listBox.ItemContainerGenerator.IndexFromContainer(listBoxItem) + 1;
-            }
 
             return 0;
         }