]> git.mjollnir.org Git - moodle.git/commit
Race condition in cron for Scheduled-Capture (very very big sites and order review...
authorethem <ethem>
Mon, 10 Mar 2008 11:05:46 +0000 (11:05 +0000)
committerethem <ethem>
Mon, 10 Mar 2008 11:05:46 +0000 (11:05 +0000)
commit25a215b9cb88265716075da1e0d836bf3d97404e
tree234222574fcce846e36e2b218e1bf985c16da1dc
parentd650e1a3158d7ae473341e9f54824f673175aae4
Race condition in cron for Scheduled-Capture (very very big sites and order review enabled only).

In one cron, 250-500 orders may be processed (based on 5 minutes).
If an admin sets cron time up smaller than 5 minutes and
250-500> new transactions are made after last cron executed, it can be blocked.
Authorize cron sets up an_lastcron every time when admin/cron.php executes.
This must be set up after blocking check code.

As result, if pending orders aren't accepted with in 30 days at payment management page, they expired and users cannot enrol.
When an admin enabled order review, he is guaranteed 'Payment managers accept/deny transactions manually'.
Scheduled-Capture is designed for forgotten orders only ;)
enrol/authorize/enrol.php