tspence / csharp-searchlight

A lightweight, secure framework for a fluent REST API with robust, secure searching features.
Apache License 2.0
11 stars 7 forks source link

Remove dependency on System.Dynamic.Linq.Core #100

Closed tspence closed 2 years ago

tspence commented 2 years ago

We use System.Dynamic.Linq.Core for one thing only: Dynamic sorting. I implemented this in a rush early on because I didn't have the time to ponder out the syntax of Expression.Lambda - so let's see if we can reduce complexity.

This may also improve performance a little bit because using Dynamic Linq Core involved converting our parsed data back into a string which would then be re-parsed in their engine for sorting.