GetAllForProperty returns sorted results

Author: ithielnor

Griddly.Mvc/GriddlyResult.cs

@@ -46,7 +46,7 @@ public SortField[] GetSortFields(NameValueCollection items)
 
         public abstract long GetCount();
 
-        public abstract IEnumerable<P> GetAllForProperty<P>(string propertyName);
+        public abstract IEnumerable<P> GetAllForProperty<P>(string propertyName, SortField[] sortFields);
     }
 
     public abstract class GriddlyResult<T> : GriddlyResult
@@ -180,7 +180,7 @@ public override void ExecuteResult(ControllerContext context)
 
         public abstract void PopulateSummaryValues(GriddlySettings<T> settings);
 
-        public override IEnumerable<P> GetAllForProperty<P>(string propertyName)
+        public override IEnumerable<P> GetAllForProperty<P>(string propertyName, SortField[] sortFields)
         {
             throw new NotImplementedException();
         }

Griddly.Mvc/Results/DapperResult.cs

@@ -79,12 +79,12 @@ public override void PopulateSummaryValues(GriddlySettings<T> settings)
             }
         }
 
-        public override IEnumerable<P> GetAllForProperty<P>(string propertyName)
+        public override IEnumerable<P> GetAllForProperty<P>(string propertyName, SortField[] sortFields)
         {
             if (propertyName.Contains("."))
                 throw new ArgumentException($"Property name may not contain a period. \"{propertyName}\"", "propertyName");
 
-            string sql = string.Format("SELECT {0} as _val FROM ({1}) [_proj]", propertyName, _sql);
+            string sql = $"SELECT {propertyName} as _val FROM ({_sql}) [_proj] {(_fixedSort ? "" : $"ORDER BY {BuildSortClause(sortFields) ?? "CURRENT_TIMESTAMP"}")}";
 
             try
             {

Griddly.Mvc/Results/MapQueryableResult.cs

@@ -40,9 +40,9 @@ public override void PopulateSummaryValues(GriddlySettings<TOut> settings)
                 _result.PopulateSummaryValue(c);
         }
 
-        public override IEnumerable<P> GetAllForProperty<P>(string propertyName)
+        public override IEnumerable<P> GetAllForProperty<P>(string propertyName, SortField[] sortFields)
         {
-            return _result.GetAllForProperty<P>(propertyName);
+            return _result.GetAllForProperty<P>(propertyName, sortFields);
         }
 
         public override long GetCount()

Griddly.Mvc/Results/QueryableResult.cs

@@ -93,9 +93,10 @@ internal void PopulateSummaryValue(GriddlyColumn c)
             }
         }
 
-        public override IEnumerable<P> GetAllForProperty<P>(string propertyName)
+        public override IEnumerable<P> GetAllForProperty<P>(string propertyName, SortField[] sortFields)
         {
-            return _result.Select<P>(propertyName, null);
+            return ApplySortFields(_result, sortFields)
+                .Select<P>(propertyName, null);
         }
 
         public override long GetCount()

Griddly/Controllers/HomeController.cs

@@ -16,7 +16,7 @@ public static ActionResult HandleCustomExport(GriddlyResult result, NameValueCol
         {
             return new JsonResult()
             {
-                Data = result.GetAllForProperty<long?>("Id"),
+                Data = result.GetAllForProperty<long?>("Id", null),
                 JsonRequestBehavior = JsonRequestBehavior.AllowGet,
             };
         }