Submit button not working- if else statement

Question

I want my submit button to,when clicked, run an if else statement. In the field 'complete' in my users table, if the value is 6 or above an email should send (I have this code), if not then it should display an error or redirect to an error page (which I can create). This is my attempt but the error message shows even when the value is 6

$usr_email = 'usr_email';
$user_name = 'user_name';
{
$sqltest = "SELECT complete From users where user_id =
".intval($_SESSION['user_id']);
}
if (isset($_POST['doSend'])) {

function getOne($query){
    $res = mysql_query($query);
    if (!$res) {
        trigger_error("db: ".mysql_error()." in ".$query);
        return FALSE;
    }
}
    if ($row = mysql_fetch_row($res)) {
        return $row[0];
    }

$isSending = getOne($sqltest);
$isSending === false;
if($isSending >= 6){
require_once "Mail.php";

$from = "<xxx>";
$to = "$usr_email";
$subject = "Hi";
$body = hello ";

$host = "ssl://smtp.gmail.com";
$port = "465";
$username = "xxx";
$password = "xxx";

$headers = array ('From' => $from,
    'To' => $to,
    'Subject' => $subject);
$smtp = Mail::factory('smtp',
    array ('host' => $host,
        'port' => $port,
        'auth' => true,
        'username' => $username,
        'password' => $password));

$mail = $smtp->send($to, $headers, $body);

if (PEAR::isError($mail)) {
    echo("<p>" . $mail->getMessage() . "</p>");
} else {
    header ("Location: error.php");
}
}
}

` <form action="myaccount.php" method="post" name="regForm" id="regForm" >` It has no target? — user1296762
– user1296762, Commented Apr 18, 2012 at 19:24

Jack · Accepted Answer · 2012-04-18 20:44:38Z

2

Change

<p align="center">
    <input name="doSend" type="submit" id="doSend" value="Submit Application">
</p>

to

<form name="input" action="myaccount.php" method="post">
    <input name="doSend" type="submit" id="doSend" value="Submit Application">
</form>

You need to add an else in the outer if statement.

$isSending = getOne($sqltest);
$isSending === false;
if($isSending >= 6){
    require_once "Mail.php";

    $from = "<xxx>";
    $to = "$usr_email";
    $subject = "Hi";
    $body = hello ";

    $host = "ssl://smtp.gmail.com";
    $port = "465";
    $username = "xxx";
    $password = "xxx";

    $headers = array ('From' => $from,
        'To' => $to,
        'Subject' => $subject);
    $smtp = Mail::factory('smtp',
        array ('host' => $host,
            'port' => $port,
            'auth' => true,
            'username' => $username,
            'password' => $password));

    $mail = $smtp->send($to, $headers, $body);

    if (PEAR::isError($mail)) {
        echo("<p>" . $mail->getMessage() . "</p>");
    } else {
        echo("<p>Message successfully sent!</p>");
    }
}
else
    header ("Location: error.php");

edited Apr 18, 2012 at 20:44

answered Apr 18, 2012 at 19:25

Jack

5,85812 gold badges52 silver badges76 bronze badges

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

9 Comments

user1296762 Over a year ago

Thanks now it just redirects to error page even when the value is 6 :( See edit

user1296762 Over a year ago

Sorry the error page is the page it redirects to if the number is less than 6. Its not an actual error

Jack Over a year ago

You are certain that getOne($sqltest) outputs a 6 or greater? Try removing this line $isSending === false;

user1296762 Over a year ago

Still doing the same. I will add some more of the mail code to my question see if that is affecting the else statement

Jack Over a year ago

Your if statement is evaluating correctly. If PEAR::isError($mail) returns false, that means there's no error and goes to the else. Your else shouldn't contain the header ("Location: error.php");

|

e-- · Accepted Answer · 2012-04-18 19:35:34Z

2

It looks like you don't have a form around it? How about this:

<p align="center">
<form action="<?=$_SERVER['PHP_SELF']?>" method="POST">
    <input name="doSend" type="submit" id="doSend" value="Submit Application">
</form>
</p>

@your comment: even if it is higher then 6? Like this?

$isSending = getOne(9);

edited Apr 18, 2012 at 19:35

answered Apr 18, 2012 at 19:24

e--

1981 silver badge15 bronze badges

2 Comments

user1296762 Over a year ago

Thanks now it just redirects to the error page even when the value is above 6. See edit

user1296762 Over a year ago

I get this when editing that line

db: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '9'

jeroen · Accepted Answer · 2012-04-18 20:51:24Z

0

Your function definition terminates before the second if statement:

function getOne($query){
    $res = mysql_query($query);
    if (!$res) {
        trigger_error("db: ".mysql_error()." in ".$query);
        return FALSE;
    }
}
if ($row = mysql_fetch_row($res)) {
    return $row[0];
}

should be:

function getOne($query){
    $res = mysql_query($query);
    if (!$res) {
        trigger_error("db: ".mysql_error()." in ".$query);
        return FALSE;
    }
    if ($row = mysql_fetch_row($res)) {
        return $row[0];
    }
}

As it is now, the value obtained from the sql query is never returned.

answered Apr 18, 2012 at 20:51

jeroen

91.8k22 gold badges117 silver badges132 bronze badges

Comments

Robert Louis Murphy · Accepted Answer · 2012-04-20 18:19:34Z

0

In most if not all browsers, a submit button does nothing if not in a <form/>, does your html have a <form>Tag Around your <input/></form>? If not, wrap your input tag in a form and it will work.

edited Apr 20, 2012 at 18:19

answered Apr 18, 2012 at 19:21

Robert Louis Murphy

1,6581 gold badge17 silver badges29 bronze badges

3 Comments

Madbreaks Over a year ago

This is not an answer, please put questions in a comment on the original post.

Robert Louis Murphy Over a year ago

Just 'cause this isn't Jeopardy doesn't mean that's not an answer.

Madbreaks Over a year ago

The fact that it doesn't answer the question means it's not an answer.

Collectives™ on Stack Overflow

Submit button not working- if else statement

4 Answers 4

9 Comments

2 Comments

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

9 Comments

2 Comments

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related