Fixes to serialization of columns and deserialisation of InList filters.

Improvements to sub query filter selector

InABox.Core/CoreUtils.cs

@@ -478,8 +478,17 @@ namespace InABox.Core
                     return Guid.Empty;
                 }
 
-            if (value is IEnumerable<object> objList && type == typeof(Guid))
-                return objList.Select(x => ChangeType<Guid>(x)).ToArray();
+            if (value is IEnumerable<object> objList)
+            {
+                if(type == typeof(Guid))
+                {
+                    return objList.Select(x => ChangeType<Guid>(x)).ToArray();
+                }
+                else if (type.IsArray)
+                {
+                    return objList.Select(x => ChangeType(x, type.GetElementType())).ToArray();
+                }
+            }
 
             if (value is byte[] && type == typeof(Guid))
                 return new Guid(value as byte[]);

InABox.Core/Query/Column.cs

@@ -753,29 +753,19 @@ namespace InABox.Core
     {
         public override void WriteJson(JsonWriter writer, object? value, JsonSerializer serializer)
         {
-            if(value is null)
+            if(!(value is IColumn col))
             {
                 writer.WriteNull();
                 return;
             }
 
-            var property = (CoreUtils.GetPropertyValue(value, "Expression") as Expression)
-                ?? throw new Exception("'Column.Expression' may not be null");
-
-            var prop = CoreUtils.ExpressionToString(value.GetType().GenericTypeArguments[0], property, true);
-
-            var name = CoreUtils.GetPropertyValue(value, "Property") as string;
-
             writer.WriteStartObject();
             
             writer.WritePropertyName("$type");
             writer.WriteValue(value.GetType().FullName);
-            
-            writer.WritePropertyName("Expression");
-            writer.WriteValue(prop);
 
             writer.WritePropertyName("Property");
-            writer.WriteValue(name);
+            writer.WriteValue(col.Name);
 
             writer.WriteEndObject();
         }

InABox.Core/Query/Filter.cs

@@ -2075,8 +2075,16 @@ namespace InABox.Core
                     var wrapper = Serialization.Deserialize<SubQuerySerializationWrapper>((string)val);
                     val = Activator.CreateInstance(typeof(SubQuery<>).MakeGenericType(CoreUtils.GetEntity(wrapper.Type)), null, null);
                     Serialization.DeserializeInto(wrapper.SerializedSubquery, val);
+                    result.Value = val;
+                }
+                else if(op == Operator.InList || op == Operator.NotInList)
+                {
+                    result.Value = CoreUtils.ChangeType(val, result.Type.MakeArrayType());
+                }
+                else
+                {
+                    result.Value = CoreUtils.ChangeType(val, result.Type);
                 }
