Can't get ereg() to work... Help!

Anon

Well, I've run into a bit of a problem... I'm putting together my first PHP/MySQL combo page. I've got a form to input URLs, which in turn, are inserted into a database, and displayed on my page. I'm trying to check the input from the form to make sure "http://" was typed, and some other stuff. Here's where the bug seems to be hiding:

if (ereg(".", $url_new) == 1 && ereg(".", $url_description_new) == 1) {
// Check url_new for the "http://" part.
if (ereg("\http://", $url_new) == 1 && ereg("^\http://", $url_new) == 0) {
header ("Location: list_edit.php?do=add&bad=no_http");
}
}

This should be checking if something is in both $url_new and $url_description_new, then checking for http://. If http:// isn't there, it should redirect to an error message on the form. Unfortunately, even if there is no http:// in the address, it is still put into the database.. Any ideas?

Anon

Not sure this is the entire problem, but you might start by reducing the redundant checks. I.e. if you want to ensure that $url_new begins with 'http://', then just check that first:

if (! ereg( '^http://', $url_new))
   { // bad_http
   Location(...
   }
elseif ( $url_description_new == '' )
   { // missing description
   Location(...
   }

Now that you're down to a single condition at a time, it should be easier to troubleshoot each one individually.

Anon

Oh-and put something distinctive in the 'bad=' so you can tell which condition is being rejected.

Anon

Unfortunately, that's the problem. It's not redirecting at all. The form also has some other fields -- they are required, but $url_new and $url_description_new aren't.. I'll fool around with all the 'if's and see if I can get it to work. 🙂

Kirk Parker wrote:

Oh-and put something distinctive in the 'bad=' so you can tell which condition is being rejected.

Anon

Figured it out! I forgot the all important exit(); after the header (" Location: Error page"). The ereg()'s were working all along. 🙂