Validating input and returning errors with PHP and XML

Question

I have a form with several input fields, i would like to go through and check to make sure they are all present before continuing.

This is the code i am using

if(isset($_POST['url'])){ $url = $_POST['url']; } else { echo "<error>no url</error></data></xml>"; exit(); }
if(isset($_POST['username'])){ $username = $_POST['username']; } else { echo "<error>no username</error></data></xml>"; exit(); }
if(isset($_POST['password'])){ $password = $_POST['password']; } else { echo "<error>no password</error></data></xml>"; exit(); }
if(isset($_POST['cachename'])){ $cachename = $_POST['cachename']; } else { echo "<error>no cachename</error></data></xml>"; exit(); }
if(isset($_POST['lat'])){ $lat = $_POST['lat']; } else { echo "<error>no lat</error></data></xml>"; exit(); }
if(isset($_POST['long'])){ $long = $_POST['long']; } else { echo "<error>no long</error></data></xml>"; exit(); }
if(isset($_POST['message'])){ $message = $_POST['message']; } else { echo "<error>no message</error></data></xml>"; exit(); }
if(isset($_POST['notes'])){ $notes = $_POST['notes']; } else { echo "<error>no notes</error></data></xml>"; exit(); }
if(isset($_POST['tags'])){ $tags = $_POST['tags']; } else { echo "<error>no tags</error></data></xml>"; exit(); }

The problem im getting, is even when i JUST enter a URL, it returns "no lat". Even when i fill in everything down to notes, it still returns "no lat"

Any ideas?

Ólafur Waage · Accepted Answer · 2009-09-20 17:47:28Z

1

Check values in $_POST

echo "<pre>";
print_r($_POST);
echo "</pre>";

Make sure every post variable is set and the names match.

answered Sep 20, 2009 at 17:47

Ólafur Waage

70.2k22 gold badges147 silver badges199 bronze badges

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

Comments

brianreavis · Accepted Answer · 2009-09-20 18:00:19Z

0

Not that this will fix your problem (see Ólafur's comment), but, here's a more automated way of performing validation on all of your fields:

$required = array('url', 'username', 'password', 'cachename', 
                  'lat', 'long', 'message', 'notes', 'tags');

while (list($i, $require)=each($required)){
    if(empty($_POST[$require])){
        die('<error>no ' . $require . '</error></data></xml>');
    }else{
        $$require = $_POST[$require];
    }
}

PS: empty() is often better better to use than isset(). An empty string will return true with the isset() function.

answered Sep 20, 2009 at 18:00

brianreavis

11.5k3 gold badges45 silver badges50 bronze badges

Collectives™ on Stack Overflow

Validating input and returning errors with PHP and XML

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related