undefined index $_POST, but is defined

Question

I have a $_POST index defined but get the error-message that it's not defined.

Here's the code:

 if (isset($_POST['submit']) AND ($_POST['userid']) AND ($_POST['firstname']) AND ($_POST['lastname']) AND ($_POST['address']) AND ($_POST['ZIP']) AND ($_POST['phonenumber']) AND ($_POST['mail']) AND ($_POST['group']))
       {
        $result = $this->sendEmail(
           $_POST['userid'], 'mail@mymail.com', 'mail@mymail.com', $_POST['firstname'], $_POST['lastname'], $_POST['address'], $_POST['ZIP'], $_POST['phonenumber'], $_POST['mail'], $_POST['group']
        );

Please be specific! , which of the above parameter is undefined, try by printing $_POST array and check whether all elements you have given in the if are coming or not. — sandip, Jan 02 '13 at 08:55
the `$userid` is undefined. The rest works. I applied also isset on each `$_POST` but then the function doesn't send the email and returns no error. — Krystian, Jan 02 '13 at 09:07
Do you have a value inside **each** key of the `$_POST` variable? — j0k, Jan 02 '13 at 09:33

score 1 · Answer 1 · answered Jan 02 '13 at 08:56

Apply isset() on each $_POST inside your if:

if (isset($_POST['submit']) AND 
    isset($_POST['userid']) AND
    isset($_POST['firstname']) AND
    isset($_POST['lastname']) AND
    isset($_POST['address']) AND
    isset($_POST['ZIP']) AND
    isset($_POST['phonenumber']) AND
    isset($_POST['mail']) AND
    isset($_POST['group']))
{
  $result = $this->sendEmail(
    $_POST['userid'],
    'mail@mymail.com',
    'mail@mymail.com',
    $_POST['firstname'],
    $_POST['lastname'],
    $_POST['address'],
    $_POST['ZIP'],
    $_POST['phonenumber'],
    $_POST['mail'],
    $_POST['group']
);

Kaspar Mary · Answer 2 · 2013-01-04T05:42:54.007

1

You can use like this also.

if (isset($_POST['submit']) && isset($_POST['userid']) && isset($_POST['firstname']) && isset($_POST['lastname']) && isset($_POST['address']) && isset($_POST['ZIP']) && isset($_POST['phonenumber']) && isset($_POST['mail']) && isset($_POST['group']))

           {
               $result = $this->sendEmail(
               $_POST['userid'], 'mail@mymail.com', 'mail@mymail.com', $_POST['firstname'], $_POST['lastname'], $_POST['address'], $_POST['ZIP'], $_POST['phonenumber'], $_POST['mail'], $_POST['group']
            );
          }

edited Jan 04 '13 at 05:42

answered Jan 02 '13 at 08:59

Kaspar Mary

126
1
11

If I manipulate the form except the button, I will pass this script and occur a lot of bugs. You need to check all the posts – Ron van der Heijden Jan 02 '13 at 09:54

score 0 · Answer 3 · answered Jan 02 '13 at 08:56

You can write like this-

 if (isset($_POST['submit']) AND 
        isset($_POST['userid']) AND $_POST['userid'] != "" AND 
        isset($_POST['firstname']) AND $_POST['firstname'] != "" AND 
        isset($_POST['lastname']) AND $_POST['lastname']!= "" AND 
        isset($_POST['lastname']) AND $_POST['lastname']!= "" AND 
        isset($_POST['address']) AND $_POST['address'] != "" AND 
        isset($_POST['ZIP']) AND $_POST['ZIP'] != "" AND 
        isset($_POST['phonenumber']) AND $_POST['phonenumber'] != "" AND 
        isset($_POST['mail']) AND $_POST['mail'] != "" AND 
        isset($_POST['group']) AND $_POST['group'] != "")
       {
        $result = $this->sendEmail(
           $_POST['userid'], 'mail@mymail.com', 'mail@mymail.com', $_POST['firstname'], $_POST['lastname'], $_POST['address'], $_POST['ZIP'], $_POST['phonenumber'], $_POST['mail'], $_POST['group']
        );

Hi Suresh, I tried your suggestion, but also here without success :( — Krystian, Jan 02 '13 at 09:17

score 0 · Answer 4 · answered Jan 02 '13 at 09:59

First off, I recommend checking on the html itself, I assume that the method is set to POST since the rest are working, but double check to be safe! Then, another big mistake is that you screwed up when typing the name field on an input. check all names on the html to be sure they're spelled correctly. Also, make sure you have all your closing tags and opening tags, as well as opening/closing quotes.

Also don't forget to add the isset() to all the things above. Also you could create a better function like,

function checkSet($var){
       if(isset($var) && $var != ""){
            return $var;
       }else{
            return false;
       }
}

this will check if it's and empty string, and if it's defined. Very useful. You can also use it to trim by doing return trim($var) if you fancy.

undefined index $_POST, but is defined

4 Answers4