[RESOLVED] E-mail Array Values

[rsmith]

Hey all,
I have an array that is created through a form. When a user selects the elements and submits the form he's given the chance to review his selections. Each selection is printer on a separate line. Here's the script:

<input type="checkbox" name="hardware[]">Monitor
<input type="checkbox" name="hardware[]">Computer
<input type="checkbox" name="hardware[]">Keyboard
<input type="checkbox" name="hardware[]">Mouse
<input type="checkbox" name="hardware[]">Printer

<?
     $hardware = $_POST['hardware']

 //Iterate through the array and display each item on a new line.
      foreach($hardware AS $hwItem){
        echo $hwItem."\n";
      }
?>

 Like I said this works great on the review page but in an effort to make things paperless for the company, I also need to have these values e-mailed to a staff member. I'm using PHP mailer and need to know how to make this show up correctly in the e-mail.

 What needs to go into this part of the PHP Mailer script:

     $mail->Body = "";

TIA for any and all help and I hope this makes sense.

Ryan

[NogDog]

You will probably want to assign values to each checkbox:

<input type="checkbox" name="hardware[]" value="Monitor">Monitor
<input type="checkbox" name="hardware[]" value="Computer">Computer
<input type="checkbox" name="hardware[]" value="Keyboard">Keyboard
<input type="checkbox" name="hardware[]" value="Mouse">Mouse
<input type="checkbox" name="hardware[]" value="Printer">Printer

To add them to the message body:

if(isset($_POST['hardware'] and count($_POST['hardware']))
{
   $mail->Body .= "Hardware: " . implode(', ', $_POST['hardware']);
}
else
{
   $mail->Body .= "(No hardware specified)";
}

[rsmith]

Thanks Nog. I do have values assigned I just didn't feel like retyping them. I tried using the implode function but got this error:

Warning: implode() [function.implode]: Bad arguments. in c:\Inetpub\wwwroot\newhire\submitRequest.php on line 53

The hardware array is already referenced as simply the $hardware variable as it is assigned by using $hardware = $_POST['hardware'];

Here is my line 53:

$mail->Body = "Requestor: ".$reqBy."\nRequested On: ".$reqDate."\nEmployee Name: ".$empName."\nHardware\n" .implode(', ', $hardware);"\n\nFinal Approval by: ".$approvedBy."";

[NogDog]

Firstly, you have a semi-colon after the call to implode() where it looks like you want a concatenation ".".

The call to implode() looks OK, so if it's still failing after fixing the above problem, then I suspect $hardware is not an array. Try assigning it as:

$hardware = (!empty($_POST['hardware']) && is_array($_POST['hardware'])) ?
      $_POST['hardware'] : array();

[rsmith]

I made the change regarding concatenation. Still getting the same issue. The way the Hardware array is built follows this flow:

1. index.php contains the initial form and checkboxes

2. review.php uses the $hardware = $_POST['hardware']; call to pull the array into a variable.

3. review.php retains the array by putting the $hardware array as the value of a hidden form element:

<input type="hidden" name="hardware" value="<? echo $hardware; ?>

4. submit.php uses the same $hardware = $_POST['hardware']; call to pull the array into the variable that is passed to the implode() command.

Can I not pass an array as a hidden attribute value?

[NogDog]

rsmith wrote:

Can I not pass an array as a hidden attribute value?

No. At least, not that simply. You could do something like:

foreach($hardware as $hw)
{
   echo "<input type='hidden' name='hardware[]' value='$hw'>\n";
}

Alternatively, you might be able to do it with one hidden field if you [man]serialize/man the array before outputting it as the field value, then [man]unserialize/man it in the receiving script.

[rsmith]

Nog, you rock! Thanks a lot for all your help man! I appreciate it!