Skip to content

Commit 22bd112

Browse files
committed
Fixes for issues in PRs
1 parent 802893e commit 22bd112

File tree

4 files changed

+7
-12
lines changed

4 files changed

+7
-12
lines changed

Mvc.JQuery.Datatables/DataTablesResult.cs

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,8 @@ public abstract class DataTablesResult : ActionResult
1616
/// <typeparam name="TTransform"></typeparam>
1717
/// <param name="q">A queryable for the data. The properties of this can be marked up with [DataTablesAttribute] to control sorting/searchability/visibility</param>
1818
/// <param name="dataTableParam"></param>
19-
/// <param name="transform">//a transform for custom column rendering e.g. to do a custom date row => new { CreatedDate = row.CreatedDate.ToString("dd MM yy") } </param>
2019
/// <returns></returns>
21-
public static DataTablesResult<TSource> Create<TSource, TTransform>(IQueryable<TSource> q, DataTablesParam dataTableParam,
22-
Func<TSource, TTransform> transform, ArrayOutputType? arrayOutput = null)
23-
{
24-
return Create(q, dataTableParam, transform, new ResponseOptions<TSource>() { ArrayOutputType = arrayOutput });
25-
}
26-
20+
2721
public static DataTablesResult<TSource> Create<TSource>(IQueryable<TSource> q, DataTablesParam dataTableParam,
2822
ArrayOutputType? arrayOutput = null)
2923
{
@@ -67,6 +61,7 @@ public static DataTablesResult<TSource> Create<TSource>(IQueryable<TSource> q, D
6761

6862
private static DataTablesData ApplyOutputRules<TSource>(DataTablesData sourceData, ResponseOptions<TSource> responseOptions)
6963
{
64+
responseOptions = responseOptions ?? new ResponseOptions<TSource>() {ArrayOutputType = ArrayOutputType.BiDimensionalArray};
7065
DataTablesData outputData = sourceData;
7166

7267
switch (responseOptions.ArrayOutputType)

Mvc.Jquery.DataTables.Tests/FilterTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public void Setup()
3333
dataTablesParam = new DataTablesParam();
3434
columns = DataTablesTypeInfo<SomeModel>.Properties.Select((p, i) =>
3535
Tuple.Create(i, new DataTablesPropertyInfo(p.PropertyInfo, new DataTablesAttributeBase[]{}))).ToArray();
36-
dataTablesParam.sSearchColumns = new List<string>(columns.Select(c => null as string));
36+
dataTablesParam.sSearchValues = new List<string>(columns.Select(c => null as string));
3737
dataTablesParam.bSearchable = new List<bool>(columns.Select(c => true));
3838

3939
}
@@ -58,7 +58,7 @@ public void SearchQueryTests(string searchString, Type colType, bool returnsResu
5858
{
5959
var col = columns.First(c => c.Item2.Type == colType).Item1;
6060

61-
dataTablesParam.sSearchColumns[col] = searchString;
61+
dataTablesParam.sSearchValues[col] = searchString;
6262
var result = new DataTablesResult<SomeModel>(queryable, dataTablesParam);
6363

6464
var data = result.Data;

Mvc.Jquery.DataTables.Tests/Fixtures/Linq.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ public static IEnumerable TestCases
8787
dataTablesParam = GetEmptyParam(propertyCount);
8888
dataTablesParam.iSortingCols = 1;
8989
dataTablesParam.iSortCol[0] = 2;
90-
dataTablesParam.sSearchColumns[3] = "25~35";
90+
dataTablesParam.sSearchValues[3] = "25~35";
9191
dataTablesParam.iDisplayStart = 6;
9292
yield return new TestCaseData(dataTablesParam)
9393
.Returns(new int[] { 17, 21, 25, 77, 81 })
@@ -107,7 +107,7 @@ static DataTablesParam GetEmptyParam(int columns)
107107
returnVar.bSearchable.AddRange(Enumerable.Repeat(true, columns));
108108
returnVar.bSortable.AddRange(Enumerable.Repeat(true, columns));
109109
returnVar.iSortCol.AddRange(Enumerable.Repeat(0, columns));
110-
returnVar.sSearchColumns.AddRange(Enumerable.Repeat("", columns));
110+
returnVar.sSearchValues.AddRange(Enumerable.Repeat("", columns));
111111
returnVar.sSortDir.AddRange(Enumerable.Repeat<string>(null, columns));
112112

113113
return returnVar;

Mvc.Jquery.DataTables.Tests/Fixtures/ModelBinding.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public void TestDataTablesIModelBinder()
9191
Assert.That(res.iSortingCols, Is.EqualTo(1), "iSortingCols");
9292
Assert.That(res.sEcho, Is.EqualTo(1), "sEcho");
9393
Assert.That(res.sSearch,Is.EqualTo(""),"sSearch");
94-
Assert.That(res.sSearchColumns, Is.EqualTo(Enumerable.Repeat("", 9)), "sSearchColumns");
94+
Assert.That(res.sSearchValues, Is.EqualTo(Enumerable.Repeat("", 9)), "sSearchColumns");
9595
Assert.That(res.sSortDir, Is.EqualTo(Enumerable.Repeat<string>(null, 9).ReplaceAtIndex("asc", 0)), "sSortDir");
9696
Assert.That(res.bEscapeRegex, Is.EqualTo(false), "bEscapeRegex");
9797
}

0 commit comments

Comments
 (0)