Skip to content

Commit

Permalink
Fixed issue with filters on number columns, the values come through f…
Browse files Browse the repository at this point in the history
…rom the client as number, so casting as string doesn't work
  • Loading branch information
d4nt committed Jan 27, 2018
1 parent 00cdb40 commit 7a80fff
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Engine/FilterNode.cs
Expand Up @@ -50,7 +50,7 @@ public override void UpdateSettings(Dictionary<string, object> settings)
FilterCompareColumnIndex = Convert.ToInt32(settings["FilterCompareColumnIndex"]);

if (settings.ContainsKey("FilterValue1"))
FilterValue1 = (string)settings["FilterValue1"];
FilterValue1 = settings["FilterValue1"].ToString();
else
FilterValue1 = null;

Expand Down
2 changes: 1 addition & 1 deletion Tests/FilterTests.cs
Expand Up @@ -23,7 +23,7 @@ private string NodesJsonGreaterThan
""Type"": ""Filter"",
""FilterColumnIndex"": 0,
""Operator"": ""GreaterThan"",
""FilterValue1"": ""5""
""FilterValue1"": 5
}
]";
}
Expand Down

0 comments on commit 7a80fff

Please sign in to comment.