-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Utrs 99 #4
Utrs 99 #4
Conversation
…e returning to this point, so committing anyway.
actualy, that's UTRS 101 |
$firstuserid = "null"; | ||
$seconduserid = null; | ||
} else { | ||
$firstuserid = User::getUserByUsername($username)->getUserId(); | ||
$seconduserid = User::getUserByUsername($username)->getUserId(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will result in a second SQL query. $seconduserid = $firstuserid;
should suffice, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, it should. I considered that, but I went for what was closest to the original. your suggestion was my first thought too, and IMO a lot cleaner
public function sendEmail($bodytemplate, $subject, $admin = false){ | ||
$success = false; | ||
try { | ||
if (!$admin){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be if ($admin === false)
closed again until I have it merged against the current master, because of the PDO transition |
script to close UTRS 99.
Prerequisites:
Actions to actually close UTRS 99:
Quite some changes:
Add script (clean_requests.php) to send reminders when an appeal is AWAITING USER for 5 days or more, and to send close notification and close appeal 5 days after the reminder is sent.
Under the hood:
Because the changes are largish, good beta testing is prefered on this patch