-
Bug
-
Resolution: Fixed
-
Minor
-
3.9.12, 3.10.9, 3.11.5, 3.11.6, 4.0
-
MOODLE_310_STABLE, MOODLE_311_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
-
MOODLE_311_STABLE
-
MDL-73948-master-v4 -
Currently, log live report polls new records with a condition timecreated > :date https://github.com/moodle/moodle/blob/0f94f2929606559cc7a07387aca1bb0c1030e8d5/report/loglive/classes/table_log.php#L318 where :date is the last poll time https://github.com/moodle/moodle/blob/0f94f2929606559cc7a07387aca1bb0c1030e8d5/report/loglive/classes/table_log_ajax.php#L50 .
This breaks the report when using logstores that are designed to queue entries and then insert them in batches. Eg https://github.com/catalyst/moodle-logstore_standardqueued
I believe there also may be an edge case with logstore_standard when a record was inserted with a delay due to high load, so its timecreated would be less than until and therefore it wouldn't be displayed by the report.
It seems like this can be addressed by polling records by id instead of timecreated.