-                result.Value = CoreUtils.ChangeType(val, result.Type);
             }
             else if(data.TryGetValue("FilterConstant", out var filterConstant))
             {

inabox.wpf/DynamicGrid/DynamicGridFilterEditor.xaml

@@ -6,7 +6,8 @@
         xmlns:wpf="clr-namespace:InABox.Wpf"
         xmlns:local="clr-namespace:InABox.DynamicGrid"
         mc:Ignorable="d"
-        Title="DynamicGridFilterEditor" Height="450" Width="800">
+        Title="DynamicGridFilterEditor" Height="450" Width="800"
+                    WindowStartupLocation="CenterScreen">
     <Grid x:Name="Grid">
         <Grid.RowDefinitions>
             <RowDefinition Height="*" />

inabox.wpf/DynamicGrid/Editors/FilterEditor/FilterEditorWindow.xaml

@@ -7,7 +7,8 @@
         xmlns:wpf="clr-namespace:InABox.Wpf"
         xmlns:sf="http://schemas.syncfusion.com/wpf"
         mc:Ignorable="d"
-        Title="FilterEditorWindow" Height="450" Width="800">
+        Title="FilterEditorWindow" Height="450" Width="800"
+                    WindowStartupLocation="CenterScreen">
     <Grid x:Name="Grid">
         <Grid.RowDefinitions>
             <RowDefinition Height="*"/>

inabox.wpf/DynamicGrid/Editors/FilterEditor/Nodes/ValueNodes/SubQueryNodePopup.xaml

@@ -6,7 +6,8 @@
         xmlns:local="clr-namespace:InABox.DynamicGrid"
         xmlns:wpf="clr-namespace:InABox.Wpf"
         mc:Ignorable="d"
-        Title="Edit Subquery" Height="450" Width="800">
+        Title="Edit Subquery" Height="450" Width="800"
+                    WindowStartupLocation="CenterScreen">
     <Grid>
         <Grid.RowDefinitions>
             <RowDefinition Height="*"/>
@@ -29,7 +30,18 @@
                     <Label Grid.Row="1" Grid.Column="0" Content="Return Value:" Margin="0,0,5,0"/>
 
                     <ComboBox x:Name="Type" Grid.Row="0" Grid.Column="1" Margin="0,0,0,5" Padding="5"/>
-                    <ComboBox x:Name="Property" Grid.Row="1" Grid.Column="1" Margin="0,0,0,5" Padding="5"/>
+                    <Button x:Name="Property" Grid.Row="1" Grid.Column="1" Margin="0,0,0,5" Padding="5,5,5,5"
+                            VerticalAlignment="Stretch" HorizontalContentAlignment="Left" VerticalContentAlignment="Center"
+                            BorderBrush="#ACACAC"
+                            Click="Property_Click"
+                            Content=" ">
+                        <Button.Background>
+                            <LinearGradientBrush StartPoint="0,0" EndPoint="0,1">
+                                <GradientStop Color="#F0F0F0" Offset="0.0"/>
+                                <GradientStop Color="#E5E5E5" Offset="1.0"/>
+                            </LinearGradientBrush>
+                        </Button.Background>
+                    </Button>
                 </Grid>
             </ScrollViewer>
         </Border>

inabox.wpf/DynamicGrid/Editors/FilterEditor/Nodes/ValueNodes/SubQueryNodePopup.xaml.cs

@@ -21,6 +21,7 @@ namespace InABox.DynamicGrid
     /// </summary>
     public partial class SubQueryNodePopup : ThemableWindow
     {
+        private string[] ColumnNames = [];
 
         public Type? QueryType
         {
@@ -42,12 +43,14 @@ namespace InABox.DynamicGrid
             }
         }
 
+        private string? _column;
         public string? Column
         {
-            get => Property.SelectedItem as string;
+            get => _column;
             set
             {
-                Property.SelectedItem = value;
+                _column = value;
+                Property.Content = value.IsNullOrWhiteSpace() ? " " : value;
             }
         }
         public IFilter? Filter
@@ -106,19 +109,24 @@ namespace InABox.DynamicGrid
         private void UpdateQueryType()
         {
             var type = (Type.SelectedItem as TypeEntry)?.Type;
+            Column = null;
 
-            Property.Items.Clear();
             if(type != null)
             {
-                foreach (var property in CoreUtils.PropertyList(type, x => true, true).Keys)
-                {
-                    Property.Items.Add(property);
-                }
                 Filter = Activator.CreateInstance(typeof(Filter<>).MakeGenericType(type)) as IFilter;
+                Property.IsEnabled = true;
+
+                var properties = DatabaseSchema.Properties(type)
+                    .Where(x => x.IsSerializable)
+                    .Select(x => x.Name)
+                    .ToList();
+                properties.Sort();
+                ColumnNames = properties.ToArray();
             }
             else
             {
                 Filter = null;
+                Property.IsEnabled = false;
             }
         }
 
@@ -149,5 +157,15 @@ namespace InABox.DynamicGrid
             DialogResult = true;
             Close();
         }
+
+        private void Property_Click(object sender, RoutedEventArgs e)
+        {
+            var type = (Type.SelectedItem as TypeEntry)?.Type;
+
+            if(type is not null && DynamicGridColumnNameSelectorGrid.SelectColumnName(type, ColumnNames, out var value))
+            {
+                Column = value.IsNullOrWhiteSpace() ? null : value;
+            }
+        }
     }
 }