Add some checks before adding the user in the base

This commit is contained in:
Némunaire 2012-06-19 19:35:40 +02:00
parent 36a57c9bc0
commit a74e9778d5
2 changed files with 14 additions and 5 deletions

View File

@ -10,8 +10,16 @@ if (count($_POST))
if (empty($email) || empty($login))
die("Veuillez remplir tous les champs avant de valider le formulaire.");
if (!preg_match("#^.+@[a-zA-Z0-9\._-]+\.[a-zA-Z]{2,5}$#", $email))
die("Votre adresse électronique ne semble pas valide.");
$us = User::getUsers();
$user = User::new_User($email, $login);
if (array_key_exists($user->getId(), $us))
die("Un utilisateur avec la même adresse électronique existe déjà.");
function mail_utf8($to, $subject = '(No subject)',
$message = '', $header = '')
{
@ -22,7 +30,6 @@ if (count($_POST))
}
$email = $_POST['email'];
echo $email;
$subject = "[Nemubot] Confirmation d'inscription";
$headers = "From: Nemubot <bot@nemunai.re>\n";
$message = "Bonjour,\n"
@ -32,7 +39,7 @@ if (count($_POST))
."pour confirmer :\n"
. "http://".$_SERVER["SERVER_NAME"]
. dirname($_SERVER["REQUEST_URI"]) . "/subscribeConfirmation.php?id="
. dirname($_SERVER["REQUEST_URI"]) . "/subscriptionConfirmation.php?id="
. $user->getId()
."\n\n Si ce n'est pas le cas, merci de supprimer cet email\n"
@ -41,6 +48,8 @@ if (count($_POST))
if (mail_utf8($email, $subject, $message, $headers))
{
$us[] = $user;
User::setUsers($us);
echo "Un courrier électronique vient de vous être envoyé";
}
else

View File

@ -4,14 +4,14 @@ include("User.class.php");
$users = User::getUsers();
if (in_array($_GET['id'], $users))
if (!array_key_exists($_GET['id'], $users))
{
echo "Cet utilisateur n'existe pas";
echo "Cet utilisateur n'existe pas !";
}
else
{
$users[$_GET['id']]->set_validated(true);
echo "Utilisateur confirmé !"
echo "Utilisateur confirmé !";
}
?>