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

(79 errors, 72 warnings)

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

admin/tool/legacyfilesmigration/batchmigrate.php
(#39) Short array syntax must be used to define arrays
(#61) Short array syntax must be used to define arrays
(#95) Short array syntax must be used to define arrays
admin/tool/legacyfilesmigration/classes/privacy/provider.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#19) Incorrect @package tag for file provider.php. Expected tool_legacyfilesmigration, found local_zz_last_actions.
(#24) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#38) There must not be a space before the colon in a return type declaration
(#41) File must end with a newline character
admin/tool/legacyfilesmigration/cli/generate_clis.php
(#1) Missing @copyright tag
(#1) Missing @license tag
(#1) Missing docblock for file generate_clis.php
(#1) Moodle boilerplate not found
(#8) Short array syntax must be used to define arrays
(#13) There should be a comma after the last array item in a multi-line array.
(#15) Short array syntax must be used to define arrays
(#20) There should be a comma after the last array item in a multi-line array.
(#35) Expected "if (...) {\n"; found "if(...){\n"
(#39) Expected "foreach (...) {\n"; found "foreach(...){\n"
(#40) Expected 1 space before "?"; 0 found
(#40) Expected 1 space before ":"; 0 found
(#40) Expected 1 space after ":"; 0 found
(#40) Expected 1 space before "?"; 0 found
(#40) Expected 1 space after "?"; 0 found
(#40) Expected 1 space before ":"; 0 found
(#40) Expected 1 space after ":"; 0 found
(#40) Line exceeds maximum limit of 180 characters; contains 226 characters
(#41) Expected "if (...) {\n"; found "if(...){\n"
(#42) Line exceeds 132 characters; contains 175 characters
admin/tool/legacyfilesmigration/cli/migrate_legacyfiles.php
(#1) Missing @copyright tag
(#1) Missing @license tag
(#1) Missing docblock for file migrate_legacyfiles.php
(#1) Moodle boilerplate not found
(#8) Inline comments must start with a capital letter, digit or 3-dots sequence
(#8) Inline comments must end in full-stops, exclamation marks, or question marks
(#9) Short array syntax must be used to define arrays
(#15) Short array syntax must be used to define arrays
(#19) There should be a comma after the last array item in a multi-line array.
(#31) Short array syntax must be used to define arrays
(#31) Expected 1 space before "=>"; 0 found
(#31) Expected 1 space after "=>"; 0 found
(#32) Expected "if (...) {\n"; found "if(...){\n"
(#38) Expected "if (...) {\n"; found "if(...){\n"
(#39) Short array syntax must be used to define arrays
(#39) Expected 1 space before "=>"; 0 found
(#39) Expected 1 space after "=>"; 0 found
(#40) Expected "if (...) {\n"; found "if(...){\n"
(#52) Expected "if (...) {\n"; found "if(...){\n"
(#55) Expected "} else {\n"; found "}else{\n"
(#56) Expected 1 space before "?"; 0 found
(#56) Expected 1 space after "?"; 0 found
(#56) Commas (,) must be followed by white space.
(#56) Expected 1 space before ":"; 0 found
(#56) Expected 1 space after ":"; 0 found
(#58) File must end with a newline character
admin/tool/legacyfilesmigration/lang/en/tool_legacyfilesmigration.php
(#20) Incorrect @package tag for file tool_legacyfilesmigration.php. Expected tool_legacyfilesmigration, found 2014 unistra {@link http://unistra.fr}.
(#28) The string key "coursenotfound" is not in the correct order, it should be before "usernotfound"
(#30) The string key "migrateselected" is not in the correct order, it should be before "unknown"
(#35) The string key "listnotmigrated" is not in the correct order, it should be before "notsupported"
(#38) The string key "notmigratedtitle" is not in the correct order, it should be before "supported"
(#39) The string key "notmigratedintro" is not in the correct order, it should be before "notmigratedtitle"
(#41) The string key "courseid" is not in the correct order, it should be before "owner"
(#43) The string key "migratetable" is not in the correct order, it should be before "select"
(#44) The string key "backtoindex" is not in the correct order, it should be before "migratetable"
(#47) The string key "batchmigrate" is not in the correct order, it should be before "confirmbatchmigrate"
(#49) The string key "migratecoursesummary" is not in the correct order, it should be before "unchoosenowner"
(#50) The string key "migratecoursesuccess" is not in the correct order, it should be before "migratecoursesummary"
(#52) The string key "migrateprogress" is not in the correct order, it should be before "viewcourse"
(#54) The string key "migrateall" is not in the correct order, it should be before "migrationfailed"
(#57) The string key "empty_owner" is not in the correct order, it should be before "preselect_owner"
(#58) The string key "coursidnotcollapse" is not in the correct order, it should be before "empty_owner"
(#60) The string key "foldername" is not in the correct order, it should be before "toollegacyfilesmigrationsettings"
(#62) The string key "copyallcoursefiles" is not in the correct order, it should be before "foldername_desc"
(#64) Unexpected comment found. Auto-fixing will not work after this comment
(#64) Inline comments must start with a capital letter, digit or 3-dots sequence
(#64) Inline comments must end in full-stops, exclamation marks, or question marks
(#64) No space found before comment text; expected "// cli traductions" but found "//cli traductions"
(#65) The string key "cli_help" is not in the correct order, it should be before "copycoursefilesforall"
(#75) Expected 1 space before "="; 0 found
(#75) Expected 1 space after "="; 0 found
(#76) The string key "cli_coursenotfound" is not in the correct order, it should be before "cli_usernotfound"
(#76) Expected 1 space before "="; 0 found
(#76) Expected 1 space after "="; 0 found
(#77) Expected 1 space before "="; 0 found
(#77) Expected 1 space after "="; 0 found
(#78) Expected 1 space before "="; 0 found
(#78) Expected 1 space after "="; 0 found
(#79) The string key "cli2_help" is not in the correct order, it should be before "cli_success"
(#89) Expected 1 space before "="; 0 found
(#89) Expected 1 space after "="; 0 found
(#90) The string key "cli2_nolog" is not in the correct order, it should be before "cli_nocoursetomigrate"
(#91) Unexpected string end, it should be a line feed after a semicolon
(#91) File must end with a newline character
admin/tool/legacyfilesmigration/listnotmigrated.php
(#1) Missing @copyright tag
(#1) Missing @license tag
(#1) Missing docblock for file listnotmigrated.php
(#34) Short array syntax must be used to define arrays
(#45) Short array syntax must be used to define arrays
(#57) File must end with a newline character
admin/tool/legacyfilesmigration/locallib.php
(#36) Short array syntax must be used to define arrays
(#66) Short array syntax must be used to define arrays
(#76) Short array syntax must be used to define arrays
(#119) Short array syntax must be used to define arrays
(#128) Missing docblock for class tool_legacyfilesmigration_tools
(#128) Expected 1 space before opening brace; found 0
(#140) Short array syntax must be used to define arrays
(#156) Short array syntax must be used to define arrays
(#157) Short array syntax must be used to define arrays
(#165) Commas (,) must be followed by white space.
(#168) Short array syntax must be used to define arrays
(#175) Short array syntax must be used to define arrays
(#193) Short array syntax must be used to define arrays
(#196) Missing docblock for function string_format_by_object
(#198) Logical operator "or" is prohibited; use "||" instead
(#200) Short array syntax must be used to define arrays
(#201) Short array syntax must be used to define arrays
(#221) Missing docblock for class legacyfiles_migrate_manager
(#222) Missing docblock for function migrate_files
(#228) Spaces must be used to indent lines; tabs are not allowed
(#228) Tab found within whitespace
(#228) This comment is 65% valid code; is this commented out code?
(#228) Line indented incorrectly; expected at least 8 spaces, found 1
(#236) Inline comments must end in full-stops, exclamation marks, or question marks
(#248) Spaces must be used to indent lines; tabs are not allowed
(#248) Tab found within whitespace
(#248) Line indented incorrectly; expected at least 16 spaces, found 2
(#289) Short array syntax must be used to define arrays
(#297) Short array syntax must be used to define arrays
(#340) Inline comments must end in full-stops, exclamation marks, or question marks
admin/tool/legacyfilesmigration/migrateablecoursesbatchform.php
(#17) Missing @license tag
(#47) Short array syntax must be used to define arrays
(#48) Short array syntax must be used to define arrays
(#49) Short array syntax must be used to define arrays
admin/tool/legacyfilesmigration/migrateablecoursestable.php
(#17) Missing @license tag
(#42) Expected "bool" but found "boolean" for @var tag in member variable comment
(#65) Short array syntax must be used to define arrays
(#74) Short array syntax must be used to define arrays
(#76) Short array syntax must be used to define arrays
(#77) Short array syntax must be used to define arrays
(#124) Short array syntax must be used to define arrays
admin/tool/legacyfilesmigration/paginationform.php
(#44) Short array syntax must be used to define arrays
admin/tool/legacyfilesmigration/renderer.php
(#26) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#29) Missing docblock for class tool_legacyfilesmigration_renderer
(#77) Short array syntax must be used to define arrays
(#120) Short array syntax must be used to define arrays
(#126) Short array syntax must be used to define arrays
(#168) Short array syntax must be used to define arrays
(#186) Short array syntax must be used to define arrays
(#239) Short array syntax must be used to define arrays
admin/tool/legacyfilesmigration/settings.php
(#20) Incorrect @package tag for file settings.php. Expected tool_legacyfilesmigration, found 2017 unistra {@link http://unistra.fr}.
admin/tool/legacyfilesmigration/version.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#21) Incorrect @package tag for file version.php. Expected tool_legacyfilesmigration, found tool.

Javascript coding style problems

(50 errors, 33 warnings)

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

admin/tool/legacyfilesmigration/module.js
(#1) Identifier 'tool_legacyfilesmigration' is not in camel case. (camelcase)
(#2) Identifier 'init_migrate_table' is not in camel case. (camelcase)
(#5) 'checkboxes' is not defined. (no-undef)
(#6) 'checkboxes' is not defined. (no-undef)
(#8) 'rowelement' is not defined. (no-undef)
(#10) 'rowelement' is not defined. (no-undef)
(#12) 'rowelement' is not defined. (no-undef)
(#16) 'rowelement' is not defined. (no-undef)
(#18) 'rowelement' is not defined. (no-undef)
(#20) 'rowelement' is not defined. (no-undef)
(#28) 'checkboxes' is not defined. (no-undef)
(#29) 'checkboxes' is not defined. (no-undef)
(#30) 'rowelement' is not defined. (no-undef)
(#32) 'rowelement' is not defined. (no-undef)
(#35) 'checkboxes' is not defined. (no-undef)
(#36) 'checkboxes' is not defined. (no-undef)
(#37) 'rowelement' is not defined. (no-undef)
(#39) 'rowelement' is not defined. (no-undef)
(#46) 'checkboxes' is not defined. (no-undef)
(#47) 'checkboxes' is not defined. (no-undef)
(#48) 'rowelement' is not defined. (no-undef)
(#52) 'checkboxes' is not defined. (no-undef)
(#53) 'checkboxes' is not defined. (no-undef)
(#54) 'rowelement' is not defined. (no-undef)
(#59) Identifier 'preselect_owner' is not in camel case. (camelcase)
(#60) Identifier 'preselect_owner' is not in camel case. (camelcase)
(#60) Missing space before opening brace. (space-before-blocks)
(#61) Expected space(s) after "if". (keyword-spacing)
(#61) Missing space before opening brace. (space-before-blocks)
(#62) Unexpected alert. (no-alert)
(#62) 'M.str' is restricted from being used. Use "core/str" module or M.util.get_string() (no-restricted-properties)
(#69) Missing space before opening brace. (space-before-blocks)
(#73) Expected space(s) after "if". (keyword-spacing)
(#73) Missing space before opening brace. (space-before-blocks)
(#74) 'vartextfield' is not defined. (no-undef)
(#75) Expected space(s) after "if". (keyword-spacing)
(#75) 'vartextfield' is not defined. (no-undef)
(#75) Missing space before opening brace. (space-before-blocks)
(#76) 'vartextfield' is not defined. (no-undef)
(#76) A space is required after ','. (comma-spacing)
(#79) Expected space(s) before "else". (keyword-spacing)
(#79) Expected space(s) after "if". (keyword-spacing)
(#79) Missing space before opening brace. (space-before-blocks)
(#80) Missing semicolon. (semi)
(#86) 'checkboxes' is not defined. (no-undef)
(#87) 'oinputs' is not defined. (no-undef)
(#91) 'checkboxes' is not defined. (no-undef)
(#97) 'copycoursefilesentry' is not defined. (no-undef)
(#98) Expected space(s) after "if". (keyword-spacing)
(#98) 'copycoursefilesentry' is not defined. (no-undef)
(#98) 'copycoursefilesentry' is not defined. (no-undef)
(#98) Missing space before opening brace. (space-before-blocks)
(#99) Expected space(s) after "if". (keyword-spacing)
(#99) 'copycoursefilesentry' is not defined. (no-undef)
(#99) Missing space before opening brace. (space-before-blocks)
(#104) 'owners' is not defined. (no-undef)
(#105) Expected space(s) after "if". (keyword-spacing)
(#105) 'owners' is not defined. (no-undef)
(#105) 'owners' is not defined. (no-undef)
(#105) Missing space before opening brace. (space-before-blocks)
(#107) 'owners' is not defined. (no-undef)
(#108) Expected space(s) before "else". (keyword-spacing)
(#108) Expected space(s) after "else". (keyword-spacing)
(#110) 'owners' is not defined. (no-undef)
(#112) Expected space(s) after "if". (keyword-spacing)
(#112) Unexpected space after unary operator '-'. (space-unary-ops)
(#112) Missing space before opening brace. (space-before-blocks)
(#113) This line has a length of 147. Maximum allowed is 132. (max-len)
(#113) 'selectownerassociatedusername' is not defined. (no-undef)
(#114) 'selectownerassociatedusername' is not defined. (no-undef)
(#115) Expected space(s) before "else". (keyword-spacing)
(#115) Expected space(s) after "else". (keyword-spacing)
(#123) 'operation' is not defined. (no-undef)
(#124) 'coursesinput' is not defined. (no-undef)
(#125) 'coursesinput' is not defined. (no-undef)
(#126) 'copycoursefilesinput' is not defined. (no-undef)
(#127) 'copycoursefilesinput' is not defined. (no-undef)
(#128) 'ownersinput' is not defined. (no-undef)
(#129) 'ownersinput' is not defined. (no-undef)
(#131) Unexpected alert. (no-alert)
(#131) 'M.str' is restricted from being used. Use "core/str" module or M.util.get_string() (no-restricted-properties)
(#137) 'e' is defined but never used. (no-unused-vars)
(#143) Missing semicolon. (semi)

CSS problems

(35 errors, 1 warnings)

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

admin/tool/legacyfilesmigration/styles.css
(#1) Expected newline before "}" (block-closing-brace-newline-before)
(#2) Expected newline before "}" (block-closing-brace-newline-before)
(#4) Expected newline before "}" (block-closing-brace-newline-before)
(#5) Expected newline before "}" (block-closing-brace-newline-before)
(#7) Expected newline before "}" (block-closing-brace-newline-before)
(#8) Expected newline before "}" (block-closing-brace-newline-before)
(#9) Expected newline before "}" (block-closing-brace-newline-before)
(#11) Expected newline before "}" (block-closing-brace-newline-before)
(#9) Expected single space before "}" of a single-line block (block-closing-brace-space-before)
(#11) Expected single space before "}" of a single-line block (block-closing-brace-space-before)
(#1) Expected newline after "{" (block-opening-brace-newline-after)
(#2) Expected newline after "{" (block-opening-brace-newline-after)
(#4) Expected newline after "{" (block-opening-brace-newline-after)
(#5) Expected newline after "{" (block-opening-brace-newline-after)
(#7) Expected newline after "{" (block-opening-brace-newline-after)
(#8) Expected newline after "{" (block-opening-brace-newline-after)
(#9) Expected newline after "{" (block-opening-brace-newline-after)
(#11) Expected newline after "{" (block-opening-brace-newline-after)
(#9) Expected single space after "{" of a single-line block (block-opening-brace-space-after)
(#11) Expected single space after "{" of a single-line block (block-opening-brace-space-after)
(#8) Expected single space before "{" (block-opening-brace-space-before)
(#11) Expected single space before "{" (block-opening-brace-space-before)
(#4) Expected "#ffeecc" to be "#fec" (color-hex-length)
(#11) Expected single space after ";" in a single-line declaration block (declaration-block-semicolon-space-after)
(#11) Expected single space after ";" in a single-line declaration block (declaration-block-semicolon-space-after)
(#11) Expected single space after ";" in a single-line declaration block (declaration-block-semicolon-space-after)
(#11) Expected single space after ";" in a single-line declaration block (declaration-block-semicolon-space-after)
(#11) Expected single space after ";" in a single-line declaration block (declaration-block-semicolon-space-after)
(#11) Expected no more than 1 declaration (declaration-block-single-line-max-declarations)
(#9) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#11) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#11) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#11) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#11) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#11) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#11) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)

PHPDocs style problems

(4 errors, 0 warnings)

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

admin/tool/legacyfilesmigration/batchmigrate.php
(#20) Invalid inline phpdocs tag @package found
admin/tool/legacyfilesmigration/locallib.php
(#162) Phpdocs for function tool_legacyfilesmigration_tools::tool_legacyfilesmigration_migrate_course has incomplete parameters list
admin/tool/legacyfilesmigration/renderer.php
(#91) Phpdocs for function tool_legacyfilesmigration_renderer::render_tool_legacyfilesmigration_batchoperationconfirm has incomplete parameters list
(#110) Phpdocs for function tool_legacyfilesmigration_renderer::course_list_page has incomplete parameters list

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

(1 errors, 1 warnings)

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

Problems running grunt
Task "stylelint:css" failed. Use --force to continue.

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]