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
(763 errors, 514 warnings)
This section shows the coding style problems detected in the code by phpcs [More info]
- admin/tool/mergeusers/classes/event/user_merged.php
- (#22) Incorrect @package tag for file user_merged.php. Expected tool_mergeusers, found tool.
- (#30) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#50) Missing docblock for function init
- (#52) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#52) Inline comments must end in full-stops, exclamation marks, or question marks
- admin/tool/mergeusers/classes/event/user_merged_failure.php
- (#20) Incorrect @package tag for file user_merged_failure.php. Expected tool_mergeusers, found tool.
- (#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#40) Missing docblock for function get_name
- (#44) Missing docblock for function get_description
- admin/tool/mergeusers/classes/event/user_merged_success.php
- (#20) Incorrect @package tag for file user_merged_success.php. Expected tool_mergeusers, found tool.
- (#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#40) Missing docblock for function get_name
- (#44) Missing docblock for function get_description
- (#49) Missing docblock for function get_log_id
- (#53) Missing docblock for function get_new_user_id
- (#57) Missing docblock for function get_old_user_id
- admin/tool/mergeusers/classes/local/observer/keptuser.php
- (#1) No one-line description found in phpdocs for docblock of file keptuser.php
- (#28) Missing docblock for class keptuser
- admin/tool/mergeusers/classes/local/observer/olduser.php
- (#1) No one-line description found in phpdocs for docblock of file olduser.php
- (#29) Missing docblock for class olduser
- (#39) "require_once" must be immediately followed by an open parenthesis
- (#43) Inline comments must end in full-stops, exclamation marks, or question marks
- (#45) Expected "if (...) {\n"; found "if(...){\n"
- (#60) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#60) Inline comments must end in full-stops, exclamation marks, or question marks
- (#60) No space found before comment text; expected "// do nothing; aborting, given that the image does not exist" but found "//do nothing; aborting, given that the image does not exist"
- (#63) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#66) Short array syntax must be used to define arrays
- (#66) Expected 1 space before "=>"; 0 found
- (#66) Expected 1 space after "=>"; 0 found
- admin/tool/mergeusers/classes/local/observer/update_user_profiles_on_merging_success.php
- (#38) "require_once" must be immediately followed by an open parenthesis
- (#55) Expected "try {\n...} catch (...) {\n"; found "try {\n...} catch (...) {"
- (#63) Empty CATCH statement detected
- (#63) Empty CATCH statement must have a comment to explain why the exception is not handled
- (#63) Closing brace must be on a line by itself
- admin/tool/mergeusers/classes/privacy/provider.php
- (#27) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#29) Missing docblock for class provider
- (#37) There must not be a space before the colon in a return type declaration
- admin/tool/mergeusers/classes/tool_mergeusers_config.php
- (#1) No one-line description found in phpdocs for docblock of file tool_mergeusers_config.php
- (#18) Incorrect @package tag for file tool_mergeusers_config.php. Expected tool_mergeusers, found tool.
- (#69) "include" must be immediately followed by an open parenthesis
- (#72) "include" must be immediately followed by an open parenthesis
- (#97) Opening brace should be on the same line as the declaration
- admin/tool/mergeusers/classes/tool_mergeusers_logger.php
- (#1) No one-line description found in phpdocs for docblock of file tool_mergeusers_logger.php
- (#18) Incorrect @package tag for file tool_mergeusers_logger.php. Expected tool_mergeusers, found tool.
- (#25) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
- (#25) "require_once" must be immediately followed by an open parenthesis
- (#29) "require_once" must be immediately followed by an open parenthesis
- (#54) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#54) Inline comments must end in full-stops, exclamation marks, or question marks
- (#54) No space found before comment text; expected "// to get it" but found "//to get it"
- (#57) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#57) Inline comments must end in full-stops, exclamation marks, or question marks
- (#57) No space found before comment text; expected "// exception is thrown on any error" but found "//exception is thrown on any error"
- (#64) Function print_error() has been deprecated; use throw new moodle_exception() (using lang strings only if meant to be shown to final user)() instead
- (#79) Line exceeds 132 characters; contains 158 characters
- (#84) Short array syntax must be used to define arrays
- (#85) Short array syntax must be used to define arrays
- (#90) Short array syntax must be used to define arrays
- (#101) Public method name "tool_mergeusers_logger::getDetail" must be in lower-case letters only
- (#103) Short array syntax must be used to define arrays
- admin/tool/mergeusers/cli/climerger.php
- (#1) No one-line description found in phpdocs for docblock of file climerger.php
- (#18) Incorrect @package tag for file climerger.php. Expected tool_mergeusers, found tool.
- (#27) "require_once" must be immediately followed by an open parenthesis
- (#34) "require_once" must be immediately followed by an open parenthesis
- (#35) "require_once" must be immediately followed by an open parenthesis
- (#39) Short array syntax must be used to define arrays
- (#65) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#65) Inline comments must end in full-stops, exclamation marks, or question marks
- (#71) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#71) Inline comments must end in full-stops, exclamation marks, or question marks
- (#72) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#72) Inline comments must end in full-stops, exclamation marks, or question marks
- (#72) No space found before comment text; expected "// may abort execution if database is not supported" but found "//may abort execution if database is not supported"
- (#75) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#75) Inline comments must end in full-stops, exclamation marks, or question marks
- (#79) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#79) Inline comments must end in full-stops, exclamation marks, or question marks
- (#79) No space found before comment text; expected "// collects and performs user mergings" but found "//collects and performs user mergings"
- (#82) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#82) Inline comments must end in full-stops, exclamation marks, or question marks
- admin/tool/mergeusers/config/config.php
- (#1) No one-line description found in phpdocs for docblock of file config.php
- (#18) Incorrect @package tag for file config.php. Expected tool_mergeusers, found tool.
- (#25) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#27) Inline doc block comments are not allowed; use "// Comment." instead
- (#35) Short array syntax must be used to define arrays
- (#37) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#37) Inline comments must end in full-stops, exclamation marks, or question marks
- (#42) Short array syntax must be used to define arrays
- (#56) Short array syntax must be used to define arrays
- (#57) Short array syntax must be used to define arrays
- (#58) Short array syntax must be used to define arrays
- (#59) Short array syntax must be used to define arrays
- (#61) Short array syntax must be used to define arrays
- (#62) Short array syntax must be used to define arrays
- (#63) Short array syntax must be used to define arrays
- (#65) Short array syntax must be used to define arrays
- (#66) Short array syntax must be used to define arrays
- (#67) Short array syntax must be used to define arrays
- (#69) Short array syntax must be used to define arrays
- (#70) Short array syntax must be used to define arrays
- (#71) Short array syntax must be used to define arrays
- (#73) Short array syntax must be used to define arrays
- (#73) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#73) Inline comments must end in full-stops, exclamation marks, or question marks
- (#73) No space found before comment text; expected "// both fields are user.id values" but found "//both fields are user.id values"
- (#74) Short array syntax must be used to define arrays
- (#75) Short array syntax must be used to define arrays
- (#77) Short array syntax must be used to define arrays
- (#78) Short array syntax must be used to define arrays
- (#79) Short array syntax must be used to define arrays
- (#79) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#79) Inline comments must end in full-stops, exclamation marks, or question marks
- (#81) Short array syntax must be used to define arrays
- (#82) Short array syntax must be used to define arrays
- (#83) Short array syntax must be used to define arrays
- (#83) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#83) Inline comments must end in full-stops, exclamation marks, or question marks
- (#85) Short array syntax must be used to define arrays
- (#86) Short array syntax must be used to define arrays
- (#87) Short array syntax must be used to define arrays
- (#87) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#87) Inline comments must end in full-stops, exclamation marks, or question marks
- (#89) Short array syntax must be used to define arrays
- (#90) Short array syntax must be used to define arrays
- (#91) Short array syntax must be used to define arrays
- (#93) Short array syntax must be used to define arrays
- (#93) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#93) Inline comments must end in full-stops, exclamation marks, or question marks
- (#94) Short array syntax must be used to define arrays
- (#95) Short array syntax must be used to define arrays
- (#97) Short array syntax must be used to define arrays
- (#97) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#97) Inline comments must end in full-stops, exclamation marks, or question marks
- (#98) Short array syntax must be used to define arrays
- (#99) Short array syntax must be used to define arrays
- (#101) Short array syntax must be used to define arrays
- (#101) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#101) Inline comments must end in full-stops, exclamation marks, or question marks
- (#101) No space found before comment text; expected "// mdl_scorscoetrac_usescosco_uix (unique)" but found "//mdl_scorscoetrac_usescosco_uix (unique)"
- (#102) Short array syntax must be used to define arrays
- (#103) Short array syntax must be used to define arrays
- (#105) Short array syntax must be used to define arrays
- (#105) Inline comments must end in full-stops, exclamation marks, or question marks
- (#105) No space found before comment text; expected "// UNIQUE KEY mdl_assigrad_assuseatt_uix" but found "//UNIQUE KEY mdl_assigrad_assuseatt_uix"
- (#106) Short array syntax must be used to define arrays
- (#107) Short array syntax must be used to define arrays
- (#109) Short array syntax must be used to define arrays
- (#109) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#109) Inline comments must end in full-stops, exclamation marks, or question marks
- (#110) Short array syntax must be used to define arrays
- (#111) Short array syntax must be used to define arrays
- (#113) Short array syntax must be used to define arrays
- (#113) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#113) Inline comments must end in full-stops, exclamation marks, or question marks
- (#114) Short array syntax must be used to define arrays
- (#115) Short array syntax must be used to define arrays
- (#117) Short array syntax must be used to define arrays
- (#117) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#117) Inline comments must end in full-stops, exclamation marks, or question marks
- (#117) No space found before comment text; expected "// unique key mdl_wikipage_subtituse_uix" but found "//unique key mdl_wikipage_subtituse_uix"
- (#118) Short array syntax must be used to define arrays
- (#119) Short array syntax must be used to define arrays
- (#121) Short array syntax must be used to define arrays
- (#121) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#121) Inline comments must end in full-stops, exclamation marks, or question marks
- (#121) No space found before comment text; expected "// unique key mdl_wikisubw_wikgrouse_uix" but found "//unique key mdl_wikisubw_wikgrouse_uix"
- (#122) Short array syntax must be used to define arrays
- (#123) Short array syntax must be used to define arrays
- (#125) Short array syntax must be used to define arrays
- (#126) Short array syntax must be used to define arrays
- (#127) Short array syntax must be used to define arrays
- (#129) Short array syntax must be used to define arrays
- (#130) Short array syntax must be used to define arrays
- (#131) Short array syntax must be used to define arrays
- (#133) Short array syntax must be used to define arrays
- (#134) Short array syntax must be used to define arrays
- (#135) Short array syntax must be used to define arrays
- (#137) Short array syntax must be used to define arrays
- (#138) Short array syntax must be used to define arrays
- (#139) Short array syntax must be used to define arrays
- (#147) Short array syntax must be used to define arrays
- (#148) Short array syntax must be used to define arrays
- (#149) Short array syntax must be used to define arrays
- (#149) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#149) Inline comments must end in full-stops, exclamation marks, or question marks
- (#149) No space found before comment text; expected "// compound index" but found "//compound index"
- (#150) Short array syntax must be used to define arrays
- (#151) Short array syntax must be used to define arrays
- (#152) Short array syntax must be used to define arrays
- (#153) Short array syntax must be used to define arrays
- (#153) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#153) Inline comments must end in full-stops, exclamation marks, or question marks
- (#153) No space found before comment text; expected "// may appear compound index" but found "//may appear compound index"
- (#158) Short array syntax must be used to define arrays
- admin/tool/mergeusers/db/access.php
- (#1) Missing @copyright tag
- (#1) Missing @license tag
- (#1) Missing docblock for file access.php
- (#1) Moodle boilerplate not found
- (#3) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#3) Short array syntax must be used to define arrays
- (#4) Short array syntax must be used to define arrays
- (#8) Short array syntax must be used to define arrays
- (#11) There should be a comma after the last array item in a multi-line array.
- admin/tool/mergeusers/db/events.php
- (#1) No one-line description found in phpdocs for docblock of file events.php
- (#18) Incorrect @package tag for file events.php. Expected tool_mergeusers, found tool.
- (#30) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#30) Short array syntax must be used to define arrays
- (#31) Short array syntax must be used to define arrays
- (#34) There should be a comma after the last array item in a multi-line array.
- (#36) Short array syntax must be used to define arrays
- (#39) There should be a comma after the last array item in a multi-line array.
- (#41) Short array syntax must be used to define arrays
- (#44) There should be a comma after the last array item in a multi-line array.
- (#45) There should be a comma after the last array item in a multi-line array.
- admin/tool/mergeusers/db/install.php
- (#1) No one-line description found in phpdocs for docblock of file install.php
- admin/tool/mergeusers/db/upgrade.php
- (#1) No one-line description found in phpdocs for docblock of file upgrade.php
- (#28) Docblock tag "@global" is not recommended.
- (#52) Short array syntax must be used to define arrays
- (#55) Short array syntax must be used to define arrays
- (#56) Short array syntax must be used to define arrays
- (#57) Short array syntax must be used to define arrays
- (#58) Short array syntax must be used to define arrays
- admin/tool/mergeusers/db/upgradelib.php
- (#26) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#37) "require_once" must be immediately followed by an open parenthesis
- (#38) "require_once" must be immediately followed by an open parenthesis
- (#49) Functions must not contain multiple empty lines in a row; found 2 empty lines
- (#138) Variable "field_info" must not contain underscores.
- (#139) Variable "field_info" must not contain underscores.
- (#147) Variable "define_field" must not contain underscores.
- (#148) Variable "define_field" must not contain underscores.
- admin/tool/mergeusers/index.php
- (#1) Missing @copyright tag
- (#1) Missing @license tag
- (#1) Missing docblock for file index.php
- (#2) Moodle boilerplate not found at first line
- (#37) Inline comments must end in full-stops, exclamation marks, or question marks
- (#54) Inline comments must end in full-stops, exclamation marks, or question marks
- (#64) Inline comments must end in full-stops, exclamation marks, or question marks
- (#71) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#71) Inline comments must end in full-stops, exclamation marks, or question marks
- (#71) No space found before comment text; expected "// may abort execution if database not supported, for security" but found "//may abort execution if database not supported, for security"
- (#73) Inline comments must end in full-stops, exclamation marks, or question marks
- (#77) Inline comments must end in full-stops, exclamation marks, or question marks
- (#80) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#82) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#82) Inline comments must end in full-stops, exclamation marks, or question marks
- (#82) No space found before comment text; expected "// get and verify the userids from the selection form usig the verify_user function (second field is column)" but found "//get and verify the userids from the selection form usig the verify_user function (second field is column)"
- (#88) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#88) Inline comments must end in full-stops, exclamation marks, or question marks
- (#96) Expected 1 space before comment text but found 3; use block comment if you need indentation
- (#97) Inline comments must end in full-stops, exclamation marks, or question marks
- (#102) Inline comments must end in full-stops, exclamation marks, or question marks
- (#114) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#118) Inline comments must end in full-stops, exclamation marks, or question marks
- (#123) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#125) Inline comments must end in full-stops, exclamation marks, or question marks
- (#130) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#130) Inline comments must end in full-stops, exclamation marks, or question marks
- (#133) Inline comments must end in full-stops, exclamation marks, or question marks
- (#134) Short array syntax must be used to define arrays
- (#138) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#138) Inline comments must end in full-stops, exclamation marks, or question marks
- (#141) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#141) Inline comments must end in full-stops, exclamation marks, or question marks
- (#145) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#150) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#156) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#161) Line indented incorrectly; expected at least 4 spaces, found 0
- (#163) Inline comments must end in full-stops, exclamation marks, or question marks
- (#166) Variable "search_users" must not contain underscores.
- (#167) Variable "user_select_table" must not contain underscores.
- (#167) Variable "search_users" must not contain underscores.
- (#169) Variable "user_select_table" must not contain underscores.
- (#171) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#171) Inline comments must end in full-stops, exclamation marks, or question marks
- (#173) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#173) Inline comments must end in full-stops, exclamation marks, or question marks
- (#173) No space found before comment text; expected "// get and verify the userids from the selection form usig the verify_user function (second field is column)" but found "//get and verify the userids from the selection form usig the verify_user function (second field is column)"
- (#179) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#179) Inline comments must end in full-stops, exclamation marks, or question marks
- (#181) Inline comments must end in full-stops, exclamation marks, or question marks
- (#190) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#194) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#194) Inline comments must end in full-stops, exclamation marks, or question marks
- admin/tool/mergeusers/index_form.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for file index_form.php. Expected tool_mergeusers, found tool.
- (#31) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#31) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#31) Inline comments must end in full-stops, exclamation marks, or question marks
- (#31) 3 slashes comments are not allowed; use "// Comment." instead
- (#49) Short array syntax must be used to define arrays
- (#55) Short array syntax must be used to define arrays
- (#67) Inline comments must end in full-stops, exclamation marks, or question marks
- (#68) Short array syntax must be used to define arrays
- (#79) Short array syntax must be used to define arrays
- (#86) Short array syntax must be used to define arrays
- admin/tool/mergeusers/lang/en/tool_mergeusers.php
- (#17) Missing @copyright tag
- (#52) Expected 1 space after "="; 0 found
- (#64) The string key "mergeusers_confirm" is not in the correct order, it should be before "mergeusersadvanced_help"
- (#65) The string key "mergeusers:mergeusers" is not in the correct order, it should be before "mergeusers_confirm"
- (#73) The string key "no_saveselection" is not in the correct order, it should be before "nomergedby"
- (#102) The string key "transactions_setting" is duplicated
- (#104) The string key "transactions_setting_desc" is duplicated
- admin/tool/mergeusers/lib.php
- (#17) Missing @copyright tag
- (#24) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#28) Docblock tag "@global" is not recommended.
- (#37) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#37) No space found before comment text; expected "// method is protected; make it accessible." but found "//method is protected; make it accessible."
- (#41) Missing docblock for function tool_mergeusers_build_exceptions_options
- (#46) Short array syntax must be used to define arrays
- (#50) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#50) No space found before comment text; expected "// duplicated records make MyMoodle does not work." but found "//duplicated records make MyMoodle does not work."
- (#60) Missing docblock for function tool_mergeusers_build_quiz_options
- (#63) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#63) Inline comments must end in full-stops, exclamation marks, or question marks
- (#64) Variable "quizStrings" must be all lower-case
- (#65) Variable "quizStrings" must be all lower-case
- (#65) Line exceeds 132 characters; contains 140 characters
- (#66) Variable "quizStrings" must be all lower-case
- (#66) Line exceeds 132 characters; contains 160 characters
- (#67) Variable "quizStrings" must be all lower-case
- (#67) Line exceeds 132 characters; contains 160 characters
- (#68) Variable "quizStrings" must be all lower-case
- (#68) Line exceeds 132 characters; contains 136 characters
- (#70) Variable "quizOptions" must be all lower-case
- (#70) Short array syntax must be used to define arrays
- (#71) Variable "quizStrings" must be all lower-case
- (#72) Variable "quizStrings" must be all lower-case
- (#73) Variable "quizStrings" must be all lower-case
- (#74) Variable "quizStrings" must be all lower-case
- (#78) Variable "quizStrings" must be all lower-case
- (#80) Variable "quizOptions" must be all lower-case
- admin/tool/mergeusers/lib/autoload.php
- (#1) No one-line description found in phpdocs for docblock of file autoload.php
- (#2) Moodle boilerplate not found at first line
- (#19) Incorrect @package tag for file autoload.php. Expected tool_mergeusers, found tool.
- (#25) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#27) Variable "fileName" must be all lower-case
- (#28) Variable "fileDirname" must be all lower-case
- (#29) Short array syntax must be used to define arrays
- (#30) Variable "fileDirname" must be all lower-case
- (#31) Variable "fileDirname" must be all lower-case
- (#32) Variable "fileDirname" must be all lower-case
- (#33) Variable "fileDirname" must be all lower-case
- (#37) Variable "fileName" must be all lower-case
- (#38) "require_once" must be immediately followed by an open parenthesis
- (#38) Variable "fileName" must be all lower-case
- admin/tool/mergeusers/lib/cligathering.php
- (#1) No one-line description found in phpdocs for docblock of file cligathering.php
- (#18) Incorrect @package tag for file cligathering.php. Expected tool_mergeusers, found tool.
- (#25) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#29) "require_once" must be immediately followed by an open parenthesis
- (#54) Opening brace should be on the same line as the declaration
- (#63) Opening brace should be on the same line as the declaration
- (#75) Opening brace should be on the same line as the declaration
- (#78) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#78) No space found before comment text; expected "// ask for the source user id." but found "//ask for the source user id."
- (#84) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#84) Inline comments must end in full-stops, exclamation marks, or question marks
- (#84) No space found before comment text; expected "// if we have to exit, do it just now ;-)" but found "//if we have to exit, do it just now ;-)"
- (#90) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#90) No space found before comment text; expected "// otherwise, ask for the target user id." but found "//otherwise, ask for the target user id."
- (#96) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#96) No space found before comment text; expected "// updates related iterator fields." but found "//updates related iterator fields."
- (#108) Opening brace should be on the same line as the declaration
- (#117) Opening brace should be on the same line as the declaration
- (#126) Opening brace should be on the same line as the declaration
- admin/tool/mergeusers/lib/db/assignsubmissionquery.php
- (#1) No one-line description found in phpdocs for docblock of file assignsubmissionquery.php
- (#1) DocBlock missing a @package tag for file assignsubmissionquery.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#22) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#24) Missing docblock for interface assign_submission_query
- (#25) Opening brace should be on the same line as the declaration for interface assign_submission_query
- (#26) Missing docblock for function latest_from_assign_and_user
- (#28) Missing docblock for function all_from_assign_and_user
- admin/tool/mergeusers/lib/db/dbassignsubmission.php
- (#1) No one-line description found in phpdocs for docblock of file dbassignsubmission.php
- (#1) DocBlock missing a @package tag for file dbassignsubmission.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#26) Missing docblock for class db_assign_submission
- (#27) Missing member variable doc comment
- (#29) Missing docblock for function __construct
- (#34) Missing docblock for function latest_from_assign_and_user
- (#39) Missing docblock for function all_from_assign_and_user
- admin/tool/mergeusers/lib/db/inmemoryfindbyquery.php
- (#1) No one-line description found in phpdocs for docblock of file inmemoryfindbyquery.php
- (#1) DocBlock missing a @package tag for file inmemoryfindbyquery.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#26) Missing docblock for class in_memory_assign_submission_query
- (#27) Opening brace should be on the same line as the declaration for class in_memory_assign_submission_query
- (#28) Missing member variable doc comment
- (#30) Missing docblock for function __construct
- (#34) Missing docblock for function latest_from_assign_and_user
- (#38) Missing docblock for function all_from_assign_and_user
- admin/tool/mergeusers/lib/duplicateddata/assignsubmissionduplicateddatamerger.php
- (#1) No one-line description found in phpdocs for docblock of file assignsubmissionduplicateddatamerger.php
- (#1) DocBlock missing a @package tag for file assignsubmissionduplicateddatamerger.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#29) Missing docblock for class AssignSubmissionDuplicatedDataMerger
- (#31) Missing docblock for constant AssignSubmissionDuplicatedDataMerger::ASSIGN_SUBMISSION_WITH_CONTENT
- (#36) Missing docblock for constant AssignSubmissionDuplicatedDataMerger::ASSIGN_SUBMISSION_NEW
- (#38) Missing member variable doc comment
- (#40) Missing docblock for function __construct
- (#40) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $findbyquery.
- (#44) Missing docblock for function merge
- (#75) Missing docblock for function old_submission_has_content_and_new_has_no_content
- (#80) Missing docblock for function both_submissions_have_content
- (#85) Missing docblock for function old_user_submission_is_older_than_new_user_submission
- admin/tool/mergeusers/lib/duplicateddata/duplicateddata.php
- (#1) No one-line description found in phpdocs for docblock of file duplicateddata.php
- (#1) DocBlock missing a @package tag for file duplicateddata.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#22) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#24) Missing docblock for class DuplicatedData
- (#25) Missing member variable doc comment
- (#27) Missing member variable doc comment
- (#29) Missing docblock for function from_empty
- (#33) Missing docblock for function from_remove_and_modify
- (#37) Missing docblock for function from_remove
- (#41) Missing docblock for function __construct
- (#46) Missing docblock for function to_remove
- (#50) Missing docblock for function to_modify
- admin/tool/mergeusers/lib/duplicateddata/duplicateddatamerger.php
- (#1) No one-line description found in phpdocs for docblock of file duplicateddatamerger.php
- (#1) DocBlock missing a @package tag for file duplicateddatamerger.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#22) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#24) Missing docblock for interface DuplicatedDataMerger
- (#25) Opening brace should be on the same line as the declaration for interface DuplicatedDataMerger
- (#26) Missing docblock for function merge
- admin/tool/mergeusers/lib/gathering.php
- (#1) No one-line description found in phpdocs for docblock of file gathering.php
- (#18) Incorrect @package tag for file gathering.php. Expected tool_mergeusers, found tool.
- (#37) File must end with a newline character
- admin/tool/mergeusers/lib/merger.php
- (#1) No one-line description found in phpdocs for docblock of file merger.php
- (#18) Incorrect @package tag for file merger.php. Expected tool_mergeusers, found tool.
- (#31) Missing docblock for class Merger
- (#86) Using ${var} in strings is deprecated since PHP 8.2, use {$var} instead. Found: ${numberoperations}
- admin/tool/mergeusers/lib/mergeusersearch.php
- (#2) Moodle boilerplate not found at first line
- (#18) Missing @copyright tag
- (#23) Incorrect @package tag for file mergeusersearch.php. Expected tool_mergeusers, found tool.
- (#35) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
- (#35) "require_once" must be immediately followed by an open parenthesis
- (#39) "require_once" must be immediately followed by an open parenthesis
- (#40) "require_once" must be immediately followed by an open parenthesis
- (#47) Expected 1 space before opening brace; found 0
- (#57) Expected 1 space before opening brace; found 0
- (#61) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#61) Inline comments must end in full-stops, exclamation marks, or question marks
- (#63) Short array syntax must be used to define arrays
- (#69) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#69) Inline comments must end in full-stops, exclamation marks, or question marks
- (#71) Short array syntax must be used to define arrays
- (#77) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#77) Inline comments must end in full-stops, exclamation marks, or question marks
- (#79) Short array syntax must be used to define arrays
- (#85) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#85) Inline comments must end in full-stops, exclamation marks, or question marks
- (#87) Short array syntax must be used to define arrays
- (#93) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#93) Inline comments must end in full-stops, exclamation marks, or question marks
- (#95) Short array syntax must be used to define arrays
- (#101) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#101) Inline comments must end in full-stops, exclamation marks, or question marks
- (#103) Short array syntax must be used to define arrays
- (#109) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#109) Inline comments must end in full-stops, exclamation marks, or question marks
- (#111) Short array syntax must be used to define arrays
- (#112) Expected 1 space after "=>"; 2 found
- (#117) There should be a comma after the last array item in a multi-line array.
- (#153) Expected 1 space before opening brace; found 0
- (#157) Short array syntax must be used to define arrays
- (#163) Short array syntax must be used to define arrays
- admin/tool/mergeusers/lib/mergeusertool.php
- (#2) Moodle boilerplate not found at first line
- (#18) Missing @copyright tag
- (#23) Incorrect @package tag for file mergeusertool.php. Expected tool_mergeusers, found tool.
- (#35) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
- (#35) "require_once" must be immediately followed by an open parenthesis
- (#39) "require_once" must be immediately followed by an open parenthesis
- (#40) "require_once" must be immediately followed by an open parenthesis
- (#57) Opening brace should be on the same line as the declaration for class MergeUserTool
- (#62) Member variable "userFieldsPerTable" must be all lower-case
- (#68) Member variable "tablesToSkip" must be all lower-case
- (#73) Member variable "tablesSkipped" must be all lower-case
- (#79) Member variable "tablesWithCompoundIndex" must be all lower-case
- (#84) Member variable "sqlListTables" must be all lower-case
- (#90) Member variable "userFieldNames" must be all lower-case
- (#101) Member variable "tableMergers" must be all lower-case
- (#106) Member variable "tablesProcessedByTableMergers" must be all lower-case
- (#111) Member variable "alwaysRollback" must be all lower-case
- (#120) Docblock tag "@global" is not recommended.
- (#124) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $config.
- (#124) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $logger.
- (#125) Opening brace should be on the same line as the declaration
- (#149) Variable "tableMergers" must be all lower-case
- (#149) Short array syntax must be used to define arrays
- (#150) Variable "tablesProcessedByTableMergers" must be all lower-case
- (#150) Short array syntax must be used to define arrays
- (#151) Variable "tableName" must be all lower-case
- (#153) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#153) Inline comments must end in full-stops, exclamation marks, or question marks
- (#155) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#160) Function print_error() has been deprecated; use throw new moodle_exception() (using lang strings only if meant to be shown to final user)() instead
- (#165) Variable "tablesProcessedByTableMergers" must be all lower-case
- (#165) Variable "tablesProcessedByTableMergers" must be all lower-case
- (#166) Variable "tableMergers" must be all lower-case
- (#166) Variable "tableName" must be all lower-case
- (#168) Variable "tableMergers" must be all lower-case
- (#169) Variable "tablesProcessedByTableMergers" must be all lower-case
- (#181) Docblock tag "@global" is not recommended.
- (#182) Docblock tag "@global" is not recommended.
- (#191) Opening brace should be on the same line as the declaration
- (#199) Short array syntax must be used to define arrays
- (#201) Short array syntax must be used to define arrays
- (#202) Short array syntax must be used to define arrays
- (#212) Short array syntax must be used to define arrays
- (#217) Docblock tag "@global" is not recommended.
- (#218) Docblock tag "@global" is not recommended.
- (#225) Method name "_merge" should not be prefixed with an underscore to indicate visibility
- (#226) Opening brace should be on the same line as the declaration
- (#229) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#232) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#233) Short array syntax must be used to define arrays
- (#233) Short array syntax must be used to define arrays
- (#236) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#238) Variable "errorMessages" must be all lower-case
- (#238) Short array syntax must be used to define arrays
- (#239) Variable "actionLog" must be all lower-case
- (#239) Short array syntax must be used to define arrays
- (#245) Variable "startTime" must be all lower-case
- (#246) Variable "startTimeString" must be all lower-case
- (#246) Variable "startTime" must be all lower-case
- (#247) Variable "actionLog" must be all lower-case
- (#247) Variable "startTimeString" must be all lower-case
- (#252) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#252) Inline comments must end in full-stops, exclamation marks, or question marks
- (#253) Short array syntax must be used to define arrays
- (#257) Variable "tableName" must be all lower-case
- (#257) Variable "userFields" must be all lower-case
- (#258) Variable "tableName" must be all lower-case
- (#259) Variable "userFields" must be all lower-case
- (#260) Variable "tableName" must be all lower-case
- (#261) Variable "tableName" must be all lower-case
- (#266) Variable "tableMerger" must be all lower-case
- (#266) Variable "tableName" must be all lower-case
- (#267) Variable "tableName" must be all lower-case
- (#270) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#271) Variable "tableMerger" must be all lower-case
- (#271) Variable "actionLog" must be all lower-case
- (#271) Variable "errorMessages" must be all lower-case
- (#277) Variable "errorMessages" must be all lower-case
- (#291) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#291) Inline comments must end in full-stops, exclamation marks, or question marks
- (#292) Variable "errorMessages" must be all lower-case
- (#295) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#295) Inline comments must end in full-stops, exclamation marks, or question marks
- (#296) Variable "skippedTables" must be all lower-case
- (#296) Short array syntax must be used to define arrays
- (#298) Variable "skippedTables" must be all lower-case
- (#301) Variable "finishTime" must be all lower-case
- (#302) Variable "actionLog" must be all lower-case
- (#302) Variable "finishTime" must be all lower-case
- (#303) Variable "actionLog" must be all lower-case
- (#303) Variable "finishTime" must be all lower-case
- (#303) Variable "startTime" must be all lower-case
- (#305) Short array syntax must be used to define arrays
- (#305) Variable "skippedTables" must be all lower-case
- (#305) Variable "actionLog" must be all lower-case
- (#308) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#308) No space found before comment text; expected "// thrown controlled exception." but found "//thrown controlled exception."
- (#310) Empty CATCH statement detected
- (#314) Variable "finishTime" must be all lower-case
- (#315) Variable "errorMessages" must be all lower-case
- (#315) Variable "startTimeString" must be all lower-case
- (#316) Variable "errorMessages" must be all lower-case
- (#316) Variable "finishTime" must be all lower-case
- (#316) Variable "startTime" must be all lower-case
- (#318) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#319) Short array syntax must be used to define arrays
- (#319) Variable "errorMessages" must be all lower-case
- (#322) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#326) Docblock tag "@global" is not recommended.
- (#327) Docblock tag "@global" is not recommended.
- (#330) Opening brace should be on the same line as the declaration
- (#333) Variable "userFieldsPerTable" must be all lower-case
- (#333) Short array syntax must be used to define arrays
- (#336) Variable "tableNames" must be all lower-case
- (#338) Variable "tableNames" must be all lower-case
- (#338) Variable "tableName" must be all lower-case
- (#340) Variable "tableName" must be all lower-case
- (#346) Variable "tableName" must be all lower-case
- (#347) Variable "tableName" must be all lower-case
- (#347) Variable "tableName" must be all lower-case
- (#351) Variable "tableName" must be all lower-case
- (#356) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#357) Variable "userFields" must be all lower-case
- (#357) Variable "tableName" must be all lower-case
- (#358) Variable "tableName" must be all lower-case
- (#361) Variable "arrayUserFields" must be all lower-case
- (#361) Variable "userFields" must be all lower-case
- (#362) Variable "currentFields" must be all lower-case
- (#362) Variable "tableName" must be all lower-case
- (#362) Variable "arrayUserFields" must be all lower-case
- (#364) Variable "currentFields" must be all lower-case
- (#365) Variable "userFieldsPerTable" must be all lower-case
- (#365) Variable "tableName" must be all lower-case
- (#365) Variable "currentFields" must be all lower-case
- (#369) Variable "userFieldsPerTable" must be all lower-case
- (#371) Variable "existingCompoundIndexes" must be all lower-case
- (#372) Variable "tableName" must be all lower-case
- (#373) Variable "chosenColumns" must be all lower-case
- (#375) Variable "columnNames" must be all lower-case
- (#375) Short array syntax must be used to define arrays
- (#376) Variable "chosenColumns" must be all lower-case
- (#376) Variable "columnName" must be all lower-case
- (#377) Variable "columnNames" must be all lower-case
- (#377) Variable "columnName" must be all lower-case
- (#380) Variable "tableColumns" must be all lower-case
- (#380) Variable "tableName" must be all lower-case
- (#382) Variable "tableColumns" must be all lower-case
- (#383) Variable "columnNames" must be all lower-case
- (#384) Variable "columnNames" must be all lower-case
- (#391) Variable "columnNames" must be all lower-case
- (#392) The use of function sizeof() is forbidden; use count() instead
- (#392) Variable "columnNames" must be all lower-case
- (#393) Variable "existingCompoundIndexes" must be all lower-case
- (#393) Variable "tableName" must be all lower-case
- (#397) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#398) Variable "existingCompoundIndexes" must be all lower-case
- (#409) Public method name "MergeUserTool::checkTransactionSupport" must be in lower-case letters only
- (#410) Opening brace should be on the same line as the declaration
- (#413) Variable "transactionsSupported" must be all lower-case
- (#414) Variable "forceOnlyTransactions" must be all lower-case
- (#416) Variable "transactionsSupported" must be all lower-case
- (#416) Variable "forceOnlyTransactions" must be all lower-case
- (#421) Function print_error() has been deprecated; use throw new moodle_exception() (using lang strings only if meant to be shown to final user)() instead
- (#426) Variable "transactionsSupported" must be all lower-case
- (#436) Private method name "MergeUserTool::getCurrentUserFieldNames" must be in lower-case letters only
- (#436) Variable "tableName" must be all lower-case
- (#436) Variable "userFields" must be all lower-case
- (#437) Opening brace should be on the same line as the declaration
- (#439) Variable "tableName" must be all lower-case
- (#439) Commas (,) must be followed by white space.
- (#441) Expected "foreach (...) {\n"; found "foreach(...) {\n"
- (#442) Variable "userFields" must be all lower-case
- (#453) Private method name "MergeUserTool::updateGrades" must be in lower-case letters only
- (#462) Short array syntax must be used to define arrays
- (#465) Short array syntax must be used to define arrays
- (#479) Missing docblock for function reaggregateCompletions
- (#479) Private method name "MergeUserTool::reaggregateCompletions" must be in lower-case letters only
- (#484) Line exceeds 132 characters; contains 139 characters
- admin/tool/mergeusers/lib/table/assignsubmissiontablemerger.php
- (#1) No one-line description found in phpdocs for docblock of file assignsubmissiontablemerger.php
- (#1) DocBlock missing a @package tag for file assignsubmissiontablemerger.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#27) Missing docblock for class AssignSubmissionTableMerger
- (#29) Missing member variable doc comment
- (#30) Missing member variable doc comment
- (#32) Missing docblock for function __construct
- (#38) Missing docblock for function mergeCompoundIndex
- (#38) Public method name "AssignSubmissionTableMerger::mergeCompoundIndex" must be in lower-case letters only
- (#38) Variable "recordsToModify" must be all lower-case
- (#38) Variable "actionLog" must be all lower-case
- (#39) Variable "errorMessages" must be all lower-case
- (#43) Variable "recordsToModify" must be all lower-case
- (#44) Variable "recordsToModify" must be all lower-case
- (#53) Variable "recordsToModify" must be all lower-case
- (#60) Variable "recordsToModify" must be all lower-case
- (#65) Variable "recordsToModify" must be all lower-case
- (#66) Variable "recordsToModify" must be all lower-case
- (#70) Variable "actionLog" must be all lower-case
- (#70) Variable "errorMessages" must be all lower-case
- (#73) Missing docblock for function get_records_to_be_updated
- (#73) Variable "fieldName" must be all lower-case
- (#76) Variable "fieldName" must be all lower-case
- admin/tool/mergeusers/lib/table/generictablemerger.php
- (#2) Moodle boilerplate not found at first line
- (#18) Missing @copyright tag
- (#21) Incorrect @package tag for file generictablemerger.php. Expected tool_mergeusers, found tool.
- (#26) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
- (#34) Opening brace should be on the same line as the declaration for class GenericTableMerger
- (#35) Missing docblock for constant GenericTableMerger::CHUNK_SIZE
- (#44) Missing docblock for function __construct
- (#45) Opening brace should be on the same line as the declaration
- (#57) Public method name "GenericTableMerger::getTablesToSkip" must be in lower-case letters only
- (#58) Opening brace should be on the same line as the declaration
- (#59) Short array syntax must be used to define arrays
- (#59) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#70) Variable "actionLog" must be all lower-case
- (#70) Variable "errorMessages" must be all lower-case
- (#71) Opening brace should be on the same line as the declaration
- (#72) Variable "fieldName" must be all lower-case
- (#73) Variable "recordsToUpdate" must be all lower-case
- (#73) Variable "fieldName" must be all lower-case
- (#74) Variable "recordsToUpdate" must be all lower-case
- (#75) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#75) Inline comments must end in full-stops, exclamation marks, or question marks
- (#75) No space found before comment text; expected "// this userid is not present in these table and field names" but found "//this userid is not present in these table and field names"
- (#79) Variable "recordsToUpdate" must be all lower-case
- (#79) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#79) Inline comments must end in full-stops, exclamation marks, or question marks
- (#80) Variable "recordsToModify" must be all lower-case
- (#83) Variable "fieldName" must be all lower-case
- (#84) Variable "fieldName" must be all lower-case
- (#84) Variable "recordsToModify" must be all lower-case
- (#85) Variable "actionLog" must be all lower-case
- (#85) Variable "errorMessages" must be all lower-case
- (#88) Variable "recordsToModify" must be all lower-case
- (#88) Variable "fieldName" must be all lower-case
- (#88) Variable "actionLog" must be all lower-case
- (#88) Variable "errorMessages" must be all lower-case
- (#112) Docblock tag "@global" is not recommended.
- (#113) Docblock tag "@global" is not recommended.
- (#121) Protected method name "GenericTableMerger::mergeCompoundIndex" must be in lower-case letters only
- (#121) Variable "recordsToModify" must be all lower-case
- (#121) Variable "actionLog" must be all lower-case
- (#122) Variable "errorMessages" must be all lower-case
- (#123) Opening brace should be on the same line as the declaration
- (#130) Short array syntax must be used to define arrays
- (#132) Variable "itemArr" must be all lower-case
- (#132) Short array syntax must be used to define arrays
- (#133) Variable "idsToRemove" must be all lower-case
- (#133) Short array syntax must be used to define arrays
- (#134) Variable "resObj" must be all lower-case
- (#135) Short array syntax must be used to define arrays
- (#137) Variable "resObj" must be all lower-case
- (#140) Variable "itemArr" must be all lower-case
- (#140) Variable "resObj" must be all lower-case
- (#142) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#142) Inline comments must end in full-stops, exclamation marks, or question marks
- (#142) No space found before comment text; expected "// free memory" but found "//free memory"
- (#144) Variable "itemArr" must be all lower-case
- (#144) Variable "otherInfo" must be all lower-case
- (#145) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#145) Inline comments must end in full-stops, exclamation marks, or question marks
- (#145) No space found before comment text; expected "// iff we have only one result and it is from the current user => update record" but found "//iff we have only one result and it is from the current user => update record"
- (#146) The use of function sizeof() is forbidden; use count() instead
- (#146) Variable "otherInfo" must be all lower-case
- (#147) Variable "otherInfo" must be all lower-case
- (#148) Variable "recordsToModify" must be all lower-case
- (#148) Variable "otherInfo" must be all lower-case
- (#148) Variable "otherInfo" must be all lower-case
- (#150) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#151) Inline comments must end in full-stops, exclamation marks, or question marks
- (#151) No space found before comment text; expected "// confirm both records exist, preventing problems from inconsistent data in database" but found "//confirm both records exist, preventing problems from inconsistent data in database"
- (#152) Variable "otherInfo" must be all lower-case
- (#152) Variable "otherInfo" must be all lower-case
- (#153) Variable "idsToRemove" must be all lower-case
- (#153) Variable "otherInfo" must be all lower-case
- (#153) Variable "otherInfo" must be all lower-case
- (#157) Variable "itemArr" must be all lower-case
- (#157) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#157) No space found before comment text; expected "// free memory" but found "//free memory"
- (#159) Variable "idsToRemove" must be all lower-case
- (#160) Variable "recordsToModify" must be all lower-case
- (#161) Variable "recordsToModify" must be all lower-case
- (#165) Variable "idsToRemove" must be all lower-case
- (#165) Variable "actionLog" must be all lower-case
- (#165) Variable "errorMessages" must be all lower-case
- (#167) Variable "idsToRemove" must be all lower-case
- (#167) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#167) Inline comments must end in full-stops, exclamation marks, or question marks
- (#167) No space found before comment text; expected "// free memory" but found "//free memory"
- (#177) Docblock tag "@global" is not recommended.
- (#178) Docblock tag "@global" is not recommended.
- (#185) Protected method name "GenericTableMerger::cleanRecordsOnCompoundIndex" must be in lower-case letters only
- (#185) Variable "idsToRemove" must be all lower-case
- (#185) Variable "actionLog" must be all lower-case
- (#185) Variable "errorMessages" must be all lower-case
- (#186) Opening brace should be on the same line as the declaration
- (#187) Variable "idsToRemove" must be all lower-case
- (#191) Variable "idsToRemove" must be all lower-case
- (#192) Variable "someIdsToRemove" must be all lower-case
- (#193) Variable "someIdsToRemove" must be all lower-case
- (#193) Variable "actionLog" must be all lower-case
- (#193) Variable "errorMessages" must be all lower-case
- (#197) Missing docblock for function cleanRecords
- (#197) Protected method name "GenericTableMerger::cleanRecords" must be in lower-case letters only
- (#197) Variable "idsToRemove" must be all lower-case
- (#197) Variable "actionLog" must be all lower-case
- (#197) Variable "errorMessages" must be all lower-case
- (#200) Variable "idsToRemove" must be all lower-case
- (#204) Variable "tableName" must be all lower-case
- (#205) Variable "idsGoByebye" must be all lower-case
- (#205) Variable "idsToRemove" must be all lower-case
- (#206) Variable "tableName" must be all lower-case
- (#206) Variable "idsGoByebye" must be all lower-case
- (#209) Variable "actionLog" must be all lower-case
- (#211) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#211) Inline comments must end in full-stops, exclamation marks, or question marks
- (#212) Variable "errorMessages" must be all lower-case
- (#215) Variable "idsGoByebye" must be all lower-case
- (#215) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#215) Inline comments must end in full-stops, exclamation marks, or question marks
- (#215) No space found before comment text; expected "// free memory" but found "//free memory"
- (#228) Protected method name "GenericTableMerger::updateAllRecords" must be in lower-case letters only
- (#228) Variable "recordsToModify" must be all lower-case
- (#228) Variable "fieldName" must be all lower-case
- (#228) Variable "actionLog" must be all lower-case
- (#228) Variable "errorMessages" must be all lower-case
- (#229) Opening brace should be on the same line as the declaration
- (#230) Variable "recordsToModify" must be all lower-case
- (#231) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#231) Inline comments must end in full-stops, exclamation marks, or question marks
- (#235) Variable "recordsToModify" must be all lower-case
- (#237) Variable "fieldName" must be all lower-case
- (#237) Variable "actionLog" must be all lower-case
- (#237) Variable "errorMessages" must be all lower-case
- (#241) Missing docblock for function updateRecords
- (#241) Protected method name "GenericTableMerger::updateRecords" must be in lower-case letters only
- (#241) Variable "recordsToModify" must be all lower-case
- (#241) Variable "fieldName" must be all lower-case
- (#241) Variable "actionLog" must be all lower-case
- (#241) Variable "errorMessages" must be all lower-case
- (#241) Line exceeds 132 characters; contains 136 characters
- (#243) Variable "tableName" must be all lower-case
- (#244) Variable "idString" must be all lower-case
- (#244) Variable "recordsToModify" must be all lower-case
- (#245) Variable "updateRecords" must be all lower-case
- (#245) Variable "tableName" must be all lower-case
- (#246) Variable "fieldName" must be all lower-case
- (#247) Variable "idString" must be all lower-case
- (#250) Variable "updateRecords" must be all lower-case
- (#251) Variable "errorMessages" must be all lower-case
- (#254) Variable "actionLog" must be all lower-case
- (#254) Variable "updateRecords" must be all lower-case
- (#256) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#259) Variable "deleteRecord" must be all lower-case
- (#259) Variable "tableName" must be all lower-case
- (#260) Variable "fieldName" must be all lower-case
- (#262) Variable "deleteRecord" must be all lower-case
- (#263) Variable "errorMessages" must be all lower-case
- (#266) Variable "actionLog" must be all lower-case
- (#266) Variable "deleteRecord" must be all lower-case
- (#280) Protected method name "GenericTableMerger::getOtherFieldsOnCompoundIndex" must be in lower-case letters only
- (#280) Variable "userField" must be all lower-case
- (#280) Variable "compoundIndex" must be all lower-case
- (#281) Opening brace should be on the same line as the declaration
- (#282) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#283) The use of function sizeof() is forbidden; use count() instead
- (#283) Variable "compoundIndex" must be all lower-case
- (#284) Variable "compoundIndex" must be all lower-case
- (#284) Variable "compoundIndex" must be all lower-case
- (#286) Variable "userField" must be all lower-case
- (#289) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#289) Inline comments must end in full-stops, exclamation marks, or question marks
- (#290) Variable "compoundIndex" must be all lower-case
- (#293) Missing docblock for function get_records_to_be_updated
- (#293) Variable "fieldName" must be all lower-case
- (#297) Variable "fieldName" must be all lower-case
- admin/tool/mergeusers/lib/table/quizattemptsmerger.php
- (#22) "require_once" must be immediately followed by an open parenthesis
- (#23) "require_once" must be immediately followed by an open parenthesis
- (#25) Missing @copyright tag
- (#53) Incorrect @package tag for class QuizAttemptsMerger. Expected tool_mergeusers, found tool.
- (#60) Opening brace should be on the same line as the declaration for class QuizAttemptsMerger
- (#84) Opening brace should be on the same line as the declaration
- (#95) Public method name "QuizAttemptsMerger::getTablesToSkip" must be in lower-case letters only
- (#96) Opening brace should be on the same line as the declaration
- (#97) Short array syntax must be used to define arrays
- (#108) Variable "actionLog" must be all lower-case
- (#108) Variable "errorMessages" must be all lower-case
- (#109) Opening brace should be on the same line as the declaration
- (#113) Variable "actionLog" must be all lower-case
- (#116) Variable "actionLog" must be all lower-case
- (#116) Variable "errorMessages" must be all lower-case
- (#122) Variable "actionLog" must be all lower-case
- (#122) Variable "errorMessages" must be all lower-case
- (#125) Variable "actionLog" must be all lower-case
- (#125) Variable "errorMessages" must be all lower-case
- (#140) Variable "actionLog" must be all lower-case
- (#140) Variable "errorMessages" must be all lower-case
- (#141) Opening brace should be on the same line as the declaration
- (#144) Variable "tableName" must be all lower-case
- (#150) Variable "tableName" must be all lower-case
- (#156) Variable "allAttempts" must be all lower-case
- (#156) Short array syntax must be used to define arrays
- (#159) Variable "allAttempts" must be all lower-case
- (#162) Short array syntax must be used to define arrays
- (#163) Variable "tableName" must be all lower-case
- (#168) Short array syntax must be used to define arrays
- (#170) Variable "attemptsByQuiz" must be all lower-case
- (#170) Short array syntax must be used to define arrays
- (#172) Short array syntax must be used to define arrays
- (#175) Variable "allAttempts" must be all lower-case
- (#176) Variable "attemptsByQuiz" must be all lower-case
- (#181) Variable "attemptsByQuiz" must be all lower-case
- (#195) Expected 1 space before comment text but found 3; use block comment if you need indentation
- (#196) Expected 1 space before comment text but found 3; use block comment if you need indentation
- (#209) Inline comments must end in full-stops, exclamation marks, or question marks
- (#216) Short array syntax must be used to define arrays
- (#222) Variable "updateSql" must be all lower-case
- (#223) Variable "updateSql" must be all lower-case
- (#224) Variable "actionLog" must be all lower-case
- (#224) Variable "updateSql" must be all lower-case
- (#226) Variable "errorMessages" must be all lower-case
- (#235) Variable "updateAll" must be all lower-case
- (#235) Variable "tableName" must be all lower-case
- (#239) Variable "updateAll" must be all lower-case
- (#240) Variable "actionLog" must be all lower-case
- (#240) Variable "updateAll" must be all lower-case
- (#242) Variable "errorMessages" must be all lower-case
- (#248) Variable "actionLog" must be all lower-case
- (#261) Protected method name "QuizAttemptsMerger::updateAllRecords" must be in lower-case letters only
- (#261) Variable "recordsToModify" must be all lower-case
- (#261) Variable "fieldName" must be all lower-case
- (#261) Variable "actionLog" must be all lower-case
- (#261) Variable "errorMessages" must be all lower-case
- (#262) Opening brace should be on the same line as the declaration
- (#263) Variable "recordsToModify" must be all lower-case
- (#263) Variable "fieldName" must be all lower-case
- (#263) Variable "actionLog" must be all lower-case
- (#263) Variable "errorMessages" must be all lower-case
- (#264) Variable "recordsToModify" must be all lower-case
- (#264) Variable "actionLog" must be all lower-case
- (#269) Docblock tag "@global" is not recommended.
- (#273) Protected method name "QuizAttemptsMerger::updateAllQuizzes" must be in lower-case letters only
- (#273) Variable "actionLog" must be all lower-case
- (#274) Opening brace should be on the same line as the declaration
- (#282) Variable "actionLog" must be all lower-case
- (#286) Missing docblock for function updateQuizzes
- (#286) Protected method name "QuizAttemptsMerger::updateQuizzes" must be in lower-case letters only
- (#286) Variable "actionLog" must be all lower-case
- (#287) Opening brace should be on the same line as the declaration
- (#292) Variable "sqlQuizzes" must be all lower-case
- (#297) Variable "sqlQuizzes" must be all lower-case
- (#300) Variable "actionLog" must be all lower-case
- admin/tool/mergeusers/lib/table/tablemerger.php
- (#2) Moodle boilerplate not found at first line
- (#18) Missing @copyright tag
- (#21) Incorrect @package tag for file tablemerger.php. Expected tool_mergeusers, found tool.
- (#56) Opening brace should be on the same line as the declaration for interface TableMerger
- (#57) Missing docblock for constant TableMerger::PRIMARY_KEY
- (#67) Public method name "TableMerger::getTablesToSkip" must be in lower-case letters only
- (#77) Variable "errorMessages" must be all lower-case
- (#77) Variable "actionLog" must be all lower-case
- admin/tool/mergeusers/lib/userreviewtable.php
- (#2) Moodle boilerplate not found at first line
- (#19) Missing @copyright tag
- (#22) Incorrect @package tag for file userreviewtable.php. Expected tool_mergeusers, found tool.
- (#34) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
- (#38) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#38) Inline comments must end in full-stops, exclamation marks, or question marks
- (#50) Opening brace should be on the same line as the declaration for class UserReviewTable
- (#68) Opening brace should be on the same line as the declaration
- (#73) Inline comments must end in full-stops, exclamation marks, or question marks
- (#91) Opening brace should be on the same line as the declaration
- (#92) Inline comments must end in full-stops, exclamation marks, or question marks
- (#93) Short array syntax must be used to define arrays
- (#95) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#95) Inline comments must end in full-stops, exclamation marks, or question marks
- (#105) Short array syntax must be used to define arrays
- (#110) There should be a comma after the last array item in a multi-line array.
- (#116) Inline comments must end in full-stops, exclamation marks, or question marks
- (#117) Short array syntax must be used to define arrays
- (#119) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#119) Inline comments must end in full-stops, exclamation marks, or question marks
- (#124) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#124) Inline comments must end in full-stops, exclamation marks, or question marks
- (#132) Inline comments must end in full-stops, exclamation marks, or question marks
- (#133) Short array syntax must be used to define arrays
- (#135) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#135) Inline comments must end in full-stops, exclamation marks, or question marks
- (#140) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#140) Inline comments must end in full-stops, exclamation marks, or question marks
- admin/tool/mergeusers/lib/userselecttable.php
- (#2) Moodle boilerplate not found at first line
- (#19) Missing @copyright tag
- (#22) Incorrect @package tag for file userselecttable.php. Expected tool_mergeusers, found tool.
- (#34) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
- (#38) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#38) Inline comments must end in full-stops, exclamation marks, or question marks
- (#50) Opening brace should be on the same line as the declaration for class UserSelectTable
- (#63) Opening brace should be on the same line as the declaration
- (#76) Opening brace should be on the same line as the declaration
- (#77) Inline comments must end in full-stops, exclamation marks, or question marks
- (#78) Short array syntax must be used to define arrays
- (#83) Short array syntax must be used to define arrays
- (#96) Short array syntax must be used to define arrays
- (#98) Short array syntax must be used to define arrays
- (#98) Line exceeds 132 characters; contains 149 characters
- (#99) Short array syntax must be used to define arrays
- (#99) Line exceeds 132 characters; contains 149 characters
- (#100) Short array syntax must be used to define arrays
- (#101) Short array syntax must be used to define arrays
- (#102) Short array syntax must be used to define arrays
- (#103) Short array syntax must be used to define arrays
- admin/tool/mergeusers/log.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for file log.php. Expected tool_mergeusers, found tool.
- (#30) Inline comments must end in full-stops, exclamation marks, or question marks
- (#48) Function print_error() has been deprecated; use throw new moodle_exception() (using lang strings only if meant to be shown to final user)() instead
- (#48) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#48) Inline comments must end in full-stops, exclamation marks, or question marks
- (#48) No space found before comment text; expected "// aborts execution" but found "//aborts execution"
- (#51) Short array syntax must be used to define arrays
- (#58) Short array syntax must be used to define arrays
- admin/tool/mergeusers/renderer.php
- (#1) No one-line description found in phpdocs for docblock of file renderer.php
- (#1) DocBlock missing a @package tag for file renderer.php. Expected @package tool_mergeusers
- (#2) Moodle boilerplate not found at first line
- (#18) Missing @license tag
- (#25) "require_once" must be immediately followed by an open parenthesis
- (#26) "require_once" must be immediately followed by an open parenthesis
- (#32) Incorrect @package tag for class tool_mergeusers_renderer. Expected tool_mergeusers, found tool.
- (#38) Opening brace should be on the same line as the declaration for class tool_mergeusers_renderer
- (#54) Opening brace should be on the same line as the declaration
- (#66) Short array syntax must be used to define arrays
- (#75) Opening brace should be on the same line as the declaration
- (#76) Short array syntax must be used to define arrays
- (#77) Short array syntax must be used to define arrays
- (#78) Short array syntax must be used to define arrays
- (#79) Short array syntax must be used to define arrays
- (#104) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $ust.
- (#105) Opening brace should be on the same line as the declaration
- (#117) Inline comments must end in full-stops, exclamation marks, or question marks
- (#119) Short array syntax must be used to define arrays
- (#139) Opening brace should be on the same line as the declaration
- (#149) Opening brace should be on the same line as the declaration
- (#165) Opening brace should be on the same line as the declaration
- (#192) Opening brace should be on the same line as the declaration
- (#206) Functions must not contain multiple empty lines in a row; found 2 empty lines
- (#212) Short array syntax must be used to define arrays
- (#213) Short array syntax must be used to define arrays
- (#227) Short array syntax must be used to define arrays
- (#235) Line exceeds 132 characters; contains 151 characters
- (#248) Opening brace should be on the same line as the declaration
- (#264) Opening brace should be on the same line as the declaration
- (#267) Short array syntax must be used to define arrays
- (#282) Opening brace should be on the same line as the declaration
- (#287) Short array syntax must be used to define arrays
- (#291) Short array syntax must be used to define arrays
- (#293) Short array syntax must be used to define arrays
- (#294) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#294) Inline comments must end in full-stops, exclamation marks, or question marks
- (#294) No space found before comment text; expected "// failure icon" but found "//failure icon"
- (#295) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#295) Inline comments must end in full-stops, exclamation marks, or question marks
- (#295) No space found before comment text; expected "// ok icon" but found "//ok icon"
- (#301) Short array syntax must be used to define arrays
- (#302) Short array syntax must be used to define arrays
- (#308) There should be a comma after the last array item in a multi-line array.
- (#311) Short array syntax must be used to define arrays
- (#314) Short array syntax must be used to define arrays
- (#328) Short array syntax must be used to define arrays
- (#330) Short array syntax must be used to define arrays
- admin/tool/mergeusers/review_form.php
- (#2) Moodle boilerplate not found at first line
- (#27) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#27) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#27) Inline comments must end in full-stops, exclamation marks, or question marks
- (#27) 3 slashes comments are not allowed; use "// Comment." instead
- (#29) Missing @copyright tag
- (#29) Missing @license tag
- (#32) DocBlock missing a @package tag for class reviewuserform. Expected @package tool_mergeusers
- (#33) Opening brace should be on the same line as the declaration for class reviewuserform
- (#42) Member variable "review_step" must not contain underscores.
- (#44) Missing docblock for function __construct
- (#44) Variable "review_step" must not contain underscores.
- (#45) Opening brace should be on the same line as the declaration
- (#46) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#46) Inline comments must end in full-stops, exclamation marks, or question marks
- (#46) No space found before comment text; expected "// just before parent's construct" but found "//just before parent's construct"
- (#49) Variable "review_step" must not contain underscores.
- (#59) Opening brace should be on the same line as the declaration
- (#60) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#61) Inline comments must end in full-stops, exclamation marks, or question marks
- (#68) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#68) Inline comments must end in full-stops, exclamation marks, or question marks
- (#71) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#71) Inline comments must end in full-stops, exclamation marks, or question marks
- (#74) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#75) Inline comments must end in full-stops, exclamation marks, or question marks
- (#77) Short array syntax must be used to define arrays
- admin/tool/mergeusers/select_form.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for file select_form.php. Expected tool_mergeusers, found tool.
- (#27) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
- (#27) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#27) Inline comments must end in full-stops, exclamation marks, or question marks
- (#27) 3 slashes comments are not allowed; use "// Comment." instead
- (#37) Missing docblock for function __construct
- (#37) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $ust.
- (#38) Opening brace should be on the same line as the declaration
- (#39) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#39) Inline comments must end in full-stops, exclamation marks, or question marks
- (#39) No space found before comment text; expected "// just before parent's construct" but found "//just before parent's construct"
- (#42) Functions must not contain multiple empty lines in a row; found 2 empty lines
- (#55) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#55) Inline comments must end in full-stops, exclamation marks, or question marks
- (#58) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#58) Inline comments must end in full-stops, exclamation marks, or question marks
- (#61) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#61) Inline comments must end in full-stops, exclamation marks, or question marks
- (#69) Inline comments must start with a capital letter, digit or 3-dots sequence
- (#69) Inline comments must end in full-stops, exclamation marks, or question marks
- admin/tool/mergeusers/settings.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for file settings.php. Expected tool_mergeusers, found tool.
- (#34) Inline doc block comments are not allowed; use "// Comment." instead
- (#76) Short array syntax must be used to define arrays
- admin/tool/mergeusers/tests/assign_submission_duplicated_test.php
- (#1) No one-line description found in phpdocs for docblock of file assign_submission_duplicated_test.php
- (#1) DocBlock missing a @package tag for file assign_submission_duplicated_test.php. Expected @package tool_mergeusers
- (#17) Missing @license tag
- (#27) Unit test assign_submission_duplicated_test should be declared as final.
- (#27) PHPUnit class "assign_submission_duplicated_test" does not have any namespace. It is recommended to add it to the "tool_mergeusers" namespace, using more levels if needed, in order to match the code being tested
- (#27) Missing docblock for class assign_submission_duplicated_test
- (#37) Test method test_remove_old_ignore_new() is missing any coverage information, own or at class level
- (#37) Test method test_remove_old_ignore_new() is missing a return type
- (#40) There should be a comma after the last array item in a multi-line array.
- (#43) There should be a comma after the last array item in a multi-line array.
- (#44) There should be a comma after the last array item in a multi-line array.
- (#56) Missing docblock for function remove_old_ignore_new_data_provider in testcase
- (#56) Data provider method "remove_old_ignore_new_data_provider" will need to be converted to static in future.
- (#56) Data provider method "remove_old_ignore_new_data_provider" must return an array, a Generator or an Iterable.
- (#62) There should be a comma after the last array item in a multi-line array.
- (#68) There should be a comma after the last array item in a multi-line array.
- (#74) There should be a comma after the last array item in a multi-line array.
- (#80) There should be a comma after the last array item in a multi-line array.
- (#81) There should be a comma after the last array item in a multi-line array.
- (#93) Test method test_update_old_and_remove_new() is missing any coverage information, own or at class level
- (#93) Test method test_update_old_and_remove_new() is missing a return type
- (#96) There should be a comma after the last array item in a multi-line array.
- (#99) There should be a comma after the last array item in a multi-line array.
- (#100) There should be a comma after the last array item in a multi-line array.
- (#111) Missing docblock for function update_old_and_remove_new_data_provider in testcase
- (#111) Data provider method "update_old_and_remove_new_data_provider" will need to be converted to static in future.
- (#111) Data provider method "update_old_and_remove_new_data_provider" must return an array, a Generator or an Iterable.
- (#117) There should be a comma after the last array item in a multi-line array.
- (#123) There should be a comma after the last array item in a multi-line array.
- (#129) There should be a comma after the last array item in a multi-line array.
- (#141) Test method test_update_first_and_remove_last() is missing any coverage information, own or at class level
- (#141) Test method test_update_first_and_remove_last() is missing a return type
- (#145) There should be a comma after the last array item in a multi-line array.
- (#148) There should be a comma after the last array item in a multi-line array.
- (#149) There should be a comma after the last array item in a multi-line array.
- (#160) Missing docblock for function update_first_and_remove_last_data_provider in testcase
- (#160) Data provider method "update_first_and_remove_last_data_provider" will need to be converted to static in future.
- (#160) Data provider method "update_first_and_remove_last_data_provider" must return an array, a Generator or an Iterable.
- (#167) There should be a comma after the last array item in a multi-line array.
- (#173) There should be a comma after the last array item in a multi-line array.
- (#179) There should be a comma after the last array item in a multi-line array.
- (#184) Missing docblock for function get_assign_submission_submitted in testcase
- (#192) Missing docblock for function get_assign_submission_submitted_by_date in testcase
- (#201) Missing docblock for function get_assign_submission_new in testcase
- (#209) Missing docblock for function get_assign_submission_draft_by_date in testcase
- (#216) Missing docblock for function get_assign_submission_draft in testcase
- (#224) Missing docblock for function get_assign_submission_reopened in testcase
- (#232) Missing docblock for function get_assign_submission in testcase
- admin/tool/mergeusers/tests/assign_test.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for file assign_test.php. Expected tool_mergeusers, found tool.
- (#34) Unit test assign_test should be declared as final.
- (#34) PHPUnit class "assign_test" does not have any namespace. It is recommended to add it to the "tool_mergeusers" namespace, using more levels if needed, in order to match the code being tested
- (#53) Test method test_mergenonconflictingassigngrades() is missing any coverage information, own or at class level
- (#53) Test method test_mergenonconflictingassigngrades() is missing a return type
- (#81) Variable "user_remove" must not contain underscores.
- (#81) Short array syntax must be used to define arrays
- (#82) Variable "user_remove" must not contain underscores.
- admin/tool/mergeusers/tests/clioptions_test.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for class clioptions_test. Expected tool_mergeusers, found tool.
- (#25) Unit test clioptions_test should be declared as final.
- (#25) PHPUnit class "clioptions_test" does not have any namespace. It is recommended to add it to the "tool_mergeusers" namespace, using more levels if needed, in order to match the code being tested
- (#27) The setUp() method in unit tests must always call to parent::setUp().
- (#33) The tearDown() method in unit tests must always call to parent::tearDown().
- (#44) Test method test_alwaysrollback() is missing any coverage information, own or at class level
- (#44) Test method test_alwaysrollback() is missing a return type
- (#48) Variable "user_remove" must not contain underscores.
- (#49) Variable "user_keep" must not contain underscores.
- (#52) Variable "user_keep" must not contain underscores.
- (#52) Variable "user_remove" must not contain underscores.
- (#55) Variable "user_remove" must not contain underscores.
- (#55) Short array syntax must be used to define arrays
- (#55) Variable "user_remove" must not contain underscores.
- (#56) Variable "user_remove" must not contain underscores.
- (#58) Variable "user_keep" must not contain underscores.
- (#58) Short array syntax must be used to define arrays
- (#58) Variable "user_keep" must not contain underscores.
- (#59) Variable "user_keep" must not contain underscores.
- (#61) Variable "user_remove_2" must not contain underscores.
- (#70) Variable "user_keep" must not contain underscores.
- (#70) Variable "user_remove_2" must not contain underscores.
- (#78) Test method test_debugdb() is missing any coverage information, own or at class level
- (#78) Test method test_debugdb() is missing a return type
- (#82) Variable "user_remove" must not contain underscores.
- (#83) Variable "user_keep" must not contain underscores.
- (#86) Variable "user_keep" must not contain underscores.
- (#86) Variable "user_remove" must not contain underscores.
- (#90) Variable "user_remove" must not contain underscores.
- (#90) Short array syntax must be used to define arrays
- (#90) Variable "user_remove" must not contain underscores.
- (#91) Variable "user_remove" must not contain underscores.
- (#93) Variable "user_keep" must not contain underscores.
- (#93) Short array syntax must be used to define arrays
- (#93) Variable "user_keep" must not contain underscores.
- (#94) Variable "user_keep" must not contain underscores.
- (#96) Variable "user_remove_2" must not contain underscores.
- (#103) Variable "user_keep" must not contain underscores.
- (#103) Variable "user_remove_2" must not contain underscores.
- admin/tool/mergeusers/tests/enrolments_test.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for class enrolments_test. Expected tool_mergeusers, found tool.
- (#25) Unit test enrolments_test should be declared as final.
- (#25) PHPUnit class "enrolments_test" does not have any namespace. It is recommended to add it to the "tool_mergeusers" namespace, using more levels if needed, in order to match the code being tested
- (#29) The setUp() method in unit tests must always call to parent::setUp().
- (#41) Test method test_mergeenrolments() is missing any coverage information, own or at class level
- (#41) Test method test_mergeenrolments() is missing a return type
- (#45) Variable "user_remove" must not contain underscores.
- (#46) Variable "user_keep" must not contain underscores.
- (#53) Short array syntax must be used to define arrays
- (#53) Expected 1 space before "=>"; 0 found
- (#53) Expected 1 space after "=>"; 0 found
- (#53) Expected 1 space before "=>"; 0 found
- (#53) Expected 1 space after "=>"; 0 found
- (#54) Short array syntax must be used to define arrays
- (#54) Expected 1 space before "=>"; 0 found
- (#54) Expected 1 space after "=>"; 0 found
- (#54) Expected 1 space before "=>"; 0 found
- (#54) Expected 1 space after "=>"; 0 found
- (#55) Short array syntax must be used to define arrays
- (#55) Expected 1 space before "=>"; 0 found
- (#55) Expected 1 space after "=>"; 0 found
- (#55) Expected 1 space before "=>"; 0 found
- (#55) Expected 1 space after "=>"; 0 found
- (#59) Short array syntax must be used to define arrays
- (#59) Expected 1 space before "=>"; 0 found
- (#59) Expected 1 space after "=>"; 0 found
- (#62) Variable "user_remove" must not contain underscores.
- (#63) Variable "user_remove" must not contain underscores.
- (#64) Variable "user_keep" must not contain underscores.
- (#65) Variable "user_keep" must not contain underscores.
- (#68) Variable "user_remove" must not contain underscores.
- (#71) Short array syntax must be used to define arrays
- (#74) Variable "user_keep" must not contain underscores.
- (#77) Short array syntax must be used to define arrays
- (#80) Variable "user_keep" must not contain underscores.
- (#80) Variable "user_remove" must not contain underscores.
- (#83) Variable "user_remove" must not contain underscores.
- (#83) Short array syntax must be used to define arrays
- (#83) Variable "user_remove" must not contain underscores.
- (#84) Variable "user_remove" must not contain underscores.
- (#87) Variable "user_keep" must not contain underscores.
- (#90) Short array syntax must be used to define arrays
- admin/tool/mergeusers/tests/quiz_test.php
- (#19) Missing @copyright tag
- (#22) Incorrect @package tag for class quiz_test. Expected tool_mergeusers, found tool.
- (#27) Unit test quiz_test should be declared as final.
- (#27) PHPUnit class "quiz_test" does not have any namespace. It is recommended to add it to the "tool_mergeusers" namespace, using more levels if needed, in order to match the code being tested
- (#34) The setUp() method in unit tests must always call to parent::setUp().
- (#50) Short array syntax must be used to define arrays
- (#52) There should be a comma after the last array item in a multi-line array.
- (#55) Short array syntax must be used to define arrays
- (#57) There should be a comma after the last array item in a multi-line array.
- (#62) Short array syntax must be used to define arrays
- (#80) Short array syntax must be used to define arrays
- (#84) There should be a comma after the last array item in a multi-line array.
- (#90) Short array syntax must be used to define arrays
- (#91) Short array syntax must be used to define arrays
- (#105) Short array syntax must be used to define arrays
- (#106) Short array syntax must be used to define arrays
- (#107) Short array syntax must be used to define arrays
- (#107) There should be a comma after the last array item in a multi-line array.
- (#130) Short array syntax must be used to define arrays
- (#131) Short array syntax must be used to define arrays
- (#132) Short array syntax must be used to define arrays
- (#132) There should be a comma after the last array item in a multi-line array.
- (#170) Test method test_mergeconflictingquizattempts() is missing any coverage information, own or at class level
- (#170) Test method test_mergeconflictingquizattempts() is missing a return type
- (#188) Variable "user_remove" must not contain underscores.
- (#188) Short array syntax must be used to define arrays
- (#189) Variable "user_remove" must not contain underscores.
- (#197) Test method test_mergenonconflictingquizattempts() is missing any coverage information, own or at class level
- (#197) Test method test_mergenonconflictingquizattempts() is missing a return type
- (#216) Variable "user_remove" must not contain underscores.
- (#216) Short array syntax must be used to define arrays
- (#217) Variable "user_remove" must not contain underscores.
- admin/tool/mergeusers/tests/user_profile_field_info_test.php
- (#16) Boilerplate comment must be followed by a single blank line or end of file
- (#30) Unit test user_profile_field_info_test should be declared as final.
- (#30) PHPUnit class "user_profile_field_info_test" does not have any namespace. It is recommended to add it to the "tool_mergeusers" namespace, using more levels if needed, in order to match the code being tested
- (#30) Missing @copyright tag
- (#30) Missing @license tag
- (#30) Missing docblock for class user_profile_field_info_test
- (#32) The setUp() method in unit tests must always call to parent::setUp().
- (#35) "require_once" must be immediately followed by an open parenthesis
- (#45) Test method test_create_user_profile_category() is missing any coverage information, own or at class level
- (#49) Variable "old_id" must not contain underscores.
- (#60) Variable "old_id" must not contain underscores.
- (#71) Test method test_user_profile_fields_are_created() is missing any coverage information, own or at class level
- (#94) Test method test_profile_fields_are_updated_on_merge_success() is missing any coverage information, own or at class level
- (#143) Variable "old_user" must not contain underscores.
- (#144) Variable "new_user" must not contain underscores.
- (#151) Variable "new_user" must not contain underscores.
- (#152) Variable "old_user" must not contain underscores.
- (#191) Line exceeds 132 characters; contains 136 characters
- admin/tool/mergeusers/version.php
- (#17) Missing @copyright tag
- admin/tool/mergeusers/view.php
- (#17) Missing @copyright tag
- (#20) Incorrect @package tag for file view.php. Expected tool_mergeusers, found tool.
- (#30) Inline comments must end in full-stops, exclamation marks, or question marks
- (#64) There should be a comma after the last array item in a multi-line array.
Javascript coding style problems
(27 errors, 13 warnings)
This section shows the coding style problems detected in the code by eslint [More info]
- admin/tool/mergeusers/module.js
- (#21) Identifier 'tool_mergeusers' is not in camel case. (camelcase)
- (#22) Identifier 'init_select_table' is not in camel case. (camelcase)
- (#22) Unexpected space before function parentheses. (space-before-function-paren)
- (#23) Unexpected space before function parentheses. (space-before-function-paren)
- (#24) 'radiobuttons' is not defined. (no-undef)
- (#25) 'radiobuttons' is not defined. (no-undef)
- (#25) Unexpected space before function parentheses. (space-before-function-paren)
- (#25) Missing space before opening brace. (space-before-blocks)
- (#28) 'current' is not defined. (no-undef)
- (#29) There should be no space after this paren. (space-in-parens)
- (#29) 'current' is not defined. (no-undef)
- (#29) There should be no space before this paren. (space-in-parens)
- (#30) 'target' is not defined. (no-undef)
- (#31) 'lastselected' is not defined. (no-undef)
- (#32) 'lastvalue' is not defined. (no-undef)
- (#32) 'lastselected' is not defined. (no-undef)
- (#34) 'target' is not defined. (no-undef)
- (#35) 'lastselected' is not defined. (no-undef)
- (#36) 'lastvalue' is not defined. (no-undef)
- (#36) 'lastselected' is not defined. (no-undef)
- (#39) Comments should not begin with a lowercase character. (capitalized-comments)
- (#40) 'id' is not defined. (no-undef)
- (#41) 'radiobutton' is not defined. (no-undef)
- (#41) 'target' is not defined. (no-undef)
- (#41) 'id' is not defined. (no-undef)
- (#42) 'radiobutton' is not defined. (no-undef)
- (#44) Comments should not begin with a lowercase character. (capitalized-comments)
- (#45) There should be no space after this paren. (space-in-parens)
- (#45) 'lastvalue' is not defined. (no-undef)
- (#45) 'lastvalue' is not defined. (no-undef)
- (#45) 'id' is not defined. (no-undef)
- (#46) 'last' is not defined. (no-undef)
- (#46) Operator '+' must be spaced. (space-infix-ops)
- (#46) 'target' is not defined. (no-undef)
- (#46) Operator '+' must be spaced. (space-infix-ops)
- (#46) 'lastvalue' is not defined. (no-undef)
- (#47) 'last' is not defined. (no-undef)
- (#49) 'lastselected' is not defined. (no-undef)
- (#49) 'id' is not defined. (no-undef)
- (#54) Missing semicolon. (semi)
CSS problems
(1 errors, 0 warnings)
This section shows CSS problems detected by stylelint [More info]
- admin/tool/mergeusers/styles.css
- (#39) Expected single space before "{" (block-opening-brace-space-before)
PHPDocs style problems
(16 errors, 0 warnings)
This section shows the phpdocs problems detected in the code by local_moodlecheck [More info]
- admin/tool/mergeusers/classes/tool_mergeusers_logger.php
- (#71) Phpdocs for function tool_mergeusers_logger::get has incomplete parameters list
- admin/tool/mergeusers/index_form.php
- (#31) Found comment starting with three or more slashes
- admin/tool/mergeusers/lib/merger.php
- (#37) Phpdocs for function Merger::__construct has incomplete parameters list
- admin/tool/mergeusers/lib/mergeusertool.php
- (#451) Phpdocs for function MergeUserTool::updateGrades has incomplete parameters list
- admin/tool/mergeusers/lib/table/quizattemptsmerger.php
- (#270) Phpdocs for function QuizAttemptsMerger::updateAllQuizzes has incomplete parameters list
- admin/tool/mergeusers/renderer.php
- (#143) Phpdocs for function tool_mergeusers_renderer::render_user_review_table has incomplete parameters list
- admin/tool/mergeusers/review_form.php
- (#27) Found comment starting with three or more slashes
- admin/tool/mergeusers/select_form.php
- (#27) Found comment starting with three or more slashes
- admin/tool/mergeusers/tests/assign_submission_duplicated_test.php
- (#29) Phpdocs for function assign_submission_duplicated_test::test_remove_old_ignore_new has incomplete parameters list
- (#85) Phpdocs for function assign_submission_duplicated_test::test_update_old_and_remove_new has incomplete parameters list
- (#134) Phpdocs for function assign_submission_duplicated_test::test_update_first_and_remove_last has incomplete parameters list
- admin/tool/mergeusers/tests/assign_test.php
- (#87) Phpdocs for function assign_test::get_user_assign_grade has incomplete parameters list
- admin/tool/mergeusers/tests/quiz_test.php
- (#74) Phpdocs for function quiz_test::add_quiz_to_course has incomplete parameters list
- (#113) Phpdocs for function quiz_test::get_user_quiz_grade has incomplete parameters list
- (#138) Phpdocs for function quiz_test::submit_quiz_attempt has incomplete parameters list
- admin/tool/mergeusers/tests/user_profile_field_info_test.php
- (#136) Phpdocs for function user_profile_field_info_test::trigger_user_merged_success_event 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]