[HTML] $_POST error

Status
Not open for further replies.
brsy

brsy

nah mang
May 12, 2011
1,530
272
Okay, I know using POST is very exploitable, but I created a PHP function to filter them out. I just need some help with a quick error. Here's my page that contains my form:

Index.php
PHP: 
<html>
    <head>
        <link rel="stylesheet" type="text/css" href="css/global.css" />
        <title>Administrative Panel: Login</title>
    </head>
    <body>
        <div class="loginContainer">
            <div class="loginBox">
                <div class="loginHeader">
                    <div class="loginDivider">
                                <form action="submit.php" method="post">
                                    <input type="text" class="loginInput" placeholder="Username..." name="username" />
                                    <input type="password" class="loginInput" placeholder="************" name="password" />
                                    <input type="submit" class="loginSubmit" value="" name="submit" />
                                </form>
 
                    </div>
                </div>
            </div>
        </div>
    </body>
</html>

Submit.php
PHP: 
<?php
//include('../class/class.configuration.php');
require_once('/class/class.module.php');
    $username = $_POST['username'];
    $password = $_POST['password'];
 
    echo "$password . $username";
?>
 
Sort by date Sort by votes
Kryptos

Kryptos

prjRev.com
Jul 21, 2010
2,205
1,252
PHP: 
<?php
//include('../class/class.configuration.php');
require_once('/class/class.module.php');
    $username = $_POST['username'];
    $password = $_POST['password'];
 
if(isset($_POST['submit'])) {
    if(isset($username) && isset($password)) {
    echo "$password . $username";
  }else { echo 'Please fill in all fields'; } 
}
?>

That should do what you want.
 
Upvote 0 Downvote
brsy

brsy

nah mang
May 12, 2011
1,530
272
Hmm, that works quite nicely... So if I wanted to compare the password with the one in the database, I would change the "echo blah blah blah" to SELECT `password` FROM `users` WHERE `password` = "$password"

Afterwards, I would do another if statement for the error handling.
Am I correct?
 
Upvote 0 Downvote
Kryptos

Kryptos

prjRev.com
Jul 21, 2010
2,205
1,252
Twizt said:
Hmm, that works quite nicely... So if I wanted to compare the password with the one in the database, I would change the "echo blah blah blah" to SELECT `password` FROM `users` WHERE `password` = "$password"

Afterwards, I would do another if statement for the error handling.
Am I correct?
Click to expand...

You'd want to so a MySQL query, yes. Just not that query.
 
Upvote 0 Downvote
brsy

brsy

nah mang
May 12, 2011
1,530
272
Is this the proper query?

PHP: 
$select = SELECT `password` FROM `users` WHERE `username` = "$username"
$sellectArray = mysql_fecth_array($data)
if($selectArray != $password) {
echo "y u no put right password?!";
}
Afterwards, I would then use
 
Upvote 0 Downvote
Kryptos

Kryptos

prjRev.com
Jul 21, 2010
2,205
1,252
Twizt said:
Is this the proper query?

PHP: 
$select = "SELECT `password` FROM `users` WHERE `username` = '$username' LIMIT 1"
$selectArray = mysql_fetch_array($data)
if($selectArray['password'] != $password) {
echo "y u no put right password?!";
}
else{
echo 'yay';
}
Afterwards, I would then use
Click to expand...

The query is a string, you have to put it between quotes.
 
Upvote 0 Downvote
brsy

brsy

nah mang
May 12, 2011
1,530
272
Is there a specific website you learned PHP from? I might want to check it out to further develop my knowledge
 
Upvote 0 Downvote
Markshall

Markshall

Русский Стандарт
Contributor
Dec 18, 2010
2,639
2,397
You must be registered for see links

You must be registered for see links


I never learned from these because I already knew a lot in PHP, but these two channels are excellent. From PHPAcademy, Alex is my favourite guy. He explains everything in such detail.
 
Upvote 0 Downvote
Status
Not open for further replies.

Users who are viewing this thread

Total: 2 (members: 0, guests: 2)
☀️  Switch to Light Theme

Latest posts

Top