Prechecker results: error


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

(0 errors, 14 warnings)

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

mod/checklist/tests/dates_test.php
(#47) Test method test_import() is missing any coverage information, own or at class level
mod/checklist/tests/group_completion_email_test.php
(#137) Test method test_group_completion_emails() is missing any coverage information, own or at class level
mod/checklist/tests/privacy_provider_test.php
(#121) Test method test_get_metadata() is missing any coverage information, own or at class level
(#183) Test method test_get_contexts_for_userid() is missing any coverage information, own or at class level
(#212) Test method test_export_for_context() is missing any coverage information, own or at class level
(#247) Test method test_delete_data_for_all_users_in_context() is missing any coverage information, own or at class level
(#308) Test method test_delete_data_for_user() is missing any coverage information, own or at class level
(#432) Test method test_get_users_in_context() is missing any coverage information, own or at class level
(#455) Test method test_delete_data_for_users() is missing any coverage information, own or at class level
mod/checklist/tests/student_comment_test.php
(#110) Test method test_create_student_comment() is missing any coverage information, own or at class level
(#120) Test method test_update_student_comment_new() is missing any coverage information, own or at class level
(#125) Test method test_external_function_create() is missing any coverage information, own or at class level
(#166) Test method test_external_function_update() is missing any coverage information, own or at class level
(#201) Test method test_can_add_student_comments() is missing any coverage information, own or at class level

Javascript coding style problems

(1 errors, 69 warnings)

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

mod/checklist/updatechecks24.js
(#1) Expected space or tab after '/*' in comment. (spaced-comment)
(#1) 'M' is already defined as a built-in global variable. (no-redeclare)
(#2) Expected space or tab after '/*' in comment. (spaced-comment)
(#2) 'window' is already defined as a built-in global variable. (no-redeclare)
(#3) Identifier 'mod_checklist' is not in camel case. (camelcase)
(#4) Unexpected space before function parentheses. (space-before-function-paren)
(#6) Unexpected space before function parentheses. (space-before-function-paren)
(#7) Expected variable declaration to be on a new line. (one-var-declaration-per-line)
(#23) Unexpected space before function parentheses. (space-before-function-paren)
(#30) Expected variable declaration to be on a new line. (one-var-declaration-per-line)
(#36) Operator '<' must be spaced. (space-infix-ops)
(#51) Unexpected space before function parentheses. (space-before-function-paren)
(#56) Identifier 'check_click' is not in camel case. (camelcase)
(#56) Unexpected space before function parentheses. (space-before-function-paren)
(#57) Expected variable declaration to be on a new line. (one-var-declaration-per-line)
(#78) Unexpected space before function parentheses. (space-before-function-paren)
(#94) Identifier 'update_progress_bar' is not in camel case. (camelcase)
(#94) Unexpected space before function parentheses. (space-before-function-paren)
(#107) Operator '>' must be spaced. (space-infix-ops)
(#110) Operator '<' must be spaced. (space-infix-ops)
(#123) Operator '>' must be spaced. (space-infix-ops)
(#126) Operator '<' must be spaced. (space-infix-ops)
(#138) Identifier 'update_server' is not in camel case. (camelcase)
(#138) Unexpected space before function parentheses. (space-before-function-paren)
(#139) Expected variable declaration to be on a new line. (one-var-declaration-per-line)
(#140) Operator '<' must be spaced. (space-infix-ops)
(#154) Unexpected space before function parentheses. (space-before-function-paren)
(#160) Identifier 'send_update_batch' is not in camel case. (camelcase)
(#160) Unexpected space before function parentheses. (space-before-function-paren)
(#161) Expected variable declaration to be on a new line. (one-var-declaration-per-line)
(#161) Expected variable declaration to be on a new line. (one-var-declaration-per-line)
(#173) Operator '<' must be spaced. (space-infix-ops)
(#187) Unexpected space before function parentheses. (space-before-function-paren)
(#190) Unexpected alert. (no-alert)
(#193) Unexpected space before function parentheses. (space-before-function-paren)
(#195) Unexpected alert. (no-alert)
(#209) Identifier 'show_spinner' is not in camel case. (camelcase)
(#209) Unexpected space before function parentheses. (space-before-function-paren)
(#213) Identifier 'hide_spinner' is not in camel case. (camelcase)
(#213) Unexpected space before function parentheses. (space-before-function-paren)
mod/checklist/yui/buttons/buttons.js
(#1) Expected space or tab after '/*' in comment. (spaced-comment)
(#1) 'M' is already defined as a built-in global variable. (no-redeclare)
(#2) 'YUI' is not defined. (no-undef)
(#2) Unexpected space before function parentheses. (space-before-function-paren)
(#4) Identifier 'toggle_values' is not in camel case. (camelcase)
(#4) Identifier 'col_clicked' is not in camel case. (camelcase)
(#4) Identifier 'row_clicked' is not in camel case. (camelcase)
(#6) Identifier 'toggle_values' is not in camel case. (camelcase)
(#8) Unexpected space before function parentheses. (space-before-function-paren)
(#14) Identifier 'toggle_values' is not in camel case. (camelcase)
(#17) Identifier 'toggle_values' is not in camel case. (camelcase)
(#20) Identifier 'col_clicked' is not in camel case. (camelcase)
(#20) Unexpected space before function parentheses. (space-before-function-paren)
(#22) Comments should not begin with a lowercase character. (capitalized-comments)
(#24) Comments should not begin with a lowercase character. (capitalized-comments)
(#27) Comments should not begin with a lowercase character. (capitalized-comments)
(#28) Unexpected space before function parentheses. (space-before-function-paren)
(#35) Comments should not begin with a lowercase character. (capitalized-comments)
(#36) Unexpected space before function parentheses. (space-before-function-paren)
(#44) Identifier 'row_clicked' is not in camel case. (camelcase)
(#44) Unexpected space before function parentheses. (space-before-function-paren)
(#46) Comments should not begin with a lowercase character. (capitalized-comments)
(#48) Comments should not begin with a lowercase character. (capitalized-comments)
(#51) Comments should not begin with a lowercase character. (capitalized-comments)
(#52) Unexpected space before function parentheses. (space-before-function-paren)
(#59) Comments should not begin with a lowercase character. (capitalized-comments)
(#60) Unexpected space before function parentheses. (space-before-function-paren)
(#68) Comments should not begin with a lowercase character. (capitalized-comments)
(#70) Identifier 'mod_checklist' is not in camel case. (camelcase)
(#72) Unexpected space before function parentheses. (space-before-function-paren)

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

(1 errors, 0 warnings)

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

mod/checklist/db/upgrade.php
'return true;' not found

Third party library modification problems

(0 errors, 0 warnings)

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


Missing changes in external functions or backup support for new detected tables or columns

(0 errors, 0 warnings)

This section shows potential problems detected when there are new database structures added in a patch and it is detected that nothing has been changed related with external functions (to be used by WS) or backup and restore. It must be checked that everything is correct and nothing is being missed in those areas. [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]