Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 1.8.4, 1.9
-
Component/s: General
-
Labels:None
-
Environment:moodle with registry cache activated.
-
Database:Any
-
Affected Branches:MOODLE_18_STABLE, MOODLE_19_STABLE
-
Fixed Branches:MOODLE_18_STABLE, MOODLE_19_STABLE
Description
Current implementation of add_to_log() updates user lastaccess field by issuing the next instruction:
$res = $db->Execute('UPDATE '. $CFG->prefix .'user
SET lastip=\''. $REMOTE_ADDR .'\', lastaccess=\''. $timenow .'\'
WHERE id = \''. $userid .'\' AND '.$timenow.' - lastaccess > 60');
This method doesn't take care of registry caché. update_record() should be used to update that fields.
One of the actual problems has to do with messaging subsystem and email notifications being sent because of inconsistency of rcache.
Thanks in advance 
Activity
- All
- Comments
- History
- Activity
- Source
- Test Sessions
Hide
Eloy Lafuente (stronk7)
added a comment -
Because that table isn't ever accessed by "id", so I think it isn't ever rcached. 
Show
Eloy Lafuente (stronk7)
added a comment - Because that table isn't ever accessed by "id", so I think it isn't ever rcached.
Hi Juan,
well spotted! B-)
I've added some code to reset the corresponding rcache record. Should be ok now... applied to 18_STABLE, 19_STABLE and HEAD.
Thanks for the report and ciao