search

srobotta / moodle-report_ldapaccounts

GNU General Public License v3.0
0 stars 1 forks source link

Csv files should probably use moodle temp dir #2

Closed danmarsden closed 12 months ago

danmarsden commented 1 year ago

https://github.com/srobotta/moodle-report_ldapaccounts/blob/master/classes/user_table.php#L329

These are temp files so should probably be stored in moodles normal temp dir so that clean up and deletion of old files can happen automatically.

Not an approval blocker

danmarsden commented 1 year ago

Same with logfiles: https://github.com/srobotta/moodle-report_ldapaccounts/blob/master/classes/ldap.php#L232