Duplicate PK Check on saving - Alert not firing

Question

I'm trying to check for duplicate pks when trying to insert a new record. I want to show an error if it does, but i'm not sure where i am going wrong/if this is the right way to go.

Please see below. I put a breakpoint at the function AgentSave_Click and saw its its not going into the IF statement

<asp:Button ID="AgentSave" runat="server" CausesValidation="true" OnClick="AgentSave_Click" Style="margin-left: 0px" Text="Save"   />

     protected void AgentSave_Click(object sender, EventArgs e)
    {
        try
        {
            if (AccountNumber.Text.Trim().Equals("select PRIMARYKEYNAME from TABLEXYZ"))
            {

                Response.Write("<script type=\"text/javascript\">" + "window.alert('ERROR: The Account Number entered is already assigned to an Agent.');" + "</script>");
                AccountNumber.Focus();
            }
            else
            {.....

score 2 · Accepted Answer · answered Nov 11 '13 at 17:25

2

OnClientClick property is meant to execute some javascript code on client before data is send back to server.

According your answer you are trying to execute server code. So you should replace onclientclick with onclick="duplicate_PK" which gets executed on server.

Regards, Uros

answered Nov 11 '13 at 17:25

Uroš Goljat

1,806
14
15

Goljat:exactly spot on..!! – Sajad Karuthedath Nov 11 '13 at 17:48
your code doesn't enter the if statement because you are comparing textbox value to a quasi sql statement. – Uroš Goljat Nov 12 '13 at 19:20

Sajad Karuthedath · Answer 2 · 2013-11-11T18:32:46.733

0

<asp:Button ID="AgentSave" runat="server"  OnClick="AgentSave_Click" Style="margin-left: 0px" Text="Save"   />

remove CausesValidation="true" or chANGE TO CausesValidation="false"

edited Nov 11 '13 at 18:32

answered Nov 11 '13 at 18:26

Sajad Karuthedath

14,987
4
32
49

i need this button to validate certain fields on the form when being enter like the PK field and other. y remove the validation? – New2This Nov 11 '13 at 18:28
ok.. check this [link](http://stackoverflow.com/questions/14191235/i-must-add-causesvalidation-false-to-every-asp-button-to-work-why) and reply – Sajad Karuthedath Nov 11 '13 at 18:31

Duplicate PK Check on saving - Alert not firing

2 Answers2