Skip to content

Conversation

@seguemark
Copy link
Contributor

=== Response Options ===
Added ResponseOptions parameters to DataTablesResult method calls
Changed arrayOutput method signatures to wrap arrayOutput into ResponseOptions

=== DT_RowID ===
Added DT_RowID to ResponseOptions, allows selection of DT_RowID column in each DataTablesResult.Create call - implicitly sets ResponseOptions.ArrayOutputType to ArrayOfObjects.
Added DataTablesRowIdAttribute to set DT_RowID at the class level; also allows exclusion of the attributed property from being emitted under its normal column name
ResponseOptions.DT_RowID -overrides- DataTablesRowIdAttribute
If a class property has DataTablesRowIdAttribute and DataTablesExcludeAttribute, then DataTablesRowIdAttribute will have no effect
DataTablesRowIdAttribute and DT_RowID only affect conversion of a row to a Dictionary (via DataTablesTypeInfo.ToDictionary)

Added ResponseOptions parameters to DataTablesResult method calls
Changed arrayOutput method signatures to wrap arrayOutput into ResponseOptions

=== DT_RowID ===
Added DT_RowID to ResponseOptions, allows selection of DT_RowID column in each DataTablesResult.Create call - implicitly sets ResponseOptions.ArrayOutputType to ArrayOfObjects.
Added DataTablesRowIdAttribute to set DT_RowID at the class level; also allows exclusion of the attributed property from being emitted under its normal column name
ResponseOptions.DT_RowID -overrides- DataTablesRowIdAttribute
If a class property has DataTablesRowIdAttribute and DataTablesExcludeAttribute, then DataTablesRowIdAttribute will have no effect
DataTablesRowIdAttribute and DT_RowID only affect conversion of a row to a Dictionary (via DataTablesTypeInfo.ToDictionary)
…ry parameters (e.g. columns[#][orderable]) as well as still supporting the old-style (e.g. bSortable_#). As part of this, added support for searching columns by name (columns[#]data]) instead of just by index.
mcintyre321 added a commit that referenced this pull request Aug 17, 2015
Added ResponseOptions to DataTablesResult method calls to pass in additional response configuration.  Added support for emitting DT_RowID in ArrayOfObjects output.
@mcintyre321 mcintyre321 merged commit 5b81a3b into mcintyre321:master Aug 17, 2015
@mcintyre321
Copy link
Owner

FYI I had to remove one of the overloads on DataTables.Create as it was causing compilation issues in the solution.

@seguemark
Copy link
Contributor Author

Yeah I think I ran into the same thing when I didn't pass in a full response options object. I've made some more enhancements including filtering on IEnumerable columns and will be generating a pull request when in done with this project and an sure everything is working right.

I love your library; I had created something similar for jqGrid several years ago and recently ported it to DataTables but I thin yours is a bit easier to use and solves some limitations mine had. I'll have to do a side-by-side comparison and see what else I can port over to your library.

-Mark

Sent from my Verizon Wireless 4G LTE smartphone

-------- Original message --------
From: Harry McIntyre notifications@github.com
Date: 08/27/2015 4:27 AM (GMT-05:00)
To: "mcintyre321/mvc.jquery.datatables" mvc.jquery.datatables@noreply.github.com
Cc: Mark Shapiro Mark.Shapiro@seguetech.com
Subject: Re: [mvc.jquery.datatables] Added ResponseOptions to DataTablesResult method calls to pass in additional response configuration. Added support for emitting DT_RowID in ArrayOfObjects output. (#123)

FYI I had to remove one of the overloads on DataTables.Create as it was causing compilation issues in the solution.


Reply to this email directly or view it on GitHubhttps://github.com//pull/123#issuecomment-135338958.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants