From fd88068f1920fe98cc72fcee564c363d31ab21a5 Mon Sep 17 00:00:00 2001 From: Andreas Date: Thu, 3 Oct 2019 13:38:38 +0200 Subject: [PATCH] small fixes to the code --- dev_mvc/controller/db/DBUser.php | 3 +++ dev_mvc/model/testactions/TA_CreateDB.php | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dev_mvc/controller/db/DBUser.php b/dev_mvc/controller/db/DBUser.php index 4e0d96f..6b36bf9 100644 --- a/dev_mvc/controller/db/DBUser.php +++ b/dev_mvc/controller/db/DBUser.php @@ -42,6 +42,9 @@ class DBUser extends Database //Email adres is niet in gebruik, return false return $user; } + else if($query->rowCount() == 0){ + trigger_error("Email $email not found in DB", E_USER_ERROR); + } else{ //Email is al in gebruik of komt meer dan een keer voor. Beide gevallen zijn een probleem dus return true. trigger_error("Multiple users for email $email returned by DB, value should be unique", E_USER_ERROR); diff --git a/dev_mvc/model/testactions/TA_CreateDB.php b/dev_mvc/model/testactions/TA_CreateDB.php index 6585e23..f9e10bb 100644 --- a/dev_mvc/model/testactions/TA_CreateDB.php +++ b/dev_mvc/model/testactions/TA_CreateDB.php @@ -56,8 +56,8 @@ class TA_CreateDB extends TestAction{ `email` varchar(256) NOT NULL, `password` varchar(256) NOT NULL, `reg_date` datetime NOT NULL DEFAULT CURRENT_TIMESTAMP, - `login_date` datetime NOT NULL, - `reg_ip` varchar(256) NOT NULL, + `login_date` datetime NOT NULL DEFAULT CURRENT_TIMESTAMP, + `reg_ip` varchar(256) NOT NULL DEFAULT '127.0.0.1', `permissions` int(11) NOT NULL DEFAULT '-1', `active` tinyint(1) DEFAULT '0', PRIMARY KEY (`ID`)