Php age check form - php

Question

I'm giving my first steps on php and I got this project where, although I understand what is being done, for some reason it does not work.
$app->post('/age/', function($lang) use($app) {
$action = $app->request()->post('action');
$remember = $app->request->post('remember') === 'on';
if ($action === 'Yes') {
setcookie('AgeCheck', 'true', time()+60*60*24*30, '/'); //Expire in 1 month
$_SESSION['age'] = 'true';
$app->redirect(urldecode($app->request()->get('return')) ?: "/$lang");
}
else if ($action === 'Enter') {
$limit = (60 * 60 * 24 * 365) * 18;
$dob = $app->request->post('dob');
if ($lang === 'us') {
$dob = \DateTime::createFormFormat('mdy', $dob);
$limit = (60 * 60 * 24 * 365) * 21;
}
else
{
$dob = \DateTime::createFromFormat('dmy', $dob);
}
if ((time() - $dob->getTimestamp()) >= $limit)
{
setcookie('AgeCheck', 1, (60*60*24*365)/12); //Expire in 1 month
$_SESSION['age'] = 1;
$app->redirect(urldecode($app->request()->get('return')) ?: "/$lang");
}
}
$app->redirect("/$lang/underage");
This should, in theory,redirect the user to the proper page if he passes the age restriction, but it always redirects him to the underage page.
The issue is, most definitely, in here: if ((time() - $dob->getTimestamp()) >= $limit)
What am I doing wrong?

Answer 1

Always check every line of your code.
Your key condition is:
if ((time() - $dob->getTimestamp()) >= $limit)
{
setcookie('DrambuieAgeCheck', 1, (60*60*24*365)/12); //Expire in 1 month
$_SESSION['age'] = 1;
$app->redirect(urldecode($app->request()->get('return')) ?: "/$lang");
}
So before that, just echo/var_dump $dob->getTimestamp(), and $limit. Everything will be clear for you.
If something wrong with $dob, check what is returned by $_POST['dob'].
Example:
var_dump($dob->getTimestamp());
var_dump($app->request->post('dob'));
var_dump($dob->getTimestamp());
var_dump(time());
echo 'condition result:';
var_dump(((time() - $dob->getTimestamp()));
if ((time() - $dob->getTimestamp()) >= $limit)
You will see most important variables in condition, and condition it self

Answer 2

Figured the issue. Everything was correct, but it wasn't even entering the if, since the $action was not enter, but submit. I'l still keep #Daimos answer as correct.
The more interesting part here is that ((time() - $dob->getTimestamp()) >= $limit) is actually true(in this context). So maybe the $action is not Enter? – Andrew
This was the key to solve it, after I've tried everything and checked that it was all correct.