2010-10-01 2 views
0

Я создал простую форму отправки, в которой пользователь вводит свой адрес электронной почты и нажимает кнопку отправки, а адрес электронной почты отправляется мне по электронной почте для использования в почтовой рассылке список.Эта простая форма контакта не будет работать должным образом

По какой-то причине он держит дает мне ту же ошибку, и не будет на самом деле отправить по электронной почте:

Извините, но это, кажется, не является действительным адресом электронной почты.

Вот мой код:

<?php define('access', true); ?> 
<?php 
    // Configuration 
    $emailTo = '[email protected]'; // The e-mail address 
    $mySite = 'test Mailing List'; // The name of the website 
    $subject = 'test | New Mailing Lister!'; 
    $msg = ''; 

    // If the form is submitted 
    if(isset($_POST['submit'])) { 
     // Check that the email entered is valid and not empty 
     if(trim($_POST['email']) == '') { 
      $hasError = true; 
      $msg = '<p class="error">Sorry but that doesn\'t seem to be a valid e-mail address.</p>'; 
     } else if (!eregi("^[A-Z0-9._%-][email protected][A-Z0-9._%-]+\.[A-Z]{2,4}$", trim($_POST['email']))) { 
      $hasError = true; 
      $msg = '<p class="error">Sorry but that doesn\'t seem to be a valid e-mail address.</p>'; 
     } else { 
      $email = trim($_POST['email']); 
     } 

     // Are we error free? If so send away 
     if(!isset($hasError)) { 
      $body = "Someone has signed up for the test mailing list, the e-mail address they used is:\n\n<strong>$email</strong>"; 
      $headers = 'From: ' . $mySite . ' <' . $emailTo . '>' . "\r\n" . 'Reply-To: ' . $email; 
      $msg = '<p class="thanks">Thanks for subscribing to test\'s services mailing list. We\'ll keep you posted on service updates!</p>'; 

      mail($emailTo, $subject, $body, $headers); 
      $emailSent = true; 
     } 
    } 
?> 
<?php include('includes/html_header.php'); ?> 
    <?php include('includes/header.php'); ?> 

    <div id="container"> 
     <?php if(!isset($_GET['id'])) { ?> 
     <div id="container_column-1"> 
      <h1>test</h1> 
      <p>Li Europan lingues es membres del sam familie. Lor separat existentie es un myth. Por scientie, musica, sport etc, litot Europa usa li sam vocabular. Li lingues differe solmen in li grammatica, li pronunciation e li plu commun vocabules. Omnicos directe al desirabilite de un nov lingua franca: On refusa continuar payar custosi traductores. At solmen va esser necessi far uniform grammatica, pronunciation e plu sommun paroles. Ma quande lingues coalesce, li grammatica del resultant lingue es plu simplic e regulari quam ti del coalescent lingues. Li nov lingua franca va esser plu simplic e regulari quam li existent Europan lingues. It va esser tam simplic quam Occidental in fact, it va esser Occidental. A un Angleso it va semblar un simplificat Angles, quam un skeptic Cambridge amico dit me que Occidental es. Li Europan lingues es membres del sam familie.</p> 
     </div> 

     <div id="container_column-2"> 
      <h1>Coming <span>soon</span>!</h1> 
      <p>We have an exciting new service to offer to our customers. We're not able to announce anything right now, but watch this space for updates in the very near future!</p> 
      <p>Alternatively, you can enter your e-mail address below to be informed of any new services and offers we have. Don't worry, we won't sell on your e-mail address and you won't receive any junk mail!</p> 

      <form method="post" action="<?php echo $_SERVER['PHP_SELF']; ?>"> 
       <label>Be the first to hear about new services!</label> 
       <input type="text" size="35" name="mailinglist" id="mailinglist" title="Enter your e-mail address" /> 
       <button type="submit" name="submit">&raquo;</button> 
      </form> 
      <?php if(isset($_POST['submit'])) { echo $msg; } ?> 
     </div> 
     <?php 
      } else { 
       include($_GET['id'] . ".php"); 
      } 
     ?> 
    </div> 

    <?php include('includes/footer.php'); ?> 
<?php include('includes/html_footer.php'); ?> 
+0

Проверьте, что страница отправляет на сервер. Ключ POST «email» явно пуст. –

+0

Неясно, на каком шаге в инструкции if ловят. В целях тестирования измените сообщение об ошибке одного из двух и посмотрите, что ему не нравится. – OpenDataAlex

ответ

1

Это довольно легко увидеть. Ваша ошибка здесь:

<input type="text" size="35" name="mailinglist" id="mailinglist" title="Enter your e-mail address" /> 

это должно быть так:

<input type="text" size="35" name="email" id="mailinglist" title="Enter your e-mail address" /> 
+0

Черт, почему я это не замечаю. Я должен пробовать все остальное, кроме этого! Благодаря! – Chrish

+0

У меня были некоторые аналогичные проблемы с построением моих форм и всегда с заполненными формами вводами из моего db и из сообщений, но через некоторое время вы привыкнете к нему, и, возможно, вы даже решите использовать codeigniter, который имеет определенную функциональность специально создавать формы – ITroubs

2

, если вы включите ошибки repoting

<?php 
error_reporting(E_ALL); 
ini_set('display_errors','on'); 

, то вы бы в конечном итоге с сообщением об ошибке:

Notice: Undefined index: email in %script% on line %line% 

и то потому что у вас нет электронной почты в вашей форме. вместо этого вы назвали его mailinglist

поэтому измените имя поля на адрес электронной почты или измените свой код, чтобы задать поле mailinglist.

1

изменение trim($_POST['email']) к trim($_POST['mailinglist']).

Смежные вопросы