If condition doesn't allow to leave empty fields

Question

I have the contact form, where you can add multiple recipients, but when i uploaded it on a server, there was these errors:

Notice: Undefined index: recipient in /var/www/vhosts/partizanas.lt/httpdocs/brief/contactform.php on line 17 Notice: Undefined index: recipient0 in /var/www/vhosts/partizanas.lt/httpdocs/brief/contactform.php on line 18 Notice: Undefined index: recipient1 in /var/www/vhosts/partizanas.lt/httpdocs/brief/contactform.php on line 19 Notice: Undefined index: recipient2 in /var/www/vhosts/partizanas.lt/httpdocs/brief/contactform.php on line 20 Notice: Undefined index: recipient3 in /var/www/vhosts/partizanas.lt/httpdocs/brief/contactform.php on line 21 Notice: Undefined index: email in /var/www/vhosts/partizanas.lt/httpdocs/brief/contactform.php on line 22"

i wrote the if condition which fixes the errors:

if (isset($_POST['Submit']))    
{    

$formproc->AddRecipient($_POST["recipient"]); 
$formproc->AddRecipient($_POST["recipient0"]); 
$formproc->AddRecipient($_POST["recipient1"]); 
$formproc->AddRecipient($_POST["recipient2"]); 
$formproc->AddRecipient($_POST["recipient3"]);  
$formproc->AddRecipient($_POST["email"]); 
}

but the problem is, the form isnt sent if there are empty input fields. And i want it to send the form when not all the fields are filled.

Any advice how to do it?

Test the keys. $value = isset($_POST["recipient"]) ? $_POST["recipient"] : ''; (basic example) — Alex Tartan
– Alex Tartan, Commented Jun 10, 2015 at 7:50
is this a copy paste? it seems you don't have input elements in your form named as recipient, recipient0... and so on — Drixson Oseña
– Drixson Oseña, Commented Jun 10, 2015 at 7:50
try if (!empty($_POST['recepient'])) $formproc->AddRecipient($_POST["recipient"]); — chanafdo
– chanafdo, Commented Jun 10, 2015 at 8:23

Community · Accepted Answer · 2017-05-23 12:22:21Z

1

Check this answer

Relying on the default value of an uninitialized variable is problematic in the case of including one file into another which uses the same variable name. It is also a major security risk with register_globals turned on. E_NOTICE level error is issued in case of working with uninitialized variables, however not in the case of appending elements to the uninitialized array. isset() language construct can be used to detect if a variable has been already initialized.

edited May 23, 2017 at 12:22

CommunityBot

11 silver badge

answered Jun 10, 2015 at 7:51

Babu Siva M

457 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

SuperDJ · Accepted Answer · 2015-06-10 08:18:12Z

1

if (isset($_POST['Submit']))    
{    
if(isset($_POST["recipient"]) && $_POST["recipient"] !="" ){$formproc->AddRecipient($_POST["recipient"]);}
if(isset($_POST["recipient1"]) && $_POST["recipient"] !="" ){$formproc->AddRecipient($_POST["recipient1"]);}
if(isset($_POST["recipient2"]) && $_POST["recipient"] !="" ){$formproc->AddRecipient($_POST["recipient2"]);}
if(isset($_POST["recipient3"]) && $_POST["recipient"] !="" ){$formproc->AddRecipient($_POST["recipient3"]);}
$formproc->AddRecipient($_POST["email"]); 
}

edited Jun 10, 2015 at 8:18

SuperDJ

7,69911 gold badges44 silver badges80 bronze badges

answered Jun 10, 2015 at 8:04

Sajin Das

211 bronze badge

Comments

Gene · Accepted Answer · 2015-06-10 09:21:10Z

Consider the following code to solve your problem:

if (isset($_POST['Submit']))    
{    

    if(!empty($_POST["recipient"])) {
        $formproc->AddRecipient($_POST["recipient"]); 
    }

    if(!empty($_POST["recipient0"])) {
        $formproc->AddRecipient($_POST["recipient0"]); 
    }

    if(!empty($_POST["recipient1"])) {
        $formproc->AddRecipient($_POST["recipient1"]); 
    }

    if(!empty($_POST["recipient2"])) {
        $formproc->AddRecipient($_POST["recipient2"]); 
    }

    if(!empty($_POST["recipient3"])) {
        $formproc->AddRecipient($_POST["recipient3"]);  
    }

    if(!empty($_POST["email"])) {
        $formproc->AddRecipient($_POST["email"]); 
    }
}

The idea here is to not execute the function AddRecipient() IF the input field is empty. !empty() function does that job for you.

Collectives™ on Stack Overflow

If condition doesn't allow to leave empty fields

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related