Form action overriding onSubmit?

My form data is being processed by the form action to register data before it is verified by the onSubmit validInfo(). Inputs that do not meet the requirements is being processed...

<form id="Form" name="Form" action="Register-Data.php" method="POST" enctype="multipart/form-data" onSubmit="return validInfo()">
			<label>Email:</label>
			<input type="text" placeholder="Enter Email" id="User" name="User" maxlength="30" style="width:352px;height:30px;" required><br>
			<label>Password:</label>
			<input type="password" placeholder="Enter Password" id="Pass" name="Pass" maxlength="20" style="width:305px;height:30px;" required><br>
			<label>Birthday:</label>
			<select id="Month" name="Month" required>
				<option value="1">January</option>
				<option value="2">February</option>
				<option value="3">March</option>
				<option value="4">April</option>
				<option value="5">May</option>
				<option value="6">June</option>
				<option value="7">July</option>
				<option value="8">August</option>
				<option value="9">September</option>
				<option value="10">October</option>
				<option value="11">November</option>
				<option value="12">December</option>
			</select>
			<select id="Day" name="Day" required>
				<option value="1">1</option>
				<option value="2">2</option>
				<option value="3">3</option>
				<option value="4">4</option>
				<option value="5">5</option>
				<option value="6">6</option>
				<option value="7">7</option>
				<option value="8">8</option>
				<option value="9">9</option>
				<option value="10">10</option>
				<option value="11">11</option>
				<option value="12">12</option>
				<option value="13">13</option>
				<option value="14">14</option>
				<option value="15">15</option>
				<option value="16">16</option>
				<option value="17">17</option>
				<option value="18">18</option>
				<option value="19">19</option>
				<option value="20">20</option>
				<option value="21">21</option>
				<option value="22">22</option>
				<option value="23">23</option>
				<option value="24">24</option>
				<option value="25">25</option>
				<option value="26">26</option>
				<option value="27">27</option>
				<option value="28">28</option>
				<option value="29">29</option>
				<option value="30">30</option>
				<option value="31">31</option>
			</select>
			<select id="Year" name="Year" required>
				<option value="2018">2018</option>
				<option value="2017">2017</option>
				<option value="2016">2016</option>
				<option value="2015">2015</option>
				<option value="2014">2014</option>
				<option value="2013">2013</option>
				<option value="2012">2012</option>
				<option value="2011">2011</option>
				<option value="2010">2010</option>
				<option value="2009">2009</option>
				<option value="2008">2008</option>
				<option value="2007">2007</option>
				<option value="2006">2006</option>
				<option value="2005">2005</option>
				<option value="2004">2004</option>
				<option value="2003">2003</option>
				<option value="2002">2002</option>
				<option value="2001">2001</option>
				<option value="2000">2000</option>
				<option value="1999">1999</option>
				<option value="1998">1998</option>
				<option value="1997">1997</option>
				<option value="1996">1996</option>
				<option value="1995">1995</option>
				<option value="1994">1994</option>
				<option value="1993">1993</option>
				<option value="1992">1992</option>
				<option value="1991">1991</option>
				<option value="1990">1990</option>
				<option value="1989">1989</option>
				<option value="1988">1988</option>
				<option value="1987">1987</option>
				<option value="1986">1986</option>
				<option value="1985">1985</option>
				<option value="1984">1984</option>
				<option value="1983">1983</option>
				<option value="1982">1982</option>
				<option value="1981">1981</option>
				<option value="1980">1980</option>
				<option value="1979">1979</option>
				<option value="1978">1978</option>
				<option value="1977">1977</option>
				<option value="1976">1976</option>
				<option value="1975">1975</option>
				<option value="1974">1974</option>
				<option value="1973">1973</option>
				<option value="1972">1972</option>
				<option value="1971">1971</option>
				<option value="1970">1970</option>
				<option value="1969">1969</option>
				<option value="1968">1968</option>
				<option value="1967">1967</option>
				<option value="1966">1966</option>
				<option value="1965">1965</option>
				<option value="1964">1964</option>
				<option value="1963">1963</option>
				<option value="1962">1962</option>
				<option value="1961">1961</option>
				<option value="1960">1960</option>
				<option value="1959">1959</option>
				<option value="1958">1958</option>
				<option value="1957">1957</option>
				<option value="1956">1956</option>
				<option value="1955">1955</option>
				<option value="1954">1954</option>
				<option value="1953">1953</option>
				<option value="1952">1952</option>
				<option value="1951">1951</option>
				<option value="1950">1950</option>
				<option value="1949">1949</option>
				<option value="1948">1948</option>
				<option value="1947">1947</option>
				<option value="1946">1946</option>
				<option value="1945">1945</option>
				<option value="1944">1944</option>
				<option value="1943">1943</option>
				<option value="1942">1942</option>
				<option value="1941">1941</option>
				<option value="1940">1940</option>
				<option value="1939">1939</option>
				<option value="1938">1938</option>
				<option value="1937">1937</option>
				<option value="1936">1936</option>
				<option value="1935">1935</option>
				<option value="1934">1934</option>
				<option value="1933">1933</option>
				<option value="1932">1932</option>
				<option value="1931">1931</option>
				<option value="1930">1930</option>
				<option value="1929">1929</option>
				<option value="1928">1928</option>
				<option value="1927">1927</option>
				<option value="1926">1926</option>
				<option value="1925">1925</option>
				<option value="1924">1924</option>
				<option value="1923">1923</option>
				<option value="1922">1922</option>
				<option value="1921">1921</option>
				<option value="1920">1920</option>
				<option value="1919">1919</option>
				<option value="1918">1918</option>
				<option value="1917">1917</option>
				<option value="1916">1916</option>
				<option value="1915">1915</option>
				<option value="1914">1914</option>
				<option value="1913">1913</option>
				<option value="1912">1912</option>
				<option value="1911">1911</option>
				<option value="1910">1910</option>
				<option value="1909">1909</option>
				<option value="1908">1908</option>
				<option value="1907">1907</option>
				<option value="1906">1906</option>
				<option value="1905">1905</option>
				<option value="1904">1904</option>
				<option value="1903">1903</option>
				<option value="1902">1902</option>
				<option value="1901">1901</option>
				<option value="1900">1900</option>
			</select>
			<br><label>Select a Gender</label><br>
			<label>M:</label>
			<input type="radio" id="radio" name="Gender" value="Male">
			<label>F:</label>
			<input type="radio" id="radio" name="Gender" value="Female">
			<label>N/A:</label>
			<input type="radio" id="radio" name="Gender" value="Other" checked><br>
			<button type="submit" style="position:relative;width:400px;height:50px;">Register</button>
		</form>
		<script>
			function validateEmail( email ) {
				"use strict";
				var re = /^(([^<>()[\]\\.,;:\s@\"]+(\.[^<>()[\]\\.,;:\s@\"]+)*)|(\".+\"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;
				var orig = 
					<?php
					$servername = 'localhost';
					$username = 'root';
					$password = 'loremipsum';
					$dbname = 'database1'; 
					$conn = new mysqli( $servername, $username, $password, $dbname );
					$email = $_POST[ 'User' ];
					$orig = "SELECT FROM userid WHERE email='$email';";
					$result = mysql_query($orig);
					if(mysql_num_rows($result) > 0) {
						return false;
					} else {
						return true;
					}
					?>;
				if ( ( re.test( email ) == true ) && ( orig == true ) ) {
					return true;
				} else {
					return false;
				}
			}

			function validatePassword( pass ) {
				"use strict";
				var rgx = /(?=.*\d)(?=.*[a-z])(?=.*[A-Z]).{8,20}/;
				return rgx.test( pass );
			}

			function validateBirthday( month, day, year ) {
				"use strict";
				var d = new Date();
				var minyear = d.getFullYear() - 13;
				var minmon = d.getMonth() + 1;
				var minday = d.getDay();
				if ( year > minyear ) {
					return false;
				} else if ( year === minyear ) {
					if ( month < minmon ) {
						return true;
					} else {
						if ( month === minmon ) {
							if ( day <= minday ) {
								return true;
							} else {
								return false;
							}
						} else {
							return false;
						}
					}
				} else {
					return true;
				}
			}

			function validInfo() {
				var User = Form.User.value;
				var Pass = Form.Pass.value;
				var Month = Form.Month.value;
				var Day = Form.Day.value;
				var Year = Form.Year.value;
				if ( ( validateEmail( User ) === true ) && ( validatePassword( Pass ) === true ) && ( validateBirthday( Month, Day, Year ) === true ) ) {
					return true;
				} else {
					alert( "Form data is incorrect!" );
					return false;
				}
			}
		</script>

Open in new window

Caden PangHS Sophomore Web DeveloperAsked:
Who is Participating?
 
leakim971Connect With a Mentor PluritechnicianCommented:
You should use Ajax to check if the email address is present in your database instead using PHP inside your code, the way you code it is not good and broke your javascript.
Additionaly you should submit the page at the end, once the email from your database is present. If the email is present, we submit the page
We don't put the onsubmit in the form itself or with the register button, we submit the page only if the last test (checking presence of the email in database is a success) with javascript code.

so the form :
<form id="Form" name="Form" action="Register-Data.php" method="POST" enctype="multipart/form-data">
the register button (please note the onclick) :
<button type="submit" style="position:relative;width:400px;height:50px;" onclick="return validInfo()">Register</button>

and the code :
 
       // https://gist.github.com/Xeoncross/7663273
        function ajax(url, callback, data, x) {
            try {
                x = new(this.XMLHttpRequest || ActiveXObject)('MSXML2.XMLHTTP.3.0');
                x.open(data ? 'POST' : 'GET', url, 1);
                x.setRequestHeader('X-Requested-With', 'XMLHttpRequest');
                x.setRequestHeader('Content-type', 'application/x-www-form-urlencoded');
                x.onreadystatechange = function () {
                    x.readyState > 3 && callback && callback(JSON.parse(x.responseText), x);
                };
                x.send(data)
            } catch (e) {
                window.console && console.log(e);
            }
        };

        function formNotValid() {
            alert( "Form data is incorrect!" );
        }

        function formValid() {
            document.getElementById("Form").submit();
        }

        function validateEmail( email ) {
            "use strict";
            var re = /^(([^<>()[\]\\.,;:\s@\"]+(\.[^<>()[\]\\.,;:\s@\"]+)*)|(\".+\"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;
            if ( re.test( email ) == true )
                ajax("orig.php?email="+encodeURIComponent(email), function(result) {
                    console.log(result);
                    if(result.email_found_in_DB)
                        formValid();
                    else
                        formNotValid();
                }, "GET");
            else
                formNotValid();
        }

        function validatePassword( pass ) {
            "use strict";
            var rgx = /(?=.*\d)(?=.*[a-z])(?=.*[A-Z]).{8,20}/;
            return rgx.test( pass );
        }

        function validateBirthday( month, day, year ) {
            "use strict";
            var d = new Date();
            var minyear = d.getFullYear() - 13;
            var minmon = d.getMonth() + 1;
            var minday = d.getDay();
            if ( year > minyear ) {
                return false;
            } else if ( year === minyear ) {
                if ( month < minmon ) {
                    return true;
                } else {
                    if ( month === minmon ) {
                        if ( day <= minday ) {
                            return true;
                        } else {
                            return false;
                        }
                    } else {
                        return false;
                    }
                }
            } else {
                return true;
            }
        }

        function validInfo() {
            var User = Form.User.value;
            var Pass = Form.Pass.value;
            var Month = Form.Month.value;
            var Day = Form.Day.value;
            var Year = Form.Year.value;
            if ( ( validatePassword( Pass ) === true ) && ( validateBirthday( Month, Day, Year ) === true ) )
                validateEmail(User);
            else
                formNotValid();
            return false;
        }

Open in new window


we need orig.php in the same folder of your page to check email presence in the database :
<?php
    header('Content-type:application/json;charset=utf-8');
    $email = $_GET["email"];

    $servername = 'localhost';
    $username = 'root';
    $password = 'loremipsum';
    $dbname = 'database1';
    $conn = new mysqli( $servername, $username, $password, $dbname );
    $email = $_POST[ 'User' ];
    $orig = "SELECT FROM userid WHERE email='$email';";
    $result = mysql_query($orig);
    if(mysql_num_rows($result) > 0)
        echo '{"email_found_in_DB":true,"email":"' . $email . '"}';
    else
        echo '{"email_found_in_DB":false,"email":"' . $email . '"}';

Open in new window


full page :
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Title</title>
    <script>

        // https://gist.github.com/Xeoncross/7663273
        function ajax(url, callback, data, x) {
            try {
                x = new(this.XMLHttpRequest || ActiveXObject)('MSXML2.XMLHTTP.3.0');
                x.open(data ? 'POST' : 'GET', url, 1);
                x.setRequestHeader('X-Requested-With', 'XMLHttpRequest');
                x.setRequestHeader('Content-type', 'application/x-www-form-urlencoded');
                x.onreadystatechange = function () {
                    x.readyState > 3 && callback && callback(JSON.parse(x.responseText), x);
                };
                x.send(data)
            } catch (e) {
                window.console && console.log(e);
            }
        };

        function formNotValid() {
            alert( "Form data is incorrect!" );
        }

        function formValid() {
            document.getElementById("Form").submit();
        }

        function validateEmail( email ) {
            "use strict";
            var re = /^(([^<>()[\]\\.,;:\s@\"]+(\.[^<>()[\]\\.,;:\s@\"]+)*)|(\".+\"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;
            if ( re.test( email ) == true )
                ajax("orig.php?email="+encodeURIComponent(email), function(result) {
                    console.log(result);
                    if(result.email_found_in_DB)
                        formValid();
                    else
                        formNotValid();
                }, "GET");
            else
                formNotValid();
        }

        function validatePassword( pass ) {
            "use strict";
            var rgx = /(?=.*\d)(?=.*[a-z])(?=.*[A-Z]).{8,20}/;
            return rgx.test( pass );
        }

        function validateBirthday( month, day, year ) {
            "use strict";
            var d = new Date();
            var minyear = d.getFullYear() - 13;
            var minmon = d.getMonth() + 1;
            var minday = d.getDay();
            if ( year > minyear ) {
                return false;
            } else if ( year === minyear ) {
                if ( month < minmon ) {
                    return true;
                } else {
                    if ( month === minmon ) {
                        if ( day <= minday ) {
                            return true;
                        } else {
                            return false;
                        }
                    } else {
                        return false;
                    }
                }
            } else {
                return true;
            }
        }

        function validInfo() {
            var User = Form.User.value;
            var Pass = Form.Pass.value;
            var Month = Form.Month.value;
            var Day = Form.Day.value;
            var Year = Form.Year.value;
            if ( ( validatePassword( Pass ) === true ) && ( validateBirthday( Month, Day, Year ) === true ) )
                validateEmail(User);
            else
                formNotValid();
            return false;
        }

    </script>
</head>
<body>
<form id="Form" name="Form" action="Register-Data.php" method="POST" enctype="multipart/form-data">
    <label>Email:</label>
    <input type="text" placeholder="Enter Email" id="User" name="User" maxlength="30" style="width:352px;height:30px;" required><br>
    <label>Password:</label>
    <input type="password" placeholder="Enter Password" id="Pass" name="Pass" maxlength="20" style="width:305px;height:30px;" required><br>
    <label>Birthday:</label>
    <select id="Month" name="Month" required>
        <option value="1">January</option>
        <option value="2">February</option>
        <option value="3">March</option>
        <option value="4">April</option>
        <option value="5">May</option>
        <option value="6">June</option>
        <option value="7">July</option>
        <option value="8">August</option>
        <option value="9">September</option>
        <option value="10">October</option>
        <option value="11">November</option>
        <option value="12">December</option>
    </select>
    <select id="Day" name="Day" required>
        <option value="1">1</option>
        <option value="2">2</option>
        <option value="3">3</option>
        <option value="4">4</option>
        <option value="5">5</option>
        <option value="6">6</option>
        <option value="7">7</option>
        <option value="8">8</option>
        <option value="9">9</option>
        <option value="10">10</option>
        <option value="11">11</option>
        <option value="12">12</option>
        <option value="13">13</option>
        <option value="14">14</option>
        <option value="15">15</option>
        <option value="16">16</option>
        <option value="17">17</option>
        <option value="18">18</option>
        <option value="19">19</option>
        <option value="20">20</option>
        <option value="21">21</option>
        <option value="22">22</option>
        <option value="23">23</option>
        <option value="24">24</option>
        <option value="25">25</option>
        <option value="26">26</option>
        <option value="27">27</option>
        <option value="28">28</option>
        <option value="29">29</option>
        <option value="30">30</option>
        <option value="31">31</option>
    </select>
    <select id="Year" name="Year" required>
        <option value="2018">2018</option>
        <option value="2017">2017</option>
        <option value="2016">2016</option>
        <option value="2015">2015</option>
        <option value="2014">2014</option>
        <option value="2013">2013</option>
        <option value="2012">2012</option>
        <option value="2011">2011</option>
        <option value="2010">2010</option>
        <option value="2009">2009</option>
        <option value="2008">2008</option>
        <option value="2007">2007</option>
        <option value="2006">2006</option>
        <option value="2005">2005</option>
        <option value="2004">2004</option>
        <option value="2003">2003</option>
        <option value="2002">2002</option>
        <option value="2001">2001</option>
        <option value="2000">2000</option>
        <option value="1999">1999</option>
        <option value="1998">1998</option>
        <option value="1997">1997</option>
        <option value="1996">1996</option>
        <option value="1995">1995</option>
        <option value="1994">1994</option>
        <option value="1993">1993</option>
        <option value="1992">1992</option>
        <option value="1991">1991</option>
        <option value="1990">1990</option>
        <option value="1989">1989</option>
        <option value="1988">1988</option>
        <option value="1987">1987</option>
        <option value="1986">1986</option>
        <option value="1985">1985</option>
        <option value="1984">1984</option>
        <option value="1983">1983</option>
        <option value="1982">1982</option>
        <option value="1981">1981</option>
        <option value="1980">1980</option>
        <option value="1979">1979</option>
        <option value="1978">1978</option>
        <option value="1977">1977</option>
        <option value="1976">1976</option>
        <option value="1975">1975</option>
        <option value="1974">1974</option>
        <option value="1973">1973</option>
        <option value="1972">1972</option>
        <option value="1971">1971</option>
        <option value="1970">1970</option>
        <option value="1969">1969</option>
        <option value="1968">1968</option>
        <option value="1967">1967</option>
        <option value="1966">1966</option>
        <option value="1965">1965</option>
        <option value="1964">1964</option>
        <option value="1963">1963</option>
        <option value="1962">1962</option>
        <option value="1961">1961</option>
        <option value="1960">1960</option>
        <option value="1959">1959</option>
        <option value="1958">1958</option>
        <option value="1957">1957</option>
        <option value="1956">1956</option>
        <option value="1955">1955</option>
        <option value="1954">1954</option>
        <option value="1953">1953</option>
        <option value="1952">1952</option>
        <option value="1951">1951</option>
        <option value="1950">1950</option>
        <option value="1949">1949</option>
        <option value="1948">1948</option>
        <option value="1947">1947</option>
        <option value="1946">1946</option>
        <option value="1945">1945</option>
        <option value="1944">1944</option>
        <option value="1943">1943</option>
        <option value="1942">1942</option>
        <option value="1941">1941</option>
        <option value="1940">1940</option>
        <option value="1939">1939</option>
        <option value="1938">1938</option>
        <option value="1937">1937</option>
        <option value="1936">1936</option>
        <option value="1935">1935</option>
        <option value="1934">1934</option>
        <option value="1933">1933</option>
        <option value="1932">1932</option>
        <option value="1931">1931</option>
        <option value="1930">1930</option>
        <option value="1929">1929</option>
        <option value="1928">1928</option>
        <option value="1927">1927</option>
        <option value="1926">1926</option>
        <option value="1925">1925</option>
        <option value="1924">1924</option>
        <option value="1923">1923</option>
        <option value="1922">1922</option>
        <option value="1921">1921</option>
        <option value="1920">1920</option>
        <option value="1919">1919</option>
        <option value="1918">1918</option>
        <option value="1917">1917</option>
        <option value="1916">1916</option>
        <option value="1915">1915</option>
        <option value="1914">1914</option>
        <option value="1913">1913</option>
        <option value="1912">1912</option>
        <option value="1911">1911</option>
        <option value="1910">1910</option>
        <option value="1909">1909</option>
        <option value="1908">1908</option>
        <option value="1907">1907</option>
        <option value="1906">1906</option>
        <option value="1905">1905</option>
        <option value="1904">1904</option>
        <option value="1903">1903</option>
        <option value="1902">1902</option>
        <option value="1901">1901</option>
        <option value="1900">1900</option>
    </select>
    <br><label>Select a Gender</label><br>
    <label>M:</label>
    <input type="radio" id="radio" name="Gender" value="Male">
    <label>F:</label>
    <input type="radio" id="radio" name="Gender" value="Female">
    <label>N/A:</label>
    <input type="radio" id="radio" name="Gender" value="Other" checked><br>
    <button type="submit" style="position:relative;width:400px;height:50px;" onclick="return validInfo()">Register</button>
</form>
</body>
</html>

Open in new window

0
 
Caden PangHS Sophomore Web DeveloperAuthor Commented:
Excellent
0
 
Caden PangHS Sophomore Web DeveloperAuthor Commented:
Actually, upon implementation, this new code has the same result as when I executed my original code...
0
 
leakim971PluritechnicianCommented:
double check for JS error in your page or share a link to test your implementation.
0
 
Caden PangHS Sophomore Web DeveloperAuthor Commented:
Never mind, I spent to much time messing around with javascript for validation. I just did it all in php.
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.