Tim Hunt
Tim Hunt
Moodle coding style rules say: 1. A method MUST have a PHP doc comment ... 2. Unless it is an overridden method, in which case it SHOULD NOT have a...
On our system (in AWS) admin/tool/objectfs/missing_files.php is not practically usable. It just sits there for a bit then gives a gateway timeout. This is a pity, becuase we would like...
Both of these should be allowed (at least judging by the comment at https://github.com/moodlehq/moodle-cs/blob/main/moodle/Sniffs/Commenting/InlineCommentSniff.php#L191-L194) but I am currently getting them reported as errors in my code: ```php /** @var assign...
Add to performance part of peer review checklist. Inspired by the fact that https://moodle.atlassian.net/browse/MDL-80815 got integrated with seemingly no thought about performance.
In https://tracker.moodle.org/browse/MDL-65818, Sam created a new admin_setting_encryptedpassword, which can store settings encrypted in the database. Given what this plugin is doing, there are some of its settings which are secrets...
In https://tracker.moodle.org/browse/MDL-65818, Sam created a new admin_setting_encryptedpassword, which can store settings encrypted in the database. Given what this plugin is doing, there are some of its settings which are secrets...