[Solved] Client Side Validation Problem

[John]

I have a rad grid that i'm doing a custom inline edit on, and when I press the save button, i'm trying to do a client side validation of some of the fields.  But the OnClientClick event seems to be completely blocking the post back.  Any ideas?

protected void RadGrid1_ItemDataBound(object sender, Telerik.Web.UI.GridItemEventArgs e)
{

            if (e.Item is GridEditFormItem && e.Item.IsInEditMode)
            {
                 Button bs = editFormItem.FindControl("btnSave") as Button;
                bs.OnClientClick = "return validateChanges('" + m_Value+ "');";
            }
}

and the javascript function right now I just have
function validateChanges(val)
{
     return true;
}

but it never posts.  Any ideas?

my asp button is this:

<asp:Button ID="btnSave" CssClass="buttonSmall" runat="server" Text="Save" CommandName="Update"/>

always works till i put in the OnClientClick property

[John]

Well I found a solution, so I think this is a bug on telerik, but maybe its by design.

When using the onclientclick to do custom javascript validation, it prevents the UpdateCommand from ever firing, so this is how I solved it.

button.OnClientClick="return validateChanges('" + value + "', '" + button.ClientID + "');

and then in the javascript I do this:

function validateChanges(val, buttonID)
{
     if (val == somecondition)
     {
          var id = buttonID.replace(/_/g,"$");
          _doPostBack(id,'');
     }
     {
           return false; //though doesn't matter since it never worked before
      }
}

If there is a better solution out there, please let me know, but this is the only way I could get it to work.

[Andy F.]

Assume that the control will ADD more code to the client-side event.  In other words, change your OnClientClick to something that will permit a fall-thru in case everything is perfect:

bs.OnClientClick = "if (!validateChanges('" + m_Value+ "')) return false;";

In the case where your validation returns TRUE, the control's additional event code gets a chance to operate.