[Solved] Possible bug in deprecated "OnRowSelected"?

[Joan Vilariño]

Hello.

I was having an applicationwide error "StackOverFlowError" when loading dynamically grids inside a <div> using ajax. This happened without any change to the code by my side, so I was suspecting it had something to do with Telerik.

Finally I found the cause... I was using the deprecated "OnRowSelected" event in my grids. When I have changed it to OnRowSelect it works flawless...

Being a deprecated method, is not as important as another bugs, but I post this so you are aware that it can break "legacy" forms like the ones I was using. I had to go all through my solution to change OnRowSelected calls to the new OnRowSelect.

Here is the code that crashes the view:

.ClientEvents(events => events
    .OnRowDataBound(() => { %>
                 function(e) {
                   PopupRowDataBound("idCentroServicio",e);
                 } <%
     })
     .OnRowSelected(() => { %>
                 function (e) {
                   PopupRowSelected(e,"idCentroServicio",'<%=Model.Controller %>','<%=Model.Action %>','<%=Model.CallBack %>');
                 } <%
     })

I hope it helps to fix it

Cheers

[Atanas Korchev]

Hi Joan Vilariño,

Indeed there was a recursive call in the deprecated method:

[Obsolete("Use OnRowSelect instead")]
public GridClientEventsBuilder OnRowSelected(Action onRowSelectedInlineCode)
{
      return OnRowSelected(onRowSelectedInlineCode);
}
should be

[Obsolete("Use OnRowSelect instead")]
public GridClientEventsBuilder OnRowSelected(Action onRowSelectedInlineCode)
{
      return OnRowSelect(onRowSelectedInlineCode);
}

I have fixed this bug. Your Telerik points have been updated.

Regards,
Atanas Korchev
the Telerik team

[John]

Indeed, I was just about to post this bug.