Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 1 down vote favorite

On this site Site there are 4 checkboxes. At the moment the checkbox values are not shown in email. What has to be done to get the checkbox values in the email?

This is the whole code kontakt.php sendmail.class.php

kontakt.php:

<form action="kontakt.php" method="post">
    <label for="name">Ihr Name:</label>
    <input type="text" class="get1" name="name" value="<?php echo htmlspecialchars($_POST['name']); ?>" />
    <br />
    <label for="phone">Telefonnummer (bitte nur Ziffern):</label>
    <input type="text" class="get1" name="phone" value="<?php echo htmlspecialchars($_POST['phone']); ?>" />
    <br />
    <label for="email">E-Mail-Adresse:</label>
    <input type="text" class="get1" name="email" value="<?php echo htmlspecialchars($_POST['email']); ?>" />
    <br />
    <label for="message">Ihre Mitteilung an uns:</label>
    <textarea cols="20" rows="5" name="message"><?php echo htmlspecialchars($_POST['message']); ?></textarea>
    <br />
    <label>Welche Leistungen interessieren Sie besonders?</label>
    <label style="display:inline;" for="dachstuehle">Dachstühle</label>  
    <input class="kaestchen" type="checkbox" name="interessen[]" id="dachstuehle" value="dachstuehle" />
    <label style="display:inline;" for="holzbau">Holzbau</label>
    <input class="kaestchen" type="checkbox" name="interessen[]" id="holzbau" value="holzbau" />
    <label style="display:inline;" for="innenausbau">Innenausbau</label>
    <input class="kaestchen" type="checkbox" name="interessen[]" id="innenausbau" value="innenausbau" />
    <label style="display:inline;" for="trocken">Trockenbau</label>
    <input class="kaestchen" type="checkbox" name="interessen[]" id="trockenbau" value="trockenbau" />
    <br /><br />
    <input type="hidden" value="1" name="s" />
    <input type="submit" value="Nachricht versenden" name="submit" />
</form>

sendmail.class.php:

<?php
class sendMail {
    var $to = 'email'; // set contact email
    var $name = '';
    var $phone = '';
    var $email = '';
    var $body = '';
    var $interessen = '';
    var $error = array();
    var $headers = array();

    function parseBody() {
        $message = '<html><body>';
        $message .= '<table rules="all" style="border-color: #666;" cellpadding="10">';
        $message .= '<tr style="background-color: #eee;"><td><strong>Name:</strong> </td><td>' . $this->name . '</td></tr>';
        $message .= "<tr><td><strong>Telefonnummer:</strong> </td><td>" . $this->phone . "</td>    </tr>";
        $message .= "<tr><td><strong>E-Mail-Adresse:</strong> </td><td>" . $this->email . "</td></tr>";
        $message .= "<tr><td><strong>Text:</strong> </td><td>" . $this->body . "</td></tr>";
        $message .= "<tr><td><strong>Interessiert an:</strong> </td><td>" . $this->interessen . "</td></tr>";
        $message .= "</table>";
        $message .= "</body></html>";
        $this->body = $message;
    }

    function send() {
        if ($this->error) {
            return FALSE;
        }

        if (mail($this->to, '=?utf-8?B?' . base64_encode($this->subject) . '?=', $this->body, $this->headers)) {
            return TRUE;
        } else {
            $this->error[] = 'Fehler beim senden';
            return FALSE;
        }
    }

}
?>

This is the whole code kontakt.php sendmail.class.php

share|improve this question
 
How do you want them displayed in the email?? Comma Seperated? –  KyleK Jun 12 at 16:00
 
interessen will be an array, looks like you're treating it as a string? And in your full class I don't see where you are setting that value. –  somedev Jun 12 at 16:00
 
In your sendmail.class.php, you do not retrieve $_POST['interessen'], which is an array. You will need to add that into the function that loads post values, and call implode() on it to return a string. –  Michael Berkowski Jun 12 at 16:00
 
yes comma separated! –  Daniel Hernandez Jun 12 at 16:00
 
Note that if you have the opportunity to make the change, it is recommended to update your code to PHP5 style classes, rather than the old PHP4 style using the var keyword. php.net/manual/en/language.oop5.basic.php –  Michael Berkowski Jun 12 at 16:01
show 4 more comments

1 Answer

up vote 1 down vote accepted

Simple...

So just put in your setParams() method like this.....

function setParams($post)
  {
  $this->interessen = implode(", ",$post['interessen']);
  //REST OF SETPARAMS STUFF...leave alone
 }

Then in your parseBody()...

Make sure the line that echoes the 'interessen'..is like this...

 $message .= "<tr><td><strong>Interessiert an:</strong> </td><td>". $this->interessen ."</td>

Which it is in your example above.

share|improve this answer
 
OK I check it and if it works I will accept your answer. –  Daniel Hernandez Jun 12 at 16:05
 
Just one question: where goes $string = implode(",",$this->interessen);? I'm not that advanced in PHP. –  Daniel Hernandez Jun 12 at 16:07
 
It gives me error message: Warning: implode(): Invalid arguments passed in /mnt/web3/a2/69/53523169/htdocs/das-bauteam/php/sendmail.class.php on line 53 Warning: Cannot modify header information - headers already sent by (output started at /mnt/web3/a2/69/53523169/htdocs/das-bauteam/php/sendmail.class.php:53) in /mnt/web3/a2/69/53523169/htdocs/das-bauteam/kontakt.php on line 11 –  Daniel Hernandez Jun 12 at 16:15
1  
Checked your sendmail class code, and you need to do this before the headers are sent....also its not being declared as a post variable....Ill provide code in a miinute –  KyleK Jun 12 at 16:23
 
There you go...:) –  KyleK Jun 12 at 16:33
show 3 more comments

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.