From 69eb41e716e9aeca3adcf0812ef3025319701667 Mon Sep 17 00:00:00 2001
From: Brian Lagunas <835562+brianlagunas@users.noreply.github.com>
Date: Fri, 6 Sep 2024 14:00:23 -0600
Subject: [PATCH] supported custom field name for dashboard filter connections

---
 .../Filters/Bindings/DashboardDataFilterBinding.cs  |  9 ++++++---
 .../Bindings/DashboardDateFilterBindingTarget.cs    |  2 +-
 src/Reveal.Sdk.Dom/Filters/DashboardDataFilter.cs   |  2 +-
 .../Extensions/IFilterBindingsExtensions.cs         | 13 ++++++++++---
 4 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDataFilterBinding.cs b/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDataFilterBinding.cs
index 75e05dd9..b6a1c288 100644
--- a/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDataFilterBinding.cs
+++ b/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDataFilterBinding.cs
@@ -5,13 +5,16 @@ public sealed class DashboardDataFilterBinding : Binding<FieldBindingSource, Das
         internal DashboardDataFilterBinding() : this(null) { }
 
         public DashboardDataFilterBinding(DashboardDataFilter dataFilter)
+            : this(dataFilter, dataFilter.FieldName) { }
+
+        public DashboardDataFilterBinding(DashboardDataFilter dataFilter, string fieldName)
         {
-            Source = new FieldBindingSource() { FieldName = dataFilter?.FieldName };
+            Source = new FieldBindingSource() { FieldName = fieldName };
             Operator = BindingOperatorType.Equals;
             Target = new DashboardDataFilterBindingTarget()
             {
-                DashboardFilterId = dataFilter?.Id,
-                DashboardFilterFieldName = dataFilter?.FieldName
+                DashboardFilterId = dataFilter.Id,
+                DashboardFilterFieldName = dataFilter.FieldName
             };
         }
     }
diff --git a/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDateFilterBindingTarget.cs b/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDateFilterBindingTarget.cs
index d000dd87..90d69eb2 100644
--- a/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDateFilterBindingTarget.cs
+++ b/src/Reveal.Sdk.Dom/Filters/Bindings/DashboardDateFilterBindingTarget.cs
@@ -6,7 +6,7 @@ namespace Reveal.Sdk.Dom.Filters
     public sealed class DashboardDateFilterBindingTarget : BindingTarget
     {
         [JsonProperty("GlobalFilterId")]
-        public string DashboardFilterId { get; set; }
+        public string DashboardFilterId { get; set; } = "_date";
 
         [JsonProperty("GlobalFilterFieldName")]
         public string DashboardFilterFieldName { get; set; }
diff --git a/src/Reveal.Sdk.Dom/Filters/DashboardDataFilter.cs b/src/Reveal.Sdk.Dom/Filters/DashboardDataFilter.cs
index a0b70d53..020730c3 100644
--- a/src/Reveal.Sdk.Dom/Filters/DashboardDataFilter.cs
+++ b/src/Reveal.Sdk.Dom/Filters/DashboardDataFilter.cs
@@ -26,7 +26,7 @@ public DashboardDataFilter(string fieldName, string title, DataSourceItem dataSo
         {
             SchemaTypeName = SchemaTypeNames.TabularGlobalFilterType;
             FieldName = fieldName;
-            Title = title;
+            Title = title ?? fieldName;
             DataDefinition.DataSourceItem = dataSourceItem;
             DataDefinition.Fields = dataSourceItem?.Fields.Clone();
         }
diff --git a/src/Reveal.Sdk.Dom/Visualizations/Extensions/IFilterBindingsExtensions.cs b/src/Reveal.Sdk.Dom/Visualizations/Extensions/IFilterBindingsExtensions.cs
index 57abae7a..d28432b3 100644
--- a/src/Reveal.Sdk.Dom/Visualizations/Extensions/IFilterBindingsExtensions.cs
+++ b/src/Reveal.Sdk.Dom/Visualizations/Extensions/IFilterBindingsExtensions.cs
@@ -6,14 +6,21 @@ public static class IFilterBindingsExtensions
     {
         public static T ConnectDashboardFilter<T>(this T visualization, DashboardFilter dashboardFilter)
             where T : IFilterBindings
+        {
+            return visualization.ConnectDashboardFilter(dashboardFilter, null);
+        }
+
+        public static T ConnectDashboardFilter<T>(this T visualization, DashboardFilter dashboardFilter, string fieldName)
+            where T : IFilterBindings
         {
             if (dashboardFilter is DashboardDateFilter)
             {
-                visualization.FilterBindings.Add(new DashboardDateFilterBinding("Date"));
+                visualization.FilterBindings.Add(new DashboardDateFilterBinding(fieldName ?? "Date"));
             }
-            else
+            else if (dashboardFilter is DashboardDataFilter dataFilter)
             {
-                visualization.FilterBindings.Add(new DashboardDataFilterBinding(dashboardFilter as DashboardDataFilter));
+                var binding = fieldName == null ? new DashboardDataFilterBinding(dataFilter) : new DashboardDataFilterBinding(dataFilter, fieldName);
+                visualization.FilterBindings.Add(binding);
             }
             return visualization;
         }