[RESOLVED] checkbox email to selected/all

[roscor]

Hi hope some one can help.

I get all my mebers off my database and display using this

<?PHP
include("dbinfo.inc.php");
    $conn = mysql_connect($host, $username, $password) or die(mysql_error());
    mysql_select_db($database,$conn) or die(mysql_error());  

$query = "SELECT * from landlordlo" or die (mysql_error());
 $result = mysql_query($query) or die ("Could not run query.");

$num_rows = mysql_num_rows($result);


echo "<font face='Verdana' size='2' color='#FF0000'><font size='1' face='Verdana'>(&nbsp;$num_rows&nbsp;)</a></td>&nbsp;Members Found on Database</font>";

}
?>
<table width="600" border="0" cellspacing="0" cellpadding="0" align="centre" >
<tr>
   <td width="100" align="center"><font face="Verdana" size="2" color="#000000">Check Box:</font></td>
	<td width="200" align="center"><font face="Verdana" size="2" color="#000000">Username:</font></td>
	 <td width="300" align="center"><font face="Verdana" size="2" color="#000000">Email</font></td>  

</tr>
</tr>
</table>



<table border="0" width="600" height="30"  bordercolor="#111111">


<tr>

  <td style="border-style:solid; border-width:1px; border-color:#000000;" width="600" height="100" valign="top"  bgcolor="#FFFFFF" bordercolor="#111111">

  <div class="scrollable" style="height:400px; width:600; overflow:auto;">


 <table cellSpacing="0" cellPadding="1" border="0" id="RecentFeedback" class="itemTable" width="600" height="20" align="top"> 
<td  align="center"><font face="Verdana" size="2" color="#000000"></td>
<td  align="center"><font face="Verdana" size="2" color="#000000"></td>
<td  align="center"><font face="Verdana" size="2" color="#000000"></td>
</tr>
<?
$color1 = '#EEEEEE';
$color2 = '#FFFFFFF';


$i = 0;

while ($row = mysql_fetch_array($result)) {

$color = (($i%2 == 0 || $i == 0) ? $color1 : $color2);

$user=$row['username'];
$email=$row['email'];


?>

<form action="adminemail.php" method="post">
<font face="Verdana" size="2" color="#000000">
<th><input type="checkbox" name="email"  value="<? echo $email; ?>l" MULTIPLE> <? echo "<td bgcolor=$color align='center' size='100'><font face='Verdana' size='2' color='#000000'>$email</td>"; ?></th>
<th><input type="hidden" value="<? echo $user; ?>" name="username"><? echo "<td bgcolor=$color align='center' size='200'><font face='Verdana' size='2' color='#000000'>$user</td>"; ?></th>

</font>

</tr>

<?
$i++;
}
echo "</table>";
?>

&nbsp;<input type="checkbox" name="checkall" onclick="checkUncheckAll(this);"/>select/unselect<br /><input type="submit" value ="send"></form>  

</body>


</html>

This then diplays all the records held. I want to use the checkbox to select/unselect as required. This is the sent to

adminemail.php

<?PHP session_start();

$hostname = $_SERVER['REMOTE_ADDR']; 
echo $hostname;
if ($hostname='???????????') { 

$message=$_POST['message'];
 if($_POST['submit']=="send") {
   $errcount = 0;
   $error1="<span style='color:red'>";      

   //check if someone has filled out a form
   //validate name
    if(preg_match('/^[a-zA-Z0-9_-]{1,}$/i', $_POST['message'])){ 
        $message  = $_POST['message']; 
    }
        else {
        $error1 .= "Required!";
        $errcount++;  

        }    

if ($errcount !=0) {
     //displays errors in form boxes
}
    else {
   $message=$_POST['message'];

$headers = "From: admin email\r\n";
$headers = "Reply-to: admin email\r\n";
$headers = "BCC: admin email\r\n";
$headers .= 'Content-Type: text/html';

$subject = "Welcome";

$rc = mail($email, $subject, nl2br($message),$headers);

}
if ($rc) {

die(print "<font face ='verdana' size='2'>Mail successfully sent!");
}
else {
die(print "<font face ='verdana' size='2'>Failed to send mail");
}
}
}

?>

<tr>
<th width="180" valign="top" bgcolor="#CCCCCC" class="sideBody" scope="col"><p align="left"><span class="sideHeads">ADMIN AREA</span><br>
</th>
<th colspan="2" valign="top" bgcolor="#FFFFFF" class="bodycopy" scope="col" align="left">
<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
<table width="100%" border="0" cellpadding="2" cellspacing="2" bgcolor="#FFFFFF">
<tr>
<th width="100%" valign="top" scope="col"><div align="left" class="bodycopy">
Enter your message below&nbsp;<? echo $error1; ?><br><textarea name="message" value="<? echo $message; ?>" cols="80" rows="20"><? echo $message; ?></textarea><br><input name="submit" type="submit" value="send" /><input type='button' value='Retry' onClick='history.go(-1)'><br>
</span>
</div></th>

</tr>
</table>
</form>

How can I tranfer all selected records to adminemail.php using the checkbox, I have been trying for two days using sessions etc but with no joy. I am sure it will be quite simple but some things dont come so easily to us all,

thanks roscor

[wilku]

I'm not sure if I understand you. You want to send an email to people checked on the first page, right?

If so, make POST varibles arrays:

<input type="checkbox" name="email[]" value="<?php echo $email;?>">
<input type="hidden" name="user[]" value="<?php echo $username;?>">

On the second page use var_dump($POST) to see in what format your data comes and according to this write your code to process the data (probably involving mail function inside foreach loop on one of the $POST[] varibles).

