Jump to content

what error to display from php


Panta

Recommended Posts

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'>&times;")
                        .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'>&times;")
                        .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'>&times;")
                        .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

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

12 minutes ago, justsomeguy said:

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

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

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 by Gabrielphp
  • Like 1
Link to comment
Share on other sites

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

  • Like 1
Link to comment
Share on other sites

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 account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...