[RESOLVED] email array checking validity of email address

[verN]

I had the following working but since I changed the email variable to an array it doesn't work. email[] This is part of an edit form where emails in the mailing lists can get updated however, before they get updated i need to check that each of the email addresses are valid.But the code doesn't work for the array.

//checks for a email address through POST.
         if ((isset($_POST['email']))) { 
             $eAdd=$_POST['email'];
		 }
 if (eregi ('^[[:alnum:]][a-z0-9_\.\-]*@[a-z0-9\.\-]+\.[a-z]{2,4}$', stripslashes($_POST['email']))) {
	          //stores the email address
        	 $eAdd= trim($_POST['email']); 
             } else {
        	 $error[]= 'Please enter a valid email address!';
        	 }

thanks

[leatherback]

//checks for a email address through POST.
         if ((isset($_POST['email']))) {
             // $eAdd is an ARRAY with email address
             $eAdd=$_POST['email'];
         }
// we loop through the array
foreach($eAdd as $key => $email)
  {

  if (eregi ('^[[:alnum:]][a-z0-9_\.\-]*@[a-z0-9\.\-]+\.[a-z]{2,4}$', stripslashes($email))) {
              //stores the email address
             $eAdd[$key]= trim($email);
             } else {
             $error[]= "Email address $key is invalid. Please enter a valid email address!";
             } 
} //edit: Forgot the ending bracket

[verN]

I still have the same problem i think it may be due to the other checking that I do below, to check that the email address is unique and to ensure that it has been updated. When I entered an invalid email address no message appeared

thanks in advance

<?php
// checks for a email address through POST.
         if ((isset($_POST['email']))) { 
             $eAdd=$_POST['email'];
		 }

      // checks whether or not to process the form
     if(isset($_POST['update'])) {   


      // we loop through the array 
     foreach($eAdd as $key => $email) 
     { 

        if (eregi ('^[[:alnum:]][a-z0-9_\.\-]*@[a-z0-9\.\-]+\.[a-z]{2,4}$', stripslashes($email))) { 
          //stores the email address 
         $eAdd[$key]= trim($email); 
         } else { 
         $error[]= "Email address $key is invalid. Please enter a valid email address!"; 
         } 
     }



    // there's no errors 	 
    if (empty($error)) {


  	// Make sure the email address is available and there isn't a duplicate.
	foreach($eAdd as $key => $ckemail) {
	$dup = "SELECT seminarID FROM mailinglists WHERE seminarID='$id' AND emailAddress='$ckemail'";
	$dup_result = mysql_query ($dup,$dbc) or die (mysql_error());
	}
	// if there isn't a duplicate email address continue
	if (mysql_num_rows($dup_result) == 0) {


	// This is used to make data safe before sending a query to MySQL. 	
	$id =mysql_real_escape_string($id);


    $updatesMade = false;
    // update the database.

		$updateM_query = "UPDATE mailinglists SET emailAddress='$eAdd[$i]' WHERE seminarID='$id' AND emailAddress='$oldEmail[$i]'";
		//echo '<b>'.$id.'</b><br>';
		//echo ''.$eAdd[$i].'<br>';
		$updateM_result=mysql_query($updateM_query,$dbc) or die (mysql_error());
		//echo $updateM_query;			 
		// email addresss is inserted


		if (mysql_affected_rows() >= 1) { 
			$updatesMade = true; 
		}

			// If updates have been made, print a message. 
            if($updatesMade = true) {

                // Print a message.
				$changes[]="The changes have been updated.";	
            }

			} else { // The email address is not available.
		$error[]= "The email address  <font color='black'>$ckemail</font>   has already been added to the database!";
	        }		 

	} else {
	  $error = NULL; 
	}
 }
?>

[leatherback]

What php/mysql errors do you get? Do the queries run?

I see no code for displaying error messages when invalid email addresses are entered, so it is not surprising that you get no output..

[verN]

yeh sorry I didn't include it, the following outputs the errors. Since errors are stored in an array..

<?php
      // print the error messages
      if(!empty($error)) { 
	      echo '<p class="errorheader">Error!</p>
         <p class="error">The following error(s) occurred:<br />';
	    // prints each error that occurred
	    foreach ($error as $msg) { 
		  echo " - $msg<br />\n";
	    }
	       echo '</p><p class="error">Please try again.</p>';
      }
?> 

// there's no errors 	 
        if (empty($error)) {


  	// Make sure the email address is available and there isn't a duplicate.
    for($i = 0; $i < sizeof($eAdd); $i++) {
	$dup = "SELECT seminarID FROM mailinglists WHERE seminarID='$id' AND emailAddress='$eAdd[$i]'";
	$dup_result = mysql_query ($dup,$dbc) or die (mysql_error());


	// if there isn't a duplicate email address continue
	if (mysql_num_rows($dup_result) == 0) {


	// This is used to make data safe before sending a query to MySQL. 	
	$id =mysql_real_escape_string($id);

    $updatesMade = false;
    // update the database.

		$updateM_query = "UPDATE mailinglists SET emailAddress='$eAdd[$i]' WHERE seminarID='$id' AND emailAddress='$oldEmail[$i]'";
		//echo '<b>'.$id.'</b><br>';
		//echo ''.$eAdd[$i].'<br>';
		$updateM_result=mysql_query($updateM_query,$dbc) or die (mysql_error());
		//echo $updateM_query;			 
		// email addresss is inserted


		if (mysql_affected_rows() >= 1) { 
			$updatesMade = true; 
		}

			// If updates have been made, print a message. 
            if($updatesMade = true) {

                // Print a message.
				$changes[]="The changes have been updated.";	
            }

			} else { // The email address is not available.
		$error[]= "The email address  <font color='black'>$eAdd[$i]</font>   has already been added to the database!";
	        }		 
    }
	} else {
	  $error = NULL; 
	}
 }
?>

Problem 1 is when i include an invalid email address, no message gets ouputted.
Problem 2 is when i update one email address, it gets updated but the for the other email addresses the follwoing error appears
Error!

The following error(s) occurred:
- The email address [e-mail] has already been added to the database!

Please try again.

I think this may be due to the array and the for loop it runs through and I have tried to make chnages to it but to no avail.

tnaks

[verN]

repeat