Registration.php - code improvement #187
Conversation
Rather than directly use $_POST and $_GET inside functions of the class, I have added some parameters to class functions. It's far more easy to know now input data needed by each functions - registerNewUser($user_name, $user_email, $user_password, $user_password_repeat, $captcha) - verifyNewUser($user_id, $user_activation_hash) - sendVerificationEmail($user_id, $user_email, $user_activation_hash) Some private variables have been removed because not use anymore. At user creation, I record inside database the current datetime and the ip address used
|
I really like your work! Thanks ;) |
Registration.php - code improvement
|
@panique I agree! |
|
@devplanete Sorry man, but this this broke the code of
|
|
This seems to be an impossible combination of table coloumn type & default value, please check this StackOverflow-question/answer: How do you set a default value for a MySQL Datetime column?. |
|
Btw another little thing:
|
|
I'd say this script should retain compatibility with MySQL 5.1. |
|
I have just reinstall my mysql server, so I use the last version of it. I will fix the issue in a few minutes... |
|
Fixed by @devplanete at #189. |
Rather than directly use $_POST and $_GET inside functions of the class, I have added some parameters to class functions.
It's far more easy to know now input data needed by each functions
Some private variables have been removed because not use anymore.
At user creation, I record inside database the current datetime and the ip address used