moodleou / moodle-report_customsql

A Moodle report plugin that lets you easily create simple reports that can be expressed as a single SQL query
48 stars 100 forks source link

PHP 8.1 - strftime is deprecated #143

Closed sharpchi closed 6 months ago

sharpchi commented 1 year ago

There are a few lines where strftime is used, but got this exception in testing on PHP8.1

`Scheduled task failed: Ad-hoc database queries: run scheduled reports task (report_customsql\task\run_reports),Unknown error type: Function strftime() is deprecated in /var/www/html/report/customsql/locallib.php on line 611 Backtrace:

The date() function should be a fairly simple replacement (with format adjustments).

abias commented 6 months ago

I think this issue was solved by https://github.com/moodleou/moodle-report_customsql/commit/a97a9e55196b95e2124444aff776c074cf6f95ef and could be closed.

timhunt commented 6 months ago

Thanks Alex.