Another remark:

if ($hostname='???????????')

this is always true. Use == instead. (I think it's a typo anyway 😉 )

[roscor]

Hi Wilku,

many thanks for putting me on the right track.

//var_dump($_POST);
foreach($email as $c)
      {
          echo ''.$c.'<br />';
      }  

this outputs the emails with a line break, as required just what is selected off the checkbox page.

I am trying in now to see if it only emails those selected,

Cheers,

roscor.

[roscor]

I have tried the above but when i select multiple records off checkbox page not all the emails are being sent out on adminemail.php

I'm very stuck!!

[liquorvicar]

Can you post your latest code?

How are you adding the emails to your message body?
You need something like this:

foreach($email as $c)
      {
          $message.= $c.'<br />';
      } 

[roscor]

hi liquorvicar,

here is what I am using at the moment,

var_dump($_POST);
foreach($email as $key => $value)
   echo $value .'<br>';

{


$headers = "From: admin email\r\n";
$headers = "Reply-to: admin email\r\n";
$headers .= 'Content-Type: text/html';
$message = "General message";
$subject = "Welcome To Uk Tenant History";

//send mail;
$rc = mail($value, $subject, nl2br($message),$headers);

}
if ($rc) {

die(print "<font face ='verdana' size='2'>Mail successfully sent!<br><h4>Here are your results.</h4>\n <b>$numrows emails are being sent out!</b><p> <b>Email:</b> $value<br>\n</font>");
}
else {

die(print "<font face ='verdana' size='2'>Failed to send mail");
}
}


?>

when using var_dump($_POST) i can see the array details like such

array(2) { ["email"]=>  array(3) { [0]=>  string(26) "[e-mail]" [1]=>  string(21) "[e-mail]" [2]=>  string(20) "[e-mail]" } ["username"]=>  string(8) "????" }

This result is as selected using checkboxes on the submission page. The page doe's die() as expected and then displays the three(or how many selected emails there are) stating they have been sent, i previously used this page and selected out of my database, used a while loop with $numrows etc but i wanted to select any specific member and email them if required. The problem being that only the last email in the array is being sent, any ideas, i am well out of my depth. i have been trawling this forum and PHP.net with no joy as i am not sure how to proceed,

thanks for looking(do i need to re-think how I am going about this)

roscor

[liquorvicar]

It may be because of your ordering of statements in this bit:

foreach($email as $key => $value)
   echo $value .'<br>';

{ 

Usually, PHP blocks are written

statement {
    do action
}

but for single-line blocks you can write them as

statement
    do action

I suspect this is what's happenign in your script; the loop is only acting on the echo and then taking the last value of $value and processing your email block with that value.

ps as a stylistic point, if there's nothing in your header or subject that changes in the loop, try setting these values outside the loop, i.e.

$headers = "From: admin email\r\n";
$headers = "Reply-to: admin email\r\n";
$headers .= 'Content-Type: text/html';
$message = "General message";
$subject = "Welcome To Uk Tenant History"; 

foreach (etc...

[roscor]

liquorvicar cheers,
I did as you said and put,

$headers = "From: admin email\r\n";
$headers = "Reply-to: admin email\r\n";
$headers .= 'Content-Type: text/html';
$message = "General message";
$subject = "Welcome To Uk Tenant History";


foreach($email as $key => $value) 
//send mail;
$rc = mail($value, $subject, nl2br($message),$headers);

This has worked and sent seperate emails.

just trying to get the output displaying $value(emails sent) in

die(print "<font face ='verdana' size='2'>Mail successfully sent!<br><h4>Here are your results.</h4>\n <b>$numrows emails are being sent out!</b><p> <b>Email:</b> $value<br>\n</font>"); 

as this only displays the last item email as before. changing $value to $rc in die() displays 1.

roscor

[liquorvicar]

I wasn't suggesting using the one-line syntax for logical blocks, just explaining why your script wasn't working! You should definitely use the curly brackets in your foreach clause.

foreach($email as $key => $value) {
   $rc = mail($value, $subject, nl2br($message),$headers); 
   if ($rc) {
      echo($value.' sent<br />');
   }
}

[roscor]

I got rid of die() as it was not needed and used this which works great,

foreach($email as $key => $value) {
   $rc = mail($value, $subject, nl2br($message),$headers);
   if ($rc) {
      echo($value.' <font face ="verdana" size="2">Mail successfully sent!<br />');

}
else {
die(print "<font face ='verdana' size='2'>Failed to send mail<br>Go <a href='frames css.php'>Back</a> and try again</font>");
}

many thanks for your help and advice,

regards

roscor

[NZ_Kiwis]

I have this...

<input type="checkbox" name="emails[]" value="<?php echo $row['email']; ?>">

<?php

var_dump($_POST['emails']);
foreach($email as $value)
   echo $value .'<br>';


echo "<hr>";


?>

But get this error:

array(3) { [0]=> string(20) "[e-mail]" [1]=> string(23) "[e-mail]" [2]=> string(18) "[e-mail]" }

Warning: Invalid argument supplied for foreach() in /home/rlnz/public_html/admin/send.php on line 4

[bradgrafelman]

$email isn't the same thing as $_POST['email'] (at least, if it is, you've got a bigger problem...).

[NZ_Kiwis]

so where does $email come from?

[bradgrafelman]

shrug Dunno... maybe register_globals is turned on for him.

[NZ_Kiwis]

but what would I use??

[bradgrafelman]

Well, obviously you'd want to use $_POST['emails'], since var_dump()'ing it shows the array you're after.