In a JSP Page, onclick of submit, it is submitting twice.

Question

Hello! everybody...
Please help me.. its urgent!
I have a jsp page, after i enter values in textboxes and click on submit, i am calling a javascript function wherein i am submitting the form.
The problem is, it is submitting twice and inserting the record in the database twice.
here is my code.
anybody pls let me know where the problem is?
Thanks in advance.
/*Javascript Function for onsubmit*/
function addVersion()
                         alert('hi');
                         this.VersionDynaForm.action = "./Version.do?param=add";
                         this.VersionDynaForm.submit();
<html:form action="/Version.do" styleId="VersionDynaForm" onsubmit= "return addVersion()">
<input type="hidden" name="actionType" value="">
     <table cellspacing=0 cellpadding=2 border=0 class="table_view" id="TABLE3" >
          <tr class="table_data">
                    <td class="smalltext" width="200px" align="left"><b>Year<span style="color:Red;">*</span></b></td>
                    <td width="150px">
                    <select id="FLP_YEAR" name="FLP_YEAR" class="mediumtext" style="width:120px; height:30px" onchange="FLP_YEARChanged()">
                    <option value="">Select</option>
                         <%
                         ArrayList year = new ArrayList<Version>();
                         year = (ArrayList)session.getAttribute("years");
                         if(year!=null)
                    for(int i=0;i<year.size();i++){                                 
                         Version temp1 = (Version)year.get(i);
                         if(temp1.getFLP_YEAR().equals(Element6)){
     %>
          <OPTION selected value='<%=temp1.getFLP_YEAR()%>'><%=temp1.getFLP_YEAR()%></OPTION>
                    <%}
                         else{%>
                         <OPTION value='<%=temp1.getFLP_YEAR()%>'><%=temp1.getFLP_YEAR()%></OPTION>
                    <%}
                    %>
                    </select>
               </td>
               <td width="150px"></td>
               <td width="200px"></td>
               <td width="200px"></td>     
               <td width="150px"></td>
               <td width="150px"></td>
                    <td width="200px"></td>
     </tr>
     <tr class="table_data">
                    <td class="smalltext" width="200px" align="left"><b>Business Line<span style="color:Red;">*</span></b></td>
                    <td widht="150px">
                    <select id="LPA_BUSINESS_LINES" name="LPA_BUSINESS_LINES" class="mediumtext" style="width:120px; height:30px" onchange="alert('hi');testChanged()">
                    <option value="">Select</option>
                         <%
                         ArrayList lst1 = new ArrayList<Version>();
                         lst1 = (ArrayList)session.getAttribute("mnemonic1");
                         if(lst1!=null)
                    for(int i=0;i<lst1.size();i++){                                 
                         Version temp1 = (Version)lst1.get(i);
                         if(temp1.getLPA_BUSINESS_LINES().equals(Element1)){
     %>
          <OPTION selected value='<%=temp1.getLPA_BUSINESS_LINES()%>'> <%=temp1.getLPA_BUSINESS_LINES()%></OPTION>
                    <%}
                         else{%>
                         <OPTION value='<%=temp1.getLPA_BUSINESS_LINES()%>'> <%=temp1.getLPA_BUSINESS_LINES()%></OPTION>
                    <%}
                    %>
                    </select>
               </td>
               <td width="150px"></td>
               <td width="200px"></td>
               <td width="200px"></td>     
               <td width="150px"></td>
               <td width="150px"></td>
                    <td width="200px"></td>
     </tr>
     <tr class="table_data" align="left">
                    <td class="smalltext" width="200px" align="left"><b>RC Number<span style="color:Red;">*</span></b></td>
                    <td width="150px">
                    <select id="LPA_RC_NUMBER" name="LPA_RC_NUMBER" class="mediumtext" style="width:120px; height:30px" onchange="LPA_RC_NUMBERChanged()">
                    <option value="">Select</option>
                         <%
                         ArrayList lst2 = new ArrayList<Version>();
                         lst2 = (ArrayList)session.getAttribute("rcmnemonic");
                         if(lst2!=null && Element2!=null)
                    for(int i=0;i<lst2.size();i++){                                 
                         Version temp2 = (Version)lst2.get(i);
                              if(temp2.getLPA_RC_NUMBER().equals(Element2)){
     %>
          <OPTION selected value='<%=temp2.getLPA_RC_NUMBER()%>'> <%=temp2.getLPA_RC_NUMBER()%></OPTION>
                    <%}
                         else{%>
                         <OPTION value='<%=temp2.getLPA_RC_NUMBER()%>'> <%=temp2.getLPA_RC_NUMBER()%></OPTION>
                    <%}
                    %>
                    </select>
                    </td>
                    <td class="smalltext" width="200px" align="left"><b>Version<span style="color:Red;">*</span></b></td>
                    <td width="150px">
                    <select id="LPA_VERSION_ID" name="LPA_VERSION_ID" class="mediumtext" style="width:120px; height:30px" onchange="LPA_VERSION_IDChanged()">
                    <option value="">Select</option>
                         <%
                         ArrayList ver = new ArrayList<Version>();
                         ver = (ArrayList)session.getAttribute("version");
                         if(ver!=null && Element5!=null)
                    for(int i=0;i<ver.size();i++){                                 
                         Version temp3 = (Version)ver.get(i);
                              if(temp3.getLPA_VERSION_ID().equals(Element5)){
     %>
          <OPTION selected value='<%=temp3.getLPA_VERSION_ID()%>'> <%=temp3.getLPA_VERSION_NO()%></OPTION>
                    <%}
                         else{%>
                         <OPTION value='<%=temp3.getLPA_VERSION_ID()%>'> <%=temp3.getLPA_VERSION_NO()%></OPTION>
                    <%}
                    %>
                    </select>
                    </td>
               <td width="150px"></td>
               <td width="200px"></td>
               <td width="200px"></td>     
               <td width="150px"></td>
     </tr>
     <tr class="table_data" >
               <td class="smalltext" width="200px" align="left"><b>Project<span style="color:Red;">*</span></b></td>
                    <td width="150px">
                    <select id="LPA_PROJECT_NAME" name="LPA_PROJECT_NAME" class="mediumtext" style="width:120px; height:30px" onchange="LPA_PROJECT_NAMEChanged()">
                    <option value="">Select</option>
                         <%
                         ArrayList lst3 = new ArrayList<Version>();
                         lst3 = (ArrayList)session.getAttribute("project");
                         if(lst3!=null && Element3!=null)
                    for(int i=0;i<lst3.size();i++){                                 
                         Version temp2 = (Version)lst3.get(i);
                              if(temp2.getLPA_PROJECT_NAME().equals(Element3)){
     %>
          <OPTION selected value='<%=temp2.getLPA_PROJECT_NAME()%>'> <%=temp2.getLPA_PROJECT_NAME()%></OPTION>
                    <%}
                         else{%>
                         <OPTION value='<%=temp2.getLPA_PROJECT_NAME()%>'> <%=temp2.getLPA_PROJECT_NAME()%></OPTION>
                    <%}
                    %>
                    </select>
               </td>     
               <td align="left" class="smalltext" width="150px"><b>Version Name</b><span style="color:Red;">*</span></td>
               <td><input type="text" class="smalltext" property="LPA_VERSION_NAME" name="LPA_VERSION_NAME" value="" maxlength="30"/></td>
               <td width="150px"></td>
               <td width="200px"></td>
               <td width="200px"></td>     
               <td width="200px"></td>
     </tr>     
</table>          
<table cellspacing=0 cellpadding=2 border=0 class="table_input" id="TABLE2" >     
          <tr class="table_input">
               <td height="10px" colspan="5" class="table_top_td" align = "right" >
               <INPUT id="button" class="mediumtext" style="width:80px; height:20px;" type="submit" value="NewVersion" class="btn" >
               <INPUT id="button" class="mediumtext" style="width:80px; height:20px;" type="button" value="Update" class="btn" name="method" onclick="javascript:return updateFunction();">
               <INPUT id="button" class="mediumtext" style="width:80px; height:20px;" type="button" value=Cancel class="btn" name="method" onclick="history.go(-1)" >
               </td>
          </tr>
</table>

Answer 1

hi
change the type="button" in the following line
<INPUT id="button" class="mediumtext" style="width:80px; height:20px;" type="submit" value="NewVersion" class="btn" >
and call the javascript function on onClick in above line and dont call it from form(tat u hav done), remove it from form
then it wont submit twice
-venkat