Prechecker results: error


Overview

(0 errors, 0 warnings)

This section shows the general problems detected when processing the code [More info]


PHP lint problems

(0 errors, 0 warnings)

This section shows php lint problems in the code detected by php -l [More info]


PHP coding style problems

(12 errors, 36 warnings)

This section shows the coding style problems detected in the code by phpcs [More info]

auth/basic/auth.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#40) Short array syntax must be used to define arrays
(#104) Short array syntax must be used to define arrays
(#170) Expected 0 spaces after "user_login"; 1 found
(#189) Short array syntax must be used to define arrays
(#229) Short array syntax must be used to define arrays
(#244) Short array syntax must be used to define arrays
(#266) Short array syntax must be used to define arrays
auth/basic/classes/form/savepassword.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#38) Short array syntax must be used to define arrays
(#41) Short array syntax must be used to define arrays
(#44) Short array syntax must be used to define arrays
auth/basic/classes/privacy/provider.php
(#19) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#24) Import statements must not begin with a leading backslash
(#47) There must not be a space before the colon in a return type declaration
(#61) There must not be a space before the colon in a return type declaration
(#72) There should be a comma after the last array item in a multi-line array.
(#102) There should be a comma after the last array item in a multi-line array.
(#111) There should be a comma after the last array item in a multi-line array.
(#117) There should be a comma after the last array item in a multi-line array.
(#178) There should be a comma after the last array item in a multi-line array.
auth/basic/db/upgrade.php
(#25) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#49) Short array syntax must be used to define arrays
(#50) Short array syntax must be used to define arrays
(#50) Short array syntax must be used to define arrays
(#81) Inline comments must end in full-stops, exclamation marks, or question marks
auth/basic/lang/en/auth_basic.php
(#29) The string key "onlybasic" is not in the correct order, it should be before "send401_help"
(#31) The string key "debug" is not in the correct order, it should be before "onlybasic_help"
(#35) Unexpected comment found. Auto-fixing will not work after this comment
(#36) Unexpected comment found. Auto-fixing will not work after this comment
(#37) Unexpected comment found. Auto-fixing will not work after this comment
(#38) The string key "privacy:metadata:auth_basic_master_password" is not in the correct order, it should be before "send401_cancel"
(#42) The string key "masterpassword" is not in the correct order, it should be before "privacy:metadata:masterpassword"
(#45) The string key "auth_basic_not_enabled" is not in the correct order, it should be before "password"
(#51) The string key "masterpassword_desc" is not in the correct order, it should be before "whitelistonly"
(#53) The string key "generated_masterpassword" is not in the correct order, it should be before "menusettings"
(#56) The string key "regeneratepassword" is not in the correct order, it should be before "savepassword"
(#60) The string key "timecreated" is not in the correct order, it should be before "uses"
auth/basic/masterpassword.php
(#58) Short array syntax must be used to define arrays
(#93) Short array syntax must be used to define arrays
(#102) Short array syntax must be used to define arrays
(#116) Short array syntax must be used to define arrays
auth/basic/settings.php
(#32) Short array syntax must be used to define arrays
auth/basic/tests/auth_basic_test.php
(#29) Unit test auth_basic_test should be declared as final.
(#29) PHPUnit class "auth_basic_test" does not have any namespace. It is recommended to add it to the "auth_basic" namespace, using more levels if needed, in order to match the code being tested
(#32) The setUp() method in unit tests must always call to parent::setUp().
(#36) Test method test_login_user() is missing any coverage information, own or at class level
(#36) Test method test_login_user() is missing a return type

Javascript coding style problems

(0 errors, 0 warnings)

This section shows the coding style problems detected in the code by eslint [More info]


CSS problems

(0 errors, 0 warnings)

This section shows CSS problems detected by stylelint [More info]


PHPDocs style problems

(0 errors, 0 warnings)

This section shows the phpdocs problems detected in the code by local_moodlecheck [More info]


Update savepoints problems

(0 errors, 0 warnings)

This section shows problems detected with the handling of upgrade savepoints [More info]


Third party library modification problems

(0 errors, 0 warnings)

This section shows problems detected with the modification of third party libraries [More info]


grunt changes

(0 errors, 0 warnings)

This section shows files built by grunt and not commited [More info]


shifter problems

(0 errors, 0 warnings)

This section shows problems detected by shifter [More info]


Mustache template problems

(0 errors, 0 warnings)

This section shows problems detected in mustache templates [More info]


Gherkin .feature problems

(0 errors, 0 warnings)

This section shows problems detected in behat .feature files [More info]