robinvanderknaap / MvcJqGrid

Fluent jqGrid Html Helper for ASP.NET MVC
www.playground.webpirates.nl/mvcjqgrid
GNU Lesser General Public License v3.0
101 stars 74 forks source link

EnableTreeGrid and JSON data #38

Open mikea314 opened 10 years ago

mikea314 commented 10 years ago

MvcJqGrid should always render datatype = _dataType And Also: If EnableTreeGrid then treedatatype = _dataType

instead of : // Datatype script.AppendLine(_enabledTreeGrid ? string.Format("treedatatype:'{0}',", _dataType.ToString().ToLower()) : string.Format("datatype:'{0}',", _dataType.ToString().ToLower()));

mikea314 commented 10 years ago

In Grid.cs, please can you change: // Datatype script.AppendLine(_enabledTreeGrid ? string.Format("treedatatype:'{0}',", _dataType.ToString().ToLower()) : string.Format("datatype:'{0}',", _dataType.ToString().ToLower()));

To: // Datatype script.AppendLine(string.Format("datatype:'{0}',", _dataType.ToString().ToLower()));

// TreeGrid Datatype if (_enabledTreeGrid) { script.AppendLine(string.Format("treedatatype:'{0}',", _dataType.ToString().ToLower())); }

mikea314 commented 9 years ago

This was closed, but it doesn't appear to have been fixed ?

robinvanderknaap commented 9 years ago

I don't know why this was closed. I'll re-open it. Could you submit a pull request with the change?

mikea314 commented 9 years ago

Done

Resolve Issue #38: datatype needs to be set, even if we are setting a treedatatype https://github.com/robinvanderknaap/MvcJqGrid/pull/57

57 opened a minute ago by mikea314

https://github.com/robinvanderknaap/MvcJqGrid/issues?q=is%3Apr+is%3Aopen+author%3Amikea314

On Wed, Jun 3, 2015 at 4:45 PM, Robin van der Knaap < notifications@github.com> wrote:

I don't know why this was closed. I'll re-open it. Could you submit a pull request with the change?

— Reply to this email directly or view it on GitHub https://github.com/robinvanderknaap/MvcJqGrid/issues/38#issuecomment-108464613 .