Panta Posted April 24, 2017 Share Posted April 24, 2017 please am not very good with javascript. i need this to display error that the email is already in use. but it keeps displaying success message. the php script is working fine. just the javascript side $(function() { $("#registerForm").find("input,textarea,select").jqBootstrapValidation({ preventSubmit: true, submitError: function($form, event, errors) { // additional error messages or events }, submitSuccess: function($form, event) { event.preventDefault(); // prevent default submit behaviour // get values from FORM var firstname = $("input#firstname").val(); var lastname = $("input#lastname").val(); var linked = $("input#linked").val(); var email = $("input#email").val(); var phone = $("input#phone").val(); var gender = $("select#gender").val(); var accountname = $("input#accountname").val(); var accountnumber = $("input#accountnumber").val(); var address = $("input#address").val(); var bankname = $("select#bankname").val(); var namenow = firstname; // For Success/Failure Message // Check for white space in name for Success/Fail message if (namenow.indexOf(' ') >= 0) { namenow = firstname.split(' ').slice(0, -1).join(' '); } $.ajax({ url: "././member/register.php", type: "POST", data: { firstname: firstname, lastname: lastname, linked: linked, email: email, phone: phone, gender: gender, bankname: bankname, accountnumber: accountnumber, accountname: accountname, address: address }, cache: false, success: function(result) { // Success message if(result=="1"){ $('#success').html("<div class='alert alert-danger'>"); $('#success > .alert-danger').html("<button type='button' class='close' data-dismiss='alert' aria-hidden='true'>×") .append("</button>"); $('#success > .alert-danger').append("<strong>Sorry " + namenow + ", it seems that my mail server is not responding. Please try again later!"); $('#success > .alert-danger').append('</div>'); //clear all fields $('#registerForm').trigger("reset"); } else{ $('#success').html("<div class='alert alert-success'>"); $('#success > .alert-success').html("<button type='button' class='close' data-dismiss='alert' aria-hidden='true'>×") .append("</button>"); $('#success > .alert-success') .append("<strong>Your Registeration is successful. Check your email/spam to validate. </strong>"); $('#success > .alert-success') .append('</div>'); //clear all fields $('#registerForm').trigger("reset");} }, error: function() { // Fail message $('#success').html("<div class='alert alert-danger'>"); $('#success > .alert-danger').html("<button type='button' class='close' data-dismiss='alert' aria-hidden='true'>×") .append("</button>"); $('#success > .alert-danger').append("<strong>Sorry " + namenow + ", it seems that my mail server is not responding. Please try again later!"); $('#success > .alert-danger').append('</div>'); //clear all fields $('#registerForm').trigger("reset"); }, }) }, filter: function() { return $(this).is(":visible"); }, }); $("a[data-toggle=\"tab\"]").click(function(e) { e.preventDefault(); $(this).tab("show"); }); }); /*When clicking on Full hide fail/success boxes */ $('#firstname').focus(function() { $('#success').html(''); }); <?php session_start(); //start the session for the page date_default_timezone_set("Africa/Lagos"); include("../include/db.php"); //include database file include("../include/settings.php"); //include configuration file //Check if page was entered by a submit button $firstname=$_POST['firstname']; //Get username !!FROM FORM!! $firstname = ereg_replace(" ", "", $firstname); //take away all spaces from username (if any) !!FROM FORM!! $lastname=$_POST['lastname']; //Get name $linked=$_POST['linked']; //Get name $email=$_POST['email']; //Get email !!FROM FORM!! $email = ereg_replace(" ", "", $email); //take away all spaces from email (if any) !!FROM FORM!! $phone=$_POST['phone']; //Get email $gender=$_POST['gender']; //Get email $bankname=$_POST['bankname']; //Get email $accountname=$_POST['accountname']; //Get email $accountnumber=$_POST['accountnumber']; //Get email $address=$_POST['address']; //Get email $joined=date("Y-m-d H:i:s"); //check to see if the username or email allready excists $ck=$flash->prepare("SELECT * FROM `user` WHERE `email`=:email"); //get rows where the username or email address is allready registered $ck->bindParam(':email',$email); $ck->execute(); //if email address allready excists if($ck->rowCount() > 0){ //tell the user //echo "The email you selected ($email) has all ready been registered! Please press back and choose another"; //return false; echo "1"; }else{ //IF THE USER GOT TO HERE, THEN HES FILLED OUT ALL THE FORMS, THE PASSWORDS MATCH, AND ENTERED A VALID USERNAME AND EMAIL, SO CREATE THE ACCOUNT //add account to DB srand ((double) microtime( )*1000000); $random=rand(10000,100000000); //check to see if admin has enabled email confirmation if ($_emailconfirmation == "1"){ //add the info to the database $reg = $flash->prepare("INSERT INTO `user` ( email , firstname , lastname ,referral, bank , bank_account_name , bank_account_number, phone , gender ,address , token, joined) VALUES (:email, :firstname, :lastname, :linked, :bankname, :accountname,:accountnumber,:phone,:gender,:address, :random , :joined )"); $reg->bindParam(':email',$email); $reg->bindParam(':firstname',$firstname); $reg->bindParam(':lastname',$lastname); $reg->bindParam(':linked',$linked); $reg->bindParam(':bankname',$bankname); $reg->bindParam(':accountname',$accountname); $reg->bindParam(':accountnumber',$accountnumber); $reg->bindParam(':phone',$phone); $reg->bindParam(':gender',$gender); $reg->bindParam(':address',$address); $reg->bindParam(':random',$random); $reg->bindParam(':joined',$joined); if($reg->execute()){ $to = "$email"; $subject = "Email Confirmation"; $body = "Thankyou for registering! you will now need to visit: $_sitelink/member/verifyaccount.php?actkeycode=$random&email=$email"; //send confirmation email to the user to activate their account via a link mail($to, $subject, $body, "From: $_replyemail"); //let them know it went well die("Welcome $username! You have registered successfully!<br>An email has been sent to the registered email with a link you need to vist it activate your account!"); return true;} }} ?> Link to comment Share on other sites More sharing options...
justsomeguy Posted April 24, 2017 Share Posted April 24, 2017 The PDO rowCount function doesn't return the number of rows returned by the select statement for most databases, so instead of that you should send a query where you select the count. Link to comment Share on other sites More sharing options...
Panta Posted April 24, 2017 Author Share Posted April 24, 2017 15 minutes ago, justsomeguy said: The PDO rowCount function doesn't return the number of rows returned by the select statement for most databases, so instead of that you should send a query where you select the count. please explain more Link to comment Share on other sites More sharing options...
justsomeguy Posted April 24, 2017 Share Posted April 24, 2017 https://www.w3schools.com/sql/sql_count_avg_sum.asp Link to comment Share on other sites More sharing options...
Panta Posted April 24, 2017 Author Share Posted April 24, 2017 12 minutes ago, justsomeguy said: https://www.w3schools.com/sql/sql_count_avg_sum.asp is this correct? $ck=$flash->prepare("SELECT COUNT(email) FROM `user` WHERE `email`=:email"); //get rows where the username or email address is allready registered $ck->bindParam(':email',$email); $ck->execute(); //if email address allready excists if($ck->fetchAll() > 0){ //tell the user //echo "The email you selected ($email) has all ready been registered! Please press back and choose another"; //return false; echo "1"; } Link to comment Share on other sites More sharing options...
justsomeguy Posted April 24, 2017 Share Posted April 24, 2017 fetchAll will return an array of all of the records returned, so comparing that with 0 wouldn't work. You can either just select all of the records, use fetchAll, and see if the length of the array is greater than 0, or give the count an alias and then fetch the row and access the count with the alias name like any other column. SELECT COUNT(email) AS nr_email ... Then you can get the row from that result and check the nr_email column. Link to comment Share on other sites More sharing options...
Panta Posted April 25, 2017 Author Share Posted April 25, 2017 i have done that ans is not working Link to comment Share on other sites More sharing options...
justsomeguy Posted April 25, 2017 Share Posted April 25, 2017 Well, then you haven't done it right. Show the code you're using. Link to comment Share on other sites More sharing options...
Gabrielphp Posted April 27, 2017 Share Posted April 27, 2017 (edited) Why use success and error ajax functions when you can use .done and .fail? .done and then you do all the checks in the php script even the return so that way you don't need to do that much of a javascript code in order to display an message. For example my way i use to display messages with jquery is very simple. For example index.php <div id="return_php" style="display: none;"></div> <form id="test"> <input type="text" name="something" placeholder="Insert something ..."> <input type="submit"> </form> <script> $('#test').submit(function(e){ e.preventDefault(); $.ajax({ url: 'path/to/something.php', data: $(this).serialize(), dataType: 'html', type: 'POST' }) .done(function(data){ $('#return_php').fadeIn(400).html(data).delay(5000).fadeOut(400); }) .fail(function(){ $('#return_php').fadeIn(400).html('Something went wrong with the ajax script !').delay(5000).fadeOut(400); }) }); </script> This way you just call the php page in which you do the checks and whatever stuff and then you echo an alert which will then be displayed into the return_php div. The .delay(5000) acts as a delay for the fadeOut function, after 5 seconds the message will fade away. something.php $something = $_POST['something']; if(empty($something)) { echo "Something needs to be filled in !"; //This message will be then returned into the div return_php on the index.php page } else { //Example of database try { $sql = "SELECT something FROM something WHERE something = :something"; $stmt = $db->prepare($sql); //$db is from a config file you have to include $stmt->bindParam(":something", $something); $stmt->execute(); $rowCount = $stmt->rowCount(); //This works for every case, at least i use it this way and for me works just fine. if($rowCount > 0) //let us presume that the something has more rows { echo "Ok, there is something in that something !"; } else { echo "There is nothing in something with the something you typed !"; } } catch (PDOException $e) { echo "Error: " . $e->getMesage(); //This will be showed into the return div aswell as the echo is still a part of this page, and this page is called by ajax. } } Now, this is my way to do it time to time when i need live php script calling on my page without needing my page to refresh. Hope it helped at least for your issue with displaying messages. EDIT I've read again through your post and i think i figured out what is all about. When i try to use that ereg_replace() php function is always throwing me an Uncaught error as the function doesn't exist. This may be influencing how you are selecting the data by that variable. Instead of ereg_replace use preg_replace, It does the same thing. $email = $_POST['email']; $email = preg_replace('/\s+/, '', $email); NOTE! Some of the code i wrote may or may not fit your code if you copy paste it as it doesn't contain the same variables or functions that you wrote. You may need to adapt it to your use. Edited April 27, 2017 by Gabrielphp 1 Link to comment Share on other sites More sharing options...
Panta Posted April 28, 2017 Author Share Posted April 28, 2017 13 hours ago, Gabrielphp said: Why use success and error ajax functions when you can use .done and .fail? .done and then you do all the checks in the php script even the return so that way you don't need to do that much of a javascript code in order to display an message. For example my way i use to display messages with jquery is very simple. For example index.php <div id="return_php" style="display: none;"></div> <form id="test"> <input type="text" name="something" placeholder="Insert something ..."> <input type="submit"> </form> <script> $('#test').submit(function(e){ e.preventDefault(); $.ajax({ url: 'path/to/something.php', data: $(this).serialize(), dataType: 'html', type: 'POST' }) .done(function(data){ $('#return_php').fadeIn(400).html(data).delay(5000).fadeOut(400); }) .fail(function(){ $('#return_php').fadeIn(400).html('Something went wrong with the ajax script !').delay(5000).fadeOut(400); }) }); </script> This way you just call the php page in which you do the checks and whatever stuff and then you echo an alert which will then be displayed into the return_php div. The .delay(5000) acts as a delay for the fadeOut function, after 5 seconds the message will fade away. something.php $something = $_POST['something']; if(empty($something)) { echo "Something needs to be filled in !"; //This message will be then returned into the div return_php on the index.php page } else { //Example of database try { $sql = "SELECT something FROM something WHERE something = :something"; $stmt = $db->prepare($sql); //$db is from a config file you have to include $stmt->bindParam(":something", $something); $stmt->execute(); $rowCount = $stmt->rowCount(); //This works for every case, at least i use it this way and for me works just fine. if($rowCount > 0) //let us presume that the something has more rows { echo "Ok, there is something in that something !"; } else { echo "There is nothing in something with the something you typed !"; } } catch (PDOException $e) { echo "Error: " . $e->getMesage(); //This will be showed into the return div aswell as the echo is still a part of this page, and this page is called by ajax. } } Now, this is my way to do it time to time when i need live php script calling on my page without needing my page to refresh. Hope it helped at least for your issue with displaying messages. EDIT I've read again through your post and i think i figured out what is all about. When i try to use that ereg_replace() php function is always throwing me an Uncaught error as the function doesn't exist. This may be influencing how you are selecting the data by that variable. Instead of ereg_replace use preg_replace, It does the same thing. $email = $_POST['email']; $email = preg_replace('/\s+/, '', $email); NOTE! Some of the code i wrote may or may not fit your code if you copy paste it as it doesn't contain the same variables or functions that you wrote. You may need to adapt it to your use. Thanks so much bro +rep 1 Link to comment Share on other sites More sharing options...
Gabrielphp Posted April 28, 2017 Share Posted April 28, 2017 5 hours ago, Panta said: Thanks so much bro +rep Glad i could help. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now