moodle-cs
moodle-cs copied to clipboard
moodlehq
Moodle Coding Style
This comes from #134, where it was detected that a few (minor) style details are not being checked currently. Here it's an initial list, to be discussed, analysed... - Blank...
Coming from #132, as far as `moodle-cs` is now an standalone product, we don't need these lines, previously required, in all the fixture files: ``` defined('MOODLE_INTERNAL') || die(); // Make...
Commented @ #91 maybe it would be nice to have a new standard, similar to the `moodle-extra` one for easier use by tools that are checking plugins and not core,...
See http://docs.moodle.org/dev/Coding_style#.40since The word Moodle should be there. If more that one version is given, they should be given as a comma-separated list in increasing order. --- Reporter: Tim Hunt...
This is a follow up of MDL-55580. The idea is to add the tooling for codechecker and alert developers of possible capabilities deprecation. * Add a simple deprecation warning for...
Can lead to code execution exploits if not used properly with user supplied data. There are better methods of data exchange. See also https://github.com/moodle/devdocs/pull/822
my `.phpcs.xml` looks like this ``` ``` and in `db/install.php` I have this code: ``` } else { $issuerdata->image = $ocislogourl; } $issuer = core\oauth2\api::create_issuer($issuerdata); $result = $issuer->is_valid(); if (!$result)...
It seems that both phpunit and php-code-coverage have started to accept nikic/php-parser v5, but that's leading to problems with PHP 7.4 runs, that we still support. So, with this commit,...